Re: [libvirt] [RFC PATCH v1 1/4] numa: describe siblings distances within cells

On Wed, 21 Jun 2017 21:08:30 +0200 Wim ten Have <wim.ten.have(a)oracle.com&gt; wrote: > On Mon, 19 Jun 2017 12:26:20 -0600 > Jim Fehlig <jfehlig(a)suse.com&gt; wrote: > > > On 06/12/2017 12:54 PM, Wim Ten Have wrote: > > > From: Wim ten Have <wim.ten.have(a)oracle.com&gt; > > > > > > Add libvirtd NUMA cell domain administration functionality to > > > describe underlying cell id sibling distances in full fashion > > > when configuring HVM guests. > > > > Thanks for the detailed cover letter :-). Some of that information (e.g. ACPI > > spec, SLIT table, qemu support) would be useful in this commit message. > > I'll follow up on that under v2. > > > > [below is an example of a 4 node setup] > > > > > > <cpu> > > > <numa> > > > <cell id='0' cpus='0' memory='2097152' unit='KiB'> > > > <distances> > > > <sibling id='0' value='10'/> > > > <sibling id='1' value='21'/> > > > <sibling id='2' value='31'/> > > > <sibling id='3' value='41'/> > > > </distances> > > > </cell> > > > > Others on the list are better XML designers, but the following seems to achieve > > the same and is a bit more compact > > > > <cell id='0' cpus='0' memory='2097152' unit='KiB'> > > <sibling id='0' distance='10'/> > > <sibling id='1' distance='21'/> > > <sibling id='2' distance='31'/> > > <sibling id='3' distance='41'/> > > </cell> > > > > CC danpb, who often has good ideas on schema design. > > Will give Daniel and (others) time prior submitting v2. Otherwise > follow your suggestion above for its compact and clear annotation. One thing on specific, forgot to mention previous reply, is that the selected syntax was derived from way libvirt describes host capabilities introduced under; commit 8ba0a58f8d9db9eeed003b889dfcd3451d10fbca Author: Michal Privoznik <mprivozn(a)redhat.com&gt; Date: Tue Jun 3 15:18:27 2014 +0200 "virCaps: Expose distance between host NUMA nodes"