From: Yasunori Goto <y-goto@jp.fujitsu.com>
To: Bernhard Walle <bwalle@suse.de>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
Vegard Nossum <vegard.nossum@gmail.com>, Greg KH <gregkh@suse.de>,
kexec <kexec@lists.infradead.org>,
LKML <linux-kernel@vger.kernel.org>,
Dave Hansen <dave@linux.vnet.ibm.com>,
Mariusz Kozlowski <m.kozlowski@tuxland.pl>,
Pekka Enberg <penberg@cs.helsinki.fi>,
linux-next@vger.kernel.org, Ingo Molnar <mingo@elte.hu>,
kernel-testers@vger.kernel.org, Vivek Goyal <vgoyal@redhat.com>
Subject: Re: linux-next: Tree for July 18: warning at kernel/lockdep.c:2068 trace_hardirqs_on_caller
Date: Tue, 22 Jul 2008 15:15:11 +0900 [thread overview]
Message-ID: <20080722142345.8733.E1E9C6FF@jp.fujitsu.com> (raw)
In-Reply-To: <20080721170037.1a0046b1@halley.suse.de>
Hello.
> > > > Is /proc/iomem updated upon memory hotplug event.
> > >
> > > Yes. I just checked that (yesterday).
> > >
> > > I think it would make sense to extend /sys/firmware/memmap on
> > > hot-plugging. Just because on reboot, the firmware will see that
> > > memory, too, and report it. However, we need a way to discriminate the
> > > originally firmware-provided memory map with later added memory. I'm
> > > not sure how that can be done, I have to think about it.
> >
> > Probably use another type of RAM identifier (System RAM (hotplug)).
> >
> > But the point is, if /sys/devices/system/memory also represents all
> > the physical memory present in the system then it might be not be
> > justified to create another similar interface. (Until and unless there
> > is something unique about /sys/firmware/memmap).
>
> But I don't see anything like a physical address there:
>
> /sys/devices/system/memory/memory2:
> -r--r--r-- 1 root root 4096 2008-07-21 15:45 phys_device
> -r--r--r-- 1 root root 4096 2008-07-21 15:45 phys_index
> -rw-r--r-- 1 root root 4096 2008-07-21 15:45 state
>
> (on a PPC64 machine where SUSE kernel has that interface enabled by
> default).
I wrote about them Documentation/memory-hotplug.txt. Please see it.
But I think /sys/firmware/memmap is better for kexec than using them.
They are made for each sections whose size is fixed on each architecture.
There is no information about areas which are occupied by firmware, and
its fixed size directories are not suitable to show them.
BTW, does kexec needs the information about not only hot-added normal memory
but also "hot-added occupied (reserved) memory by firmware"?
Fujitsu has ia64 box which can add memory. The information about memory
area is notified via _CRS method of ACPI. Our firmware team said that
there was no interface to notify the area which was occupied by firmware.
So, _CRS shows only normal (not-reserved) memory area. It means OS can't know
reserved memory which is hot-added.
If kexec has to know those reserved area, then it is very bad news for me. :-(
Thanks.
--
Yasunori Goto
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
prev parent reply other threads:[~2008-07-22 6:17 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20080718195352.e562a00f.sfr@canb.auug.org.au>
[not found] ` <200807190928.33978.m.kozlowski@tuxland.pl>
[not found] ` <19f34abd0807190255x304173d4wf2bfabb2d5bce511@mail.gmail.com>
[not found] ` <19f34abd0807190559y2fe5ebf9h7095793e82de3122@mail.gmail.com>
[not found] ` <20080719221723.GB5578@suse.de>
[not found] ` <19f34abd0807191527u61c5ed61kffe2279c8d46915d@mail.gmail.com>
[not found] ` <19f34abd0807191544nfd73be5nf7dde4b61992a7e8@mail.gmail.com>
[not found] ` <20080719225817.GA6264@suse.de>
[not found] ` <19f34abd0807191611y7cabf405iad307ba79591e04f@mail.gmail.com>
2008-07-20 9:01 ` linux-next: Tree for July 18: warning at kernel/lockdep.c:2068 trace_hardirqs_on_caller Dave Hansen
2008-07-20 9:35 ` Thomas Meyer
2008-07-20 13:03 ` Bernhard Walle
2008-07-20 15:44 ` Bernhard Walle
2008-07-21 13:17 ` Vivek Goyal
2008-07-21 13:25 ` Bernhard Walle
2008-07-21 13:39 ` Vivek Goyal
2008-07-21 13:47 ` Bernhard Walle
2008-07-21 14:54 ` Vivek Goyal
2008-07-21 15:00 ` Bernhard Walle
2008-07-21 15:00 ` Bernhard Walle
2008-07-22 6:15 ` Yasunori Goto [this message]
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=20080722142345.8733.E1E9C6FF@jp.fujitsu.com \
--to=y-goto@jp.fujitsu.com \
--cc=bwalle@suse.de \
--cc=dave@linux.vnet.ibm.com \
--cc=gregkh@suse.de \
--cc=kernel-testers@vger.kernel.org \
--cc=kexec@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-next@vger.kernel.org \
--cc=m.kozlowski@tuxland.pl \
--cc=mingo@elte.hu \
--cc=penberg@cs.helsinki.fi \
--cc=sfr@canb.auug.org.au \
--cc=vegard.nossum@gmail.com \
--cc=vgoyal@redhat.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