linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@linux-foundation.org>
To: Gary Hade <garyhade@us.ibm.com>
Cc: roel.kluin@gmail.com, mingo@elte.hu,
	linux-kernel@vger.kernel.org, linux-mm@kvack.org,
	y-goto@jp.fujitsu.com
Subject: Re: [PATCH] mm: get_nid_for_pfn() returns int
Date: Fri, 27 Feb 2009 16:22:49 -0800	[thread overview]
Message-ID: <20090227162249.bcd0813a.akpm@linux-foundation.org> (raw)
In-Reply-To: <20090228001400.GC7174@us.ibm.com>

On Fri, 27 Feb 2009 16:14:00 -0800
Gary Hade <garyhade@us.ibm.com> wrote:

> On Fri, Feb 27, 2009 at 01:46:16PM -0800, Andrew Morton wrote:
>
> > > It is still lingering in -mm:
> > > http://userweb.kernel.org/~akpm/mmotm/broken-out/mm-get_nid_for_pfn-returns-int.patch
> > > 
> > 
> > Should it unlinger?  I have it in the 2.6.30 pile.
> 
> Yes, that would be good. :)

What would be good?  Your answer is ambiguous.

> > Does it actually fix a demonstrable bug?  
> 
> I am not aware of anyone that has actually reproduced the
> problem.

What problem?

All I gave at present is

  From: Roel Kluin <roel.kluin@gmail.com>

  get_nid_for_pfn() returns int

  Signed-off-by: Roel Kluin <roel.kluin@gmail.com>
  Cc: Gary Hade <garyhade@us.ibm.com>

>  I do not believe that we have any systems where 
> it can be reproduced since it would require both
>   (1) a memory section with an uninitialized range of
>       pages and
>   (2) a memory remove event for that memory section.
> As far as I know, none of our systems have (1).  Yasunori Goto
> has a system with (1) but I am not sure if he can do (2).

Please send a new changelog for this patch.

If you believe this patch should be merged into 2.6.29 then please
explain why.  Please also consider whether it should be backported into
2.6.28.x and eariler.


--
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>

  reply	other threads:[~2009-02-28  0:23 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-01-18 22:36 [PATCH] mm: get_nid_for_pfn() returns int Roel Kluin
2009-01-19 17:59 ` Gary Hade
2009-01-27  6:33   ` Andrew Morton
2009-01-27 21:07     ` Gary Hade
2009-01-28  5:04       ` Yasunori Goto
2009-02-27 14:56       ` roel kluin
2009-02-27 21:33         ` Gary Hade
2009-02-27 21:46           ` Andrew Morton
2009-02-28  0:14             ` Gary Hade
2009-02-28  0:22               ` Andrew Morton [this message]
2009-02-28  3:02                 ` Gary Hade
2009-02-28  4:08                   ` Andrew Morton
2009-03-01 20:58                     ` Gary Hade

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=20090227162249.bcd0813a.akpm@linux-foundation.org \
    --to=akpm@linux-foundation.org \
    --cc=garyhade@us.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mm@kvack.org \
    --cc=mingo@elte.hu \
    --cc=roel.kluin@gmail.com \
    --cc=y-goto@jp.fujitsu.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).