Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-07-11 Thread Vlastimil Babka
On 07/11/2014 10:28 AM, Vlastimil Babka wrote: On 06/23/2014 03:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-07-11 Thread Vlastimil Babka
On 06/23/2014 03:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct async compactions for

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-07-11 Thread Vlastimil Babka
On 06/23/2014 03:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct async compactions for

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-07-11 Thread Vlastimil Babka
On 07/11/2014 10:28 AM, Vlastimil Babka wrote: On 06/23/2014 03:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Zhang Yanfei
Hello Minchan Thank you for your explain. Actually, I read the kernel with an old version. The latest upstream kernel has the behaviour like you described below. Oops, how long didn't I follow the buddy allocator change. Thanks. On 06/24/2014 07:35 AM, Minchan Kim wrote: >>> Anyway, most big

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Minchan Kim
Hello Zhang, On Mon, Jun 23, 2014 at 04:55:55PM +0800, Zhang Yanfei wrote: > Hello Minchan, > > On 06/23/2014 09:39 AM, Minchan Kim wrote: > > Hello Vlastimil, > > > > On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: > >> Async compaction aborts when it detects zone lock

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Zhang Yanfei
Hello Minchan, On 06/23/2014 09:39 AM, Minchan Kim wrote: > Hello Vlastimil, > > On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: >> Async compaction aborts when it detects zone lock contention or >> need_resched() >> is true. David Rientjes has reported that in practice, most

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Minchan Kim
Hello Zhang, On Mon, Jun 23, 2014 at 04:55:55PM +0800, Zhang Yanfei wrote: Hello Minchan, On 06/23/2014 09:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Zhang Yanfei
Hello Minchan Thank you for your explain. Actually, I read the kernel with an old version. The latest upstream kernel has the behaviour like you described below. Oops, how long didn't I follow the buddy allocator change. Thanks. On 06/24/2014 07:35 AM, Minchan Kim wrote: Anyway, most big

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-23 Thread Zhang Yanfei
Hello Minchan, On 06/23/2014 09:39 AM, Minchan Kim wrote: Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-22 Thread Minchan Kim
Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: > Async compaction aborts when it detects zone lock contention or need_resched() > is true. David Rientjes has reported that in practice, most direct async > compactions for THP allocation abort due to

Re: [PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-22 Thread Minchan Kim
Hello Vlastimil, On Fri, Jun 20, 2014 at 05:49:35PM +0200, Vlastimil Babka wrote: Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct async compactions for THP allocation abort due to need_resched().

[PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-20 Thread Vlastimil Babka
Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct async compactions for THP allocation abort due to need_resched(). This means that a second direct compaction is never attempted, which might be OK for a

[PATCH v3 05/13] mm, compaction: report compaction as contended only due to lock contention

2014-06-20 Thread Vlastimil Babka
Async compaction aborts when it detects zone lock contention or need_resched() is true. David Rientjes has reported that in practice, most direct async compactions for THP allocation abort due to need_resched(). This means that a second direct compaction is never attempted, which might be OK for a