On 08.11.24 18:33, SeongJae Park wrote:
+ David Hildenbrand
On Thu, 7 Nov 2024 15:56:12 -0800 Joanne Koong <joannelkoong@xxxxxxxxx> wrote:
In offline_pages(), do_migrate_range() may potentially retry forever if
the migration fails. Add a return value for do_migrate_range(), and
allow offline_page() to try migrating pages 5 times before erroring
out, similar to how migration failures in __alloc_contig_migrate_range()
is handled.
I'm curious if this could cause unexpected behavioral differences to memory
hotplugging users, and how '5' is chosen. Could you please enlighten me?
I'm wondering how much more often I'll have to nack such a patch. :)
On a related note: MAINTAINERS file exists for a reason.
--
Cheers,
David / dhildenb
