[Top][All Lists]

[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


> How are multiple optional parameters handled? Filled from
> left-to-right?

HMP: yes.

QMP has only named parameters.

reply via email to

[Prev in Thread] Current Thread [Next in Thread]