From: David Gibson <david@gibson.dropbear.id.au>
To: David Gibson <dgibson@redhat.com>
Cc: Igor Mammedov <imammedo@redhat.com>,
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: Mon, 27 Jun 2016 13:16:28 +1000 [thread overview]
Message-ID: <20160627031628.GD4242@voom.fritz.box> (raw)
In-Reply-To: <20160624153851.1d3f8d08@voom.fritz.box>
[-- Attachment #1: Type: text/plain, Size: 1719 bytes --]
On Fri, Jun 24, 2016 at 03:38:51PM +1000, David Gibson wrote:
> 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.
Which it looks like we've done. I've merged both these into my
ppc-for-2.7 tree, and I'll try to send a push request before the end
of my day.
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
prev parent reply other threads:[~2016-06-27 3:42 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
2016-06-27 3:16 ` David Gibson [this message]
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=20160627031628.GD4242@voom.fritz.box \
--to=david@gibson.dropbear.id.au \
--cc=bharata@linux.vnet.ibm.com \
--cc=dgibson@redhat.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.