Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Peter Hurley
On 11/04/2014 08:35 AM, Michal Nazarewicz wrote: > On Tue, Nov 04 2014, Peter Hurley wrote: >> On 11/04/2014 12:43 AM, Minchan Kim wrote: >>> Hello, >>> >>> On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: Having test_pages_isolated failure message as a warning confuses

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Michal Nazarewicz
On Tue, Nov 04 2014, Peter Hurley wrote: > On 11/04/2014 12:43 AM, Minchan Kim wrote: >> Hello, >> >> On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: >>> Having test_pages_isolated failure message as a warning confuses >>> users into thinking that it is more serious than it

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Peter Hurley
On 11/04/2014 12:43 AM, Minchan Kim wrote: > Hello, > > On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: >> Having test_pages_isolated failure message as a warning confuses >> users into thinking that it is more serious than it really is. In >> reality, if called via CMA,

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Peter Hurley
On 11/04/2014 12:43 AM, Minchan Kim wrote: Hello, On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: Having test_pages_isolated failure message as a warning confuses users into thinking that it is more serious than it really is. In reality, if called via CMA, allocation

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Michal Nazarewicz
On Tue, Nov 04 2014, Peter Hurley pe...@hurleysoftware.com wrote: On 11/04/2014 12:43 AM, Minchan Kim wrote: Hello, On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: Having test_pages_isolated failure message as a warning confuses users into thinking that it is more serious

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-04 Thread Peter Hurley
On 11/04/2014 08:35 AM, Michal Nazarewicz wrote: On Tue, Nov 04 2014, Peter Hurley pe...@hurleysoftware.com wrote: On 11/04/2014 12:43 AM, Minchan Kim wrote: Hello, On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: Having test_pages_isolated failure message as a warning

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-03 Thread Minchan Kim
Hello, On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: > Having test_pages_isolated failure message as a warning confuses > users into thinking that it is more serious than it really is. In > reality, if called via CMA, allocation will be retried so a single >

[PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-03 Thread Michal Nazarewicz
Having test_pages_isolated failure message as a warning confuses users into thinking that it is more serious than it really is. In reality, if called via CMA, allocation will be retried so a single test_pages_isolated failure does not prevent allocation from succeeding. Demote the warning

[PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-03 Thread Michal Nazarewicz
Having test_pages_isolated failure message as a warning confuses users into thinking that it is more serious than it really is. In reality, if called via CMA, allocation will be retried so a single test_pages_isolated failure does not prevent allocation from succeeding. Demote the warning

Re: [PATCH] mm: alloc_contig_range: demote pages busy message from warn to info

2014-11-03 Thread Minchan Kim
Hello, On Mon, Nov 03, 2014 at 05:57:53PM +0100, Michal Nazarewicz wrote: Having test_pages_isolated failure message as a warning confuses users into thinking that it is more serious than it really is. In reality, if called via CMA, allocation will be retried so a single test_pages_isolated