public inbox for linux-ia64@vger.kernel.org
 help / color / mirror / Atom feed
From: Simon Horman <horms@verge.net.au>
To: Jay Lan <jlan@sgi.com>
Cc: "linux-ia64@vger.kernel.org" <linux-ia64@vger.kernel.org>,
	kexec@lists.infradead.org
Subject: Re: [PATCH]IA64: do not include uncached memory to vmcore
Date: Fri, 12 Sep 2008 23:35:02 +0000	[thread overview]
Message-ID: <20080912233500.GF3832@verge.net.au> (raw)
In-Reply-To: <48CACCBA.9030905@sgi.com>

On Fri, Sep 12, 2008 at 01:10:34PM -0700, Jay Lan wrote:
> Currently a memory segment in memory map with attribute of EFI_MEMORY_UC
> is denoted as "System RAM" in /proc/iomem, while memory of attribute
> (EFI_MEMORY_WB|EFI_MEMORY_UC) is also labeled the same.
> 
> The kexec utility then includes uncached memory as part of vmcore.
> The kdump kernel may MCA when it tries to save the vmcore to a disk.
> A normal "cached" access can cause MCAs.
> 
> Since kexec assembled memory ranges with memory tagged as "System RAM",
> the uncached memory will be excluded if it is labeled differently.
> 
> Simon, since only IA64 will create "Uncached RAM" label, i do not
> make changes to other arch.
> 
> Our HP machine in the lab is dead. I am sorry that i can not test
> against other IA64 systems (than SGI's). Feedback is very much
> appreciated.
> 
> The corresponding kernel patch is needed to test this kexec patch:
> http://marc.info/?l=linux-ia64&m\x122122791230130&w=2
> This patch without the kernel patch will have no effect and do no
> harm.

Hi Jay,

both of these patches seem reasonable to me, especially from a kexec point
of view. I will try and test them on an HP RX2620 and Intel Tiger 4 and
get back to you shortly.

-- 
Simon Horman
  VA Linux Systems Japan K.K., Sydney, Australia Satellite Office
  H: www.vergenet.net/~horms/             W: www.valinux.co.jp/en


       reply	other threads:[~2008-09-12 23:35 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <48CACCBA.9030905@sgi.com>
2008-09-12 23:35 ` Simon Horman [this message]
2008-09-23 23:57   ` [PATCH]IA64: do not include uncached memory to vmcore Simon Horman

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=20080912233500.GF3832@verge.net.au \
    --to=horms@verge.net.au \
    --cc=jlan@sgi.com \
    --cc=kexec@lists.infradead.org \
    --cc=linux-ia64@vger.kernel.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