From: David Gibson <dgibson@redhat.com>
To: Igor Mammedov <imammedo@redhat.com>
Cc: Peter Krempa <pkrempa@redhat.com>,
qemu-devel@nongnu.org, Bharata Rao <bharata@linux.vnet.ibm.com>
Subject: Re: [Qemu-devel] [PATCH v2 0/2] qapi: Fix up cpu hotplug property names and add witness for cpu hotplug support
Date: Fri, 24 Jun 2016 15:38:51 +1000 [thread overview]
Message-ID: <20160624153851.1d3f8d08@voom.fritz.box> (raw)
In-Reply-To: <20160624073139.202c1f33@igors-macbook-pro.local>
[-- Attachment #1: Type: text/plain, Size: 1338 bytes --]
On Fri, 24 Jun 2016 07:31:39 +0200
Igor Mammedov <imammedo@redhat.com> wrote:
> On Fri, 24 Jun 2016 13:00:56 +1000
> David Gibson <dgibson@redhat.com> wrote:
>
> > On Thu, 23 Jun 2016 23:23:32 +0200
> > Peter Krempa <pkrempa@redhat.com> wrote:
> >
> > > Version 2:
> > > - fix typos/incompetence/drowsiness based language errors in commit
> > > message
> > > - select version 1 as prefered way
> > > - add -id suffix to all members of CpuInstanceProperties
> > > - note in qapi-schema the need to keep members in sync
> > > - fix output text field names in HMP impl
> > >
> > > Peter Krempa (2):
> > > qapi: Report support for -device cpu hotplug in query-machines
> > > qapi: keep names in 'CpuInstanceProperties' in sync with struct
> > > CPUCore
> >
> > Adding Bharata to CC.
> >
> > Igor, should I take these through my tree?
> Yep, please do so.
> It' would be even better if it's merged into master ASAP
> (for libvirt and for x86 device_add series that introduces these
> properties for x86 CPUs)
Yes, that's my intention. Just want to check what the final word is on
whether we need the extra witness, or whether checking what's returned
from query-hotpluggable-cpus is sufficient.
--
David Gibson <dgibson@redhat.com>
Senior Software Engineer, Virtualization, Red Hat
[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
next prev parent reply other threads:[~2016-06-24 5:37 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-06-23 21:23 [Qemu-devel] [PATCH v2 0/2] qapi: Fix up cpu hotplug property names and add witness for cpu hotplug support Peter Krempa
2016-06-23 21:23 ` [Qemu-devel] [PATCH v2 1/2] qapi: Report support for -device cpu hotplug in query-machines Peter Krempa
2016-06-24 6:24 ` Igor Mammedov
2016-06-23 21:23 ` [Qemu-devel] [PATCH v2 2/2] qapi: keep names in 'CpuInstanceProperties' in sync with struct CPUCore Peter Krempa
2016-06-23 22:03 ` Eric Blake
2016-06-24 5:32 ` Igor Mammedov
2016-06-24 3:00 ` [Qemu-devel] [PATCH v2 0/2] qapi: Fix up cpu hotplug property names and add witness for cpu hotplug support David Gibson
2016-06-24 5:31 ` Igor Mammedov
2016-06-24 5:38 ` David Gibson [this message]
2016-06-27 3:16 ` David Gibson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160624153851.1d3f8d08@voom.fritz.box \
--to=dgibson@redhat.com \
--cc=bharata@linux.vnet.ibm.com \
--cc=imammedo@redhat.com \
--cc=pkrempa@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).