Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Peter Wu
On Wednesday 07 August 2013 15:34:08 Jiri Kosina wrote: > On Wed, 7 Aug 2013, Peter Wu wrote: > > [..] > > I have applied Manoj's patch[1] on top of 3.11-rc4 which seem to fix the > > issue. One observation is that the new device is named /dev/hidraw1 > > instead of /dev/hidraw0. Example: > > > >

RE: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Manoj Chourasia
then the patch could solve your issue. Regards -Manoj -Original Message- From: Jiri Kosina [mailto:jkos...@suse.cz] Sent: Wednesday, August 07, 2013 7:04 PM To: Peter Wu Cc: linux-in...@vger.kernel.org; Manoj Chourasia; linux-kernel@vger.kernel.org; alno...@suse.cz Subject: Re: List corruption

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Jiri Kosina
On Wed, 7 Aug 2013, Peter Wu wrote: > > does the patch below fix the problem you are seeing? > That one is already in 3.11-rc4 as far as I can see. Also, that code can > probably simplified by moving the mutex_unlock after the out label, removing > the need to duplicate the mutex_unlock. > >

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Peter Wu
On Wednesday 07 August 2013 03:01:26 Jiri Kosina wrote: > On Tue, 6 Aug 2013, Peter Wu wrote: > > While debugging upowerd (with Logitech Unifying receiver via hidraw), > > I came across this list corruption warning. > > Peter, > > does the patch below fix the problem you are seeing? That one is

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Peter Wu
On Wednesday 07 August 2013 03:01:26 Jiri Kosina wrote: On Tue, 6 Aug 2013, Peter Wu wrote: While debugging upowerd (with Logitech Unifying receiver via hidraw), I came across this list corruption warning. Peter, does the patch below fix the problem you are seeing? That one is already

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Jiri Kosina
On Wed, 7 Aug 2013, Peter Wu wrote: does the patch below fix the problem you are seeing? That one is already in 3.11-rc4 as far as I can see. Also, that code can probably simplified by moving the mutex_unlock after the out label, removing the need to duplicate the mutex_unlock. Remember

RE: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Manoj Chourasia
then the patch could solve your issue. Regards -Manoj -Original Message- From: Jiri Kosina [mailto:jkos...@suse.cz] Sent: Wednesday, August 07, 2013 7:04 PM To: Peter Wu Cc: linux-in...@vger.kernel.org; Manoj Chourasia; linux-kernel@vger.kernel.org; alno...@suse.cz Subject: Re: List corruption

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-07 Thread Peter Wu
On Wednesday 07 August 2013 15:34:08 Jiri Kosina wrote: On Wed, 7 Aug 2013, Peter Wu wrote: [..] I have applied Manoj's patch[1] on top of 3.11-rc4 which seem to fix the issue. One observation is that the new device is named /dev/hidraw1 instead of /dev/hidraw0. Example: f(){

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-06 Thread Jiri Kosina
On Tue, 6 Aug 2013, Peter Wu wrote: > While debugging upowerd (with Logitech Unifying receiver via hidraw), > I came across this list corruption warning. Peter, does the patch below fix the problem you are seeing? --- drivers/hid/hidraw.c |2 +- 1 files changed, 1 insertions(+), 1

Re: List corruption in hidraw_release in 3.11-rc4

2013-08-06 Thread Jiri Kosina
On Tue, 6 Aug 2013, Peter Wu wrote: While debugging upowerd (with Logitech Unifying receiver via hidraw), I came across this list corruption warning. Peter, does the patch below fix the problem you are seeing? --- drivers/hid/hidraw.c |2 +- 1 files changed, 1 insertions(+), 1