From: Chandru <chandru@in.ibm.com>
To: Dave Hansen <dave@linux.vnet.ibm.com>
Cc: Benjamin Herrenschmidt <benh@kernel.crashing.org>,
linux-kernel@vger.kernel.org, mm-commits@vger.kernel.org,
chandru@linux.vnet.ibm.com, paulus@samba.org,
akpm@linux-foundation.org, Johannes Weiner <hannes@cmpxchg.org>,
"Jon M. Tollefson [imap]" <kniht@us.ibm.com>
Subject: Re: + powerpc-fix-code-for-reserved-memory-spanning-across-nodes.patch added to -mm tree
Date: Tue, 10 Feb 2009 23:57:42 +0530 [thread overview]
Message-ID: <4991C71E.9060509@in.ibm.com> (raw)
In-Reply-To: <1234288310.30155.27.camel@nimitz>
Dave Hansen wrote:
> On Mon, 2009-02-09 at 11:31 +1100, Benjamin Herrenschmidt wrote:
>
>> Dave, you've had your hands in that code more than I did lately, do
>> that
>> look ok to you ?
>>
>
> I still think that patch is bogus. I don't know how it is still around.
> I described the problem pretty completely here:
>
> http://git.kernel.org/gitweb.cgi?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=4a6186696e7f15b3ea4dafcdb64ee0703e0e4487
>
> Chandru effectively reverted my bugfix from that patch in this one.
>
>
>> (BTW. On another note, do you still have pending bug fixes that didn't
>> got in .29 ?)
>>
>
> No, I don't think so. There were some distro fixes but I couldn't
> reproduce the bugs on mainline.
>
> -- Dave
>
>
Dave,
If I remember, the first mail that I sent on this subject said
2.6.28-rc9 panics. Unless something has changed in the way
lmb's are added or are reserved, this should be also
reproducible with the mainline kernel on the machines that
had exhibited this problem earlier. I will verify your patch
on these machines again and let you know the results.
Thanks,
Chandru
next prev parent reply other threads:[~2009-02-10 18:27 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <200812250806.mBP86LPQ018399@imap1.linux-foundation.org>
2009-02-08 20:47 ` + powerpc-fix-code-for-reserved-memory-spanning-across-nodes.patch added to -mm tree Johannes Weiner
2009-02-09 0:31 ` Benjamin Herrenschmidt
2009-02-10 17:51 ` Dave Hansen
2009-02-10 18:27 ` Chandru [this message]
2009-02-13 11:01 ` Chandru
2009-02-10 11:35 ` Chandru
2009-02-10 17:59 ` Johannes Weiner
2009-02-13 11:18 ` Chandru
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=4991C71E.9060509@in.ibm.com \
--to=chandru@in.ibm.com \
--cc=akpm@linux-foundation.org \
--cc=benh@kernel.crashing.org \
--cc=chandru@linux.vnet.ibm.com \
--cc=dave@linux.vnet.ibm.com \
--cc=hannes@cmpxchg.org \
--cc=kniht@us.ibm.com \
--cc=linux-kernel@vger.kernel.org \
--cc=mm-commits@vger.kernel.org \
--cc=paulus@samba.org \
/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