Re: [PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-06 Thread Mike Turquette
Quoting Lai Jiangshan (2013-06-03 02:17:15) > The @cn is stay in @clk_notifier_list after it is freed, it cause > memory corruption. > > Example, if @clk is registered(first), unregistered(first), > registered(second), unregistered(second). > > The freed @cn will be used when @clk is

Re: [PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-06 Thread Mike Turquette
Quoting Lai Jiangshan (2013-06-03 02:17:15) The @cn is stay in @clk_notifier_list after it is freed, it cause memory corruption. Example, if @clk is registered(first), unregistered(first), registered(second), unregistered(second). The freed @cn will be used when @clk is

Re: [PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-03 Thread Sören Brinkmann
Hi Lai, On Mon, Jun 03, 2013 at 05:17:15PM +0800, Lai Jiangshan wrote: > On 06/01/2013 03:12 AM, Sören Brinkmann wrote: > > Hi, > > > > we recently encountered some kernel panics when we compiled one of our > > drivers as module and tested inserting/removing the module. > > Trying to debug this

[PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-03 Thread Lai Jiangshan
On 06/01/2013 03:12 AM, Sören Brinkmann wrote: > Hi, > > we recently encountered some kernel panics when we compiled one of our > drivers as module and tested inserting/removing the module. > Trying to debug this issue, I could reproduce it on the mainline kernel > with a dummy module. > > What

[PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-03 Thread Lai Jiangshan
On 06/01/2013 03:12 AM, Sören Brinkmann wrote: Hi, we recently encountered some kernel panics when we compiled one of our drivers as module and tested inserting/removing the module. Trying to debug this issue, I could reproduce it on the mainline kernel with a dummy module. What happens

Re: [PATCH] clk: remove the clk_notifier from clk_notifier_list before free it (was: Re: [BUG] zynq | CCF | SRCU)

2013-06-03 Thread Sören Brinkmann
Hi Lai, On Mon, Jun 03, 2013 at 05:17:15PM +0800, Lai Jiangshan wrote: On 06/01/2013 03:12 AM, Sören Brinkmann wrote: Hi, we recently encountered some kernel panics when we compiled one of our drivers as module and tested inserting/removing the module. Trying to debug this issue, I