Re: CMA broken in next-20120926

2012-10-09 Thread Thierry Reding
On Tue, Oct 09, 2012 at 01:08:30PM +0200, Bartlomiej Zolnierkiewicz wrote: > On Tuesday 09 October 2012 12:11:43 Mel Gorman wrote: > > On Tue, Oct 09, 2012 at 10:40:10AM +0200, Bartlomiej Zolnierkiewicz wrote: > > > I also need following patch to make CONFIG_CMA=y && CONFIG_COMPACTION=y > > > case

Re: CMA broken in next-20120926

2012-10-09 Thread Mel Gorman
On Tue, Oct 09, 2012 at 01:08:30PM +0200, Bartlomiej Zolnierkiewicz wrote: > On Tuesday 09 October 2012 12:11:43 Mel Gorman wrote: > > On Tue, Oct 09, 2012 at 10:40:10AM +0200, Bartlomiej Zolnierkiewicz wrote: > > > I also need following patch to make CONFIG_CMA=y && CONFIG_COMPACTION=y > > > case

Re: CMA broken in next-20120926

2012-10-09 Thread Bartlomiej Zolnierkiewicz
On Tuesday 09 October 2012 12:11:43 Mel Gorman wrote: > On Tue, Oct 09, 2012 at 10:40:10AM +0200, Bartlomiej Zolnierkiewicz wrote: > > I also need following patch to make CONFIG_CMA=y && CONFIG_COMPACTION=y case > > work: > > > > From: Bartlomiej Zolnierkiewicz > > Subject: [PATCH] mm: compaction

Re: CMA broken in next-20120926

2012-10-09 Thread Thierry Reding
On Tue, Oct 09, 2012 at 11:11:43AM +0100, Mel Gorman wrote: > On Tue, Oct 09, 2012 at 10:40:10AM +0200, Bartlomiej Zolnierkiewicz wrote: > > I also need following patch to make CONFIG_CMA=y && CONFIG_COMPACTION=y case > > work: > > > > From: Bartlomiej Zolnierkiewicz > > Subject: [PATCH] mm: comp

Re: CMA broken in next-20120926

2012-10-09 Thread Mel Gorman
On Tue, Oct 09, 2012 at 10:40:10AM +0200, Bartlomiej Zolnierkiewicz wrote: > I also need following patch to make CONFIG_CMA=y && CONFIG_COMPACTION=y case > work: > > From: Bartlomiej Zolnierkiewicz > Subject: [PATCH] mm: compaction: cache if a pageblock was scanned and no > pages were isolated -

Re: CMA broken in next-20120926

2012-10-09 Thread Bartlomiej Zolnierkiewicz
Hi, On Monday 08 October 2012 10:48:07 Mel Gorman wrote: > On Mon, Oct 08, 2012 at 05:06:54PM +0900, Minchan Kim wrote: > > Hi Mel, > > > > On Tue, Oct 02, 2012 at 04:12:17PM +0100, Mel Gorman wrote: > > > On Tue, Oct 02, 2012 at 05:03:07PM +0200, Thierry Reding wrote: > > > > On Tue, Oct 02, 20

Re: CMA broken in next-20120926

2012-10-08 Thread Mel Gorman
On Mon, Oct 08, 2012 at 05:06:54PM +0900, Minchan Kim wrote: > Hi Mel, > > On Tue, Oct 02, 2012 at 04:12:17PM +0100, Mel Gorman wrote: > > On Tue, Oct 02, 2012 at 05:03:07PM +0200, Thierry Reding wrote: > > > On Tue, Oct 02, 2012 at 03:41:35PM +0100, Mel Gorman wrote: > > > > On Tue, Oct 02, 2012

Re: CMA broken in next-20120926

2012-10-08 Thread Minchan Kim
Hi Mel, On Tue, Oct 02, 2012 at 04:12:17PM +0100, Mel Gorman wrote: > On Tue, Oct 02, 2012 at 05:03:07PM +0200, Thierry Reding wrote: > > On Tue, Oct 02, 2012 at 03:41:35PM +0100, Mel Gorman wrote: > > > On Tue, Oct 02, 2012 at 02:48:14PM +0200, Thierry Reding wrote: > > > > > So this really isn't

Re: CMA broken in next-20120926

2012-10-02 Thread Mel Gorman
On Tue, Oct 02, 2012 at 05:03:07PM +0200, Thierry Reding wrote: > On Tue, Oct 02, 2012 at 03:41:35PM +0100, Mel Gorman wrote: > > On Tue, Oct 02, 2012 at 02:48:14PM +0200, Thierry Reding wrote: > > > > So this really isn't all that new, but I just wanted to confirm my > > > > results from last week

