From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:36718) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1aLia2-0006Wo-0k for qemu-devel@nongnu.org; Tue, 19 Jan 2016 21:33:06 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1aLiZx-000325-Oh for qemu-devel@nongnu.org; Tue, 19 Jan 2016 21:33:05 -0500 Date: Wed, 20 Jan 2016 12:59:34 +1100 From: David Gibson Message-ID: <20160120015934.GG27454@voom.redhat.com> References: <1453174759-22837-1-git-send-email-david@gibson.dropbear.id.au> <87oachep1q.fsf@blackfin.pond.sub.org> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="FwyhczKCDPOVeYh6" Content-Disposition: inline In-Reply-To: <87oachep1q.fsf@blackfin.pond.sub.org> Subject: Re: [Qemu-devel] [PATCHv4 0/8] Cleanups to error reporting on ppc and spapr List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Markus Armbruster Cc: lvivier@redhat.com, thuth@redhat.com, aik@ozlabs.ru, mdroth@linux.vnet.ibm.com, qemu-devel@nongnu.org, qemu-ppc@nongnu.org --FwyhczKCDPOVeYh6 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Jan 19, 2016 at 01:23:13PM +0100, Markus Armbruster wrote: > David Gibson writes: >=20 > > Another spin of my patches to clean up a bunch of error reporting in > > the pseries machine type and target-ppc code, to better use the error > > API. > > > > Once reviewed, I hope to merge this into ppc-for-2.6 shortly. >=20 > Thanks for helping with error cleanup. >=20 > > Changes in v4: > > * Corrected a place I'd accidentally messed up the indenting > > * Dropped the patch changing the htab allocation path and some hunks > > of the migration path cleanups, since they will be obsoleted by > > other things I'm working on > > Changes in v3: > > * Adjusted a commit message for accuracy (suggest by Markus) > > * Dropped a patch which relied on a wrong guess about the behaviour > > of foreach_dynamic_sysbus_device(). > > Changes in v2: > > * Assorted minor tweaks based on review > > =20 > > David Gibson (8): > > ppc: Cleanup error handling in ppc_set_compat() > > pseries: Cleanup error handling of spapr_cpu_init() > > pseries: Clean up error handling in spapr_validate_node_memory() > > pseries: Cleanup error handling in spapr_vga_init() > > pseries: Clean up error handling in spapr_rtas_register() > > pseries: Clean up error handling in xics_system_init() > > pseries: Clean up error reporting in ppc_spapr_init() > > pseries: Clean up error reporting in htab migration functions >=20 > I think the two "Cleanup" should be spelled "Clean up". >=20 > Series > Reviewed-by: Markus Armbruster Thanks I'll make that trivial change and fold into ppc-for-2.6. --=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 --FwyhczKCDPOVeYh6 Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJWnuoGAAoJEGw4ysog2bOSR/4QAKfF5iygdzUnoN2Wjk5zPK9u TrwpfGydouyNNKZhPW/FfDZY1VXlqIWC3wJdg9jaP1fDCe38fekH/KAJ8z+bw9N7 GnKT07Zhh2H8pd9zy6LPbVZNCA5qM/jrdsvWVOtvoc42RZRVLTCOHydRsdPkubcp OUv/EiIehZSXHXwf8ny7W69JR+o/coKS5TPxBaWviqvt6tl99EMdecr9jpztF8YM GKv2WmmLedNhUjoV3WPm9dnAxAtpHfzxZu2ii0R42tw4Ti4BS+FQJWsNZLFOE3Nz aKaAcURZTYa4wBe3aUnf8JlCGeNdGbJ8vZbf6ma4c2Rf0jWY4uiXwA505RSLXhCl 2ivWh0b4N9t9/FEKU53KKyP8Px2mGX7l80EiZ+jPLdUHbZUvg0MIu0c+BNtpmp1B wCmI4YjADDQXc6AXSfEBvQwBeHA2TeWVX5U9B8o/iosqTz4pd3KsW+OPGiWfV8lM FQZvqvQkNNg0FsxTT1t1sA7AVEumBEpyPk2RIiZCw6nOzpJqxzb0WhISKwBSqohC Hge2niseV1s5FXLqr9bKR583PVGrO/TlCSnRht0QRRqys2Sz7YFuJzZYI5pu2wt7 fHNK8ic7x60ik5jA90TexLUZivXw4SfyOjKzkV90uMkWTHoEpTQ2yNLW864HW24m f5nv5Y3+835x2d8/oJar =gtn+ -----END PGP SIGNATURE----- --FwyhczKCDPOVeYh6--