[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command
From: |
Simon Ruderich |
Subject: |
Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command |
Date: |
Sun, 22 Apr 2018 11:47:28 +0200 |
User-agent: |
Mutt/1.9.5 (2018-04-13) |
On Tue, Apr 17, 2018 at 04:18:43PM -0500, Eric Blake wrote:
> Focusing on just the interface:
>
>> +++ b/qapi/misc.json
>> @@ -1185,6 +1185,26 @@
>> { 'command': 'pmemsave',
>> 'data': {'val': 'int', 'size': 'int', 'filename': 'str'} }
>>
>> +##
>> +# @pmemload:
>> +#
>> +# Load a portion of guest physical memory from a file.
>> +#
>> +# @val: the physical address of the guest to start from
>> +#
>> +# @size: the size of memory region to load
>
> Should size be an optional parameter (default read to the end of the file)?
>
>> +#
>> +# @offset: the offset in the file to start from
>
> Should offset be an optional parameter (default start reading from
> offset 0 of the file)?
>From the QAPI point-of-view making both optional seems like a
good idea, however then pmemsave should also mark its "size"
parameter optional for consistency (this is backwards compatible
as existing callers are not affected).
On the monitor API side this is more problematic as the order of
the pmemsave/pmemload parameters make optional parameters
impossible.
Personally I'd keep it as is because it's simple and consistent
with the existing interface (and can be changed in the future
without breaking compatibility).
In case you prefer those parameters to be optional I'll require
some help to implement that: I don't understand how
qapi-schema.json, hmp-commands.hx and hmp.h interact (I've to
admit I just copied that from the original patch and didn't give
it much thought as it worked fine) and would require some
pointers on how to implement optional arguments for
qapi-schema.json.
Regards
Simon
--
+ privacy is necessary
+ using gnupg http://gnupg.org
+ public key id: 0x92FEFDB7E44C32F9
- Re: [Qemu-devel] [PATCH] qmp: add pmemload command, (continued)
- [Qemu-devel] [PATCH v2 4/5] hmp: don't truncate size in hmp_memsave/hmp_pmemsave, Simon Ruderich, 2018/04/12
- Re: [Qemu-devel] [PATCH v2 4/5] hmp: don't truncate size in hmp_memsave/hmp_pmemsave, Eric Blake, 2018/04/17
- [Qemu-devel] [PATCH v2 3/5] cpus: use size_t in qmp_memsave/qmp_pmemsave, Simon Ruderich, 2018/04/12
- Re: [Qemu-devel] [PATCH v2 3/5] cpus: use size_t in qmp_memsave/qmp_pmemsave, Eric Blake, 2018/04/17
- [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command, Simon Ruderich, 2018/04/12
- Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command, Eric Blake, 2018/04/17
- Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command,
Simon Ruderich <=
- Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command, Eric Blake, 2018/04/23
- Re: [Qemu-devel] [PATCH v2 5/5] qmp: add pmemload command, Simon Ruderich, 2018/04/24
- Re: [Qemu-devel] [PATCH v2 1/5] cpus: correct coding style in qmp_memsave/qmp_pmemsave, Eric Blake, 2018/04/17
- Re: [Qemu-devel] [PATCH v2 1/5] cpus: correct coding style in qmp_memsave/qmp_pmemsave, Simon Ruderich, 2018/04/22