Re: CMA broken in next-20120926

2012-10-02 Thread Thierry Reding
On Tue, Oct 02, 2012 at 03:41:35PM +0100, Mel Gorman wrote: > On Tue, Oct 02, 2012 at 02:48:14PM +0200, Thierry Reding wrote: > > > So this really isn't all that new, but I just wanted to confirm my > > > results from last week. We'll see if bisection shows up something > > > interesting. > > > >

Re: CMA broken in next-20120926

2012-10-02 Thread Mel Gorman
On Tue, Oct 02, 2012 at 02:48:14PM +0200, Thierry Reding wrote: > > So this really isn't all that new, but I just wanted to confirm my > > results from last week. We'll see if bisection shows up something > > interesting. > > I just finished bisecting this and git reports: > > 3750280f8bd0e

Re: CMA broken in next-20120926

2012-10-02 Thread Thierry Reding
On Mon, Oct 01, 2012 at 04:24:29PM +0200, Thierry Reding wrote: > On Fri, Sep 28, 2012 at 01:43:32PM +0100, Mel Gorman wrote: > > On Fri, Sep 28, 2012 at 01:39:24PM +0200, Thierry Reding wrote: > > > On Fri, Sep 28, 2012 at 12:07:12PM +0100, Mel Gorman wrote: > > > > On Fri, Sep 28, 2012 at 12:51:1

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 01:39:24PM +0200, Thierry Reding wrote: > On Fri, Sep 28, 2012 at 12:07:12PM +0100, Mel Gorman wrote: > > On Fri, Sep 28, 2012 at 12:51:13PM +0200, Thierry Reding wrote: > > > On Fri, Sep 28, 2012 at 12:38:15PM +0200, Thierry Reding wrote: > > > > On Fri, Sep 28, 2012 at 12:

Re: CMA broken in next-20120926

2012-09-28 Thread Thierry Reding
On Fri, Sep 28, 2012 at 12:07:12PM +0100, Mel Gorman wrote: > On Fri, Sep 28, 2012 at 12:51:13PM +0200, Thierry Reding wrote: > > On Fri, Sep 28, 2012 at 12:38:15PM +0200, Thierry Reding wrote: > > > On Fri, Sep 28, 2012 at 12:32:07PM +0200, Thierry Reding wrote: > > > > On Fri, Sep 28, 2012 at 11:

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 12:51:13PM +0200, Thierry Reding wrote: > On Fri, Sep 28, 2012 at 12:38:15PM +0200, Thierry Reding wrote: > > On Fri, Sep 28, 2012 at 12:32:07PM +0200, Thierry Reding wrote: > > > On Fri, Sep 28, 2012 at 11:27:28AM +0100, Mel Gorman wrote: > > > > On Fri, Sep 28, 2012 at 11:

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 12:38:15PM +0200, Thierry Reding wrote: > > > > > > I've been running a few tests and indeed this solves the obvious problem > > that the coherent pool cannot be created at boot (which in turn caused > > the ethernet adapter to fail on Tegra). > > > > However I've been wor

Re: CMA broken in next-20120926

2012-09-28 Thread Thierry Reding
On Fri, Sep 28, 2012 at 12:38:15PM +0200, Thierry Reding wrote: > On Fri, Sep 28, 2012 at 12:32:07PM +0200, Thierry Reding wrote: > > On Fri, Sep 28, 2012 at 11:27:28AM +0100, Mel Gorman wrote: > > > On Fri, Sep 28, 2012 at 11:48:25AM +0300, Peter Ujfalusi wrote: > > > > Hi, > > > > > > > > On 09/

Re: CMA broken in next-20120926

2012-09-28 Thread Thierry Reding
On Fri, Sep 28, 2012 at 12:32:07PM +0200, Thierry Reding wrote: > On Fri, Sep 28, 2012 at 11:27:28AM +0100, Mel Gorman wrote: > > On Fri, Sep 28, 2012 at 11:48:25AM +0300, Peter Ujfalusi wrote: > > > Hi, > > > > > > On 09/28/2012 11:37 AM, Mel Gorman wrote: > > > >> I hope this patch fixes the bug

Re: CMA broken in next-20120926

2012-09-28 Thread Thierry Reding
On Fri, Sep 28, 2012 at 11:27:28AM +0100, Mel Gorman wrote: > On Fri, Sep 28, 2012 at 11:48:25AM +0300, Peter Ujfalusi wrote: > > Hi, > > > > On 09/28/2012 11:37 AM, Mel Gorman wrote: > > >> I hope this patch fixes the bug. If this patch fixes the problem > > >> but has some problem about descript

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 11:48:25AM +0300, Peter Ujfalusi wrote: > Hi, > > On 09/28/2012 11:37 AM, Mel Gorman wrote: > >> I hope this patch fixes the bug. If this patch fixes the problem > >> but has some problem about description or someone has better idea, > >> feel free to modify and resend to a

