Re: [libvirt] [PATCH] Fix guest boot failure when vcpu placement="auto" on memoryless numa node

2017-07-09 Thread Peter Krempa
On Mon, Jul 10, 2017 at 12:09:36 +0530, Nitesh Konkar wrote: > When the vcpu placement is auto and we have memoryless numa nodes on the host, > numad returns a list numa nodes with and without memory. When we try to write > it > to /sys/fs/cgroup/*/cpuset.mems it errors out as invlaid argument. >

[libvirt] [PATCH] Fix guest boot failure when vcpu placement="auto" on memoryless numa node

2017-07-09 Thread Nitesh Konkar
When the vcpu placement is auto and we have memoryless numa nodes on the host, numad returns a list numa nodes with and without memory. When we try to write it to /sys/fs/cgroup/*/cpuset.mems it errors out as invlaid argument. Signed-off-by: Nitesh Konkar --- numactl --hardware available: 4 nodes

[libvirt] [PATCH go-xml] Add support for domain clock and test code

2017-07-09 Thread zhenwei.pi
Signed-off-by: zhenwei.pi --- domain.go | 7 +++ domain_test.go | 6 ++ 2 files changed, 13 insertions(+) diff --git a/domain.go b/domain.go index b9b0f77..1d020aa 100644 --- a/domain.go +++ b/domain.go @@ -532,6 +532,12 @@ type DomainCPU struct { Features []DomainCPUFeature

[libvirt] [PATCH go-xml] Add support for domain clock and test code

2017-07-09 Thread zhenwei.pi
Signed-off-by: zhenwei.pi --- domain.go | 5 + domain_test.go | 4 2 files changed, 9 insertions(+) diff --git a/domain.go b/domain.go index b9b0f77..8bbf82b 100644 --- a/domain.go +++ b/domain.go @@ -532,6 +532,10 @@ type DomainCPU struct { Features []DomainCPUFeature `xml

Re: [libvirt] [PATCH v2] test: adding tests to virStrToDouble() inside virstringtest.

2017-07-09 Thread Julio Faracco
Hi guys, Does anybody know why my V2 patch is being sent as a reply? It is happening even if I set manually --subject-prefix="PATCH v2" and --no-chain-reply-to I had never seen that and I usually send the patch to myself before. Everything is fine. I don't believe that I'm missing any GIT paramete

[libvirt] [PATCH v2] test: adding tests to virStrToDouble() inside virstringtest.

2017-07-09 Thread Julio Faracco
There are no occurrences of tests related to Strings and Double numbers inside virstringtest.c. This commit introduces some tests to validate the conversion. The test does not include locale changes yet. Signed-off-by: Julio Faracco --- tests/virstringtest.c | 80

[libvirt] [PATCH v2] test: adding tests to virStrToDouble() inside virstringtest.

2017-07-09 Thread Julio Faracco
There are no occurrences of tests related to Strings and Double numbers inside virstringtest.c. This commit introduces some tests to validate the conversion. The test does not include locale changes yet. Signed-off-by: Julio Faracco --- tests/virstringtest.c | 80

[libvirt] [PATCH] test: adding tests to virStrToDouble() inside virstringtest.

2017-07-09 Thread Julio Faracco
There are no occurrences of tests related to Strings and Double numbers inside virstringtest.c. This commit introduces some tests to validate the conversion. The test does not include locale changes yet. Signed-off-by: Julio Faracco --- tests/virstringtest.c | 80

Re: [libvirt] [PATCH] test: adding tests to virStrToDouble() inside virstringtest.

2017-07-09 Thread Julio Faracco
Hi John, Thanks for your points. The V2 is ready... I will use %lf for now because the intention of output is show the error with a clear message. %lf is easier to check, I believe. Well, any other suggestions are welcome. 2017-07-08 10:59 GMT-03:00 John Ferlan : > > > On 06/24/2017 08:15 PM, J

Re: [libvirt] [PATCH v2 0/5] Misc cleanups for internal.h & fix tests with CLang

2017-07-09 Thread Roman Bogorodskiy
Daniel P. Berrange wrote: > When writing the fix for test suite mocking under CLang I found a > bunch of cruft in internal.h The first four patches thus cleanup > up internal.h. We then add the extra annotations requird to prevent > CLang optimizer breaking mock overrides. > > Changed in v2: >