On Thu, 2018-09-06 at 15:01 +0100, Andrew Cooper wrote: > alloc_vcpu()'s call to domain_update_node_affinity() has existed for > a decade, > but its effort is mostly wasted. > > alloc_vcpu() is called in a loop for each vcpu, bringing them into > existence. > The values of the affinity masks are still default, which is allcpus > in > general, or a processor singleton for pinned domains. > > Furthermore, domain_update_node_affinity() itself loops over all > vcpus > accumulating the masks, making it a scalability concern with large > numbers of > vcpus. > > Move it to be called once after all vcpus are constructed, which has > the same > net effect, but with fewer intermediate memory allocations and less > cpumask > arithmetic. > > Signed-off-by: Andrew Cooper <andrew.coop...@citrix.com> > Reviewed-by: Dario Faggioli <dfaggi...@suse.com>
Regards, Dario -- <<This happens because I choose it to happen!>> (Raistlin Majere) ----------------------------------------------------------------- Dario Faggioli, Ph.D, http://about.me/dario.faggioli Software Engineer @ SUSE https://www.suse.com/
signature.asc
Description: This is a digitally signed message part
_______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
