Re: [PATCH 1/2] lockdep: check the depth of subclass

[Dmitry Torokhov <dmitry.torokhov@gmail.com>]

On Wed, Oct 13, 2010 at 11:27:46AM +0900, Hitoshi Mitake wrote:
> On 10/13/10 01:03, Dmitry Torokhov wrote:
> > On Tue, Oct 12, 2010 at 12:27:01PM +0200, Peter Zijlstra wrote:
> >> On Tue, 2010-10-05 at 18:01 +0900, Hitoshi Mitake wrote:
> >>> ---
> >>>   drivers/input/mousedev.c |    2 +-
> >>>   kernel/lockdep.c         |   15 +++++++++++++++
> >>>   2 files changed, 16 insertions(+), 1 deletions(-)
> >>>
> >>> diff --git a/drivers/input/mousedev.c b/drivers/input/mousedev.c
> >>> index d528a2d..9897334 100644
> >>> --- a/drivers/input/mousedev.c
> >>> +++ b/drivers/input/mousedev.c
> >>> @@ -866,7 +866,7 @@ static struct mousedev
> *mousedev_create(struct input_dev *dev,
> >>>   	spin_lock_init(&mousedev->client_lock);
> >>>   	mutex_init(&mousedev->mutex);
> >>>   	lockdep_set_subclass(&mousedev->mutex,
> >>> -			     minor == MOUSEDEV_MIX ? MOUSEDEV_MIX : 0);
> >>> +			     minor == MOUSEDEV_MIX ? SINGLE_DEPTH_NESTING : 0);
> >>
> >> Ah good find.
> >>
> >
> > I'll pick up mousedev change for .37.
> 
> Thanks a lot, Dmitry.
> 
> Should I divide this patch into two for
> drivers/input/mousedev.c and kernel/lockdep.c?
> If you need, I'll resend second version.

No, thanks, I'll extract it on my end.

-- 
Dmitry