From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O31y9-0002yI-GL for qemu-devel@nongnu.org; Sat, 17 Apr 2010 02:57:33 -0400 Received: from [140.186.70.92] (port=44567 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O31y6-0002y4-Nh for qemu-devel@nongnu.org; Sat, 17 Apr 2010 02:57:33 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O31y3-0004iC-G7 for qemu-devel@nongnu.org; Sat, 17 Apr 2010 02:57:30 -0400 Received: from mail-pz0-f204.google.com ([209.85.222.204]:56338) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O31y3-0004hd-6T for qemu-devel@nongnu.org; Sat, 17 Apr 2010 02:57:27 -0400 Received: by pzk42 with SMTP id 42so2481343pzk.4 for ; Fri, 16 Apr 2010 23:57:24 -0700 (PDT) MIME-Version: 1.0 In-Reply-To: <1271459677-21942-1-git-send-email-atar4qemu@google.com> References: <1271459677-21942-1-git-send-email-atar4qemu@google.com> Date: Sat, 17 Apr 2010 09:57:24 +0300 Message-ID: From: Blue Swirl Content-Type: text/plain; charset=UTF-8 Subject: [Qemu-devel] Re: [patch 02/02] sparc32 use empty_slot for missing RAM List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Artyom Tarasenko Cc: qemu-devel@nongnu.org, Artyom Tarasenko On 4/17/10, Artyom Tarasenko wrote: > use empty_slot device for the RAM which is not installed > > Models without ECC don't trap when missing ram is accessed. > > Signed-off-by: Artyom Tarasenko > --- > Makefile.target | 2 +- > hw/sun4m.c | 7 ++++++- > 2 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/Makefile.target b/Makefile.target > index 95c9d23..6362927 100644 > --- a/Makefile.target > +++ b/Makefile.target > @@ -248,7 +248,7 @@ obj-sparc-y += cirrus_vga.o > else > obj-sparc-y = sun4m.o lance.o tcx.o sun4m_iommu.o slavio_intctl.o > obj-sparc-y += slavio_timer.o slavio_misc.o sparc32_dma.o > -obj-sparc-y += cs4231.o eccmemctl.o sbi.o sun4c_intctl.o > +obj-sparc-y += cs4231.o eccmemctl.o sbi.o sun4c_intctl.o empty_slot.o > endif The device could be compiled once, so please use instead Makefile.objs, conditional to CONFIG_EMPTY_SLOT which gets defined in default-configs/sparc-softmmu.mak. On the other hand, the device is not used by other boards yet, so there is no benefit at this point. > obj-arm-y = integratorcp.o versatilepb.o arm_pic.o arm_timer.o > diff --git a/hw/sun4m.c b/hw/sun4m.c > index 90e661d..fb49af5 100644 > --- a/hw/sun4m.c > +++ b/hw/sun4m.c > @@ -36,6 +36,7 @@ > #include "isa.h" > #include "fw_cfg.h" > #include "escc.h" > +#include "empty_slot.h" > #include "qdev-addr.h" > #include "loader.h" > #include "elf.h" > @@ -820,7 +821,11 @@ static void sun4m_hw_init(const struct sun4m_hwdef *hwdef, ram_addr_t RAM_size, > > /* set up devices */ > ram_init(0, RAM_size, hwdef->max_mem); > - > + /* models without ECC don't trap when missing ram is accessed */ > + if(!hwdef->ecc_base) { The indentation seems to be off and there should be a space between 'if' and '('. > + empty_slot_init(RAM_size, hwdef->max_mem - RAM_size); > + } > + > prom_init(hwdef->slavio_base, bios_name); > > slavio_intctl = slavio_intctl_init(hwdef->intctl_base, > > -- > 1.6.6.1 > >