On Wed, Aug 24, 2022 at 12:10:11PM +0200, Michal Hocko wrote: > On Wed 24-08-22 10:49:01, Mel Gorman wrote: > > +static unsigned int zonelist_iter_begin(void) > > +{ > > + if (IS_ENABLED(CONFIG_MEMORY_HOTREMOVE)) > > + return 0; > > + > > + return read_seqbegin(&zonelist_update_seq); > > +} > > + > > +static unsigned int check_retry_zonelist(unsigned int seq) > > +{ > > + if (IS_ENABLED(CONFIG_MEMORY_HOTREMOVE)) > > + return seq; > > + > > + return read_seqretry(&zonelist_update_seq, seq); > > +} > > these should be negated Yey, last minute changes that are rushed. V2 shortly. -- Mel Gorman SUSE Labs