On 5/28/25 02:21, Byungchul Park wrote:
So.. For now, this is the best option we can pick. We can do all that
you told me once struct netmem_desc has it own instance from slab.
Again, it's because the page pool fields (or netmem things) from struct
page will be gone by this series.
Mina, thoughts?
Can you please post an updated series with the approach you have in
mind? I think this series as-is seems broken vis-a-vie the
_pp_padding_map param move that looks incorrect. Pavel and I have also
commented on patch 18 that removing the ASSERTS seems incorrect as
it's breaking the symmetry between struct page and struct net_iov.
I told you I will fix it. I will send the updated series shortly for
*review*. However, it will be for review since we know this work can be
completed once the next works have been done:
Please don't forget to tag it with "RFC", otherwise nobody will
assume it's for for review only.
--
Pavel Begunkov
