Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-06 Thread Alban Browaeys
Le jeudi 05 février 2015 à 13:45 -0800, Stephen Boyd a écrit : > On 02/05/15 13:30, Alban Browaeys wrote: > >> Thanks for the information. Can you please try the patch in this other > >> thread[1]? I think you're seeing the same problem. > >> > >> [1] https://lkml.org/lkml/2015/2/5/595 > > > > This

Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Stephen Boyd
On 02/05/15 13:30, Alban Browaeys wrote: >> Thanks for the information. Can you please try the patch in this other >> thread[1]? I think you're seeing the same problem. >> >> [1] https://lkml.org/lkml/2015/2/5/595 > > This fixes both oops : the one from __clk_get and the next from > hlist_del. > G

Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Alban Browaeys
> > Thanks for the information. Can you please try the patch in this other > thread[1]? I think you're seeing the same problem. > > [1] https://lkml.org/lkml/2015/2/5/595 This fixes both oops : the one from __clk_get and the next from hlist_del. Thanks Alban -- To unsubscribe from this list:

Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Stephen Boyd
On 02/05/15 12:43, Alban Browaeys wrote: > Le jeudi 05 février 2015 à 11:30 -0800, Stephen Boyd a écrit : > >>> Signed-off-by: Alban Browaeys >>> --- >>> drivers/clk/clk.c | 17 + >>> 1 file changed, 9 insertions(+), 8 deletions(-) >>> >>> diff --git a/drivers/clk/clk.c b/drivers/

Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Alban Browaeys
Le jeudi 05 février 2015 à 11:30 -0800, Stephen Boyd a écrit : > > Signed-off-by: Alban Browaeys > > --- > > drivers/clk/clk.c | 17 + > > 1 file changed, 9 insertions(+), 8 deletions(-) > > > > diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c > > index df94668..8f33722 100644

Re: [PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Stephen Boyd
On 02/05/15 10:24, Alban Browaeys wrote: > On the second call to __set_clk_parents from of_clk_set_defaults, here > when registering the second fimc device the kernel OOPS in an "unhandled > paging request at virtual address 6b6b6b77". This in __clk_get when > dereferencing clk->owner. > > Move the

[PATCH 1/2] clk: Fix __clk_get access to already freed owner field.

2015-02-05 Thread Alban Browaeys
On the second call to __set_clk_parents from of_clk_set_defaults, here when registering the second fimc device the kernel OOPS in an "unhandled paging request at virtual address 6b6b6b77". This in __clk_get when dereferencing clk->owner. Move the clk free in the kref managed _clk_release call inst