Hi Alex, I've completely forgotten what the original problem was it was so long ago :) , let me dig through the old emails and see if I can jog my memory. My intention was to do the kfree_skb patch in my next submission but I'll see what I can do. - Martin. On 20/10/14 09:55, Alexander Aring wrote: > Hi Martin, > > On Mon, Oct 20, 2014 at 09:34:21AM +0100, Martin Townsend wrote: >> Hi Alex, >> >> On 20/10/14 08:25, Alexander Aring wrote: >>> Hi Martin, >>> >>> I did a fast quick test and it's still working. I see now the conversion >>> from to errno, but you didn't change/fix the behaviour with kfree_skb. I >>> am pretty sure this is still broken with and without your patch. >>> >>> I know you want to make smaller patches but can you please add the >>> conversion simple to this patch series? What I see now is the change >>> from if (ret != NET_RX_FOO) to errno. This change the behaviour of >>> current somewhat working state. What I mean, it's okay to change that >>> but please in a serie which contains also fixes for the kfree_skb thing. >> Is this the kfree_skb in lowpan_give_skb_to_devices? I'll add it in. > mhh no, I thought you main goal was to move out the error handling of > lowpan_process_header function (don't know the new name, maybe it is > iphc_decompression). > > - Alex -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
