From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH v10 05/11] libxl: Change default for b_info->{cpu, node}map to "not allocated" Date: Fri, 27 Jun 2014 14:07:05 +0200 Message-ID: <1403870825.8515.4.camel@Solace> References: <20140620161751.450.15846.stgit@Solace> <20140620161929.450.5352.stgit@Solace> <1403865850.25894.20.camel@kazak.uk.xensource.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============6854363265677561049==" Return-path: In-Reply-To: <1403865850.25894.20.camel@kazak.uk.xensource.com> 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: Andrew.Cooper3@citrix.com, xen-devel@lists.xen.org, Wei Liu , Ian.Jackson@citrix.com List-Id: xen-devel@lists.xenproject.org --===============6854363265677561049== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-Yu3ZlWRM1RoQIdHkG4++" --=-Yu3ZlWRM1RoQIdHkG4++ Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On ven, 2014-06-27 at 11:44 +0100, Ian Campbell wrote: > On Fri, 2014-06-20 at 18:19 +0200, Dario Faggioli wrote: > > by avoiding allocating them in libxl__domain_build_info_setdefault. > > In fact, back in 7e449837 ("libxl: provide _init and _setdefault for > > libxl_domain_build_info") and a5d30c23 ("libxl: allow for explicitly > > specifying node-affinity"), it was decided that the default for these > > fields was for them to be allocted and filled. >=20 > "allocated" >=20 > > Signed-off-by: Dario Faggioli >=20 > Acked-by: Ian Campbell >=20 > > + * updated accordingly; if it does not manage, inf->nodemap is jus= t left >=20 > info-> ? >=20 Indeed. And I'll fix these and resend, since it looks like I need to :-) > > + * alone. It is then the the subsequent call to > > + * libxl_domain_set_nodeaffinity() that enacts the actual placemen= t. > > */ > > if (libxl_defbool_val(info->numa_placement)) { > > - if (!libxl_bitmap_is_full(&info->cpumap)) { > > + if (info->cpumap.size) { >=20 > Not for now but a helper for this might be nice on day. >=20 Noted. > > LOG(ERROR, "Can run NUMA placement only if no vcpu " > > - "affinity is specified"); > > + "affinity is specified explicitly"); >=20 > OOI, this says "Can run", I think what it really means is "I will run"? > No change needed just curious. >=20 "Can run" is a bad choice of words. The idea is that, if the user specified any of those fields, NUMA node affinity will be based on that, so we don't want automatic placement to take place. So, yes, something like "NUMA placement won't run" or similar would have been more appropriate. Anyway, I've got an unrelated patch touching this code, which I'll submit as soon as this one will be in. If you're fine, I'll leave it alone in the repost of this series, and amend it there. > I'll fix the two typos on commit, assuming nothing in the remainder of > the series requires a resend. >=20 As said, I'm resending, so I'll also take care of the typos. Thanks, Dario --=20 <> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Senior Software Engineer, Citrix Systems R&D Ltd., Cambridge (UK) --=-Yu3ZlWRM1RoQIdHkG4++ 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 v2 iEYEABECAAYFAlOtXmkACgkQk4XaBE3IOsTbAgCeKexf4ZCj5vcy3Xth50Q+1C1q a2QAmgO3dcaRDEGLHmyaA3ZR6NYByMVn =Fn+5 -----END PGP SIGNATURE----- --=-Yu3ZlWRM1RoQIdHkG4++-- --===============6854363265677561049== 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 --===============6854363265677561049==--