Messages in this thread |  | | Date | Wed, 12 Feb 2025 15:22:55 +0100 | | Subject | Re: [PATCH net-next v4 0/8] bpf: cpumap: enable GRO for XDP_PASS frames | | From | Alexander Lobakin <> |
| |
From: Jakub Kicinski <kuba@kernel.org> Date: Mon, 10 Feb 2025 16:35:29 -0800
> On Mon, 10 Feb 2025 16:31:52 +0100 Alexander Lobakin wrote: >> This was rejected by Kuba in v2. >> He didn't like to have napi_id two times within napi_struct (one inside >> gro_node, one outside). > > Do you mean: > > the napi_id in gro sticks out.. > > https://lore.kernel.org/netdev/20250113130104.5c2c02e0@kernel.org/ ? > > That's more of a nudge to try harder than a "no". We explored > the alternatives, there's no perfect way to layer this. I think > Eric's suggestion is probably as clean as we can get.
You mean to cache napi_id in gro_node?
Then we get +8 bytes to sizeof(napi_struct) for little reason... Dunno, if you really prefer, I can do it that way.
OTOH you gave Acked-by on struct_group(), then Eric appeared and you took your ack back :D
Thanks, Olek
|  |