Re: [PATCH 3/3] mm: zpdesc: minor naming and comment corrections

[Yosry Ahmed <yosry.ahmed@linux.dev>]

On Tue, Sep 09, 2025 at 04:11:13PM +0100, Johannes Weiner wrote:
> On Fri, Sep 05, 2025 at 07:05:22PM +0000, Yosry Ahmed wrote:
> > On Fri, Aug 29, 2025 at 05:15:28PM +0100, Johannes Weiner wrote:
> > > zpdesc is the page descriptor used by the zsmalloc backend allocator,
> > > which in turn is used by zswap and zram. The zpool layer is gone.
> > > 
> > > Signed-off-by: Johannes Weiner <hannes@cmpxchg.org>
> > > ---
> > 
> > Why not rename zpdesc to zsdesc or zdesc while we're at it? It's a lot
> > of noise but it's just this file and zsmalloc.c IIUC. Up to you.
> 
> I'm kind of fine with zpdesc and not sure the other names are this
> much better to warrant the churn. So if you don't mind, I'd just leave
> it alone for now.

I undestand there's churn involved, I am fine either way.

> 
> > If it's just the comment changes I would squash it into the second
> > patch, but not a big deal. Either way:
> > 
> > Acked-by: Yosry Ahmed <yosry.ahmed@linux.dev>
> 
> Thanks for your review!