Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-06 Thread Vlastimil Babka
On 06/03/2016 02:47 PM, Joonsoo Kim wrote: [...] @@ -128,8 +127,12 @@ static void unset_migratetype_isolate(struct page *page, unsigned migratetype) zone->nr_isolate_pageblock--; out: spin_unlock_irqrestore(>lock, flags); - if (isolated_page) + if (isolated_page) {

Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-06 Thread Vlastimil Babka
On 06/03/2016 02:47 PM, Joonsoo Kim wrote: [...] @@ -128,8 +127,12 @@ static void unset_migratetype_isolate(struct page *page, unsigned migratetype) zone->nr_isolate_pageblock--; out: spin_unlock_irqrestore(>lock, flags); - if (isolated_page) + if (isolated_page) {

Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-03 Thread Joonsoo Kim
2016-06-03 19:23 GMT+09:00 Vlastimil Babka : > On 05/26/2016 04:37 AM, js1...@gmail.com wrote: >> >> From: Joonsoo Kim >> >> It's not necessary to initialized page_owner with holding the zone lock. >> It would cause more contention on the zone lock although

Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-03 Thread Joonsoo Kim
2016-06-03 19:23 GMT+09:00 Vlastimil Babka : > On 05/26/2016 04:37 AM, js1...@gmail.com wrote: >> >> From: Joonsoo Kim >> >> It's not necessary to initialized page_owner with holding the zone lock. >> It would cause more contention on the zone lock although it's not >> a big problem since it is

Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-03 Thread Vlastimil Babka
On 05/26/2016 04:37 AM, js1...@gmail.com wrote: From: Joonsoo Kim It's not necessary to initialized page_owner with holding the zone lock. It would cause more contention on the zone lock although it's not a big problem since it is just debug feature. But, it is better

Re: [PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-06-03 Thread Vlastimil Babka
On 05/26/2016 04:37 AM, js1...@gmail.com wrote: From: Joonsoo Kim It's not necessary to initialized page_owner with holding the zone lock. It would cause more contention on the zone lock although it's not a big problem since it is just debug feature. But, it is better than before so do it.

[PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-05-25 Thread js1304
From: Joonsoo Kim It's not necessary to initialized page_owner with holding the zone lock. It would cause more contention on the zone lock although it's not a big problem since it is just debug feature. But, it is better than before so do it. This is also preparation step

[PATCH v2 2/7] mm/page_owner: initialize page owner without holding the zone lock

2016-05-25 Thread js1304
From: Joonsoo Kim It's not necessary to initialized page_owner with holding the zone lock. It would cause more contention on the zone lock although it's not a big problem since it is just debug feature. But, it is better than before so do it. This is also preparation step to use stackdepot in