Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1236410

Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0 allocations

From Vlastimil Babka <vbabka@suse.cz>
Newsgroups linux.kernel
Subject Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0 allocations
Date 2015-09-30 16:50 +0200
Message-ID <qeqRH-7Z1-13@gated-at.bofh.it> (permalink)
References (2 earlier) <q6mrU-7dD-19@gated-at.bofh.it> <q6N8L-3wZ-5@gated-at.bofh.it> <qelp0-8sJ-7@gated-at.bofh.it> <qeq5l-6MN-43@gated-at.bofh.it> <qeqoG-7qA-27@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 09/30/2015 04:16 PM, Vitaly Wool wrote:
>>>>>
>>>>
>>>> So what do you suggest instead? A fixed number, some other heuristic?
>>>> You have pushed several times now for the series to focus on the latency
>>>> of standard high-order allocations but again I will say that it is
>>>> outside
>>>> the scope of this series. If you want to take steps to reduce the latency
>>>> of ordinary high-order allocation requests that can sleep then it should
>>>> be a separate series.
>>>
>>>
>>> I do believe https://lkml.org/lkml/2015/9/9/313 does a better job
>>
>>
>> Does a better job regarding what exactly? It does fix the CMA-specific
>> issue, but so does this patch - without affecting allocation fastpaths by
>> making them update another counter. But the issues discussed here are not
>> related to that CMA problem.
>
> Let me disagree. Guaranteeing one suitable high-order page is not
> enough, so the suggested patch does not work that well for me.
> Existing broken watermark calculation doesn't work for me either, as
> opposed to the one with my patch applied. Both solutions are related
> to the CMA issue but one does make compaction work harder and cause
> bigger latencies -- why do you think these are not related?

Well you didn't mention which issues you have with this patch. If you 
did measure bigger latencies and more compaction work, please post the 
numbers and details about the test.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0 allocations Vitaly Wool <vitalywool@gmail.com> - 2015-09-30 11:00 +0200
  Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0  allocations Vlastimil Babka <vbabka@suse.cz> - 2015-09-30 16:00 +0200
    Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0 allocations Vitaly Wool <vitalywool@gmail.com> - 2015-09-30 16:20 +0200
      Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for order-0  allocations Vlastimil Babka <vbabka@suse.cz> - 2015-09-30 16:50 +0200
        Re: [PATCH 12/12] mm, page_alloc: Only enforce watermarks for  order-0 allocations Mel Gorman <mgorman@techsingularity.net> - 2015-09-30 17:20 +0200

csiph-web