From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56671) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1bzDAo-0003U3-MX for qemu-devel@nongnu.org; Tue, 25 Oct 2016 21:38:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1bzDAn-0006M9-5o for qemu-devel@nongnu.org; Tue, 25 Oct 2016 21:38:34 -0400 Date: Wed, 26 Oct 2016 11:02:49 +1100 From: David Gibson Message-ID: <20161026000249.GJ11052@umbus.fritz.box> References: <1477370856-8940-1-git-send-email-mdroth@linux.vnet.ibm.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="vDuE8pp2+RlIQOrG" Content-Disposition: inline In-Reply-To: <1477370856-8940-1-git-send-email-mdroth@linux.vnet.ibm.com> Subject: Re: [Qemu-devel] [PATCH 00/10] spapr: option vector re-work and memory unplug support List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Roth Cc: qemu-devel@nongnu.org, nfont@linux.vnet.ibm.com, qemu-ppc@nongnu.org, jallen@linux.vnet.ibm.com, bharata@linux.vnet.ibm.com --vDuE8pp2+RlIQOrG Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Oct 24, 2016 at 11:47:26PM -0500, Michael Roth wrote: > This series is based on David's ppc-for-2.8 branch, and is also available= from: >=20 > https://github.com/mdroth/qemu/commits/spapr-hotplug-event-update >=20 > Changes since RFC: > * Submit as v1 now that PAPR Hotplug ACR is accepted > * Rebase on latest ppc-for-2.8 (with device-tree refactoring) > * address Patchew warnings > * add comments to clarify spapr->ov5/ov5_cas usage. (David) > * revise comment to clarify intent when setting spapr->ov5 > OV5_HP_EVT bit. (Bharata) > * drop internal usage of spapr_ovec_from_bitmap() in favor of > directly assigning bitmap to sPAPROptionVector instances. (David) > * standardize meaning of 'vector_len' variable through spapr_ovec_* > functions to be the byte-wise length of option vectors entries, > and not including the preceeding length byte itself. (David) > * fix spapr_ovec_populate_dt() to parse up to OV_MAXBITS bits > rather than OV_MAXBITS - 1. (David) > * fix spapr_ovec_populate_dt() encode the minimum of 1 option > vector byte instead of the max of OV_MAXBYTES in cases where > no option bits are set. (David) > * add some comments to spapr_ovec_populate_dt() to clarify what > is being encoded into length byte of ibm,architecture-vec-5 > * switch 'legacy-hotplug-events' option to > 'modern-hotplug-events' (David) > * modify rtas_event_log_to_source() to check for OV5_HP_EVT > option rather than relying on whether the hotplug source is > specifically enabled. Assert the latter in cases where > OV5_HP_EVT is set. (Bharata) > * drop global EventSource list in favor of an sPAPREventSource > list field within sPAPRMachineState (David) > * add CPU unplug hook in mc->unplug_request (Bharata) >=20 >=20 > Patches 1-4 address various deficiencies in how we currently handle option > vectors via ibm,client-architecture-support. This is done here in prepara= tion > for a new option vector bit introduced later in this series, as well as a > number of future option vector bits related to other features, but I can > break this out into a separate series if preferred. I've now merged these 4 (adjusting for a couple of tiny nits mentioned in comments). --=20 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 --vDuE8pp2+RlIQOrG Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iQIcBAEBCAAGBQJYD/KpAAoJEGw4ysog2bOSmbgQAL3OrwNXSRegOgjPothxLorO UWgue/GPi0AWOqhY1sbJY8qWlanSmtJV9gRyvHmlRDZeogKtaH5h+DKoMN5N3RyC c9OakC/NlcQ2HAT0/0b+ZHiVfgMAQAGMqKEo+pfKmSBJjsckjAGVbwuUuJUhtHmR w1ZIvCYvomGYnKycx6frKOegvhWg7ZvzVL0TWL4jEVs+AY1W+8AgofM40oiU/rU5 +uTFSLvCcxdqaOFXeV2obMtmTVT0e9T6yVz4B3YqSaykRHDU1Yz6NOaigkYCUkKl j16tHofcoQAUBqVWiUs8OsmQlrf/hbGcWTmbJD6Q+AoZmjy7OpV/TgNcDR0Tu2mE B376gAmI6bVflden6Dpq89i7K/WIIujsgQpV+2bDERnol5NXlYGGVpqsqzIqMOdr RyDbqIeINOYSgzC5b8vzF6WEB3EFlKtTmQ6EAIicaz8oj4buUdgiBOxcGX8TJySl W3k9CYzTmszKmaIEh/XIxOwRUlAraQYSmzDIEvW3ZZlWqAr1tIkFapU3euAyNKQH DO7T2KpdoaU0TPfM5MTNIrRr3wGCJnxe2CdU/IeL0uQoMT99oKuLqB/gtieIPz2w M/dlAoUSembxv1L48W3cMrgbBhqPJmR07d4imRcXdY3FPRruzFIjEl7e3JRNr0Is zCCf3iMXkjM6IlviQcIC =ILBs -----END PGP SIGNATURE----- --vDuE8pp2+RlIQOrG--