> -----Original Message----- > From: Johannes Weiner <hannes@xxxxxxxxxxx> > Sent: Saturday, September 28, 2024 6:54 AM > To: Sridhar, Kanchana P <kanchana.p.sridhar@xxxxxxxxx> > Cc: linux-kernel@xxxxxxxxxxxxxxx; linux-mm@xxxxxxxxx; > yosryahmed@xxxxxxxxxx; nphamcs@xxxxxxxxx; > chengming.zhou@xxxxxxxxx; usamaarif642@xxxxxxxxx; > shakeel.butt@xxxxxxxxx; ryan.roberts@xxxxxxx; Huang, Ying > <ying.huang@xxxxxxxxx>; 21cnbao@xxxxxxxxx; akpm@xxxxxxxxxxxxxxxxxxxx; > Zou, Nanhai <nanhai.zou@xxxxxxxxx>; Feghali, Wajdi K > <wajdi.k.feghali@xxxxxxxxx>; Gopal, Vinodh <vinodh.gopal@xxxxxxxxx> > Subject: Re: [PATCH v8 5/8] mm: zswap: Modify zswap_stored_pages to be > atomic_long_t. > > On Fri, Sep 27, 2024 at 07:16:17PM -0700, Kanchana P Sridhar wrote: > > For zswap_store() to support large folios, we need to be able to do > > a batch update of zswap_stored_pages upon successful store of all pages > > in the folio. For this, we need to add folio_nr_pages(), which returns > > a long, to zswap_stored_pages. > > > > Signed-off-by: Kanchana P Sridhar <kanchana.p.sridhar@xxxxxxxxx> > > Long for pages makes sense to me even independent of the large folios > coming in. An int is just 8TB in 4k (base) pages. > > Acked-by: Johannes Weiner <hannes@xxxxxxxxxxx> Thanks Johannes for the Acked-by's! Thanks, Kanchana
