<div dir="ltr">You're right; it should indeed be "net_hdr_word(&addr[fn_bit >> 5]);".  That's my fault for not checking the patches thoroughly enough.  I'm more than a bit surprised the compiler accepted that code as-is...<br>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On 21 January 2013 13:50, Ketan Kulkarni <span dir="ltr"><<a href="mailto:ketkulka@gmail.com" target="_blank">ketkulka@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Looking at the patch -<br>
<br>
-              addr[fn_bit >> 5];<br>
+              net_hdr_word(addr[fn_bit >> 5]);<br>
<br>
Should the above line be - (& missing in the above line??)<br>
<br>
+              net_hdr_word(&addr[fn_bit >> 5]);<br>
<br>
-Ketan<br>
<div class="HOEnZb"><div class="h5"><br>
On Mon, Jan 21, 2013 at 6:51 PM, Ketan Kulkarni <<a href="mailto:ketkulka@gmail.com">ketkulka@gmail.com</a>> wrote:<br>
> Well with this 3.7.3-1, I didnt see any crashes. I didnt change any<br>
> default config.<br>
> The router boots up well (in fact writing this mail through the same)<br>
> I also note I use IPv4 network only.<br>
><br>
> I collected dmesg after booting up and kept here (in case required) -<br>
> <a href="http://pastebin.com/p1aM1aV8" target="_blank">http://pastebin.com/p1aM1aV8</a><br>
><br>
> -Ketan<br>
><br>
><br>
> On Mon, Jan 21, 2013 at 2:31 PM, Dave Taht <<a href="mailto:dave.taht@gmail.com">dave.taht@gmail.com</a>> wrote:<br>
>> A build without that patch is now up here:<br>
>><br>
>> <a href="http://snapon.lab.bufferbloat.net/~cero2/cerowrt/wndr/3.7.3-1/" target="_blank">http://snapon.lab.bufferbloat.net/~cero2/cerowrt/wndr/3.7.3-1/</a><br>
>><br>
>> so 1/3 of the potential causes is not there....<br>
>><br>
>> On Mon, Jan 21, 2013 at 3:51 AM, Ketan Kulkarni <<a href="mailto:ketkulka@gmail.com">ketkulka@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi Dave,<br>
>>> Probably I can take a look later tonight by having serial cable - I<br>
>>> have that one handy.<br>
>>> What is the build and how to re-create the scenario?<br>
>>><br>
>><br>
>> Well, I hate to commit sources when stuff is obviously borked. If you<br>
>> want to try flashing with that you can figure out if it was us or the merge<br>
>> that went bad....<br>
>><br>
>><br>
>><br>
>><br>
>>><br>
>>> Thanks,<br>
>>> Ketan<br>
>>><br>
>>><br>
>>> On Mon, Jan 21, 2013 at 12:54 AM, Dave Taht <<a href="mailto:dave.taht@gmail.com">dave.taht@gmail.com</a>> wrote:<br>
>>> > Mine and Robert Bradley's latest attempt at killing the last ipv6<br>
>>> > instruction traps<br>
>>> > with a rollup patch from bugs 419 and 421...<br>
>>> ><br>
>>> > brick the router. It boots, the main green light goes on, then it dies.<br>
>>> > Naturally<br>
>>> > I left the serial cable at the office...<br>
>>> ><br>
>>> > Of course in the middle of this patch set revision I also updated to<br>
>>> > 3.7.3<br>
>>> > and openwrt head,<br>
>>> > so have a few more variables than usual than just us to cope with.<br>
>>> ><br>
>>> > I would not mind eyeballs on this to see if there is an obvious flaw,<br>
>>> > otherwise I will progressively bisect from what was known to work<br>
>>> > forward as fast as I can. (besides, the hope is to  submit this<br>
>>> > upstream)<br>
>>> ><br>
>>> > net_hdr_word is defined in the 902-unaligned_access_hacks patch in<br>
>>> > openwrt<br>
>>> > head...<br>
>>> ><br>
>>> > now off to find a small, sharp, pointy object...<br>
>>> ><br>
>>> > --<br>
>>> > Dave Täht<br>
>>> ><br>
>>> > Fixing bufferbloat with cerowrt:<br>
>>> > <a href="http://www.teklibre.com/cerowrt/subscribe.html" target="_blank">http://www.teklibre.com/cerowrt/subscribe.html</a><br>
>>> ><br>
>>> > _______________________________________________<br>
>>> > Cerowrt-devel mailing list<br>
>>> > <a href="mailto:Cerowrt-devel@lists.bufferbloat.net">Cerowrt-devel@lists.bufferbloat.net</a><br>
>>> > <a href="https://lists.bufferbloat.net/listinfo/cerowrt-devel" target="_blank">https://lists.bufferbloat.net/listinfo/cerowrt-devel</a><br>
>>> ><br>
>><br>
>><br>
>><br>
>><br>
>> --<br>
>> Dave Täht<br>
>><br>
>> Fixing bufferbloat with cerowrt:<br>
>> <a href="http://www.teklibre.com/cerowrt/subscribe.html" target="_blank">http://www.teklibre.com/cerowrt/subscribe.html</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Robert Bradley
</div>