Thanks Peter. Let's try, can it works on Phi platform firstly. BR Shaohe. > -----Original Message----- > From: Peter Krempa [mailto:pkrempa@xxxxxxxxxx] > Sent: Tuesday, April 25, 2017 4:33 PM > To: Feng, Shaohe <shaohe.feng@xxxxxxxxx> > Cc: libvir-list@xxxxxxxxxx; Du, Dolpher <dolpher.du@xxxxxxxxx>; Zyskowski, Robert <robert.zyskowski@xxxxxxxxx>; > Uminski, Piotr <Piotr.Uminski@xxxxxxxxx>; Daniluk, Lukasz <lukasz.daniluk@xxxxxxxxx>; Zang, Rui <rui.zang@xxxxxxxxx>; > Chen, He <he.chen@xxxxxxxxx>; jdenemar@xxxxxxxxxx > Subject: Re: [PATCH] Support CPU less NUMA node. > > On Thu, Apr 20, 2017 at 19:22:49 +0800, Feng, Shaohe wrote: > > Thanks Peter for the comments. > > Will update it in the next version. > > > > On 2017年04月20日 19:17, Peter Krempa wrote: > > > On Thu, Apr 20, 2017 at 03:49:29 +0000, Shaohe Feng wrote: > > > > Some platform NUMA node does not include cpu, such as Phi. > > > > > > > > This patch support CPU less NUMA node. > > > > But there must be one CPU cell node for a guest. > > > > Also must assign the host nodeset for this guest cell node. > > > Additionally to the review below: > > > > > > Did qemu always support this? If not, you need to add code which > > > will allow such configuration only if qemu supports this (adding a > > > capability bit etc ...). > > need to check it. > > > Also please elaborat how is this better than adding a regular numa > > > node whith cpus which are disabled. In such case you can still add > > > more cpus via the hotplug API in case you desire so. > > do you means add an extra attribute "disable" for cpus? > > such as > > > > <cell id='1' cpus="" cpu_disable="true" "memory='512000' unit='KiB'/> > > Why would you do that? You can specify cpus="some-sane-cpu-set" and then using the <vcpus> element disable the given > set. > > See http://libvirt.org/formatdomain.html#elementsCPUAllocation -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
