From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH 1 of 3 v5/leftover] libxl: enable automatic placement of guests on NUMA nodes [and 1 more messages] Date: Wed, 18 Jul 2012 02:31:34 +0200 Message-ID: <1342571494.11794.96.camel@Abyss> References: <0411b2cebd725b193465.1341932614@Solace> <20485.35590.105351.434937@mariner.uk.xensource.com> <5fa66c8b9093399e5bc3.1342458792@Solace> <20485.43293.833036.352186@mariner.uk.xensource.com> <1342556600.18848.10.camel@dagon.hellion.org.uk> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2352579148791019438==" Return-path: In-Reply-To: <1342556600.18848.10.camel@dagon.hellion.org.uk> List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: Ian Campbell Cc: Andre Przywara , Stefano Stabellini , George Dunlap , Juergen Gross , Ian Jackson , xen-devel List-Id: xen-devel@lists.xenproject.org --===============2352579148791019438== Content-Type: multipart/signed; micalg="pgp-sha1"; protocol="application/pgp-signature"; boundary="=-Tnquod/P9KVlz9RqrZWZ" --=-Tnquod/P9KVlz9RqrZWZ Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, 2012-07-17 at 21:23 +0100, Ian Campbell wrote:=20 > > I've been told that boxes with 16 NUMA nodes are coming out about > > now. > >=20 > > This algorithm will start to have unreasonable performance with > > anything bigger than that. Eg choosing 16 NUMA nodes out of 32 would > > involve searching 601,080,390 combinations. 32 out of 64 gives 10^18. > >=20 > > So I think we need a different approach :-(. >=20 > If 16 are coming out now then 32 nodes are at least a little in the > future, and 16 won't be common just yet. >=20 FWIW, I agree. Consider that 16 nodes systems exists right now only because AMD has an architecture where they're putting 2 nodes on one socket, which is something pretty special that might require some special thinking and consideration already. The point being assuming future 16, 32 and 64 nodes systems to just be like the machines we see now, only bigger, won't be something letting me sleep at night... :-D > The 4.2 freeze is starting to drag on and reworking this significantly > at this stage seems likely to take quite a while -- can we perhaps go > with the existing patches (with the known scalability limitation and > workaround of setting the explicit cpu affinity documented in the > release notes) and revisit for 4.3 and/or 4.2.1? >=20 That would be fine by me. I already said in the other e-mail how I think we can and should address the potential scalability issue, at whatever time you think it's best. I'm fine with putting a limit on the maximum number of nodes a guest should span on (and that being more a feature than a workaround, and can definitely go in the release notes) wherever you like. If it's just a matter of putting a LIBXL__MAX_GUEST_NODES (or whatever) macro in libxl_internal.h, and enforce that, I'm fine doing it right now, as I'm reposting these last three patches either way. We just need to agree on a value to write besides it. :-) Thanks and Regards, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://retis.sssup.it/people/faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-Tnquod/P9KVlz9RqrZWZ Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (GNU/Linux) iEYEABECAAYFAlAGA+YACgkQk4XaBE3IOsS1qwCdGqbqJVM5w1+7sRmTsYbJqliG ZUcAn0l1pNtvqILs1X59WJ2RcNvhF2cK =c25Z -----END PGP SIGNATURE----- --=-Tnquod/P9KVlz9RqrZWZ-- --===============2352579148791019438== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel --===============2352579148791019438==--