On 4/13/21 3:47 AM, Oscar Salvador wrote: > Currently, __alloc_contig_migrate_range can generate -EINTR, -ENOMEM or > -EBUSY, > and report them down the chain. > The problem is that when migrate_pages() reports -ENOMEM, we keep going till > we > exhaust all the try-attempts (5 at the moment) instead of bailing out. > > migrate_pages() bails out right away on -ENOMEM because it is considered a > fatal > error. Do the same here instead of keep going and retrying. > Note that this is not fixing a real issue, just a cosmetic change. Although we > can save some cycles by backing off ealier > > Signed-off-by: Oscar Salvador <osalva...@suse.de> > Acked-by: Vlastimil Babka <vba...@suse.cz> > Reviewed-by: David Hildenbrand <da...@redhat.com> > Acked-by: Michal Hocko <mho...@suse.com>
Acked-by: Mike Kravetz <mike.krav...@oracle.com> -- Mike Kravetz