Re: [PATCH 1/9] mm, swap: use unified helper for swap cache look up

[David Hildenbrand]

On 02.09.25 14:32, Chris Li wrote:
> On Tue, Sep 2, 2025 at 3:07 AM David Hildenbrand <david@xxxxxxxxxx> wrote:
> > > diff --git a/mm/swap_state.c b/mm/swap_state.c
> > > index 99513b74b5d8..ff9eb761a103 100644
> > > --- a/mm/swap_state.c
> > > +++ b/mm/swap_state.c
> > > @@ -69,6 +69,21 @@ void show_swap_cache_info(void)
> > >        printk("Total swap = %lukB\n", K(total_swap_pages));
> > >    }
> > >
> > > +/*
> >
> > While at it, proper kerneldoc?
>
> Agree, add the kerneldoc while we are at it. Those are important API
> to interact with the swap table.
>
> BTW, I already submitted a design doc for swap table to Kairui, which
> will show up as the first patch in the V2 series

Nice!

--
Cheers

David / dhildenb