Hi,

An update on the lock up problem. In my driver I've ensured that the rx processes everything in the interrupt before calling ieee802154_rx_irqsafe. Then in the tx I don't think you need any synchronisation except between my interrupt and the xmit routine but I've put some in there just to be sure and I'm still getting lockups.

So I tried ping from iputils and not the one from busy box and this time I get a load of

|ping: sendmsg: No buffer space available|


after it locks up.

Does anyone know what this means? Any suggestions as to anything else I can try?

Regards,
Martin.

On 13/02/14 09:35, Martin Townsend wrote:
Hi Alex,

I think there is a missing kfree_skb in lowpan_frag_recv, Here's a
patch, can you verify if this is the case.

Sadly I'm still seeing lock ups but as your setup is ok I'll explore
other avenues with my setup. One thing I have noticed is that the lock
ups are different now in that before some pings would get through every
now and again and if I stopped the ping and restarted with a small
payload size they would work.  Now it locks up for good which really
does make me suspect my driver, or maybe I'm missing some important
Xilinx kernel patches.

Cheers,
Martin.

  From 69c82ea3a166f09332d05db91d95f3423f7a9d06 Mon Sep 17 00:00:00 2001
From: Martin Townsend<martin.towns...@xsilon.com>
Date: Tue, 11 Feb 2014 16:33:23 +0000
Subject: [PATCH] fixed lowpan frag recv skb leak

---
   net/ieee802154/reassembly.c | 2 +-
   1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/net/ieee802154/reassembly.c b/net/ieee802154/reassembly.c
index 2cf6d99..1d5e6d8 100644
--- a/net/ieee802154/reassembly.c
+++ b/net/ieee802154/reassembly.c
@@ -377,8 +377,8 @@ int lowpan_frag_rcv(struct sk_buff *skb, const u8
frag_type)
           return ret;
       }

-    kfree_skb(skb);
   err:
+    kfree_skb(skb);
       return -1;
   }
   EXPORT_SYMBOL(lowpan_frag_rcv);

------------------------------------------------------------------------------
Android apps run on BlackBerry 10
Introducing the new BlackBerry 10.2.1 Runtime for Android apps.
Now with support for Jelly Bean, Bluetooth, Mapview and more.
Get your Android app in front of a whole new audience.  Start now.
http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk
_______________________________________________
Linux-zigbee-devel mailing list
Linux-zigbee-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-zigbee-devel

Reply via email to