On 02/10/2012 01:30 AM, Francois Romieu wrote: > Ben Greear : > [...] >> I am hacking on the 8169 code, (adding RX-ALL and RX-FCS support on top >> of my previously posted patches) so this could easily be my fault.. >> but just in case someone else has seen it, please let me know.... > > I have never met it before. Neither does the web. > > Are you hacking against davem's -next branch ? Yes. I was testing the attached patch, and when I enabled the rx-fcs, it crashed shortly after. So, my changes must be bad somehow... Probably something to do with pkt_size being 4 bytes larger when rx-fcs is enabled (I was hoping that would grab the FCS, as similar logic seems to work fine in 8139too...) Thanks, Ben -- Ben Greear Candela Technologies Inc http://www.candelatech.com