On Mon, 2015-07-13 at 15:23 +0800, Herbert Xu wrote: > The real issue here is that the recv path no longer handles shared > skbs. So either we need to fix the recv path to not touch skbs > without cloning them, or we need to get rid of the use of shared > skbs in netlink. > > In fact it looks I introduced the bug way back in > > commit a59322be07c964e916d15be3df473fb7ba20c41e > Author: Herbert Xu <herb...@gondor.apana.org.au> > Date: Wed Dec 5 01:53:40 2007 -0800 > > [UDP]: Only increment counter on first peek/recv > > I will try to mend this error :) > > Cheers,
Herbert, UDP peek support is very buggy anyway, because of deferred checksums __skb_checksum_complete() will happily manipulate csum, ip_summed, csum_complete_sw & csum_valid Ideally, peek should never touch skb (but skb->users) -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html