On 12/1/21 15:30, Liam Howlett wrote: > From: "Liam R. Howlett" <Liam.Howlett@xxxxxxxxxx> > > Signed-off-by: Liam R. Howlett <Liam.Howlett@xxxxxxxxxx> > --- > mm/memcontrol.c | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/mm/memcontrol.c b/mm/memcontrol.c > index 781605e92015..ac95b3eca557 100644 > --- a/mm/memcontrol.c > +++ b/mm/memcontrol.c > @@ -5808,7 +5808,7 @@ static unsigned long mem_cgroup_count_precharge(struct mm_struct *mm) > unsigned long precharge; > > mmap_read_lock(mm); > - walk_page_range(mm, 0, mm->highest_vm_end, &precharge_walk_ops, NULL); > + walk_page_range(mm, 0, -1, &precharge_walk_ops, NULL); Wouldn't ULONG_MAX look better? > mmap_read_unlock(mm); > > precharge = mc.precharge; > @@ -6106,9 +6106,7 @@ static void mem_cgroup_move_charge(void) > * When we have consumed all precharges and failed in doing > * additional charge, the page walk just aborts. > */ > - walk_page_range(mc.mm, 0, mc.mm->highest_vm_end, &charge_walk_ops, > - NULL); > - > + walk_page_range(mc.mm, 0, -1, &charge_walk_ops, NULL); > mmap_read_unlock(mc.mm); > atomic_dec(&mc.from->moving_account); > }
