On Tue, Dec 01, 2015 at 05:45:17PM +0100, Arnd Bergmann wrote: > On Tuesday 01 December 2015 16:40:57 Mark Brown wrote:
> > mutex_lock(&rdev->mutex); > > - for (i = 1; rdev->supply; rdev = rdev->supply->rdev, i++) > > + for (i = 1; rdev; rdev = rdev->supply->rdev, i++) > > mutex_lock_nested(&rdev->mutex, i); > I think now we have a double lock, the initial mutex_lock() needs to > be removed again. Ugh, I even made that fix before I sent but didn't actually commit it :(
signature.asc
Description: PGP signature
