[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] qapi/misc.json: Clarify about usage of QMP 'cpu

From: Kashyap Chamarthy
Subject: Re: [Qemu-devel] [PATCH] qapi/misc.json: Clarify about usage of QMP 'cpu-add'
Date: Thu, 20 Sep 2018 13:03:32 +0200
User-agent: Mutt/1.10.1 (2018-07-13)

On Wed, Sep 19, 2018 at 01:46:37PM -0300, Eduardo Habkost wrote:
> On Wed, Sep 19, 2018 at 12:22:19PM +0200, Kashyap Chamarthy wrote:
> > On Tue, Sep 18, 2018 at 11:19:38AM -0300, Eduardo Habkost wrote:


> > Noted; shall I send a reworded patch to say: "This command will be
> > deprecated in the near future"?
> I'd prefer a patch (against this file and qemu-deprecated.texi)
> saying "this command is deprecated".

Okay, I'll send a new patch (with commit summary line: "Deprecate QMP

> I don't see a reason to not deprecate the command immediately
> (instead of just promising that it will be deprecated).

Yeah, noted.


> > Yeah, to be clear, would you like me to amend the text in a different
> > phrasing?  Or is the current phrasing alright?  Maybe needs an
> > additional sentence about: "Refer the documentation of
> > 'query-hotpluggable-cpus'"?
> I'd prefer a more succinct phrasing.  e.g.:
>   "This command is deprecated.  The `device_add` command should
>   be used instead.  See the `query-hotpluggable-cpus` command for
>   details."


> The main problem is that the details I expected to see on the
> documentation of `query-hotpluggable-cpus` aren't there.
> But before writing `query-hotpluggable-cpus` documentation from
> scratch, I would like to find the answer to this:
> > 
> > > Igor, is the QMP documentation for query-hotpluggable-cpus the
> > > only documentation we have on CPU hotplug?

Yes, I'd also like to know the answer to that.


reply via email to

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