Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-06 Thread Andrew Morton
On Mon, 5 Mar 2018 16:57:40 -0800 Mike Kravetz wrote: > >>> > >>> I could not immediately come up with a good idea here. The zone lock > >>> would be the obvious choice, but I don't think we want to hold it while > >>> examining each of the page blocks. Perhaps a new lock or semaphore > >>> ass

Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-05 Thread Mike Kravetz
On 03/02/2018 05:39 PM, Mike Kravetz wrote: > On 03/02/2018 04:56 PM, Andrew Morton wrote: >> On Fri, 2 Mar 2018 16:38:33 -0800 Mike Kravetz >> wrote: >> >>> On 03/02/2018 04:06 PM, Andrew Morton wrote: On Mon, 26 Feb 2018 11:10:54 -0800 Mike Kravetz wrote: > start_isolate_pa

Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-02 Thread Mike Kravetz
On 03/02/2018 04:56 PM, Andrew Morton wrote: > On Fri, 2 Mar 2018 16:38:33 -0800 Mike Kravetz > wrote: > >> On 03/02/2018 04:06 PM, Andrew Morton wrote: >>> On Mon, 26 Feb 2018 11:10:54 -0800 Mike Kravetz >>> wrote: >>> start_isolate_page_range() is used to set the migrate type of a

Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-02 Thread Andrew Morton
On Fri, 2 Mar 2018 16:38:33 -0800 Mike Kravetz wrote: > On 03/02/2018 04:06 PM, Andrew Morton wrote: > > On Mon, 26 Feb 2018 11:10:54 -0800 Mike Kravetz > > wrote: > > > >> start_isolate_page_range() is used to set the migrate type of a > >> set of page blocks to MIGRATE_ISOLATE while attempti

Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-02 Thread Mike Kravetz
On 03/02/2018 04:06 PM, Andrew Morton wrote: > On Mon, 26 Feb 2018 11:10:54 -0800 Mike Kravetz > wrote: > >> start_isolate_page_range() is used to set the migrate type of a >> set of page blocks to MIGRATE_ISOLATE while attempting to start >> a migration operation. It assumes that only one thre

Re: [PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-03-02 Thread Andrew Morton
On Mon, 26 Feb 2018 11:10:54 -0800 Mike Kravetz wrote: > start_isolate_page_range() is used to set the migrate type of a > set of page blocks to MIGRATE_ISOLATE while attempting to start > a migration operation. It assumes that only one thread is > calling it for the specified range. This routi

[PATCH 1/1] mm: make start_isolate_page_range() fail if already isolated

2018-02-26 Thread Mike Kravetz
start_isolate_page_range() is used to set the migrate type of a set of page blocks to MIGRATE_ISOLATE while attempting to start a migration operation. It assumes that only one thread is calling it for the specified range. This routine is used by CMA, memory hotplug and gigantic huge pages. Each