[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 38/80] s390x/s390-virtio-ccw: use memdev for RAM
From: |
Igor Mammedov |
Subject: |
Re: [PATCH v4 38/80] s390x/s390-virtio-ccw: use memdev for RAM |
Date: |
Thu, 6 Feb 2020 14:15:53 +0100 |
On Wed, 5 Feb 2020 21:11:23 +0100
Halil Pasic <address@hidden> wrote:
> On Fri, 31 Jan 2020 16:09:08 +0100
> Igor Mammedov <address@hidden> wrote:
>
> > memory_region_allocate_system_memory() API is going away, so
> > replace it with memdev allocated MemoryRegion. The later is
> > initialized by generic code, so board only needs to opt in
> > to memdev scheme by providing
> > MachineClass::default_ram_id
> > and using MachineState::ram instead of manually initializing
> > RAM memory region.
> >
> > Signed-off-by: Igor Mammedov <address@hidden>
> > Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> > Reviewed-by: David Hildenbrand <address@hidden>
> > Acked-by: Cornelia Huck <address@hidden>
>
> Hi Igor!
>
> I gave
> -object memory-backend-file,id=mem
> -machine type=s390-ccw-virtio,memory-backend=mem
> a spin on s390x. Seems to largely work a expected. So I guess it is:
>
> Tested-by: Halil Pasic <address@hidden>
> Acked-by: Halil Pasic <address@hidden>
Thanks,
Could you also take a look at patches 3-7/8o that makes this possible?
(it never hurts to have second pair of eyes on a code that affects
everyone).
>
> Thanks!
>
> Halil
>
> > ---
> > CC: address@hidden
> > ---
> > hw/s390x/s390-virtio-ccw.c | 7 +++----
> > 1 file changed, 3 insertions(+), 4 deletions(-)
> >
> > diff --git a/hw/s390x/s390-virtio-ccw.c b/hw/s390x/s390-virtio-ccw.c
> > index e759eb5..a89cf4c 100644
> > --- a/hw/s390x/s390-virtio-ccw.c
> > +++ b/hw/s390x/s390-virtio-ccw.c
> > @@ -154,14 +154,12 @@ static void virtio_ccw_register_hcalls(void)
> > virtio_ccw_hcall_early_printk);
> > }
> >
> > -static void s390_memory_init(ram_addr_t mem_size)
> > +static void s390_memory_init(MemoryRegion *ram)
> > {
> > MemoryRegion *sysmem = get_system_memory();
> > - MemoryRegion *ram = g_new(MemoryRegion, 1);
> > Error *local_err = NULL;
> >
> > /* allocate RAM for core */
> > - memory_region_allocate_system_memory(ram, NULL, "s390.ram", mem_size);
> > memory_region_add_subregion(sysmem, 0, ram);
> >
> > /*
> > @@ -245,7 +243,7 @@ static void ccw_init(MachineState *machine)
> >
> > s390_sclp_init();
> > /* init memory + setup max page size. Required for the CPU model */
> > - s390_memory_init(machine->ram_size);
> > + s390_memory_init(machine->ram);
> >
> > /* init CPUs (incl. CPU model) early so s390_has_feature() works */
> > s390_init_cpus(machine);
> > @@ -471,6 +469,7 @@ static void ccw_machine_class_init(ObjectClass *oc,
> > void *data)
> > hc->plug = s390_machine_device_plug;
> > hc->unplug_request = s390_machine_device_unplug_request;
> > nc->nmi_monitor_handler = s390_nmi;
> > + mc->default_ram_id = "s390.ram";
> > }
> >
> > static inline bool machine_get_aes_key_wrap(Object *obj, Error **errp)
>