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

2014-02-11 Thread Martin Townsend
Hi Alex, I will try the patch as soon as I can, hopefully late afternoon. It's a homogeneous network of only 2 devices both booting linux from the same TFTP server. The devices are our own custom boards with a Xilinx FPGA running a PLC modem so the device driver is one I have written so there

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

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-07 Thread Martin Townsend
Hi Alex, I have tried out your new branch and I am seeing the same problem. It was happy for about an hour and then the pings became sporadic, after trying each device separately I can see that one of the devices is having trouble with pings. I have a wireshark trace from this device which is

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

2014-02-06 Thread Martin Townsend
Hi Alex, On 06/02/14 11:45, Alexander Aring wrote: > Hi Martin, > > On Thu, Feb 06, 2014 at 11:08:35AM +, Martin Townsend wrote: >> Hi Alex, >> >> No problems testing your new branch, just send me the URL, branch name and >> can you also send me the commit hash as open embedded uses this. >>

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

2014-02-06 Thread Alexander Aring
Hi Martin, On Thu, Feb 06, 2014 at 11:08:35AM +, Martin Townsend wrote: > Hi Alex, > > No problems testing your new branch, just send me the URL, branch name and > can you also send me the commit hash as open embedded uses this. > What do you mean with commit hash? My current commit id is 6f

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

2014-02-06 Thread Martin Townsend
Hi Alex, No problems testing your new branch, just send me the URL, branch name and can you also send me the commit hash as open embedded uses this. One thing I've noticed with this problem I'm seeing is that I can make it happen a lot quicker by making a SSH connection to one of the boards ov

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

2014-02-05 Thread Alexander Aring
Hi Martin, On Wed, Feb 05, 2014 at 10:08:03AM +, Martin Townsend wrote: > Hi Alex, > > I have been testing your 6lowpan_pending branch with the fragmentation > updates (commit 5e47d2f516a6cf1acf9927125392484835d93552 - I don't think > this has the spin_lock patch below) using 2 devices ping

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

2014-02-05 Thread Martin Townsend
Hi Alex, I have been testing your 6lowpan_pending branch with the fragmentation updates (commit 5e47d2f516a6cf1acf9927125392484835d93552 - I don't think this has the spin_lock patch below) using 2 devices pinging each other, one with 1000 byte ping and the other with a 777 byte ping and the goo

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

2014-02-04 Thread Alexander Aring
Hi David, thanks for your reply. On Tue, Feb 04, 2014 at 08:32:03PM -0800, David Miller wrote: > From: Alexander Aring > Date: Tue, 4 Feb 2014 11:57:53 +0100 > > > @@ -197,7 +197,9 @@ static void lowpan_fragment_timer_expired(unsigned long > > entry_addr) > > > > pr_debug("timer expired

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

2014-02-04 Thread David Miller
From: Alexander Aring Date: Tue, 4 Feb 2014 11:57:53 +0100 > @@ -197,7 +197,9 @@ static void lowpan_fragment_timer_expired(unsigned long > entry_addr) > > pr_debug("timer expired for frame with tag %d\n", entry->tag); > > + spin_lock_bh(&flist_lock); > list_del(&entry->list)