> > return pp; > > } > > What if 'pp' is NULL? > > Aside from that, this replace a lookup with 2 atomic ops, and only when > such lookup is amortized on multiple aggregated packets: I'm unsure if > it's worthy and I don't understand how that improves RR tests (where > the socket can't see multiple, consecutive skbs, AFAIK). > > Cheers, > > Paolo >
If we not release the socket in udp_gro_complete , we can reduce a udp socket Lookup when do ip demux again, it maybe more worthy. I test UDP_STREAM, find no difference, both can reach NIC's limit, 10G; so Test RR, I will do more tests -RongQing
