[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command |
Date: |
Thu, 16 Aug 2018 07:43:41 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux) |
Simon Ruderich <address@hidden> writes:
> On Wed, Aug 15, 2018 at 04:29:25PM +0200, Markus Armbruster wrote:
>>> For the HMP or the QMP interface?
>>
>> Both.
>
> Ok.
>
>>> If you think 'offset' is not necessary I can also drop it
>>> completely.
>>
>> I think it's a reasonable feature, and since you already coded it up...
>
> In that case, should size also become optional? As already
> suggested in this thread (and similar for QMP):
>
> On Fri, Aug 10, 2018 at 11:36:51AM +0100, Dr. David Alan Gilbert wrote:
>> Also, had you considered rearranging and making them optional,
>> for example if you do:
>>
>> val:l,filename:F,offset:i?,size:i?
>>
>> I think that would mean you can do the fairly obvious:
>> pmemload addr "myfile"
>>
>> with the assumption that loads the whole file.
>
> This would deviate from pmemsave/memsave, but feels more natural.
The different order or arguments in HMP is somewhat ugly. Okay if it
makes the command more pleasant to use. Up to you and Dave to decide.
If you decide to deviate, consider
filename:F,address:l,size:i?,offset:i?
> How are multiple optional parameters handled? Filled from
> left-to-right?
HMP: yes.
QMP has only named parameters.
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Eric Blake, 2018/08/09
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Dr. David Alan Gilbert, 2018/08/10
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/14
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Markus Armbruster, 2018/08/14
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/14
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Markus Armbruster, 2018/08/15
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/15
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Markus Armbruster, 2018/08/15
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/15
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command,
Markus Armbruster <=
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/16
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Markus Armbruster, 2018/08/16
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Simon Ruderich, 2018/08/16
- Re: [Qemu-devel] [PATCH v3 5/5] qmp: add pmemload command, Dr. David Alan Gilbert, 2018/08/14