Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-21 Thread Igor Mammedov
On Mon, 21 Aug 2017 19:23:35 +0800 Dou Liyang wrote: > Hi, Igor > > [...] > >> > >> Ok, I will do it right now. > > > > see commit 6b9c1dd2c for example of adding test case, > > currently test has base SRAT tables that is reused by all tests. > > When you create

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-21 Thread Dou Liyang
Hi, Igor [...] Ok, I will do it right now. see commit 6b9c1dd2c for example of adding test case, currently test has base SRAT tables that is reused by all tests. When you create testcase that produces different SRAT table, 'make check' will print warnings that ables do not match you can add

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-21 Thread Igor Mammedov
On Sun, 20 Aug 2017 19:52:11 +0800 Dou Liyang wrote: > Hi Eduardo, > > At 08/19/2017 12:48 AM, Eduardo Habkost wrote: > > On Thu, Aug 17, 2017 at 01:58:40PM +0800, Dou Liyang wrote: > >> Hi Igor, > >> > >> I tested this patch with following guests: > >> > >> 1. RHEL

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-20 Thread Dou Liyang
Hi Eduardo, At 08/19/2017 12:48 AM, Eduardo Habkost wrote: On Thu, Aug 17, 2017 at 01:58:40PM +0800, Dou Liyang wrote: Hi Igor, I tested this patch with following guests: 1. RHEL 6.5 with Linux 2.6.32 2. RHEL 7.0 with Linux 3.10.0 3. Fedora 23 with Linux 4.13.0-rc5 4. window 2003 service 5.

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-18 Thread Eduardo Habkost
On Fri, Aug 18, 2017 at 09:35:00PM +0300, Michael S. Tsirkin wrote: > On Wed, Aug 16, 2017 at 09:45:35AM +0800, Dou Liyang wrote: > > Currently, Using the fisrt node without memory on the machine makes > > QEMU unhappy. With this example command line: > > ... \ > > -m 1024M,slots=4,maxmem=32G

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-18 Thread Michael S. Tsirkin
On Wed, Aug 16, 2017 at 09:45:35AM +0800, Dou Liyang wrote: > Currently, Using the fisrt node without memory on the machine makes > QEMU unhappy. With this example command line: > ... \ > -m 1024M,slots=4,maxmem=32G \ > -numa node,nodeid=0 \ > -numa node,mem=1024M,nodeid=1 \ > -numa

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-18 Thread Eduardo Habkost
On Thu, Aug 17, 2017 at 01:58:40PM +0800, Dou Liyang wrote: > Hi Igor, > > I tested this patch with following guests: > > 1. RHEL 6.5 with Linux 2.6.32 > 2. RHEL 7.0 with Linux 3.10.0 > 3. Fedora 23 with Linux 4.13.0-rc5 > 4. window 2003 service > 5. window 7 > 6. window 10 What's the

Re: [Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-16 Thread Dou Liyang
Hi Igor, I tested this patch with following guests: 1. RHEL 6.5 with Linux 2.6.32 2. RHEL 7.0 with Linux 3.10.0 3. Fedora 23 with Linux 4.13.0-rc5 4. window 2003 service 5. window 7 6. window 10 Thanks, dou. At 08/16/2017 09:45 AM, Dou Liyang wrote: Currently, Using the fisrt node

[Qemu-devel] [PATCH v3] hw/acpi-build: Fix SRAT memory building when there is no memory in node0

2017-08-15 Thread Dou Liyang
Currently, Using the fisrt node without memory on the machine makes QEMU unhappy. With this example command line: ... \ -m 1024M,slots=4,maxmem=32G \ -numa node,nodeid=0 \ -numa node,mem=1024M,nodeid=1 \ -numa node,nodeid=2 \ -numa node,nodeid=3 \ Guest reports "No NUMA configuration