Re: [PATCH] evdev: Release eventual input device grabs when getting disconnected

[Greg KH <greg@kroah.com>]

On Sun, Mar 30, 2008 at 02:51:02PM -0700, Linus Torvalds wrote:
> 
> 
> On Sun, 30 Mar 2008, Bj?rn Steinbrink wrote:
> >
> > I can't reproduce the bug on my UP box and currently can't afford
> > crashing my SMP box (all the oopses seem to come from SMP kernels, so I
> > guess it needs SMP to crash), so while this doesn't show any new
> > problems, I can't tell whether it actually fixes anything. Testers
> > welcome!
> 
> Ok, I applied this because I will do an -rc8 today or tomorrow, but I 
> really really hope somebody can figure out what made this all start to 
> trigger. It does smell like some core device layer change, because we do 
> not seem to have a lot of changes since 2.6.24 in evdev.c and input.c that 
> seem relevant.
> 
> Greg, are there any refcounting changes that would cause the input devices 
> to be free'd earlier or something?

Earlier?  No, not that I know of at all, as long as the reference
counting logic was correct originally.  All of the problems we have been
fixing were ones where we accidentally were grabbing too many references
and then wondering why things were not getting cleaned up properly as
the kobject rework exposed these problems making them more obvious.

I haven't heard of the opposite happening.  Anything that I can try to
test for here, I have a lot of removable input devices to test with.

thanks,

greg k-h