Introduce a bunch of new virsh commands for managing checkpoints
in isolation. More commands are needed for performing incremental
backups, but these commands were easy to implement by modeling
heavily after virsh-snapshot.c (no need for checkpoint-revert,
and checkpoint-list was a lot easier
A lot of this work heavily copies from the existing snapshot APIs.
The test driver doesn't really have to do anything more than just
expose an interface into libvirt metadata, making it possible to test
saving and restoring XML, and tracking relations between multiple
checkpoints.
Signed-off-by:
A lot of this work heavily copies from the existing snapshot
APIs. The interaction with qemu during create/delete still
needs to be implemented, but this takes care of all the libvirt
metadata (saving and restoring XML, and tracking the relations
between multiple checkpoints).
Signed-off-by:
Time to actually issue the QMP transactions that create and
delete persistent checkpoints. For create, we only need one
transaction: inside, we visit all disks affected by the
checkpoint, and create a new enabled bitmap, as well as
disabling the bitmap of the parent checkpoint (if any). For
Now that various new API have been added, it is worth a landing page
that gives an overview of capturing various pieces of guest state, and
which APIs are best suited to which tasks.
Signed-off-by: Eric Blake
Reviewed-by: John Ferlan
Reviewed-by: Daniel P. Berrangé
---
docs/docs.html.in
Once a checkpoint has been created, it is desirable to estimate the
size of the disk delta that is represented between the checkpoint and
the current operation. To do this, we have to scrape information out
of QMP query-block on a request from the user.
---
src/qemu/qemu_monitor.h | 4 ++
Introduce a bunch of new public APIs related to backup checkpoints.
Checkpoints are modeled heavily after virDomainSnapshotPtr (both
represent a point in time of the guest), although a snapshot exists
with the intent of rolling back to that state, while a checkpoint
exists to make it possible to
Creating a checkpoint does not modify guest-visible state,
but does modify host resources. Rather than reuse existing
domain:write, domain:block_write, or domain:snapshot access
controls, it seems better to introduce a new access control
specific to tasks related to checkpoints and incremental
Create a new file for managing a list of checkpoint objects, borrowing
heavily from existing virDomainSnapshotObjList paradigms.
Note that while checkpoints definitely have a use case for multiple
children to a single parent (create a base snapshot, create a child
snapshot, revert to the base,
Similar to commit a487890d for qemu, a little bit of refactoring in
the snapshot delete code will make it easier to reuse functionality
for checkpoints.
Signed-off-by: Eric Blake
---
src/test/test_driver.c | 34 +-
1 file changed, 17 insertions(+), 17
Prepare for new checkpoint APIs by describing the XML that will
represent a checkpoint. The checkpoint XML is modeled heavily after
virDomainSnapshotPtr. (See the docs for more details).
Add testsuite coverage for some minimal uses of the XML (bare minimum,
the sample from html, and a full
This is a subset of v8 of incremental backup, focusing more on just
the checkpoint manipulation code (as this had fewer review comments
that still needed addressing). I think there is enough here that even
if this lands in 5.5 while the backup APIs slide into 5.6 that we'd
still have a reasonably
The remote code generator had to be taught about the new
virDomainCheckpointPtr type, at which point the remote driver code for
backups can be generated.
Signed-off-by: Eric Blake
Reviewed-by: Daniel P. Berrangé
---
src/remote/remote_daemon_dispatch.c | 20
src/remote/remote_driver.c
Add a new file checkpoint_conf.c that performs the translation to and
from new XML describing a checkpoint. The code shares a common base
class with snapshots, since a checkpoint similarly represents the
domain state at a moment in time. Add some basic testing of round trip
XML handling through
On Tue, 18 Jun 2019 14:48:11 +0200
Sylvain Bauza wrote:
> On Tue, Jun 18, 2019 at 1:01 PM Cornelia Huck wrote:
>
> > On Mon, 17 Jun 2019 11:05:17 -0600
> > Alex Williamson wrote:
> >
> > > On Mon, 17 Jun 2019 16:10:30 +0100
> > > Daniel P. Berrangé wrote:
> > >
> > > > On Mon, Jun 17,
On 6/18/19 1:39 PM, Daniel P. Berrangé wrote:
> Signed-off-by: Daniel P. Berrangé
> ---
> src/conf/virnetworkobj.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
> index 47c142998e..ed1f02f4c5 100644
> ---
On 6/18/19 1:39 PM, Daniel P. Berrangé wrote:
> Fixes for bugs spotted by coverity
>
> Daniel P. Berrangé (4):
> conf: fix leak when parsing network port XML
> conf: fix leak of directory handle when loading network ports
> conf: fix NULL deref when exporting ports
> conf: add error
Signed-off-by: Daniel P. Berrangé
---
src/conf/virnetworkobj.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
index 47c142998e..ed1f02f4c5 100644
--- a/src/conf/virnetworkobj.c
+++ b/src/conf/virnetworkobj.c
@@ -1750,6 +1750,7 @@
Signed-off-by: Daniel P. Berrangé
---
nwfilter_binding.go | 73 +
xml_test.go | 3 ++
2 files changed, 76 insertions(+)
create mode 100644 nwfilter_binding.go
diff --git a/nwfilter_binding.go b/nwfilter_binding.go
new file mode 100644
index
Hi,
This is labeled as RFC but it's more like a FYI to let people know and
comment beforehand. Shiva sent a 28 patch series last year that implements
hotplug/unplug support for PCI multifunction devices [1]. The design
motivation of his work was based in a RFC sent to this mailing list back
in
Fixes for bugs spotted by coverity
Daniel P. Berrangé (4):
conf: fix leak when parsing network port XML
conf: fix leak of directory handle when loading network ports
conf: fix NULL deref when exporting ports
conf: add error checking of UUID generation
src/conf/domain_conf.c | 12
Signed-off-by: Daniel P. Berrangé
---
src/conf/virnetworkobj.c | 8 +---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/src/conf/virnetworkobj.c b/src/conf/virnetworkobj.c
index ed1f02f4c5..fd4bf779b9 100644
--- a/src/conf/virnetworkobj.c
+++ b/src/conf/virnetworkobj.c
@@
Signed-off-by: Daniel P. Berrangé
---
src/conf/domain_conf.c | 12 ++--
1 file changed, 10 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
index ee651e7742..c69d382d70 100644
--- a/src/conf/domain_conf.c
+++ b/src/conf/domain_conf.c
@@ -30415,7
Use auto free to avoid leaking the "trustGuestRxFilters" strings
Signed-off-by: Daniel P. Berrangé
---
src/conf/virnetworkportdef.c | 26 +-
1 file changed, 9 insertions(+), 17 deletions(-)
diff --git a/src/conf/virnetworkportdef.c b/src/conf/virnetworkportdef.c
index
On Tue, Jun 18, 2019 at 06:33:01PM +0200, Christophe de Dinechin wrote:
>
>
> > On 18 Jun 2019, at 17:43, Daniel P. Berrangé wrote:
> >
> > Signed-off-by: Daniel P. Berrangé
> > ---
> > network.go | 80 ++
> > network_port.go | 233
Signed-off-by: Daniel P. Berrangé
---
domain.go | 1 +
network.go | 1 +
network_port.go | 200
xml_test.go | 3 +
4 files changed, 205 insertions(+)
create mode 100644 network_port.go
diff --git a/domain.go b/domain.go
[...]
> +
> +
> +int
> +virNetworkObjDeleteAllPorts(virNetworkObjPtr net,
> +const char *stateDir)
> +{
> +char *dir;
> +DIR *dh;
> +struct dirent *de;
> +int rc;
> +int ret = -1;
> +
> +if (!(dir = virNetworkObjGetPortStatusDir(net,
[...]
> +virNetworkPortDefPtr
> +virDomainNetDefToNetworkPort(virDomainDefPtr dom,
> + virDomainNetDefPtr iface)
> +{
> +virNetworkPortDefPtr port;
> +
> +if (iface->type != VIR_DOMAIN_NET_TYPE_NETWORK) {
> +virReportError(VIR_ERR_INTERNAL_ERROR,
> +
Signed-off-by: Jonathon Jongsma
---
src/util/virscsi.h | 11 ---
src/util/virscsihost.h | 7 ++-
src/util/virscsivhost.h | 13 +
3 files changed, 11 insertions(+), 20 deletions(-)
diff --git a/src/util/virscsi.h b/src/util/virscsi.h
index 907948455e..6cc68835b7
Signed-off-by: Jonathon Jongsma
---
src/util/virobject.h | 19 ---
1 file changed, 8 insertions(+), 11 deletions(-)
diff --git a/src/util/virobject.h b/src/util/virobject.h
index d13fc4b1ac..fe5dbe7326 100644
--- a/src/util/virobject.h
+++ b/src/util/virobject.h
@@ -19,11 +19,10
Internal headers should use #pragma once instead of the standard #ifndef
guard. Public headers still require the existing header guard.
Signed-off-by: Jonathon Jongsma
---
build-aux/header-ifdef.pl | 35 ++-
1 file changed, 26 insertions(+), 9 deletions(-)
diff
[...]
> +static virNetworkPortDefPtr
> +virNetworkPortDefParseXML(xmlXPathContextPtr ctxt)
> +{
> +virNetworkPortDefPtr def;
> +char *uuid = NULL;
> +xmlNodePtr virtPortNode;
> +xmlNodePtr vlanNode;
> +xmlNodePtr bandwidthNode;
> +xmlNodePtr addressNode;
> +char
Signed-off-by: Jonathon Jongsma
---
src/xenapi/xenapi_driver.h | 5 +
src/xenapi/xenapi_driver_private.h | 17 +++--
src/xenapi/xenapi_utils.h | 15 ++-
src/xenconfig/xen_common.h | 17 +++--
src/xenconfig/xen_sxpr.h |
Signed-off-by: Jonathon Jongsma
---
tests/qemumonitortestutils.h | 19 +++---
tests/qemusecuritytest.h | 7 +-
tests/testutils.h| 45 ++--
tests/testutilshostcpus.h| 11 ++-
tests/testutilslxc.h | 11 ++-
tests/testutilsqemu.h| 21 +++---
Signed-off-by: Jonathon Jongsma
---
src/util/virsysinfo.h | 13 +
src/util/virsysinfopriv.h | 5 +
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/src/util/virsysinfo.h b/src/util/virsysinfo.h
index c0015834bc..f30809294b 100644
--- a/src/util/virsysinfo.h
Signed-off-by: Jonathon Jongsma
---
src/vbox/vbox_MSCOMGlue.h | 11 +--
src/vbox/vbox_common.h| 161 +-
src/vbox/vbox_driver.h| 19 ++--
src/vbox/vbox_get_driver.h| 7 +-
src/vbox/vbox_glue.h | 15 ++--
Signed-off-by: Jonathon Jongsma
---
src/util/virlog.h | 55 ++-
1 file changed, 26 insertions(+), 29 deletions(-)
diff --git a/src/util/virlog.h b/src/util/virlog.h
index 775a726b06..a30b760fb4 100644
--- a/src/util/virlog.h
+++ b/src/util/virlog.h
@@
Signed-off-by: Jonathon Jongsma
---
src/util/virhash.h | 8 ++--
src/util/virhashcode.h | 7 ++-
2 files changed, 4 insertions(+), 11 deletions(-)
diff --git a/src/util/virhash.h b/src/util/virhash.h
index d2156a5ce0..ed3774e693 100644
--- a/src/util/virhash.h
+++
Signed-off-by: Jonathon Jongsma
---
src/test/test_driver.h | 7 ++-
1 file changed, 2 insertions(+), 5 deletions(-)
diff --git a/src/test/test_driver.h b/src/test/test_driver.h
index ff7803da7f..8c8a462db7 100644
--- a/src/test/test_driver.h
+++ b/src/test/test_driver.h
@@ -19,11 +19,8 @@
Signed-off-by: Jonathon Jongsma
---
src/storage/storage_backend.h | 13 -
src/storage/storage_backend_disk.h | 5 +---
src/storage/storage_backend_fs.h| 5 +---
src/storage/storage_backend_gluster.h | 5 +---
src/storage/storage_backend_iscsi.h
Signed-off-by: Jonathon Jongsma
---
src/util/virnetdev.h | 33 +++-
src/util/virnetdevbandwidth.h| 8 +++-
src/util/virnetdevbridge.h | 8 +++-
src/util/virnetdevip.h | 9 +++--
src/util/virnetdevmacvlan.h | 22
Signed-off-by: Jonathon Jongsma
---
src/util/virthread.h | 19 ---
src/util/virthreadjob.h | 6 +-
src/util/virthreadpool.h | 9 +++--
3 files changed, 12 insertions(+), 22 deletions(-)
diff --git a/src/util/virthread.h b/src/util/virthread.h
index
Signed-off-by: Jonathon Jongsma
---
src/rpc/virkeepalive.h| 9 +++-
src/rpc/virnetclient.h| 41 ---
src/rpc/virnetclientprogram.h | 15 +
src/rpc/virnetclientstream.h | 9 +++-
src/rpc/virnetdaemon.h| 21
Signed-off-by: Jonathon Jongsma
---
src/util/virfile.h | 61 ++
1 file changed, 29 insertions(+), 32 deletions(-)
diff --git a/src/util/virfile.h b/src/util/virfile.h
index 641960e2ca..094c92e2b9 100644
--- a/src/util/virfile.h
+++
Signed-off-by: Jonathon Jongsma
---
src/qemu/qemu_agent.h| 8 ++-
src/qemu/qemu_alias.h| 13 ++---
src/qemu/qemu_block.h| 17 +++---
src/qemu/qemu_blockjob.h | 9 ++--
src/qemu/qemu_capabilities.h | 23
My previous series ported a bunch of internal headers to use #pragma once as
the header guard (per
https://wiki.libvirt.org/page/BiteSizedTasks#Switch_headers_to_use_.23pragma_once).
This series
ports the rest of the internal headers. The public headers are unchanged, and
the syntax check
Signed-off-by: Jonathon Jongsma
---
src/locking/domain_lock.h | 11 ---
src/locking/lock_daemon.h | 9 +++--
src/locking/lock_daemon_config.h | 7 ++-
src/locking/lock_daemon_dispatch.h | 7 ++-
src/locking/lock_driver.h | 18 +++---
> On 18 Jun 2019, at 17:43, Daniel P. Berrangé wrote:
>
> Signed-off-by: Daniel P. Berrangé
> ---
> network.go | 80 ++
> network_port.go | 233
> network_port_compat.h | 67
> network_port_wrapper.go |
On 6/17/19 1:49 PM, Cole Robinson wrote:
> I'm happy to announce the release of virt-manager 2.2.0!
>
> virt-manager is a desktop application for managing KVM, Xen, and LXC
> virtualization via libvirt.
>
> The release can be downloaded from:
>
> http://virt-manager.org/download/
>
A note to
Signed-off-by: Jonathon Jongsma
---
src/vz/vz_driver.h | 5 +
src/vz/vz_sdk.h| 9 +++--
src/vz/vz_utils.h | 37 +
3 files changed, 21 insertions(+), 30 deletions(-)
diff --git a/src/vz/vz_driver.h b/src/vz/vz_driver.h
index
Signed-off-by: Jonathon Jongsma
---
src/vmx/vmx.h | 21 +
1 file changed, 9 insertions(+), 12 deletions(-)
diff --git a/src/vmx/vmx.h b/src/vmx/vmx.h
index 8c068b4cb2..18478ac0ae 100644
--- a/src/vmx/vmx.h
+++ b/src/vmx/vmx.h
@@ -19,14 +19,13 @@
*
*/
-#ifndef
Signed-off-by: Jonathon Jongsma
---
src/util/virerror.h | 41 +++--
src/util/virerrorpriv.h | 5 +
2 files changed, 20 insertions(+), 26 deletions(-)
diff --git a/src/util/virerror.h b/src/util/virerror.h
index 8f51510dc2..4cfb0c18dd 100644
---
Signed-off-by: Jonathon Jongsma
---
src/vmware/vmware_conf.h | 19 ---
src/vmware/vmware_driver.h | 5 +
2 files changed, 9 insertions(+), 15 deletions(-)
diff --git a/src/vmware/vmware_conf.h b/src/vmware/vmware_conf.h
index 93b424ef2f..deeed22300 100644
---
Signed-off-by: Jonathon Jongsma
---
src/util/virxml.h | 33 +++--
1 file changed, 15 insertions(+), 18 deletions(-)
diff --git a/src/util/virxml.h b/src/util/virxml.h
index 38031b62c5..6208977dd1 100644
--- a/src/util/virxml.h
+++ b/src/util/virxml.h
@@ -18,18 +18,17
Signed-off-by: Jonathon Jongsma
---
src/util/virnetlink.h | 37 +
1 file changed, 17 insertions(+), 20 deletions(-)
diff --git a/src/util/virnetlink.h b/src/util/virnetlink.h
index 1bc075a86f..6419a775e1 100644
--- a/src/util/virnetlink.h
+++
Signed-off-by: Jonathon Jongsma
---
src/util/virprobe.h | 63 +
1 file changed, 30 insertions(+), 33 deletions(-)
diff --git a/src/util/virprobe.h b/src/util/virprobe.h
index 5635e58252..80a5f48ec8 100644
--- a/src/util/virprobe.h
+++
Signed-off-by: Jonathon Jongsma
---
src/util/virarptable.h | 7 ++
src/util/viraudit.h | 15 +---
src/util/virauth.h | 8 +++---
src/util/virauthconfig.h | 9 +++
src/util/virautoclean.h | 17 ++---
src/util/virbitmap.h | 11
Signed-off-by: Jonathon Jongsma
---
src/util/virmacaddr.h | 17 +++--
src/util/virmacmap.h | 5 +
2 files changed, 8 insertions(+), 14 deletions(-)
diff --git a/src/util/virmacaddr.h b/src/util/virmacaddr.h
index 55f9d61abe..55cfd12164 100644
--- a/src/util/virmacaddr.h
+++
Signed-off-by: Jonathon Jongsma
---
src/util/vircommand.h | 11 ---
src/util/vircommandpriv.h | 7 ++-
2 files changed, 6 insertions(+), 12 deletions(-)
diff --git a/src/util/vircommand.h b/src/util/vircommand.h
index c03da43f25..c9a8d3c41c 100644
--- a/src/util/vircommand.h
Signed-off-by: Jonathon Jongsma
---
src/util/virstring.h | 37 +
1 file changed, 17 insertions(+), 20 deletions(-)
diff --git a/src/util/virstring.h b/src/util/virstring.h
index b2cd8ea53c..5e64ad1bb9 100644
--- a/src/util/virstring.h
+++
Signed-off-by: Jonathon Jongsma
---
src/util/virsocketaddr.h | 45 +++-
1 file changed, 21 insertions(+), 24 deletions(-)
diff --git a/src/util/virsocketaddr.h b/src/util/virsocketaddr.h
index 5230cddf64..b2ecb3c748 100644
--- a/src/util/virsocketaddr.h
+++
Signed-off-by: Jonathon Jongsma
---
tools/nss/libvirt_nss.h | 27 -
tools/virsh-completer.h | 6 +-
tools/virsh-console.h| 11 ++--
tools/virsh-domain-monitor.h | 7 +--
tools/virsh-domain.h | 7 +--
tools/virsh-host.h
Signed-off-by: Jonathon Jongsma
---
src/util/vircgroup.h| 10 --
src/util/vircgroupbackend.h | 15 ++-
src/util/vircgroupv1.h | 5 +
src/util/vircgroupv2.h | 5 +
4 files changed, 12 insertions(+), 23 deletions(-)
diff --git a/src/util/vircgroup.h
Signed-off-by: Jonathon Jongsma
---
src/util/virdbus.h | 22 ++
src/util/virdbuspriv.h | 11 ---
2 files changed, 14 insertions(+), 19 deletions(-)
diff --git a/src/util/virdbus.h b/src/util/virdbus.h
index c1e5de31ab..083c074d59 100644
--- a/src/util/virdbus.h
Signed-off-by: Jonathon Jongsma
---
src/util/virstorageencryption.h | 17 +++--
src/util/virstoragefile.h| 31 ++-
src/util/virstoragefilebackend.h | 9 +++--
3 files changed, 24 insertions(+), 33 deletions(-)
diff --git
Signed-off-by: Jonathon Jongsma
---
src/util/viralloc.h | 101 +---
1 file changed, 49 insertions(+), 52 deletions(-)
diff --git a/src/util/viralloc.h b/src/util/viralloc.h
index 92c71d9a1a..2b82096fde 100644
--- a/src/util/viralloc.h
+++
Signed-off-by: Jonathon Jongsma
---
src/util/virhostcpu.h | 17 +++--
src/util/virhostcpupriv.h | 11 ---
src/util/virhostdev.h | 19 ---
src/util/virhostmem.h | 7 ++-
4 files changed, 21 insertions(+), 33 deletions(-)
diff --git
Signed-off-by: Jonathon Jongsma
---
src/util/virarch.h | 41 +++--
1 file changed, 19 insertions(+), 22 deletions(-)
diff --git a/src/util/virarch.h b/src/util/virarch.h
index 638ca9f484..233b83f31b 100644
--- a/src/util/virarch.h
+++ b/src/util/virarch.h
@@
Signed-off-by: Jonathon Jongsma
---
src/phyp/phyp_driver.h | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/src/phyp/phyp_driver.h b/src/phyp/phyp_driver.h
index 52a5d2bd17..d7076e3697 100644
--- a/src/phyp/phyp_driver.h
+++ b/src/phyp/phyp_driver.h
@@ -19,9 +19,6 @@
*
Signed-off-by: Jonathon Jongsma
---
src/util/virfirewall.h | 11 ---
src/util/virfirewalld.h | 7 ++-
src/util/virfirewalldpriv.h | 7 ++-
src/util/virfirewallpriv.h | 7 ++-
4 files changed, 10 insertions(+), 22 deletions(-)
diff --git a/src/util/virfirewall.h
Signed-off-by: Jonathon Jongsma
---
src/openvz/openvz_conf.h | 21 +
src/openvz/openvz_driver.h | 11 ---
src/openvz/openvz_util.h | 5 +
3 files changed, 14 insertions(+), 23 deletions(-)
diff --git a/src/openvz/openvz_conf.h b/src/openvz/openvz_conf.h
Signed-off-by: Jonathon Jongsma
---
src/security/security_apparmor.h | 13 +
src/security/security_dac.h | 7 ++-
src/security/security_driver.h | 11 ---
src/security/security_manager.h | 13 +
src/security/security_nop.h | 7 ++-
Signed-off-by: Jonathon Jongsma
---
src/nwfilter/nwfilter_dhcpsnoop.h | 6 ++
src/nwfilter/nwfilter_driver.h| 9 +++--
src/nwfilter/nwfilter_ebiptables_driver.h | 13 +
src/nwfilter/nwfilter_gentech_driver.h| 11 ---
Signed-off-by: Jonathon Jongsma
---
src/remote/remote_daemon.h | 35 +---
src/remote/remote_daemon_config.h | 7 ++
src/remote/remote_daemon_dispatch.h | 11 -
src/remote/remote_daemon_stream.h | 7 ++
src/remote/remote_driver.h |
Signed-off-by: Jonathon Jongsma
---
src/secret/secret_driver.h | 5 +
src/secret/secret_util.h | 8 +++-
2 files changed, 4 insertions(+), 9 deletions(-)
diff --git a/src/secret/secret_driver.h b/src/secret/secret_driver.h
index b7e46ee045..3dcf420792 100644
---
Signed-off-by: Jonathon Jongsma
---
src/util/viratomic.h | 91 +---
1 file changed, 44 insertions(+), 47 deletions(-)
diff --git a/src/util/viratomic.h b/src/util/viratomic.h
index 0b2b3df081..35800dafcd 100644
--- a/src/util/viratomic.h
+++
Signed-off-by: Jonathon Jongsma
---
src/network/bridge_driver.h | 15 ++-
src/network/bridge_driver_platform.h | 15 ++-
2 files changed, 12 insertions(+), 18 deletions(-)
diff --git a/src/network/bridge_driver.h b/src/network/bridge_driver.h
index
Signed-off-by: Jonathon Jongsma
---
src/lxc/lxc_cgroup.h| 13 +---
src/lxc/lxc_conf.h | 47 +++--
src/lxc/lxc_container.h | 37 +++-
src/lxc/lxc_domain.h| 13 +---
src/lxc/lxc_driver.h| 5 +
when a disk without PR perform attach or detach operation,
need not call qemuHotplugRemoveManagedPR, otherwise, it will
print err log about PR, let us fix it.
Signed-off-by: Jie Wang
---
src/qemu/qemu_hotplug.c | 7 +--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git
Signed-off-by: Daniel P. Berrangé
---
network.go | 80 ++
network_port.go | 233
network_port_compat.h | 67
network_port_wrapper.go | 197 +
network_port_wrapper.h | 79
On Thu, Jun 13, 2019 at 04:06:59PM +0200, Ilias Stamatis wrote:
Signed-off-by: Ilias Stamatis
---
src/test/test_driver.c | 46 ++
1 file changed, 46 insertions(+)
diff --git a/src/test/test_driver.c b/src/test/test_driver.c
index 8ef843b203..9e8236a5b6
Oops, I forgot about this patch.
On Sun, Apr 28, 2019 at 01:25:05PM +0800, Yi Li wrote:
From 4af765cb6ee87eb7a131901057a8b6d0e859ac63 Mon Sep 17 00:00:00 2001
From: Yi Li
Date: Sun, 28 Apr 2019 10:29:53 +0800
Subject: [PATCH v2] storage: escape ipv6 for ceph mon hosts to librados
Hosts for
On Thu, Apr 25, 2019 at 09:44:32AM +0200, Pavel Hrdina wrote:
We need to mock virCgroupV2DevicesAvailable() in order to remove any
dependency on kernel as BPF devices might not be available.
Signed-off-by: Pavel Hrdina
---
src/util/vircgroupv2devices.h | 5 -
On Thu, Apr 25, 2019 at 09:44:31AM +0200, Pavel Hrdina wrote:
So the issue here is that you can end up with configuration where
you have cgroup v1 and v2 enabled at the same time and the devices
controllers is enabled for cgroup v1.
In cgroup v2 there is no devices controller, the device access
On Thu, 6 Jun 2019 10:15:52 -0600
Alex Williamson wrote:
> On Thu, 6 Jun 2019 09:32:24 -0600
> Alex Williamson wrote:
>
> > On Thu, 6 Jun 2019 16:44:17 +0200
> > Cornelia Huck wrote:
> >
> > > Add a rough implementation for vfio-ap.
> > >
> > > Signed-off-by: Cornelia Huck
> > > ---
> >
On Thu, Apr 25, 2019 at 09:44:30AM +0200, Pavel Hrdina wrote:
If we want to deny all devices we just need to replace any existing
program with new program with empty map.
Signed-off-by: Pavel Hrdina
---
src/util/vircgroupv2.c | 11 +++
1 file changed, 11 insertions(+)
Reviewed-by:
On Thu, Apr 25, 2019 at 09:44:29AM +0200, Pavel Hrdina wrote:
If we want to allow all devices with all permissions we need to replace
any existing program that has any rule configured, otherwise we just
need to add new rule which will for example allow read access to all
devices.
Signed-off-by:
On Tue, Jun 18, 2019 at 04:45:25PM +0200, Andrea Bolognani wrote:
> On Tue, 2019-06-18 at 15:18 +0100, Daniel P. Berrangé wrote:
> > On Tue, Jun 18, 2019 at 03:33:33PM +0200, Andrea Bolognani wrote:
> > > This replaces the existing Planet Virt Tools blog roll.
> > >
> > > Signed-off-by: Andrea
On Tue, 2019-06-18 at 16:47 +0200, Peter Krempa wrote:
> On Tue, Jun 18, 2019 at 15:18:53 +0100, Daniel Berrange wrote:
> > I'm a nack on this change as it is significantly worse than
> > what we have today.
>
> How about at least hosting the JS stuff from somewhere else on
> libvirt.org than the
On Thu, Apr 25, 2019 at 09:44:28AM +0200, Pavel Hrdina wrote:
In order to deny device we need to check if there is any entry in BPF
map and we need to load the current value from map if there is already
entry for that device. If both values are same we can remove that entry
but if they are
On Thu, Apr 25, 2019 at 09:44:27AM +0200, Pavel Hrdina wrote:
In order to allow device we need to create key and value which will be
used to update BPF map. virBPFUpdateElem() can override existing
entries in BPF map so we need to check if that entry exists in order to
track number of entries
On Thu, Apr 25, 2019 at 09:44:26AM +0200, Pavel Hrdina wrote:
Device rules are stored in BPF map that is a hash type, this function
will create a key based on major and minor id of device.
Signed-off-by: Pavel Hrdina
---
src/libvirt_private.syms | 1 +
src/util/vircgroupv2devices.c | 8
On Thu, Apr 25, 2019 at 09:44:25AM +0200, Pavel Hrdina wrote:
Signed-off-by: Pavel Hrdina
---
src/libvirt_private.syms | 1 +
src/util/vircgroupv2devices.c | 34 ++
src/util/vircgroupv2devices.h | 6 ++
3 files changed, 41 insertions(+)
diff --git
On Thu, Apr 25, 2019 at 09:44:24AM +0200, Pavel Hrdina wrote:
We need to close our FD that we have for BPF program and map in order
to let kernel remove all resources once the cgroup is removed as well.
Signed-off-by: Pavel Hrdina
---
src/libvirt_private.syms | 1 +
src/util/vircgroupv2.c
On Thu, Apr 25, 2019 at 09:44:23AM +0200, Pavel Hrdina wrote:
This function will be called for every virCgroup(Allow|Deny)* API in
order to prepare BPF program for guest. Since libvirtd can be restarted
at any point we will first try to detect existing progam, if there is
none we will create a
On Thu, Apr 25, 2019 at 09:44:22AM +0200, Pavel Hrdina wrote:
This function creates new BPF program with new empty BPF map with the
default size and attaches it to the guest cgroup.
Signed-off-by: Pavel Hrdina
---
src/libvirt_private.syms | 1 +
src/util/vircgroupv2devices.c | 53
On Tue, Jun 11, 2019 at 14:01:25 +0200, Peter Krempa wrote:
> Peter Krempa (2):
> util: error: Add API for prefixing last set error with a string
> qemu: process: Report better error when virtlogd connection fails
ping?
signature.asc
Description: PGP signature
--
libvir-list mailing list
On Thu, Apr 25, 2019 at 09:44:21AM +0200, Pavel Hrdina wrote:
This function will be called if libvirtd was restarted while some
domains were running. It will try to detect existing programs attached
to the guest cgroup.
Signed-off-by: Pavel Hrdina
---
src/libvirt_private.syms | 1 +
On Tue, Jun 18, 2019 at 15:18:53 +0100, Daniel Berrange wrote:
> On Tue, Jun 18, 2019 at 03:33:33PM +0200, Andrea Bolognani wrote:
> > This replaces the existing Planet Virt Tools blog roll.
> >
> > Signed-off-by: Andrea Bolognani
> > ---
> > docs/index.html.in | 19 +--
> >
On Tue, 2019-06-18 at 15:18 +0100, Daniel P. Berrangé wrote:
> On Tue, Jun 18, 2019 at 03:33:33PM +0200, Andrea Bolognani wrote:
> > This replaces the existing Planet Virt Tools blog roll.
> >
> > Signed-off-by: Andrea Bolognani
> > ---
> > docs/index.html.in | 19 +--
> >
1 - 100 of 172 matches
Mail list logo