From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49662) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e1u6F-0004A6-WF for qemu-devel@nongnu.org; Tue, 10 Oct 2017 08:57:38 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1e1u6E-000654-Fe for qemu-devel@nongnu.org; Tue, 10 Oct 2017 08:57:32 -0400 Date: Tue, 10 Oct 2017 23:21:59 +1100 From: David Gibson Message-ID: <20171010122159.GA10496@umbus.fritz.box> References: <1507578671-158758-1-git-send-email-imammedo@redhat.com> <20171010022137.GL2668@umbus.fritz.box> <20171010133841.70f2ba74@nial.brq.redhat.com> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="FCuugMFkClbJLl1L" Content-Disposition: inline In-Reply-To: <20171010133841.70f2ba74@nial.brq.redhat.com> Subject: Re: [Qemu-devel] [PATCH v2 00/24] generalize parsing of cpu_model (part 3/PPC) List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Igor Mammedov Cc: qemu-devel@nongnu.org, Alexander Graf , =?iso-8859-1?Q?Herv=E9?= Poussineau , "Edgar E. Iglesias" , "open list:ppce500" --FCuugMFkClbJLl1L Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Tue, Oct 10, 2017 at 01:38:41PM +0200, Igor Mammedov wrote: > On Tue, 10 Oct 2017 13:21:37 +1100 > David Gibson wrote: >=20 > > On Mon, Oct 09, 2017 at 09:50:47PM +0200, Igor Mammedov wrote: > >=20 > > Applied to ppc-for-2.11. There was a small error in 19/24, which > > reverted pnv to defaulting to POWER9 v1.0 whereas I've recently > > changed it to v2.0. Rather than requiring a respin I adjusted that > > myself, and fixed the minor conflicts in the subsequent patches. > there were conflicts in spapr code but not in pnv, so I haven't > > noticed. Yeah, that's what I figured. > I've checked commit on ppc-for-2.11 branch and it looks good. > Anyways thanks for fixing it up. No problem. >=20 > >=20 > > > Changelog: > > > - v2: > > > * rebase on dgibson/ppc-for-2.11 tree > > > * fix typo: s/pvn_chip_core_typename/pnv_chip_core_typename/ > > > * fix typo: s/pvn_core_cpu_typename/pnv_core_cpu_typename/ > > > * limit setting default cpu type to 'host' type only for spapr m= achine > > > * split out ppc_cpu_lookup_alias() movement into separate patch > > > * fix rebase conflict: add newly added power9_v2 core > > > * set "max-cpu-compat" property only if it exists, > > > in practice it should limit 'compat' hack to spapr > > > and allow to avoid including machine/spapr headers > > > in target/ppc/cpu.c > > > * do compat=3D parsing only if machine."max-cpu-compat" exists > > > that would match what current code practically does. > > > * add newline before ppc_cpu_parse_featurestr() definition > > > * pickup Acked-by/Reviewed-by where it's appropriate > > >=20 > > > this series is continuation of effort to remove boards dependency on > > > cpu_model parsing and generalizing default cpu type handling. > > > For background story look at merged:=20 > > >=20 > > > [PATCH v2 0/5] generalize parsing of cpu_model (x86/arm) > > > https://lists.gnu.org/archive/html/qemu-devel/2017-09/msg03564.html > > >=20 > > > Part 3 is independent subset that takes care of all PPC boards. > > > Series borrows the 1st 3 patches from earlier posted part 2 > > > qom: update doc comment for type_register[_static]() = =20 > > > qom: introduce type_register_static_array() = =20 > > > qom: add helper macro DEFINE_TYPES() > > > to avoid interdependency, hopefully this 3 will be merged via machine= -next, > > > but it's fine for them to go via any tree (whichever first) > > >=20 > > > Beside of switching to generic cpu_model parsing series contains a bu= nch > > > of cleanups/simplifications which reduces code a bit and normalizes > > > code that I've happend to touch to a typical QEMU patterns. > > >=20 > > > git tree for testing: > > > https://github.com/imammedo/qemu/branches cpu_init_removal_ppc_part= 3_v2 > > >=20 > > > Refference to v1: > > > https://lists.gnu.org/archive/html/qemu-devel/2017-10/msg01087.html > > >=20 > > >=20 > > > CC: Alexander Graf (supporter:ppce500) > > > CC: David Gibson (maintainer:PowerPC) > > > CC: "Herv=E9 Poussineau" (maintainer:PReP) > > > CC: "Edgar E. Iglesias" (odd fixer:virtex_= ml507) > > > CC: qemu-ppc@nongnu.org (open list:ppce500) > > >=20 > > > Igor Mammedov (24): > > > qom: update doc comment for type_register[_static]() > > > qom: introduce type_register_static_array() > > > qom: add helper macro DEFINE_TYPES() > > > ppc: mpc8544ds/e500plat: use generic cpu_model parsing > > > ppc: mac_newworld: use generic cpu_model parsing > > > ppc: mac_oldworld: use generic cpu_model parsing > > > ppc: bamboo: use generic cpu_model parsing > > > ppc: replace cpu_model with cpu_type on ref405ep,taihu boards > > > ppc: virtex-ml507: replace cpu_model with cpu_type > > > ppc: 40p/prep: replace cpu_model with cpu_type > > > ppc: spapr: replace ppc_cpu_parse_features() with > > > cpu_parse_cpu_model() > > > ppc: move '-cpu foo,compat=3Dxxx' parsing into > > > ppc_cpu_parse_featurestr() > > > ppc: spapr: define core types statically > > > ppc: spapr: use cpu type name directly > > > ppc: spapr: register 'host' core type along with the rest of core > > > types > > > ppc: spapr: use cpu model names as tcg defaults instead of aliases > > > ppc: move ppc_cpu_lookup_alias() before its first user > > > ppc: spapr: use generic cpu_model parsing > > > ppc: pnv: use generic cpu_model parsing > > > ppc: pnv: normalize core/chip type names > > > ppc: pnv: drop PnvCoreClass::cpu_oc field > > > ppc: pnv: define core types statically > > > ppc: pnv: drop PnvChipClass::cpu_model field > > > ppc: pnv: consolidate type definitions and batch register them > > >=20 > > > include/hw/ppc/pnv.h | 12 +-- > > > include/hw/ppc/pnv_core.h | 4 +- > > > include/hw/ppc/ppc.h | 2 - > > > include/hw/ppc/spapr.h | 2 - > > > include/hw/ppc/spapr_cpu_core.h | 7 +- > > > include/qom/object.h | 50 ++++++++++- > > > target/ppc/cpu-qom.h | 1 + > > > target/ppc/cpu.h | 1 - > > > hw/ppc/e500.c | 8 +- > > > hw/ppc/e500plat.c | 1 + > > > hw/ppc/mac_newworld.c | 15 ++-- > > > hw/ppc/mac_oldworld.c | 6 +- > > > hw/ppc/mpc8544ds.c | 2 + > > > hw/ppc/pnv.c | 131 ++++++++++++---------------- > > > hw/ppc/pnv_core.c | 67 +++++++-------- > > > hw/ppc/ppc.c | 25 ------ > > > hw/ppc/ppc405_uc.c | 6 +- > > > hw/ppc/ppc440_bamboo.c | 7 +- > > > hw/ppc/ppc4xx_devs.c | 4 +- > > > hw/ppc/prep.c | 12 +-- > > > hw/ppc/spapr.c | 22 ++--- > > > hw/ppc/spapr_cpu_core.c | 184 ++++++++++--------------------= ---------- > > > hw/ppc/virtex_ml507.c | 11 +-- > > > qom/object.c | 9 ++ > > > target/ppc/kvm.c | 22 ++--- > > > target/ppc/translate_init.c | 69 +++++++++++++-- > > > 26 files changed, 305 insertions(+), 375 deletions(-) > > > =20 > >=20 >=20 --=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 --FCuugMFkClbJLl1L Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEdfRlhq5hpmzETofcbDjKyiDZs5IFAlncu2QACgkQbDjKyiDZ s5Iq/BAAs6Vi1aB6Ca9tG46UEZALbphFv+b64MtiabDICR6FfzjWkCT8KKsuT/U8 SDSInIFoif0qXZhpNIN/k2nIqMmCgd0zQ5EAfvde2PuW9Fkh48ivyh6XcDomasQ1 7Wyso7/5iU3cMfPVzeoF7PlUyuBodikYHvYmI9zLF+Pf7IKeO47NGXQnBK06oDMP 1ngI2Kx2nRv23d+J5FS1ayaOrLKg/QExQASwR7vpmtoMHlThx4PpdJhcTaxrJp8t wm8jLwsZtA4OQsOuGvKzpVE0Qncik5li5f6Eq/zk9UN0GYv/l2IyGw3lvDwsJz/4 G1Uib1ZxHpDHvvY+oCr8iFdODG+ftHQoxYBJkm2MmuoegQO27T3HG9857By+b0nd UuIwiaEtWkhgfeybzvxlLuPCay4Wv9LIS+J1lftHCkAxHi+XC01Rwux3pKz33tT0 pCHWnyXiCeeCYdAejUSwfXA28bU26AZAMXxj6JCP0pQXXCwCnM2ANWfg1HZxvh/5 fFwB9tqPbOSt+jYrs1CmtUUDVg8g0+UH5iaXEA/98dLhmql/3PXDGk3Qcn7f4dqY vu1vsW/BxfyVfMtFlu126BVtZTjtrjfqYlTJnWAcRUWkoW7qJTfNx+5wK/D3I4uJ B84/YxKL7xM01yI/hEZVP/h4vamrGy7zljjyBZ2DirFHwdrUGmI= =IbUP -----END PGP SIGNATURE----- --FCuugMFkClbJLl1L--