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 <osalvador@xxxxxxx> > Acked-by: Vlastimil Babka <vbabka@xxxxxxx> > Reviewed-by: David Hildenbrand <david@xxxxxxxxxx> > Acked-by: Michal Hocko <mhocko@xxxxxxxx> Acked-by: Mike Kravetz <mike.kravetz@xxxxxxxxxx> -- Mike Kravetz
