Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-17 Thread Marcel Holtmann
Hi Jiri, > > > I have just verified that this locking scheme is indeed correct. So you > > > can add > > > > > > Signed-off-by: Jiri Kosina <[EMAIL PROTECTED]> > > > > > > if you wish to, and submit the patch to Andrew. > > I guess I don't get sent networking patches any more? > > :-) > >

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-17 Thread Marcel Holtmann
Hi Jiri, I have just verified that this locking scheme is indeed correct. So you can add Signed-off-by: Jiri Kosina [EMAIL PROTECTED] if you wish to, and submit the patch to Andrew. I guess I don't get sent networking patches any more? :-) Well, this is

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, David Miller wrote: > > I have just verified that this locking scheme is indeed correct. So you > > can add > > > > Signed-off-by: Jiri Kosina <[EMAIL PROTECTED]> > > > > if you wish to, and submit the patch to Andrew. > I guess I don't get sent networking patches any

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread David Miller
From: Jiri Kosina <[EMAIL PROTECTED]> Date: Thu, 17 May 2007 01:03:55 +0200 (CEST) > On Wed, 16 May 2007, Jiri Kosina wrote: > > > > since Jiri has a good test case for it, I leave it to him for testing. > > > If he confirms that this fixes the locking issues, then this is > > > Signed-off-by:

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Jiri Kosina wrote: > > since Jiri has a good test case for it, I leave it to him for testing. > > If he confirms that this fixes the locking issues, then this is > > Signed-off-by: Marcel Holtmann <[EMAIL PROTECTED]> > I will verify later this evening and will let you know. I

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Marcel Holtmann wrote: > since Jiri has a good test case for it, I leave it to him for testing. > If he confirms that this fixes the locking issues, then this is > Signed-off-by: Marcel Holtmann <[EMAIL PROTECTED]> I will verify later this evening and will let you know. I am

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satyam, > > > > > > (later) > > > > > > I Googled a bit to see if this problem was faced elsewhere in the > > > > > > kernel > > > > > > too. Saw the following commit by Ingo Molnar > > > > > > (9883a13c72dbf8c518814b6091019643cdb34429): > > > > > > - lock_sock(sock->sk); > > > > > > +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
On 5/16/07, Satyam Sharma <[EMAIL PROTECTED]> wrote: Hi Marcel, [...] > > > > (later) > > > > I Googled a bit to see if this problem was faced elsewhere in the kernel > > > > too. Saw the following commit by Ingo Molnar > > > > (9883a13c72dbf8c518814b6091019643cdb34429): > > > > -

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Marcel, On 5/16/07, Marcel Holtmann <[EMAIL PROTECTED]> wrote: Hi Satayam, > > > (later) > > > I Googled a bit to see if this problem was faced elsewhere in the kernel > > > too. Saw the following commit by Ingo Molnar > > > (9883a13c72dbf8c518814b6091019643cdb34429): > > > -

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satayam, > > > (later) > > > I Googled a bit to see if this problem was faced elsewhere in the kernel > > > too. Saw the following commit by Ingo Molnar > > > (9883a13c72dbf8c518814b6091019643cdb34429): > > > - lock_sock(sock->sk); > > > + local_bh_disable(); > > > +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Jiri, On 5/16/07, Jiri Kosina <[EMAIL PROTECTED]> wrote: On Fri, 11 May 2007, Satyam Sharma wrote: > (later) > I Googled a bit to see if this problem was faced elsewhere in the kernel > too. Saw the following commit by Ingo Molnar > (9883a13c72dbf8c518814b6091019643cdb34429): > -

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Fri, 11 May 2007, Satyam Sharma wrote: > (later) > I Googled a bit to see if this problem was faced elsewhere in the kernel > too. Saw the following commit by Ingo Molnar > (9883a13c72dbf8c518814b6091019643cdb34429): > - lock_sock(sock->sk); > + local_bh_disable(); > +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Fri, 11 May 2007, Satyam Sharma wrote: (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Jiri, On 5/16/07, Jiri Kosina [EMAIL PROTECTED] wrote: On Fri, 11 May 2007, Satyam Sharma wrote: (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): -

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satayam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); + bh_lock_sock_nested(sock-sk);

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
Hi Marcel, On 5/16/07, Marcel Holtmann [EMAIL PROTECTED] wrote: Hi Satayam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Satyam Sharma
On 5/16/07, Satyam Sharma [EMAIL PROTECTED] wrote: Hi Marcel, [...] (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Marcel Holtmann
Hi Satyam, (later) I Googled a bit to see if this problem was faced elsewhere in the kernel too. Saw the following commit by Ingo Molnar (9883a13c72dbf8c518814b6091019643cdb34429): - lock_sock(sock-sk); + local_bh_disable(); +

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Marcel Holtmann wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel Holtmann [EMAIL PROTECTED] I will verify later this evening and will let you know. I am

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, Jiri Kosina wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel Holtmann [EMAIL PROTECTED] I will verify later this evening and will let you know. I am

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread David Miller
From: Jiri Kosina [EMAIL PROTECTED] Date: Thu, 17 May 2007 01:03:55 +0200 (CEST) On Wed, 16 May 2007, Jiri Kosina wrote: since Jiri has a good test case for it, I leave it to him for testing. If he confirms that this fixes the locking issues, then this is Signed-off-by: Marcel

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-16 Thread Jiri Kosina
On Wed, 16 May 2007, David Miller wrote: I have just verified that this locking scheme is indeed correct. So you can add Signed-off-by: Jiri Kosina [EMAIL PROTECTED] if you wish to, and submit the patch to Andrew. I guess I don't get sent networking patches any more? :-)

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-13 Thread Greg KH
On Fri, May 11, 2007 at 06:59:31PM +0530, Satyam Sharma wrote: > [1] This is the first problem point. However, I didn't find any reason > why this particular driver's .disconnect() couldn't sleep. In fact, a > comment in include/linux/usb.h:811 says: > > "The probe() and disconnect() methods

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-13 Thread Greg KH
On Fri, May 11, 2007 at 06:59:31PM +0530, Satyam Sharma wrote: [1] This is the first problem point. However, I didn't find any reason why this particular driver's .disconnect() couldn't sleep. In fact, a comment in include/linux/usb.h:811 says: The probe() and disconnect() methods are

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-11 Thread Satyam Sharma
Hi Jiri, On 4/26/07, Jiri Kosina <[EMAIL PROTECTED]> wrote: On Mon, 23 Apr 2007, Jiri Kosina wrote: > > BUG: sleeping function called from invalid context at net/core/sock.c:1523 > > in_atomic():1, irqs_disabled():0 > > 1 lock held by khubd/180: > > #0: (old_style_rw_init#2){-.-?}, at: []

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-05-11 Thread Satyam Sharma
Hi Jiri, On 4/26/07, Jiri Kosina [EMAIL PROTECTED] wrote: On Mon, 23 Apr 2007, Jiri Kosina wrote: BUG: sleeping function called from invalid context at net/core/sock.c:1523 in_atomic():1, irqs_disabled():0 1 lock held by khubd/180: #0: (old_style_rw_init#2){-.-?}, at: [f88c5816]

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-26 Thread Jiri Kosina
On Mon, 23 Apr 2007, Jiri Kosina wrote: > > BUG: sleeping function called from invalid context at net/core/sock.c:1523 > > in_atomic():1, irqs_disabled():0 > > 1 lock held by khubd/180: > > #0: (old_style_rw_init#2){-.-?}, at: [] > > hci_sock_dev_event+0x42/0xc5 [bluetooth] [...] > OK, this

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-26 Thread Jiri Kosina
On Mon, 23 Apr 2007, Jiri Kosina wrote: BUG: sleeping function called from invalid context at net/core/sock.c:1523 in_atomic():1, irqs_disabled():0 1 lock held by khubd/180: #0: (old_style_rw_init#2){-.-?}, at: [f88c5816] hci_sock_dev_event+0x42/0xc5 [bluetooth] [...] OK, this

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-24 Thread Jiri Kosina
On Tue, 24 Apr 2007, Herbert Xu wrote: > > Hmm, *sigh*. I guess the patch below fixes the problem, but it is a > > masterpiece in the field of ugliness. And I am not sure whether it is > > completely correct either. Are there any immediate ideas for better > > solution with respect to how

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-24 Thread Jiri Kosina
On Tue, 24 Apr 2007, Herbert Xu wrote: Hmm, *sigh*. I guess the patch below fixes the problem, but it is a masterpiece in the field of ugliness. And I am not sure whether it is completely correct either. Are there any immediate ideas for better solution with respect to how struct sock

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Herbert Xu
Jiri Kosina <[EMAIL PROTECTED]> wrote: > > Hmm, *sigh*. I guess the patch below fixes the problem, but it is a > masterpiece in the field of ugliness. And I am not sure whether it is > completely correct either. Are there any immediate ideas for better > solution with respect to how struct

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Jiri Kosina
On Mon, 23 Apr 2007, Jeremy Fitzhardinge wrote: > I got this on resume; it looks like a Bluetooth and/or USB problem. > PM: Removing info for No Bus:hci0 > BUG: sleeping function called from invalid context at net/core/sock.c:1523 > in_atomic():1, irqs_disabled():0 > 1 lock held by khubd/180: >

2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Jeremy Fitzhardinge
I got this on resume; it looks like a Bluetooth and/or USB problem. PM: Removing info for No Bus:hci0 BUG: sleeping function called from invalid context at net/core/sock.c:1523 in_atomic():1, irqs_disabled():0 1 lock held by khubd/180: #0: (old_style_rw_init#2){-.-?}, at: []

2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Jeremy Fitzhardinge
I got this on resume; it looks like a Bluetooth and/or USB problem. PM: Removing info for No Bus:hci0 BUG: sleeping function called from invalid context at net/core/sock.c:1523 in_atomic():1, irqs_disabled():0 1 lock held by khubd/180: #0: (old_style_rw_init#2){-.-?}, at: [f88c5816]

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Jiri Kosina
On Mon, 23 Apr 2007, Jeremy Fitzhardinge wrote: I got this on resume; it looks like a Bluetooth and/or USB problem. PM: Removing info for No Bus:hci0 BUG: sleeping function called from invalid context at net/core/sock.c:1523 in_atomic():1, irqs_disabled():0 1 lock held by khubd/180: #0:

Re: 2.6.21-rc7: BUG: sleeping function called from invalid context at net/core/sock.c:1523

2007-04-23 Thread Herbert Xu
Jiri Kosina [EMAIL PROTECTED] wrote: Hmm, *sigh*. I guess the patch below fixes the problem, but it is a masterpiece in the field of ugliness. And I am not sure whether it is completely correct either. Are there any immediate ideas for better solution with respect to how struct sock