From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:58202) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1dAIlg-0001XV-2t for qemu-devel@nongnu.org; Mon, 15 May 2017 12:22:44 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1dAIla-0007MU-FU for qemu-devel@nongnu.org; Mon, 15 May 2017 12:22:44 -0400 Received: from 10.mo69.mail-out.ovh.net ([46.105.73.241]:40597) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1dAIla-0007Ld-90 for qemu-devel@nongnu.org; Mon, 15 May 2017 12:22:38 -0400 Received: from player699.ha.ovh.net (b9.ovh.net [213.186.33.59]) by mo69.mail-out.ovh.net (Postfix) with ESMTP id 9163D1D5D1 for ; Mon, 15 May 2017 18:22:36 +0200 (CEST) Date: Mon, 15 May 2017 18:22:31 +0200 From: Greg Kurz Message-ID: <20170515182231.7ce91bce@bahia> In-Reply-To: References: <149484833874.20089.4164801378197848306.stgit@bahia.lan> <149484840466.20089.893964776019028654.stgit@bahia.lan> <20170515151602.61dc381d@bahia> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; boundary="Sig_/gev_.S0e8r29+PbTu2jM1lS"; protocol="application/pgp-signature" Subject: Re: [Qemu-devel] [PATCH 6/6] spapr: fix migration of ICP objects from/to older QEMU List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?Q8OpZHJpYw==?= Le Goater Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org, Bharata B Rao , David Gibson --Sig_/gev_.S0e8r29+PbTu2jM1lS Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable On Mon, 15 May 2017 18:11:27 +0200 C=C3=A9dric Le Goater wrote: > >>> + int smt =3D kvmppc_smt_threads(); > >>> + int nr_servers =3D DIV_ROUND_UP(max_cpus * smt, smp_threads)= ; =20 > >> > >> may be we should reintroduce nr_servers at the machine level ?=20 > >> =20 > >=20 > > I had reintroduced it but then I realized it was only used in this > > function. =20 >=20 > nr_servers is also used when the device tree is populated with the=20 > interrupt controller nodes. No big deal. >=20 Ah yes you're right. Maybe this can be factored out in a followup patch. > C.=20 >=20 --Sig_/gev_.S0e8r29+PbTu2jM1lS Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 iEYEARECAAYFAlkZ1ccACgkQAvw66wEB28K0VwCeM3m6RR7itpDMYiyuHn1NbTpR fnUAn0Kl/x5KvQoUBuFylIUWvvcjOYdA =A/Tm -----END PGP SIGNATURE----- --Sig_/gev_.S0e8r29+PbTu2jM1lS--