> > For clarity this can be written as
> > 
> >             nr_pages = min_t(unsigned long,
> >                             folio_nr_pages(folio), page_count - i);
> >             nr_pages = min_t(unsigned long, nr_pages, max_pages);
> 
> Do you think the min_array() is less clear?  Let's see what others say.

min_array() is clear, it's the ((unsigned long[]) { ... }) that
iis nice and fancy but of difficult first read. But, as I said,
it can stay, I don't have a strong opinion, maybe I'd have done
the same.

Andi

Reply via email to