[PATCH 0/4] qemuhotplugtest: Fix timeouts
by Peter Krempa
The test takes too long for no good reason. One was caused by the
compiler inlining too much and second one was a deliberate decision.
Peter Krempa (4):
qemuDomainGetUnplugTimeout: Add G_GNUC_NO_INLINE
mock:qemuDomainGetUnplugTimeout: Remove unused attribute for '@vm'
mock: qemuDomainGetUnplugTimeout: Decrease timeout
qemuhotplugtest: detach: Remove commands which are not issued
src/qemu/qemu_hotplug.c | 2 +-
tests/qemuhotplugmock.c | 6 +++---
tests/qemuhotplugtest.c | 9 +++------
3 files changed, 7 insertions(+), 10 deletions(-)
--
2.26.0
5 years
[libvirt PATCH 0/5] Remove virRun in favor of virCommand
by Ján Tomko
Ján Tomko (5):
openvz: switch from virRun to virCommand
vmware: use virCommand instead of virRun
Remove all usage of virRun
util: remove references to virRun/virExec
util: remove virRun
src/libvirt_private.syms | 1 -
src/lxc/lxc_driver.c | 5 +-
src/openvz/openvz_driver.c | 96 ++++++++++++++------------------
src/qemu/qemu_domain.c | 16 +++---
src/security/security_apparmor.c | 11 +---
src/util/vircommand.c | 35 ------------
src/util/vircommand.h | 2 -
src/util/virnetdev.c | 24 +++-----
src/util/virprocess.c | 4 +-
src/vmware/vmware_conf.c | 9 ++-
src/vmware/vmware_driver.c | 69 ++++++++++-------------
11 files changed, 98 insertions(+), 174 deletions(-)
--
2.25.1
5 years
[all PATCH v2] gitlab: add CI job for validating DCO signoff
by Daniel P. Berrangé
This job uses the shared "check-dco" image to validate that all
commits on a branch in a developer's repo fork have a suitable
Signed-off-by statement present.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
---
This patch was against the Perl repo, but if this is approved then
I'll also apply it to *all* the other repos which currently lack a
DCO check, without reposting further patches for each repo.
For libvirt.git I'll send a patch to update its existing DCO job.
Changed in v2:
- Exclude based on project namespace, not branch names
.gitlab-ci.yml | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
create mode 100644 .gitlab-ci.yml
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
new file mode 100644
index 0000000..50dae92
--- /dev/null
+++ b/.gitlab-ci.yml
@@ -0,0 +1,16 @@
+
+stages:
+ - prebuild
+
+# Check that all commits are signed-off for the DCO.
+# Skip on "libvirt" namespace, since we only need to run
+# this test on developer's personal forks from which
+# merge requests are submitted
+check-dco:
+ stage: prebuild
+ image: registry.gitlab.com/libvirt/libvirt-ci/check-dco:master
+ script:
+ - /check-dco
+ except:
+ variables:
+ - $CI_PROJECT_NAMESPACE == 'libvirt'
--
2.25.2
5 years
[PATCH] news: Document new Xen hypervisor features
by Jim Fehlig
Signed-off-by: Jim Fehlig <jfehlig(a)suse.com>
---
docs/news.xml | 22 ++++++++++++++++++++++
1 file changed, 22 insertions(+)
diff --git a/docs/news.xml b/docs/news.xml
index 99c65447b0..074dc22e97 100644
--- a/docs/news.xml
+++ b/docs/news.xml
@@ -66,6 +66,28 @@
feedback from users on how well it works in practice.
</description>
</change>
+ <change>
+ <summary>
+ Xen: add support for e820_host hypervisor feature
+ </summary>
+ <description>
+ e820_host is a Xen-specific option only available for PV guests.
+ When enabled it provides the guest with a virtual e820 memory map
+ based on the host one. It must be enabled to allow hotplugging
+ PCI devices to PV guests, particularly when memory ballooning
+ is enabled.
+ </description>
+ </change>
+ <change>
+ <summary>
+ Xen: add support for passthrough hypervisor feature
+ </summary>
+ <description>
+ passthrough is a Xen-specific option new to Xen 4.13 that enables
+ PCI passthrough for guests. It must be enabled to allow hotplugging
+ PCI devices.
+ </description>
+ </change>
</section>
<section title="Improvements">
</section>
--
2.26.0
5 years
[PATCH v2 0/2] Include lease time option into DHCP settings
by Julio Faracco
This series is based on latest series from Nehal. It includes a new
entry called <lease/> under <range/> and <host/> from <dhcp/> scope.
This was implemented to include independent lease time for each line and
dnsmasq option. So, users are able to define one lease time for ranges
and other different for each host entry. The new syntax is simlar with:
<dhcp>
<range ...>
<lease expiry='14' unit='mins'/>
</range>
<host ...>
<lease expiry='1' unit='hours'/>
</host>
</dhcp>
It will produce a option in dnsmasq configuration file:
dhcp-range=192.168.122.2,192.168.122.254,255.255.255.0,14m
And some contents into hostsfile:
00:16:3e:77:e2:ed,192.168.122.10,a.example.com,1h
This series includes some test cases to cover lease time XML syntax
also. Now, each test case requires a hostsfile to test this specific
setting.
Julio Faracco (2):
conf: Add <lease/> option for <dhcp/> settings
tests: Add tests for <lease/> to cover dnsmasq settings
docs/schemas/basictypes.rng | 8 +
docs/schemas/network.rng | 20 +++
src/conf/network_conf.c | 159 +++++++++++++++---
src/conf/network_conf.h | 27 ++-
src/libvirt_private.syms | 3 +
src/network/bridge_driver.c | 56 +++++-
src/network/bridge_driver.h | 1 +
src/test/test_driver.c | 2 +-
src/util/virdnsmasq.c | 68 +++++---
src/util/virdnsmasq.h | 3 +
src/vbox/vbox_network.c | 16 +-
.../dhcp6-nat-network.hostsfile | 7 +
.../dhcp6-network.hostsfile | 5 +
.../dhcp6host-routed-network.hostsfile | 7 +
.../networkxml2confdata/leasetime-hours.conf | 16 ++
.../leasetime-hours.hostsfile | 2 +
tests/networkxml2confdata/leasetime-hours.xml | 19 +++
.../leasetime-infinite.conf | 16 ++
.../leasetime-infinite.hostsfile | 2 +
.../leasetime-infinite.xml | 19 +++
.../leasetime-minutes.conf | 16 ++
.../leasetime-minutes.hostsfile | 2 +
.../networkxml2confdata/leasetime-minutes.xml | 19 +++
.../leasetime-seconds.conf | 16 ++
.../leasetime-seconds.hostsfile | 2 +
.../networkxml2confdata/leasetime-seconds.xml | 19 +++
...t-network-dns-srv-record-minimal.hostsfile | 2 +
.../nat-network-dns-srv-record.hostsfile | 2 +
.../nat-network-dns-txt-record.hostsfile | 2 +
.../nat-network-mtu.hostsfile | 2 +
.../nat-network-name-with-quotes.hostsfile | 2 +
.../networkxml2confdata/nat-network.hostsfile | 2 +
.../ptr-domains-auto.hostsfile | 2 +
tests/networkxml2conftest.c | 41 ++++-
tests/networkxml2xmlin/leasetime-hours.xml | 19 +++
tests/networkxml2xmlin/leasetime-infinite.xml | 19 +++
tests/networkxml2xmlin/leasetime-minutes.xml | 19 +++
tests/networkxml2xmlin/leasetime-seconds.xml | 19 +++
tests/networkxml2xmlout/leasetime-hours.xml | 21 +++
.../networkxml2xmlout/leasetime-infinite.xml | 21 +++
tests/networkxml2xmlout/leasetime-minutes.xml | 21 +++
tests/networkxml2xmlout/leasetime-seconds.xml | 21 +++
tests/networkxml2xmltest.c | 4 +
43 files changed, 683 insertions(+), 66 deletions(-)
create mode 100644 tests/networkxml2confdata/dhcp6-nat-network.hostsfile
create mode 100644 tests/networkxml2confdata/dhcp6-network.hostsfile
create mode 100644 tests/networkxml2confdata/dhcp6host-routed-network.hostsfile
create mode 100644 tests/networkxml2confdata/leasetime-hours.conf
create mode 100644 tests/networkxml2confdata/leasetime-hours.hostsfile
create mode 100644 tests/networkxml2confdata/leasetime-hours.xml
create mode 100644 tests/networkxml2confdata/leasetime-infinite.conf
create mode 100644 tests/networkxml2confdata/leasetime-infinite.hostsfile
create mode 100644 tests/networkxml2confdata/leasetime-infinite.xml
create mode 100644 tests/networkxml2confdata/leasetime-minutes.conf
create mode 100644 tests/networkxml2confdata/leasetime-minutes.hostsfile
create mode 100644 tests/networkxml2confdata/leasetime-minutes.xml
create mode 100644 tests/networkxml2confdata/leasetime-seconds.conf
create mode 100644 tests/networkxml2confdata/leasetime-seconds.hostsfile
create mode 100644 tests/networkxml2confdata/leasetime-seconds.xml
create mode 100644 tests/networkxml2confdata/nat-network-dns-srv-record-minimal.hostsfile
create mode 100644 tests/networkxml2confdata/nat-network-dns-srv-record.hostsfile
create mode 100644 tests/networkxml2confdata/nat-network-dns-txt-record.hostsfile
create mode 100644 tests/networkxml2confdata/nat-network-mtu.hostsfile
create mode 100644 tests/networkxml2confdata/nat-network-name-with-quotes.hostsfile
create mode 100644 tests/networkxml2confdata/nat-network.hostsfile
create mode 100644 tests/networkxml2confdata/ptr-domains-auto.hostsfile
create mode 100644 tests/networkxml2xmlin/leasetime-hours.xml
create mode 100644 tests/networkxml2xmlin/leasetime-infinite.xml
create mode 100644 tests/networkxml2xmlin/leasetime-minutes.xml
create mode 100644 tests/networkxml2xmlin/leasetime-seconds.xml
create mode 100644 tests/networkxml2xmlout/leasetime-hours.xml
create mode 100644 tests/networkxml2xmlout/leasetime-infinite.xml
create mode 100644 tests/networkxml2xmlout/leasetime-minutes.xml
create mode 100644 tests/networkxml2xmlout/leasetime-seconds.xml
--
2.24.1
5 years
[all PATCH] gitlab: add CI job for validating DCO signoff
by Daniel P. Berrangé
This job uses the shared "check-dco" image to validate that all
commits on a developer's branch have a suitable Signed-off-by
statement present.
Signed-off-by: Daniel P. Berrangé <berrange(a)redhat.com>
---
This patch was against the Perl repo, but if this is approved then
I'll also apply it to *all* the other repos which currently lack a
DCO check, without reposting further patches for each repo.
For libvirt.git I'll send a patch to update its existing DCO job.
.gitlab-ci.yml | 16 ++++++++++++++++
1 file changed, 16 insertions(+)
create mode 100644 .gitlab-ci.yml
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
new file mode 100644
index 0000000..56e0e02
--- /dev/null
+++ b/.gitlab-ci.yml
@@ -0,0 +1,16 @@
+
+stages:
+ - prebuild
+
+# Check that all commits are signed-off for the DCO. Skip
+# on master branch and -maint branches, since we only need
+# to test developer's personal branches.
+check-dco:
+ stage: prebuild
+ image: registry.gitlab.com/libvirt/libvirt-ci/check-dco:master
+ script:
+ - /check-dco
+ only:
+ - branches
+ except:
+ - master
--
2.25.2
5 years
[PATCH] docs: drvqemu: trivial fix for qemu commands passthrough
by Chen Hanxiao
element <qemu:commandline> should be the child of <domain>
Signed-off-by: Chen Hanxiao <chen_han_xiao(a)126.com>
---
docs/drvqemu.html.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/drvqemu.html.in b/docs/drvqemu.html.in
index 5f412ba376..afc4ddf56d 100644
--- a/docs/drvqemu.html.in
+++ b/docs/drvqemu.html.in
@@ -578,7 +578,7 @@ mount -t cgroup none /dev/cgroup -o devices
typically, the namespace is given the name
of <code>qemu</code>. With the namespace in place, it is then
possible to add an element <code><qemu:commandline></code>
- under <code>driver</code>, with the following sub-elements
+ under <code>domain</code>, with the following sub-elements
repeated as often as needed:
</p>
<dl>
--
2.23.0
5 years
[PATCH 0/6] qemuDomainSaveImageOpen: Fix startup with corrupted save image
by Peter Krempa
This is an alternative to
https://www.redhat.com/archives/libvir-list/2020-April/msg01081.html
which actually fixes qemuDomainSaveImageOpen in a way that would make it
more obvious which value needs to be returned and also cleans up the
code some more.
Peter Krempa (6):
qemu: fix domain start with corrupted save file
virQEMUSaveData: Register autoclear function and use it in
qemuDomainSaveImageOpen
qemuDomainSaveImageOpen: Use g_autoptr for 'dom'
qemuDomainSaveImageOpen: Automatically close 'fd' if unneeded
qemuDomainSaveImageOpen: Use 'g_new0' instead of VIR_ALLOC(_N)
qemuDomainSaveImageOpen: Refactor handling of errors
src/qemu/qemu_driver.c | 82 +++++++++++++++++++++---------------------
1 file changed, 41 insertions(+), 41 deletions(-)
--
2.26.0
5 years
[libvirt PATCH v2] qemu: fix domain start with corrupted save file
by Pavel Mores
This is to fix
https://bugzilla.redhat.com/show_bug.cgi?id=1791522
With this change, if a domain comes across a corrupted save file during
boot it removes the save file and logs a warning but continues to boot
normally instead of failing to boot (with a subsequent boot attempt
succeeding).
The regression was introduced by 21ad56e932 and this change effectively
reverts the relevant part of that commit.
---
src/qemu/qemu_driver.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index e8d47a41cd..2579ef3984 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -6810,13 +6810,14 @@ qemuDomainSaveImageOpen(virQEMUDriverPtr driver,
*ret_def = def;
*ret_data = data;
+ cleanup:
return fd;
error:
virDomainDefFree(def);
virQEMUSaveDataFree(data);
VIR_FORCE_CLOSE(fd);
- return -1;
+ goto cleanup;
}
static int ATTRIBUTE_NONNULL(4) ATTRIBUTE_NONNULL(5) ATTRIBUTE_NONNULL(6)
--
2.24.1
5 years
[libvirt PATCH] qemu: fix domain start with corrupted save file
by Pavel Mores
This is to fix
https://bugzilla.redhat.com/show_bug.cgi?id=1791522
With this change, if a domain comes across a corrupted save file during boot it
removes the save file and logs a warning but continues to boot normally instead
of failing to boot (with a subsequent boot attempt succeeding).
The regression was introduced by 21ad56e932 and this change effectively reverts
the relevant part of that commit.
---
src/qemu/qemu_driver.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index e8d47a41cd..2579ef3984 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -6810,13 +6810,14 @@ qemuDomainSaveImageOpen(virQEMUDriverPtr driver,
*ret_def = def;
*ret_data = data;
+ cleanup:
return fd;
error:
virDomainDefFree(def);
virQEMUSaveDataFree(data);
VIR_FORCE_CLOSE(fd);
- return -1;
+ goto cleanup;
}
static int ATTRIBUTE_NONNULL(4) ATTRIBUTE_NONNULL(5) ATTRIBUTE_NONNULL(6)
--
2.24.1
5 years
