On Fri, 6 Dec 2024 14:49:18 +0100 Alexander Lobakin wrote: > > nit: netmem_pp is not a great name. Ain't nothing especially netmem > > about it, it's just the _current_ page pool. > > It's the page_pool of the @netmem we're processing on this iteration. > "This netmem's PP" => netmem_pp. > Current page_pool which we'll use for recycling is @pool. Heh, yes, I guess there are levels to current..ness :) Maybe instead of current the one we're servicing could be called tgt_pp and the one from iterator just pp? No big deal either way, tho, this is very nit picky and subjective...