Re: [Linux-zigbee-devel] 6lowpan: lockless tx queue of routing netlink device

2014-02-10 Thread Alexander Aring
Hi Eric, On Sun, Feb 09, 2014 at 04:41:47AM -0800, Eric Dumazet wrote: > > Please try the following fix, thanks for this report ! > > diff --git a/net/ieee802154/6lowpan.c b/net/ieee802154/6lowpan.c > index 48b25c0af4d0..069af33013c4 100644 > --- a/net/ieee802154/6lowpan.c > +++ b/net/ieee802154

Re: [Linux-zigbee-devel] 6lowpan: lockless tx queue of routing netlink device

2014-02-10 Thread Eric Dumazet
On Mon, 2014-02-10 at 18:33 +0100, Alexander Aring wrote: > Hi Eric, > > On Sun, Feb 09, 2014 at 04:41:47AM -0800, Eric Dumazet wrote: > > > > Please try the following fix, thanks for this report ! > > > > diff --git a/net/ieee802154/6lowpan.c b/net/ieee802154/6lowpan.c > > index 48b25c0af4d0..0

Re: [Linux-zigbee-devel] 6lowpan: lockless tx queue of routing netlink device

2014-02-10 Thread Alexander Aring
Hi, On Mon, Feb 10, 2014 at 10:16:26AM -0800, Eric Dumazet wrote: > On Mon, 2014-02-10 at 18:33 +0100, Alexander Aring wrote: > > Hi Eric, > > > > On Sun, Feb 09, 2014 at 04:41:47AM -0800, Eric Dumazet wrote: > > > > > > Please try the following fix, thanks for this report ! > > > > > > diff --

Re: [Linux-zigbee-devel] 6lowpan: lockless tx queue of routing netlink device

2014-02-10 Thread Eric Dumazet
On Mon, 2014-02-10 at 19:21 +0100, Alexander Aring wrote: > no problem. Maybe you can try to give me some explanation what you did > there? > > I see you make a lockdep_set_class for each tx queue and assign the > lowpan_netdev_xmit_lock_key. So why it's better to make a own > "lock_key". This

Re: [Linux-zigbee-devel] 6lowpan: lockless tx queue of routing netlink device

2014-02-10 Thread Alexander Aring
On Mon, Feb 10, 2014 at 11:26:12AM -0800, Eric Dumazet wrote: > On Mon, 2014-02-10 at 19:21 +0100, Alexander Aring wrote: > > > no problem. Maybe you can try to give me some explanation what you did > > there? > > > > I see you make a lockdep_set_class for each tx queue and assign the > > lowpan

Re: [Linux-zigbee-devel] [PATCH net] 6lowpan: add missing fragment list spinlock

2014-02-10 Thread Alexander Aring
On Fri, Feb 07, 2014 at 12:28:15PM +, Martin Townsend wrote: > Hi Alex, > > I have tried out your new branch and I am seeing the same problem. Currently there was some locking issue on a lowpan device. Maybe try the new patch! [1] I will update my branch soon, have exam-time right now. [1] h

Re: [Linux-zigbee-devel] [PATCH net] 6lowpan: add missing fragment list spinlock

2014-02-10 Thread Alexander Aring
On Mon, Feb 10, 2014 at 09:13:59PM +0100, Alexander Aring wrote: > On Fri, Feb 07, 2014 at 12:28:15PM +, Martin Townsend wrote: > > Hi Alex, > > > > I have tried out your new branch and I am seeing the same problem. > > Currently there was some locking issue on a lowpan device. Maybe try the