On Fri, 10 Jan 2025 15:08:44 +0900 Hyeonggon Yoo <hyeonggon.yoo@xxxxxx> wrote: > >> [42.hyeyoo: keep reset_zpdesc() to use struct page fields] > > > > Ummm ... why did you make this change? Now page->index still has a user > > in zsmalloc ;-( > > Ohhh.., my bad :/ > > I was thinking of a situation when someone re-arranges zpdesc fields > and forgets to reset the field that overlaps page->index now. > And when writing it did not consider that we're going to remove usages > of page->index anyway... > > Let me fix this - will send an updated version of this patch or series > (whatever way that is convenient for You and Andrew) A little fixup patch would be preferred at this stage, please. Or a new version of this patch, from which I'll create a little fixup patch.