From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vivek Goyal Subject: Re: linux-next: Tree for July 18: warning at kernel/lockdep.c:2068 trace_hardirqs_on_caller Date: Mon, 21 Jul 2008 09:39:37 -0400 Message-ID: <20080721133937.GC4451@redhat.com> References: <19f34abd0807190255x304173d4wf2bfabb2d5bce511@mail.gmail.com> <19f34abd0807190559y2fe5ebf9h7095793e82de3122@mail.gmail.com> <20080719221723.GB5578@suse.de> <19f34abd0807191527u61c5ed61kffe2279c8d46915d@mail.gmail.com> <19f34abd0807191544nfd73be5nf7dde4b61992a7e8@mail.gmail.com> <20080719225817.GA6264@suse.de> <19f34abd0807191611y7cabf405iad307ba79591e04f@mail.gmail.com> <1216544462.9311.20.camel@nimitz> <20080721131721.GB4451@redhat.com> <20080721152539.3d8684f9@halley.suse.de> Mime-Version: 1.0 Return-path: Content-Disposition: inline In-Reply-To: <20080721152539.3d8684f9@halley.suse.de> Sender: linux-next-owner@vger.kernel.org List-ID: Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Bernhard Walle Cc: Dave Hansen , Stephen Rothwell , Vegard Nossum , Pekka Enberg , Greg KH , kexec , LKML , Mariusz Kozlowski , linux-next@vger.kernel.org, Ingo Molnar , kernel-testers@vger.kernel.org On Mon, Jul 21, 2008 at 03:25:39PM +0200, Bernhard Walle wrote: > * Vivek Goyal [2008-07-21 09:17]: > > > > 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). Thanks Vivek