Re: [PATCH] mm: attempt to batch free swap entries for zap_pte_range()

[Andrew Morton]

On Wed, 7 Aug 2024 04:44:44 +0800 Barry Song <21cnbao@xxxxxxxxx> wrote:

> > > +static bool try_batch_swap_entries_free(struct swap_info_struct *p,
> >
> > Why call it "p" here and not "si" like in the other code you are touching?
> 
> that is because I found other _free_ functions are all using "p":

`p' sucks.  "pointer to something".  It's just lazy.  In this context, "si"
has meaning; lots of it.