All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Dave Hansen <dave@linux.vnet.ibm.com>
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	Mel Gorman <mgorman@suse.de>
Subject: Re: [PATCH] Debugging: Keep track of page owners
Date: Fri, 7 Dec 2012 14:44:28 -0800	[thread overview]
Message-ID: <20121207144428.98b3eaf4.akpm@linux-foundation.org> (raw)
In-Reply-To: <50C26FA7.9010000@linux.vnet.ibm.com>

On Fri, 07 Dec 2012 14:37:27 -0800
Dave Hansen <dave@linux.vnet.ibm.com> wrote:

> On 12/07/2012 02:26 PM, Andrew Morton wrote:\
> > I have cunningly divined the intention of your update and have queued
> > the below incremental.  The change to
> > pagetypeinfo_showmixedcount_print() was a surprise.  What's that there
> > for?
> 
> Do you mean to ask why it's being modified at all here in this patch?
> It's referenced in the changelog a bit.  I believe it came from Mel at
> some point.  I didn't do much to that portion, but I happily drug those
> hunks along with my forward port.  I believe it's virtually all the same
> as what you posted here:
> 
> 	https://bugzilla.kernel.org/show_bug.cgi?id=50181

I'm now deeply confused.

Two days ago I merged a version which you sent me.  It is presently at
http://ozlabs.org/~akpm/mmotm/broken-out/debugging-keep-track-of-page-owners.patch.

Today you sent a new version.  My earlier email showed the difference
between these two versions.

AFACIT that difference was undescribed.  I can see that the new version
uses the stack-tracing infrastructure, but the change to
pagetypeinfo_showmixedcount_print() is a mystery.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

WARNING: multiple messages have this Message-ID (diff)
From: Andrew Morton <akpm@linux-foundation.org>
To: Dave Hansen <dave@linux.vnet.ibm.com>
Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org,
	Mel Gorman <mgorman@suse.de>
Subject: Re: [PATCH] Debugging: Keep track of page owners
Date: Fri, 7 Dec 2012 14:44:28 -0800	[thread overview]
Message-ID: <20121207144428.98b3eaf4.akpm@linux-foundation.org> (raw)
In-Reply-To: <50C26FA7.9010000@linux.vnet.ibm.com>

On Fri, 07 Dec 2012 14:37:27 -0800
Dave Hansen <dave@linux.vnet.ibm.com> wrote:

> On 12/07/2012 02:26 PM, Andrew Morton wrote:\
> > I have cunningly divined the intention of your update and have queued
> > the below incremental.  The change to
> > pagetypeinfo_showmixedcount_print() was a surprise.  What's that there
> > for?
> 
> Do you mean to ask why it's being modified at all here in this patch?
> It's referenced in the changelog a bit.  I believe it came from Mel at
> some point.  I didn't do much to that portion, but I happily drug those
> hunks along with my forward port.  I believe it's virtually all the same
> as what you posted here:
> 
> 	https://bugzilla.kernel.org/show_bug.cgi?id=50181

I'm now deeply confused.

Two days ago I merged a version which you sent me.  It is presently at
http://ozlabs.org/~akpm/mmotm/broken-out/debugging-keep-track-of-page-owners.patch.

Today you sent a new version.  My earlier email showed the difference
between these two versions.

AFACIT that difference was undescribed.  I can see that the new version
uses the stack-tracing infrastructure, but the change to
pagetypeinfo_showmixedcount_print() is a mystery.


  reply	other threads:[~2012-12-07 22:44 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-12-07 21:24 [PATCH] Debugging: Keep track of page owners Dave Hansen
2012-12-07 21:24 ` Dave Hansen
2012-12-07 22:26 ` Andrew Morton
2012-12-07 22:26   ` Andrew Morton
2012-12-07 22:37   ` Dave Hansen
2012-12-07 22:37     ` Dave Hansen
2012-12-07 22:44     ` Andrew Morton [this message]
2012-12-07 22:44       ` Andrew Morton
2012-12-07 22:58       ` Dave Hansen
2012-12-07 22:58         ` Dave Hansen
2012-12-08  2:51   ` Steven Rostedt
2012-12-08  2:51     ` Steven Rostedt
2012-12-08  2:58 ` Steven Rostedt
2012-12-08  2:58   ` Steven Rostedt
  -- strict thread matches above, loose matches on Subject: below --
2012-12-05  1:12 Dave Hansen
2012-12-05 15:01 ` Laura Abbott
2012-12-05 17:48   ` Dave Hansen
2012-12-05 22:36     ` Andrew Morton
2012-12-07 16:37     ` Laura Abbott

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20121207144428.98b3eaf4.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=dave@linux.vnet.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mgorman@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.