On 8/8/19 6:54 PM, Jonathon Jongsma wrote:
Series looks good to me, though you may want to wait for somebody with
more experience to properly ack it.
Reviewed-by: Jonathon Jongsma
That's okay. I've taken your R-b and pushed it. These are simple enough
so if there was any problem, it would
On Thu, Aug 08, 2019 at 12:30:24PM +0200, Ilias Stamatis wrote:
> Signed-off-by: Ilias Stamatis
> ---
> src/test/test_driver.c | 40
> 1 file changed, 40 insertions(+)
Reviewed-by: Erik Skultety
--
libvir-list mailing list
libvir-list@redhat.com
https://
We don't need to access it from anywhere else, so it can be
a local variable instead of an object property.
Signed-off-by: Andrea Bolognani
---
refresh | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/refresh b/refresh
index 8f99042..89c2818 100755
--- a/refresh
+++ b/r
Instead of adding native projects first and then coming back
to add MinGW projects at a later time, add all projects in a
single step.
Signed-off-by: Andrea Bolognani
---
refresh | 22 ++
1 file changed, 10 insertions(+), 12 deletions(-)
diff --git a/refresh b/refresh
index
This reverts some of the changes made by 7130ffe0a0e9, which
is okay because they were not entirely accurate: by the time
we start looking for CROSS, we have already removed the project
name from the stem, so the OS name is indeed the first thing
you'll find there.
Signed-off-by: Andrea Bolognani
Using rfind() makes sense when we're looking for CROSS, since
we don't know exactly where it will be, but when looking for
the project we know it will be at the very beginning of the
string so we should check accordingly.
Signed-off-by: Andrea Bolognani
---
refresh | 4 ++--
1 file changed, 2 in
Andrea Bolognani (4):
refresh: Get rid of self.project_name
refresh: Improve project handling
refresh: Look for project with startswith()
refresh: Update comments
refresh | 44 +++-
1 file changed, 23 insertions(+), 21 deletions(-)
--
2.21.0
-
Haven't spotted it during review of Dan's patches.
Long story short, if there's an unexpected key in JSON parsing stops.
But in general a key is hot harmful and shouldn't cause parser to stop.
See 1/3 for more explanation.
Michal Prívozník (3):
nss: Don't stop parsing on unexpected key
nss: In
The NSS module has a compile time option which when enabled makes
ERROR() and DEBUG() print messages onto stderr. But now that the
module no longer links with libvirt, we need to include stdio.h
and define NULLSTR().
Signed-off-by: Michal Privoznik
---
tools/nss/libvirt_nss.h | 2 ++
1 file chan
If yajl_parse() fails, we try to print an error message. For
that, yajl_get_error() is used. However, its documentation say
that caller is also responsible for freeing the memory it
allocates by using yajl_free_error().
Signed-off-by: Michal Privoznik
---
tools/nss/libvirt_nss_leases.c | 7 -
Due to latest rewrite of NSS module, we are doing yajl parsing
ourselves. This means, we had to introduce couple of callback
that yajl calls. According to its documentation, a callback can
cancel parsing if it returns a zero value. Well, we do just that
in the string callback (findLeasesParserStrin
On Wed, Aug 07, 2019 at 06:56:35PM +0200, Ilias Stamatis wrote:
> The managedSave APIs according to the documentation are supposed to
> operate on a disk file. However, this might not be appropriate in the
> case of the test driver since:
>
> * It's better if the test driver keeps all its state in
On Fri, Aug 02, 2019 at 14:54:15 +0200, Paolo Bonzini wrote:
> host-passthrough documentation menions that the source and destination
> hosts are not identical in both hardware and configuration. Configuration
> actually includes microcode version and QEMU version, but this is not
> clear so make
On Fri, Aug 09, 2019 at 10:49:09AM +0200, Michal Privoznik wrote:
> Due to latest rewrite of NSS module, we are doing yajl parsing
> ourselves. This means, we had to introduce couple of callback
> that yajl calls. According to its documentation, a callback can
> cancel parsing if it returns a zero
On Fri, Aug 09, 2019 at 10:49:10AM +0200, Michal Privoznik wrote:
> The NSS module has a compile time option which when enabled makes
> ERROR() and DEBUG() print messages onto stderr. But now that the
> module no longer links with libvirt, we need to include stdio.h
> and define NULLSTR().
>
> Sig
On Fri, Aug 09, 2019 at 10:49:11AM +0200, Michal Privoznik wrote:
> If yajl_parse() fails, we try to print an error message. For
> that, yajl_get_error() is used. However, its documentation say
> that caller is also responsible for freeing the memory it
> allocates by using yajl_free_error().
>
>
Ján Tomko writes:
> One more thing, qemuxml2argvtest is using DO_TEST:
> DO_TEST("hyperv", NONE);
> DO_TEST("hyperv-off", NONE);
> DO_TEST("hyperv-panic", NONE);
> which manually enumarate the used QEMU capabilities (although there are
> none here)
>
> DO_TEST_CAPS_LATEST is the way t
Redefining a domain via virDomainDefineXML should not give different results
based on an already existing definition.
Also, there's a crasher somewhere in the code:
https://bugzilla.redhat.com/show_bug.cgi?id=1739338
This reverts commit 94b3aa55f83ada33a9fdda66068d58ef1a56c0a5
---
src/conf/domai
On 8/9/19 11:04 AM, Daniel P. Berrangé wrote:
On Fri, Aug 09, 2019 at 10:49:11AM +0200, Michal Privoznik wrote:
If yajl_parse() fails, we try to print an error message. For
that, yajl_get_error() is used. However, its documentation say
that caller is also responsible for freeing the memory it
al
On Fri, Aug 09, 2019 at 12:13:45PM +0200, Vitaly Kuznetsov wrote:
Ján Tomko writes:
One more thing, qemuxml2argvtest is using DO_TEST:
DO_TEST("hyperv", NONE);
DO_TEST("hyperv-off", NONE);
DO_TEST("hyperv-panic", NONE);
which manually enumarate the used QEMU capabilities (although
On Fri, Aug 09, 2019 at 12:15:43 +0200, Ján Tomko wrote:
> Redefining a domain via virDomainDefineXML should not give different results
> based on an already existing definition.
>
> Also, there's a crasher somewhere in the code:
> https://bugzilla.redhat.com/show_bug.cgi?id=1739338
>
> This reve
Hi
On Fri, Aug 9, 2019 at 2:15 PM Ján Tomko wrote:
>
> Redefining a domain via virDomainDefineXML should not give different results
> based on an already existing definition.
>
> Also, there's a crasher somewhere in the code:
> https://bugzilla.redhat.com/show_bug.cgi?id=1739338
>
> This reverts
On 8/8/19 4:26 PM, Jiri Denemark wrote:
Since qemuDomain{Device,}DefPostParse callbacks require qemuCaps, we
need to make sure they get the capabilities stored in the domain's
private data if the domain is running. Passing NULL may cause QEMU
capabilities probing to be triggered in case the cache
On 8/8/19 6:12 PM, Jiri Denemark wrote:
This reverts commit f38d553e2d6ec2f041cb7947b5eafcdd3b26ae65.
Gnulib's make coverage (or init-coverage, build-coverage, gen-coverage)
is not a 1-1 replacement for the original configure option. Our old
--enable-test-coverage seems to be close to gnulib's m
From: Marc-André Lureau
Since its introduction in commit
8737578d115b8328f59f3342377b0c7ac06f658a, the TPM version format is
"2.0" and not "2".
Signed-off-by: Marc-André Lureau
---
src/conf/domain_conf.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_co
On 8/9/19 2:23 PM, marcandre.lur...@redhat.com wrote:
From: Marc-André Lureau
Since its introduction in commit
8737578d115b8328f59f3342377b0c7ac06f658a, the TPM version format is
"2.0" and not "2".
Signed-off-by: Marc-André Lureau
---
src/conf/domain_conf.c | 4 ++--
1 file changed, 2 inse
On Fri, Aug 09, 2019 at 04:23:48PM +0400, marcandre.lur...@redhat.com wrote:
> From: Marc-André Lureau
>
> Since its introduction in commit
> 8737578d115b8328f59f3342377b0c7ac06f658a, the TPM version format is
> "2.0" and not "2".
>
> Signed-off-by: Marc-André Lureau
> ---
The html docs agrees wi
On Fri, Aug 02, 2019 at 11:51:05AM +0200, Ilias Stamatis wrote:
> Signed-off-by: Ilias Stamatis
> ---
> src/test/test_driver.c | 58 ++
> 1 file changed, 58 insertions(+)
>
> diff --git a/src/test/test_driver.c b/src/test/test_driver.c
> index aae9875194..7
On 8/9/19 8:29 AM, Han Han wrote:
Signed-off-by: Han Han
---
src/libvirt-admin.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
Ah, good catch. There are some other APIs that don't log all the
arguments. I'm adding them into this patch, changing the commit message
a big, ACKing an
Ján Tomko writes:
> Either leave this as-is and add the new feature in a separate test,
> or you can branch the test based on QEMU versions like we do for other
> features:
>
> DO_TEST_CAPS_VER("pv-spinlock-disabled", "2.7.0");
> DO_TEST_CAPS_VER("pv-spinlock-disabled", "4.0.0");
> DO
On 8/8/19 4:26 PM, Maxiwell S. Garcia wrote:
QEMU shows a warning message if partial NUMA mapping is set. This patch
adds a warning message in libvirt when editing the XML. It must be an
error in future, when QEMU remove this ability.
Signed-off-by: Maxiwell S. Garcia
---
src/qemu/qemu_domain
On Tue, Jul 23, 2019 at 16:47:05 -0300, Maxiwell S. Garcia wrote:
> The snapshot-create operation of running guests saves the live
> XML and uses it to replace the active and inactive domain in
> case of revert. So, the config XML is ignored by the snapshot
> process. This commit changes it and add
On Wed, Jul 31, 2019 at 14:52:17 -0500, Eric Blake wrote:
> On 7/31/19 2:45 PM, Daniel Henrique Barboza wrote:
> > Max,
> >
> > Code looks ok. Two tests (virsh-checkpoint and virsh-snapshot) are
> > failing, but they are also failing on master, thus I say this patch get a
> > pass because it didn'
On Fri, 2019-08-09 at 10:35 +0200, Andrea Bolognani wrote:
>
> Andrea Bolognani (4):
> refresh: Get rid of self.project_name
> refresh: Improve project handling
> refresh: Look for project with startswith()
> refresh: Update comments
>
> refresh | 44 +++--
On 8/9/19 9:02 AM, Jiri Denemark wrote:
> On Wed, Jul 31, 2019 at 14:52:17 -0500, Eric Blake wrote:
>> On 7/31/19 2:45 PM, Daniel Henrique Barboza wrote:
>>> Max,
>>>
>>> Code looks ok. Two tests (virsh-checkpoint and virsh-snapshot) are
>>> failing, but they are also failing on master, thus I say
The QEMU driver now supports Direct Mode for Hyper-V Synthetic timers
for Hyper-V guests.
Reviewed-by: Ján Tomko
Signed-off-by: Vitaly Kuznetsov
---
docs/news.xml | 9 +
1 file changed, 9 insertions(+)
diff --git a/docs/news.xml b/docs/news.xml
index a4d727b9c5..d63fca3b48 100644
--- a
Support 'Direct Mode' for Hyper-V Synthetic Timers in domain config.
Make it 'stimer' enlightenment option as it is not a separate thing.
Reviewed-by: Ján Tomko
Signed-off-by: Vitaly Kuznetsov
---
docs/formatdomain.html.in | 10 +++---
docs/schemas/domaincommon.rng | 16 -
src/conf/
QEMU-4.1 supports 'Direct Mode' for Hyper-V synthetic timers
(hv-stimer-direct CPU flag): Windows guests can request that timer
expiration notifications are delivered as normal interrupts (and not
VMBus messages). This is used by Hyper-V on KVM.
Signed-off-by: Vitaly Kuznetsov
---
src/qemu/qemu_
Changes since v3 [Ján Tomko]:
- Drop already merged patches.
- add "tests: qemuxml2argv: switch to DO_TEST_CAPS_LATEST for Hyper-V
tests" patch (hope I got the idea correctly).
- add hyperv-stimer-direct test to qemuxml2xml/qemuxml2argv tests.
- use VIR_CPU_x86_HV_STIMER_DIRECT instead of 'hv-sti
In particular, switch to using canonical 'hv-feature' syntax instead of
'hv_feature' aliases.
Suggested-by: Ján Tomko ~
Signed-off-by: Vitaly Kuznetsov
---
tests/qemuxml2argvdata/hyperv-off.args| 27 ---
.../hyperv-off.x86_64-latest.args | 32 +
te
From: Wim ten Have
QEMU version 2.12.1 introduced a performance feature under commit
be7773268d98 ("target-i386: add KVM_HINTS_DEDICATED performance hint")
This patch adds a new KVM feature 'hint-dedicated' to set this performance
hint for KVM guests. The feature is off by default.
To enable th
QEMU version 2.12.1 introduced a performance feature under commit
be7773268d98 ("target-i386: add KVM_HINTS_DEDICATED performance hint")
This patch adds a new KVM feature 'hint-dedicated' to set this performance
hint for KVM guests.
Wim ten Have (2):
qemu: support for kvm-hint-dedicated perform
From: Wim ten Have
Update the KVM feature tests for QEMU's kvm-hint-dedicated
performance hint.
Signed-off-by: Wim ten Have
Signed-off-by: Menno Lageman
---
tests/qemuxml2argvdata/kvm-features-off.xml | 1 +
tests/qemuxml2argvdata/kvm-features.args | 4 ++--
tests/qemuxml2argvdata/kvm-
The nwfilter XML configs are not merely examples, they are data that is
actively shipped and used in production by users.
Signed-off-by: Daniel P. Berrangé
---
examples/Makefile.am | 10 --
src/nwfilter/Makefile.inc.am | 8
Ilias Stamatis (3):
test_driver: use domain-private data to store managed image
test_driver: implement virDomainManagedSaveGetXMLDesc
test_driver: implement virDomainManagedSaveDefineXML
src/test/test_driver.c | 93 +-
1 file changed, 92 insertions(+)
The managedSave APIs according to the documentation are supposed to
operate on a disk file. However, this might not be appropriate in the
case of the test driver since:
* It's better if the test driver keeps all its state in memory only and
doesn't affect the host in any way.
* The test driver, a
Signed-off-by: Ilias Stamatis
---
src/test/test_driver.c | 40
1 file changed, 40 insertions(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 211c0d737a..a127dc69c1 100755
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -
Signed-off-by: Ilias Stamatis
---
src/test/test_driver.c | 30 ++
1 file changed, 30 insertions(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 8bd5a5296b..211c0d737a 100755
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@@ -7624,6 +76
On Thu, Aug 08, 2019 at 03:00:27PM -0300, Daniel Henrique Barboza wrote:
> (CCing Nikolay Shirokovskiy, author of 29f2b5248c )
>
> On 8/7/19 8:34 AM, Roman Bolshakov wrote:
> > An attempt to poweroff a VM from inside triggers the error for existing
> > session of virsh console and it returns with
Ilias Stamatis (2):
test_driver: implement testDomainGetBlockIoTune
test_driver: implement virDomainSetBlockIoTune
src/test/test_driver.c | 361 +
1 file changed, 361 insertions(+)
--
2.22.0
--
libvir-list mailing list
libvir-list@redhat.com
https://w
Signed-off-by: Ilias Stamatis
---
src/test/test_driver.c | 259 +
1 file changed, 259 insertions(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 14b7d683e1..009bc18a73 100755
--- a/src/test/test_driver.c
+++ b/src/test/test_driver.c
@
Signed-off-by: Ilias Stamatis
Reviewed-by: Erik Skultety
---
src/test/test_driver.c | 102 +
1 file changed, 102 insertions(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 2b5376ec28..14b7d683e1 100755
--- a/src/test/test_driver.c
++
52 matches
Mail list logo