netdev - Re: [PATCH net-next v7 1/7] netmem: introduce struct netmem

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <aHUMwHft71cB8PFY@hyeyoo>
Date: Mon, 14 Jul 2025 22:58:31 +0900
From: Harry Yoo <harry.yoo@...cle.com>
To: Vlastimil Babka <vbabka@...e.cz>
Cc: Jakub Kicinski <kuba@...nel.org>, Byungchul Park <byungchul@...com>,
        willy@...radead.org, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org, linux-mm@...ck.org,
        kernel_team@...ynix.com, almasrymina@...gle.com,
        ilias.apalodimas@...aro.org, hawk@...nel.org,
        akpm@...ux-foundation.org, davem@...emloft.net,
        john.fastabend@...il.com, andrew+netdev@...n.ch,
        asml.silence@...il.com, toke@...hat.com, tariqt@...dia.com,
        edumazet@...gle.com, pabeni@...hat.com, saeedm@...dia.com,
        leon@...nel.org, ast@...nel.org, daniel@...earbox.net,
        david@...hat.com, lorenzo.stoakes@...cle.com, Liam.Howlett@...cle.com,
        rppt@...nel.org, surenb@...gle.com, mhocko@...e.com, horms@...nel.org,
        linux-rdma@...r.kernel.org, bpf@...r.kernel.org,
        vishal.moola@...il.com, hannes@...xchg.org, ziy@...dia.com,
        jackmanb@...gle.com
Subject: Re: [PATCH net-next v7 1/7] netmem: introduce struct netmem_desc
 mirroring struct page

On Mon, Jul 14, 2025 at 03:24:02PM +0200, Vlastimil Babka wrote:
> On 7/14/25 11:42, Harry Yoo wrote:
> > On Tue, Jul 01, 2025 at 04:45:08PM -0700, Jakub Kicinski wrote:
> >> Thanks a lot, this clarifies things for me.
> > 
> > You're welcome :)
> > 
> >> Unfortunately, I still think that it's hard to judge patches 1 and 7 
> >> in context limited to this series, so let's proceed to reposting just
> >> the "middle 5" patches.
> > 
> > Could you please share your thoughts on why it's hard to judge them and
> > what's missing from the series, such as in the comments, changelog, or
> > the cover letter?
> 
> I think we moved on in the discussion since then? "middle 5" patches are now
> merged, and 1+7 was, along with more patches (that make the context less
> limited hopefully), posted as v9/v10 now?

On v7 Jakub asked [1] [2] few questions about the patch series
and I answered [3].

[1] https://lore.kernel.org/linux-mm/20250626174904.4a6125c9@kernel.org
[2] https://lore.kernel.org/linux-mm/20250627173730.15b25a8c@kernel.org 
[3] https://lore.kernel.org/linux-mm/aGHNmKRng9H6kTqz@hyeyoo

To me Jakub's comment "Thanks a lot, this clarifies things for me"
and still "hard to judge patches 1 and 7 in context limited to this series"
after reading my clarification (which wasn't part of the series)
sounded like he wants some of the clarification to be added to
later versions of the patchset.

But I don't speak for Jakub, I'll leave the answer to him.
Maybe I just misinterpreted his reply.

-- 
Cheers,
Harry / Hyeonggon