Re: CMA broken in next-20120926

2012-09-28 Thread Peter Ujfalusi
Hi, On 09/28/2012 11:37 AM, Mel Gorman wrote: >> I hope this patch fixes the bug. If this patch fixes the problem >> but has some problem about description or someone has better idea, >> feel free to modify and resend to akpm, Please. >> > > A full revert is overkill. Can the following patch be t

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
> I hope this patch fixes the bug. If this patch fixes the problem > but has some problem about description or someone has better idea, > feel free to modify and resend to akpm, Please. > A full revert is overkill. Can the following patch be tested as a potential replacement please? ---8<--- mm:

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 08:56:28AM +0100, Mel Gorman wrote: > On Fri, Sep 28, 2012 at 02:43:30PM +0900, Minchan Kim wrote: > > On Thu, Sep 27, 2012 at 03:11:59PM -0700, Andrew Morton wrote: > > > On Thu, 27 Sep 2012 13:29:11 +0200 > > > Thierry Reding wrote: > > > > > > > Hi Marek, > > > > > > >

Re: CMA broken in next-20120926

2012-09-28 Thread Mel Gorman
On Fri, Sep 28, 2012 at 02:43:30PM +0900, Minchan Kim wrote: > On Thu, Sep 27, 2012 at 03:11:59PM -0700, Andrew Morton wrote: > > On Thu, 27 Sep 2012 13:29:11 +0200 > > Thierry Reding wrote: > > > > > Hi Marek, > > > > > > any idea why CMA might be broken in next-20120926. I see that there > > >

Re: CMA broken in next-20120926

2012-09-28 Thread Peter Ujfalusi
On 09/28/2012 08:43 AM, Minchan Kim wrote: > From 24a547855fa2bd4212a779cc73997837148310b3 Mon Sep 17 00:00:00 2001 > From: Minchan Kim > Date: Fri, 28 Sep 2012 14:28:32 +0900 > Subject: [PATCH] revert mm: compaction: iron out isolate_freepages_block() > and isolate_freepages_range() > > [1] mad

Re: CMA broken in next-20120926

2012-09-27 Thread Thierry Reding
On Fri, Sep 28, 2012 at 02:43:30PM +0900, Minchan Kim wrote: > On Thu, Sep 27, 2012 at 03:11:59PM -0700, Andrew Morton wrote: > > On Thu, 27 Sep 2012 13:29:11 +0200 > > Thierry Reding wrote: > > > > > Hi Marek, > > > > > > any idea why CMA might be broken in next-20120926. I see that there > > >

Re: CMA broken in next-20120926

2012-09-27 Thread Minchan Kim
On Thu, Sep 27, 2012 at 03:11:59PM -0700, Andrew Morton wrote: > On Thu, 27 Sep 2012 13:29:11 +0200 > Thierry Reding wrote: > > > Hi Marek, > > > > any idea why CMA might be broken in next-20120926. I see that there > > haven't been any major changes to CMA itself, but there's been quite a > > b

Re: CMA broken in next-20120926

2012-09-27 Thread Andrew Morton
On Thu, 27 Sep 2012 13:29:11 +0200 Thierry Reding wrote: > Hi Marek, > > any idea why CMA might be broken in next-20120926. I see that there > haven't been any major changes to CMA itself, but there's been quite a > bit of restructuring of various memory allocation bits lately. I wasn't > able t

Re: CMA broken in next-20120926

2012-09-27 Thread Mark Brown
On Thu, Sep 27, 2012 at 05:59:19PM +0200, Marek Szyprowski wrote: > On Thursday, September 27, 2012 1:29 PM Thierry Reding wrote: > > any idea why CMA might be broken in next-20120926. I see that there > > haven't been any major changes to CMA itself, but there's been quite a > > bit of restructur

RE: CMA broken in next-20120926

2012-09-27 Thread Marek Szyprowski
Hello, On Thursday, September 27, 2012 1:29 PM Thierry Reding wrote: > any idea why CMA might be broken in next-20120926. I see that there > haven't been any major changes to CMA itself, but there's been quite a > bit of restructuring of various memory allocation bits lately. I wasn't > able to t

CMA broken in next-20120926

2012-09-27 Thread Thierry Reding
Hi Marek, any idea why CMA might be broken in next-20120926. I see that there haven't been any major changes to CMA itself, but there's been quite a bit of restructuring of various memory allocation bits lately. I wasn't able to track the problem down, though. What I see is this during boot (with