From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dario Faggioli Subject: Re: [PATCH 3/5] x86: widen NUMA nodes to be allocated from Date: Fri, 27 Feb 2015 13:55:38 +0000 Message-ID: <1425045335.10194.76.camel@citrix.com> References: <54EF315902000078000640FF@mail.emea.novell.com> <54EF33A60200007800064144@mail.emea.novell.com> <1425043639.10194.67.camel@citrix.com> <54F0734F.4020802@linaro.org> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0130331229470659552==" Return-path: Received: from mail6.bemta3.messagelabs.com ([195.245.230.39]) by lists.xen.org with esmtp (Exim 4.72) (envelope-from ) id 1YRLOH-0006iQ-Jw for xen-devel@lists.xenproject.org; Fri, 27 Feb 2015 13:55:41 +0000 In-Reply-To: <54F0734F.4020802@linaro.org> Content-Language: en-US List-Unsubscribe: , List-Post: List-Help: List-Subscribe: , Sender: xen-devel-bounces@lists.xen.org Errors-To: xen-devel-bounces@lists.xen.org To: "julien.grall@linaro.org" Cc: "Keir (Xen.org)" , Ian Campbell , Andrew Cooper , Julien Grall , Stefano Stabellini , "JBeulich@suse.com" , "xen-devel@lists.xenproject.org" List-Id: xen-devel@lists.xenproject.org --===============0130331229470659552== Content-Language: en-US Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="=-m4AGnbtRQDSYAhQNrJ8K" --=-m4AGnbtRQDSYAhQNrJ8K Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Fri, 2015-02-27 at 13:38 +0000, Julien Grall wrote: > >> Signed-off-by: Jan Beulich > >> > > Reviewed-by: Dario Faggioli > >=20 > > One question (a genuine one, i.e., I'm really not sure what I'm saying > > is correct). > >=20 > > After this series, vcpu_to_node() (defined in xen/include/xen/numa.h) i= s > > left with only one use, in xen/arch/arm/domain.c, besides of course > > being used to implement domain_to_node() (still in > > xen/include/xen/numa.h). > >=20 > > So, provided ARM people (and I'm Cc-ing them) can get rid of that, can > > that macro be removed all together, and domain_to_node(d) be defined > > after d->node_affinity... something like: > Given the changes made by Jan on x86, I think we could replace > vcpu_to_node by MEMF_no_owner. >=20 I expected this to be the case. Happy to hear it is! :-) > FWIW, we don't have any NUMA support on ARM currently. >=20 I know. Thanks and Regards, Dario --=-m4AGnbtRQDSYAhQNrJ8K 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 iEYEABECAAYFAlTwd1cACgkQk4XaBE3IOsQWfQCfX3OtBcRSgdF5NWNZNT7W75I6 K9EAoKV0dSTmI9aDwhgVyo+0q/Ckk6Xy =mgjF -----END PGP SIGNATURE----- --=-m4AGnbtRQDSYAhQNrJ8K-- --===============0130331229470659552== 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 --===============0130331229470659552==--