Re: [PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-13 Thread Michal Hocko
On Tue 11-12-18 14:53:12, Oscar Salvador wrote: > v1 -> v2: > - Keep branch to decrease refcount and print out > the failed pfn/page > - Modified changelog per Michal's feedback > - move put_page() out of the if/else branch > > --- > >From

Re: [PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-12 Thread Wei Yang
On Tue, Dec 11, 2018 at 02:53:12PM +0100, Oscar Salvador wrote: >v1 -> v2: >- Keep branch to decrease refcount and print out > the failed pfn/page >- Modified changelog per Michal's feedback >- move put_page() out of the if/else branch > >--- >>From

Re: [PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-11 Thread Oscar Salvador
On Wed, 2018-12-12 at 03:35 +, Wei Yang wrote: > I see the above code is wrapped with CONFIG_DEBUG_VM on current Linus > tree. > This is removed by someone else? Yes, e8abbd69957288 ("mm, memory_hotplug: be more verbose for memory offline failures") got rid of the CONFIG_DEBUG_VM. This commit

Re: [PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-11 Thread Wei Yang
On Tue, Dec 11, 2018 at 02:53:12PM +0100, Oscar Salvador wrote: >v1 -> v2: >- Keep branch to decrease refcount and print out > the failed pfn/page >- Modified changelog per Michal's feedback >- move put_page() out of the if/else branch > >--- >>From

Re: [PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-11 Thread Wei Yang
On Tue, Dec 11, 2018 at 02:53:12PM +0100, Oscar Salvador wrote: >v1 -> v2: >- Keep branch to decrease refcount and print out > the failed pfn/page >- Modified changelog per Michal's feedback >- move put_page() out of the if/else branch > >--- >>From

[PATCH v2] mm, memory_hotplug: Don't bail out in do_migrate_range prematurely

2018-12-11 Thread Oscar Salvador
v1 -> v2: - Keep branch to decrease refcount and print out the failed pfn/page - Modified changelog per Michal's feedback - move put_page() out of the if/else branch --- >From f81da873be9a5b7845249d1e62a423f054c487d5 Mon Sep 17 00:00:00 2001 From: Oscar Salvador