From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50167) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YaWOb-0004Ev-1T for qemu-devel@nongnu.org; Tue, 24 Mar 2015 17:30:00 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1YaWOZ-0000BI-RJ for qemu-devel@nongnu.org; Tue, 24 Mar 2015 17:29:56 -0400 Received: from mail-wi0-x22b.google.com ([2a00:1450:400c:c05::22b]:36683) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1YaWOZ-0000A0-HU for qemu-devel@nongnu.org; Tue, 24 Mar 2015 17:29:55 -0400 Received: by wibg7 with SMTP id g7so86166288wib.1 for ; Tue, 24 Mar 2015 14:29:55 -0700 (PDT) Sender: Paolo Bonzini Message-ID: <5511D74F.9060908@redhat.com> Date: Tue, 24 Mar 2015 22:29:51 +0100 From: Paolo Bonzini MIME-Version: 1.0 References: In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH for-2.3] m68k: memory: Replace memory_region_init_ram with memory_region_allocate_system_memory List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?RGlyayBNw7xsbGVy?= , qemu-devel@nongnu.org On 24/03/2015 22:26, Dirk Müller wrote: > Commit 0b183fc871:"memory: move mem_path handling to > memory_region_allocate_system_memory" split memory_region_init_ram and > memory_region_init_ram_from_file. Also it moved mem-path handling a step > up from memory_region_init_ram to memory_region_allocate_system_memory. > > Therefore for any board that uses memory_region_init_ram directly, > -mem-path is not supported. > > Fix this by replacing memory_region_init_ram with > memory_region_allocate_system_memory. > > Signed-off-by: Dirk Mueller > --- > hw/m68k/an5206.c | 3 +-- > hw/m68k/dummy_m68k.c | 3 ++- > hw/m68k/mcf5208.c | 3 +-- > 3 files changed, 4 insertions(+), 5 deletions(-) > > diff --git a/hw/m68k/an5206.c b/hw/m68k/an5206.c > index f1f1350..f63ab2b 100644 > --- a/hw/m68k/an5206.c > +++ b/hw/m68k/an5206.c > @@ -50,8 +50,7 @@ static void an5206_init(MachineState *machine) > env->rambar0 = AN5206_RAMBAR_ADDR | 1; > > /* DRAM at address zero */ > - memory_region_init_ram(ram, NULL, "an5206.ram", ram_size, &error_abort); > - vmstate_register_ram_global(ram); > + memory_region_allocate_system_memory(ram, NULL, "an5206.ram", ram_size); > memory_region_add_subregion(address_space_mem, 0, ram); > > /* Internal SRAM. */ > diff --git a/hw/m68k/dummy_m68k.c b/hw/m68k/dummy_m68k.c > index 278f4c0..6e5fa50 100644 > --- a/hw/m68k/dummy_m68k.c > +++ b/hw/m68k/dummy_m68k.c > @@ -42,7 +42,8 @@ static void dummy_m68k_init(MachineState *machine) > env->vbr = 0; > > /* RAM at address zero */ > - memory_region_init_ram(ram, NULL, "dummy_m68k.ram", ram_size, > &error_abort); > + memory_region_allocate_system_memory(ram, NULL, "dummy_m68k.ram", > + ram_size); > vmstate_register_ram_global(ram); You need to remove vmstate_register_ram_global too. Paolo > memory_region_add_subregion(address_space_mem, 0, ram); > > diff --git a/hw/m68k/mcf5208.c b/hw/m68k/mcf5208.c > index a01a445..326a42d 100644 > --- a/hw/m68k/mcf5208.c > +++ b/hw/m68k/mcf5208.c > @@ -218,8 +218,7 @@ static void mcf5208evb_init(MachineState *machine) > /* TODO: Configure BARs. */ > > /* DRAM at 0x40000000 */ > - memory_region_init_ram(ram, NULL, "mcf5208.ram", ram_size, &error_abort); > - vmstate_register_ram_global(ram); > + memory_region_allocate_system_memory(ram, NULL, "mcf5208.ram", ram_size); > memory_region_add_subregion(address_space_mem, 0x40000000, ram); > > /* Internal SRAM. */ >