Re: [External] [PATCH v2] mm/list_lru: optimize condition of exiting the loop

2020-11-02 Thread Muchun Song
On Wed, Oct 28, 2020 at 10:17 PM Hui Su wrote: > > In list_lru_walk(), nr_to_walk type is 'unsigned long', > so nr_to_walk won't be '< 0'. > > In list_lru_walk_node(), nr_to_walk type is 'unsigned long', > so *nr_to_walk won't be '< 0' too. > > We can use '!nr_to_walk' instead of 'nr_to_walk <= 0'

Re: [PATCH v2] mm/list_lru: optimize condition of exiting the loop

2020-11-02 Thread Pankaj Gupta
> In list_lru_walk(), nr_to_walk type is 'unsigned long', > so nr_to_walk won't be '< 0'. > > In list_lru_walk_node(), nr_to_walk type is 'unsigned long', > so *nr_to_walk won't be '< 0' too. > > We can use '!nr_to_walk' instead of 'nr_to_walk <= 0', which > is more precise. > > Signed-off-by: Hui

Re: [PATCH v2] mm/list_lru: optimize condition of exiting the loop

2020-11-02 Thread Vlastimil Babka
On 10/28/20 3:16 PM, Hui Su wrote: In list_lru_walk(), nr_to_walk type is 'unsigned long', so nr_to_walk won't be '< 0'. In list_lru_walk_node(), nr_to_walk type is 'unsigned long', so *nr_to_walk won't be '< 0' too. We can use '!nr_to_walk' instead of 'nr_to_walk <= 0', which is more precise.