On Wed, Jul 11, 2018 at 11:06 PM, Jesper Dangaard Brouer <bro...@redhat.com> wrote: > On Wed, 11 Jul 2018 19:05:20 +0000 > Saeed Mahameed <sae...@mellanox.com> wrote: > >> On Wed, 2018-07-11 at 17:01 +0200, Jesper Dangaard Brouer wrote: >> > Only driver sfc actually uses this, but I don't have this NIC, so I >> > tested this on mlx5, with my own changes to make it use >> > netif_receive_skb_list(), >> > but I'm not ready to upstream the mlx5 driver change yet. >> >> >> Thanks Jesper for sharing this, should we look forward to those patches >> or do you want us to implement them ? > > Well, I would prefer you to implement those. I just did a quick > implementation (its trivially easy) so I have something to benchmark > with. The performance boost is quite impressive! > > One reason I didn't "just" send a patch, is that Edward so-fare only > implemented netif_receive_skb_list() and not napi_gro_receive_list(). > And your driver uses napi_gro_receive(). This sort-of disables GRO for > your driver, which is not a choice I can make. Interestingly I get
But we can apply such patch on the mlx5 code path which comes into play when GRO is disabled, right? or there's just one code path there. Can you send your patch as RFC to the list for the purpose of experimentation? > around the same netperf TCP_STREAM performance. I assume we can get > even better perf if we "listify" napi_gro_receive.
