[PATCH v2] clk: Don't hold prepare_lock across debugfs creation

2014-09-10 Thread Mike Turquette
Quoting Stephen Boyd (2014-09-05 17:00:00) > On 09/04, Stephen Boyd wrote: > > > > I don't understand why we need to hold the prepare lock around > > the kref_put(), so I changed the flow so that we don't do this > > when we unregister a clock. > > Ok we hold the prepare mutex to make sure get an

[PATCH v2] clk: Don't hold prepare_lock across debugfs creation

2014-09-05 Thread Stephen Boyd
On 09/04, Stephen Boyd wrote: > > I don't understand why we need to hold the prepare lock around > the kref_put(), so I changed the flow so that we don't do this > when we unregister a clock. Ok we hold the prepare mutex to make sure get and put are serialized. Good. Here's the interdiff to move

[PATCH v2] clk: Don't hold prepare_lock across debugfs creation

2014-09-04 Thread Stephen Boyd
Rob Clark reports a lockdep splat that involves the prepare_lock chained with the mmap semaphore. == [ INFO: possible circular locking dependency detected ] 3.17.0-rc1-00050-g07a489b #802 Tainted: GW --