[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 05/34] add memdev backend infrastructure
From: |
Peter Crosthwaite |
Subject: |
Re: [Qemu-devel] [PATCH v3 05/34] add memdev backend infrastructure |
Date: |
Fri, 30 May 2014 01:39:05 +1000 |
On Fri, May 30, 2014 at 1:25 AM, Paolo Bonzini <address@hidden> wrote:
> Il 29/05/2014 16:05, Peter Crosthwaite ha scritto:
>
>>> > + path = object_get_canonical_path_component(OBJECT(backend));
>>> > + memory_region_init_ram(&backend->mr, OBJECT(backend), path,
>>
>> Passing the full canonical path as the name of memory region is
>> redundant as that information is already passed via the owner
>> argument. It should just be a shorthand.
>>
>
> It's not the full canonical path, it's basically the id.
>
Fair enough. Although its still redundant though isn't it?
Should we ever clean up messages etc in memory api to include both
owner and name this is going to come out:
"foo/bar/baz:baz"
or some such.
Then again, if we are only worried about getting a sane shorthand,
perhaps what Igor has done here is a sane default for the memory API
when passed a NULL name?
Regards,
Peter
> Paolo
>
[Qemu-devel] [PATCH v3 04/34] vl.c: daemonize before guest memory allocation, Igor Mammedov, 2014/05/27
[Qemu-devel] [PATCH v3 06/34] vl.c: extend -m option to support options for memory hotplug, Igor Mammedov, 2014/05/27