In a few places (e.g. device attach/detach/update) we are given a
device XML, parse it but then need a copy of parsed data so that
the original can be passed to function handling the request over
inactive XML and the copy is then passed to function handling the
operation over live XML. Note, both
Notable changes:
* drop parted and XFS headers.
Signed-off-by: Andrea Bolognani
---
Pushed under the CI refresh rule.
Test pipeline: https://gitlab.com/abologna/libvirt/-/pipelines/441266800
Full patch:
https://gitlab.com/libvirt/libvirt/-/commit/4c7316b2f00c847bc5dc9c8218c2585d2d04636a
In order to hopefully address [libvirt#147] at long last, I've picked
up Roman's patches from 2018 and attempted to forward-port them.
More specifically, I've used the [roolebo/hvf-domain] branch as a
starting point, since it seems to contain a few improvements over
[v2] and was just easier to
From: Roman Bolshakov
QEMU supports Hypervisor.framework since 2.12 as hvf accel.
Hypervisor.framework provides a lightweight interface to run a virtual
cpu on macOS without the need to install third-party kernel
extensions (KEXTs).
It's supported since macOS 10.10 on machines with Intel VT-x
From: Roman Bolshakov
It's worth to make the domain type a little bit more visible than a row
in news. An example of hvf domain is available on QEMU driver page.
While at it, mention Hypervisor.framework on index page.
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
From: Roman Bolshakov
Many domain elements have "QEMU and KVM only" or "QEMU/KVM since x.y.z"
remarks. Most of the elements work for HVF domain, so it makes sense to
add respective notices for HVF domain.
All the elements have been manually tested.
Signed-off-by: Roman Bolshakov
From: Roman Bolshakov
With this change virsh domcapabilites shows:
https://gitlab.com/libvirt/libvirt/-/issues/147
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 25 ++---
src/qemu/qemu_process.c | 2 +-
2 files
We need to use a hardcoded list of capabilities because we don't
yet have proper replies files obtained from QEMU running on actual
macOS machines.
Signed-off-by: Andrea Bolognani
---
.../hvf-aarch64-virt-headless.args| 48 +
.../hvf-aarch64-virt-headless.xml |
From: Roman Bolshakov
Signed-off-by: Roman Bolshakov
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index
From: Roman Bolshakov
It replaces hardcoded checks that select accelCPU/accelCPUModels
(formerly known as kvmCPU/kvmCPUModels) for KVM. It'll be cleaner to use
the function when multiple accelerators are supported in qemu driver.
Explicit KVM domain checks should be done only when a feature is
From: Roman Bolshakov
The function should be used to check if qemu capabilities include a
hardware acceleration, i.e. accel is not TCG.
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 16 +---
1 file changed, 13 insertions(+), 3
From: Roman Bolshakov
This makes possible to add more accelerators by touching less code and
reduces code duplication.
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 14 --
1 file changed, 12 insertions(+), 2 deletions(-)
diff
From: Roman Bolshakov
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 1 +
src/qemu/qemu_capabilities.h | 1 +
2 files changed, 2 insertions(+)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
index
From: Roman Bolshakov
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
NEWS.rst | 8
1 file changed, 8 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index 313b4f52b6..29a39da9dd 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -47,6 +47,14 @@ v8.0.0 (unreleased)
Signed-off-by: Andrea Bolognani
---
NEWS.rst | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/NEWS.rst b/NEWS.rst
index 29a39da9dd..2e1b576979 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -53,7 +53,8 @@ v8.0.0 (unreleased)
since 2.12.
It's supported on machines with
Apple Silicon (aarch64) has HVF support, but there is no
32-bit Intel hardware that is HVF-capable.
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/src/qemu/qemu_capabilities.c b/src/qemu/qemu_capabilities.c
Signed-off-by: Andrea Bolognani
---
tests/testutilsqemu.c | 40
1 file changed, 40 insertions(+)
diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c
index 7fdb82daec..a75995c77a 100644
--- a/tests/testutilsqemu.c
+++ b/tests/testutilsqemu.c
@@
From: Roman Bolshakov
There's no QMP command for querying if hvf is supported, therefore we
use sysctl interface that tells if Hypervisor.framework works/available
on the host.
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 33
On 04/01/22 6:27 pm, Ani Sinha wrote:
On Tue, 4 Jan 2022, Rohit Kumar wrote:
This change adds the domain name in the error and debug logs during
monitor IO processing so that we may infer which VM experienced
errors such as IO or socket hangup. This may help in debugging
monitor IO errors.
Mon, 3 Jan 2022 03:18:11 -0800 Andrea Bolognani :
> The fact that we still QEMU_AUDIO_DRV and SDL_AUDIODRIVER in the
> service file even after your changes goes against this principle.
These variables are probably stale.
But, I just looked briefly at the code and could not figure out within
From: Roman Bolshakov
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
docs/docs.html.in | 3 +
docs/index.html.in | 3 +-
docs/macos.html.in | 229 +
3 files changed, 234 insertions(+), 1 deletion(-)
create mode 100644
From: Roman Bolshakov
With more acceleration types, KVM should be used only in error messages
related to KVM.
Signed-off-by: Roman Bolshakov
Signed-off-by: Andrea Bolognani
---
src/qemu/qemu_capabilities.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
On 04/01/22 7:27 pm, Daniel P. Berrangé wrote:
On Tue, Jan 04, 2022 at 11:47:39AM +, John Levon wrote:
On Sat, Dec 11, 2021 at 07:57:47PM -0800, “Divya wrote:
From: Divya Garg
VM XML accepts target.port but this does not get passed while building the qemu
command line for this VM.
On Tue, 4 Jan 2022, Rohit Kumar wrote:
>
> On 03/01/22 7:12 pm, Ani Sinha wrote:
> >
> > On Wed, 22 Dec 2021, Rohit Kumar wrote:
> >
> > > This patch is to determine the VM which had IO or socket hangup error.
> > > Accessing directly vm->def->name inside qemuMonitorIO() or
> > >
On Tue, Jan 04, 2022 at 15:30:00 +0530, Ani Sinha wrote:
> On Tue, 4 Jan 2022, Rohit Kumar wrote:
> > On 03/01/22 7:12 pm, Ani Sinha wrote:
> > > On Wed, 22 Dec 2021, Rohit Kumar wrote:
[...]
> > > > @@ -694,6 +702,7 @@ qemuMonitorOpenInternal(virDomainObj *vm,
> > > > mon->fd = fd;
> > >
On Tue, 4 Jan 2022, Peter Krempa wrote:
> On Tue, Jan 04, 2022 at 15:30:00 +0530, Ani Sinha wrote:
> > On Tue, 4 Jan 2022, Rohit Kumar wrote:
> > > On 03/01/22 7:12 pm, Ani Sinha wrote:
> > > > On Wed, 22 Dec 2021, Rohit Kumar wrote:
>
> [...]
>
> > > > > @@ -694,6 +702,7 @@
The virDomainTPMDefFormat() function can't fail really. There's
no point in it returning an integer then. Make it return void and
fix both places which check for its retval.
Signed-off-by: Michal Privoznik
---
src/conf/domain_conf.c | 12 +---
1 file changed, 5 insertions(+), 7
On Tue, Jan 04, 2022 at 09:14:28 +0100, Michal Privoznik wrote:
> Make the tpm-*.xml files symlinks to their respective input XMLs
> from qemuxml2argvdata/ directory. This uncovers a bug in our
> formatter which formats an invalid XML if both
> and elements are present for
> . This is going to
On Tue, Jan 04, 2022 at 09:31:39 +0100, Michal Prívozník wrote:
> On 1/4/22 09:30, Peter Krempa wrote:
> > On Tue, Jan 04, 2022 at 09:14:28 +0100, Michal Privoznik wrote:
> >> Make the tpm-*.xml files symlinks to their respective input XMLs
> >> from qemuxml2argvdata/ directory. This uncovers a
On 03/01/22 10:12 pm, Peter Krempa wrote:
On Wed, Dec 22, 2021 at 22:39:21 -0800, Rohit Kumar wrote:
This patch is to determine the VM which had IO or socket hangup error.
Accessing directly vm->def->name inside qemuMonitorIO() or qemuMonitorSend()
might leads to illegal access as we are out
On Tue, Jan 04, 2022 at 09:14:27 +0100, Michal Privoznik wrote:
> We already have the input xml because of xml2arg test. However,
> the corresponding xml2xml test case is missing. Make the expected
> XML a symlink to the input XML and clean the latter up a bit.
>
> Signed-off-by: Michal Privoznik
On Tue, Jan 04, 2022 at 09:45:30 +0100, Peter Krempa wrote:
> On Tue, Jan 04, 2022 at 14:10:49 +0530, Rohit Kumar wrote:
> >
> > On 03/01/22 10:12 pm, Peter Krempa wrote:
> > > On Wed, Dec 22, 2021 at 22:39:21 -0800, Rohit Kumar wrote:
> > > > This patch is to determine the VM which had IO or
On 04/01/22 2:17 pm, Peter Krempa wrote:
On Tue, Jan 04, 2022 at 09:45:30 +0100, Peter Krempa wrote:
On Tue, Jan 04, 2022 at 14:10:49 +0530, Rohit Kumar wrote:
On 03/01/22 10:12 pm, Peter Krempa wrote:
On Wed, Dec 22, 2021 at 22:39:21 -0800, Rohit Kumar wrote:
This patch is to determine
On Mon, Jan 03, 2022 at 03:59:42PM -0700, Jim Fehlig wrote:
> + * libxl: Fix potential deadlock and crash (CVE-2021-4147)
> +
> +A rouge guest could continuously reboot itself and cause libvirtd on the
*rogue
although introducing a virDomainSetColor() or virDomainApplyMakeup()
API could be
The element formatting is handled in
virDomainTPMDefFormat() which uses the "old style" - appending
strings directly into the output buffer. With this, it's easy to
get conditions that tell when an element has ended wrong. In this
particular case, if both and
are to be formatted the current
Please note that the test suite is temporarily broken after 2/4 but
fixed in 3/4. This could be resolved be swapping those two patches, but
I figured I keep the order to demonstrate the bug. However, I can do the
swap if desired.
Michal Prívozník (4):
qemuxml2xmltest: Introduce
Make the tpm-*.xml files symlinks to their respective input XMLs
from qemuxml2argvdata/ directory. This uncovers a bug in our
formatter which formats an invalid XML if both
and elements are present for
. This is going to be addressed in the next commit.
Signed-off-by: Michal Privoznik
---
We already have the input xml because of xml2arg test. However,
the corresponding xml2xml test case is missing. Make the expected
XML a symlink to the input XML and clean the latter up a bit.
Signed-off-by: Michal Privoznik
---
tests/qemuxml2argvdata/tpm-emulator-spapr.xml | 74
Changing means changing the guest ABI and as
such must be prevented on both restoring from a file or
migration.
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2035888
Signed-off-by: Michal Privoznik
---
src/conf/domain_conf.c | 14 ++
1 file changed, 14 insertions(+)
diff
On 03/01/22 7:12 pm, Ani Sinha wrote:
On Wed, 22 Dec 2021, Rohit Kumar wrote:
This patch is to determine the VM which had IO or socket hangup error.
Accessing directly vm->def->name inside qemuMonitorIO() or qemuMonitorSend()
might leads to illegal access as we are out of 'vm' context and
On 1/4/22 09:30, Peter Krempa wrote:
> On Tue, Jan 04, 2022 at 09:14:28 +0100, Michal Privoznik wrote:
>> Make the tpm-*.xml files symlinks to their respective input XMLs
>> from qemuxml2argvdata/ directory. This uncovers a bug in our
>> formatter which formats an invalid XML if both
>> and
On 1/4/22 09:33, Peter Krempa wrote:
> https://www.libvirt.org/hacking.html#preparing-patches
>
> "If you're going to submit multiple patches, the automated tests must
> pass after each patch, not just after the last one."
>
Alright then. Let me swap those two patches and post v2.
Michal
On Tue, Jan 04, 2022 at 09:29:15 +0100, Michal Privoznik wrote:
> Changing means changing the guest ABI and as
> such must be prevented on both restoring from a file or
> migration.
>
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=2035888
> Signed-off-by: Michal Privoznik
> ---
>
On Tue, Jan 04, 2022 at 14:10:49 +0530, Rohit Kumar wrote:
>
> On 03/01/22 10:12 pm, Peter Krempa wrote:
> > On Wed, Dec 22, 2021 at 22:39:21 -0800, Rohit Kumar wrote:
> > > This patch is to determine the VM which had IO or socket hangup error.
> > > Accessing directly vm->def->name inside
On Tue, Jan 04, 2022 at 09:14:29 +0100, Michal Privoznik wrote:
> The element formatting is handled in
> virDomainTPMDefFormat() which uses the "old style" - appending
> strings directly into the output buffer. With this, it's easy to
> get conditions that tell when an element has ended wrong. In
On Tue, Jan 04, 2022 at 09:14:30 +0100, Michal Privoznik wrote:
> The virDomainTPMDefFormat() function can't fail really. There's
> no point in it returning an integer then. Make it return void and
> fix both places which check for its retval.
>
> Signed-off-by: Michal Privoznik
> ---
>
v2 of:
https://listman.redhat.com/archives/libvir-list/2022-January/msg00047.html
diff to v1:
- Pushed 1/4 from the original series, because it was acked and
independent of the rest.
- Swapped two patches to make the test suite pass after each single
patch.
- Renamed variable in 1/3 (3/4 in the
The element formatting is handled in
virDomainTPMDefFormat() which uses the "old style" - appending
strings directly into the output buffer. With this, it's easy to
get conditions that tell when an element has ended wrong. In this
particular case, if both and
are to be formatted the current
Make the tpm-*.xml files symlinks to their respective input XMLs
from qemuxml2argvdata/ directory. Neither of the XMLs relies on
autofill of any TPM data.
Signed-off-by: Michal Privoznik
---
.../tpm-emulator-tpm2-enc.xml | 12 -
.../tpm-emulator-tpm2-pstate.xml
The virDomainTPMDefFormat() function can't fail really. There's
no point in it returning an integer then. Make it return void and
fix both places which check for its retval.
And while at it, turn @def into a const pointer to make it
obvious the function does not modify passed struct.
On Tue, Jan 04, 2022 at 10:04:59 +0100, Michal Privoznik wrote:
> Make the tpm-*.xml files symlinks to their respective input XMLs
> from qemuxml2argvdata/ directory. Neither of the XMLs relies on
> autofill of any TPM data.
>
> Signed-off-by: Michal Privoznik
> ---
Reviewed-by: Peter Krempa
On 04/01/22 3:56 pm, Ani Sinha wrote:
On Tue, 4 Jan 2022, Peter Krempa wrote:
On Tue, Jan 04, 2022 at 15:30:00 +0530, Ani Sinha wrote:
On Tue, 4 Jan 2022, Rohit Kumar wrote:
On 03/01/22 7:12 pm, Ani Sinha wrote:
On Wed, 22 Dec 2021, Rohit Kumar wrote:
[...]
@@ -694,6 +702,7 @@
On Mon, Dec 20, 2021 at 12:02:41PM +0100, Michal Privoznik wrote:
> The virt_socket_lib is built from virnetsocket.c (among others).
> But this file includes virprobe.h which includes libvirt_probes.h
> which is a generated file. But this dependency is not recorded in
> meson which may lead to a
On Tue, Dec 21, 2021 at 03:22:59PM +0100, Michal Privoznik wrote:
> As encryption norms get more strict it's easy to fall on the
> insecure side. For instance, so far we are generating 2048 bits
> long prime for Diffie-Hellman keys. Some systems consider this
> not long enough. While we may just
On Mon, Dec 20, 2021 at 08:23:18PM -0700, Jim Fehlig wrote:
> Set a launch secret in guest memory using the sev-inject-launch-secret
> QMP API. Only supported with qemu >= 6.0.0 and SEV-enabled guests in a
> paused state.
>
> Signed-off-by: Jim Fehlig
> ---
>
> Daniel already r-b V3 of this
On Mon, Dec 20, 2021 at 08:23:17PM -0700, Jim Fehlig wrote:
> The 'sev-inject-launch-secret' qmp command is only available with
> qemu >= 6.0.0. Introduce a capability for sev-inject-launch-secret.
>
> Signed-off-by: Jim Fehlig
> ---
> src/qemu/qemu_capabilities.c | 2 ++
>
This prevents starting any daemons with improper logging settings. This is
desirable on its own, but will be even more beneficial when more functions start
reporting errors and failing on them, coming up in following patches
Signed-off-by: Martin Kletzander
---
src/locking/lock_daemon.c | 15
Signed-off-by: Martin Kletzander
---
src/util/virlog.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/util/virlog.c b/src/util/virlog.c
index bf791d901a24..e368cada6024 100644
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -178,6 +178,7 @@ virLogSetDefaultOutputToFile(const char
Signed-off-by: Martin Kletzander
---
src/util/virlog.c | 8 +---
src/util/virlog.h | 2 +-
2 files changed, 6 insertions(+), 4 deletions(-)
diff --git a/src/util/virlog.c b/src/util/virlog.c
index 90d3d7c5cb53..bf791d901a24 100644
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -201,7
The only difference is that we are not going to be guaranteed that the mutex is
normal (as opposed to recursive, although there is no system known to me that
would default to recursive mutexes), but that was done only to find occasional
errors (during runtime, back in 2010, commit 336fd879c00b).
This unclutters the top-level docs directory.
Signed-off-by: Andrea Bolognani
Reviewed-by: Ani Sinha
---
docs/api.rst | 6 ++---
docs/goals.html.in| 2 +-
docs/{ => images}/event_loop_simple.png | Bin
docs/{ =>
We have a subdirectory specifically for CSS files now, so it makes
sense to have the stylesheet that defines fonts to be there too.
Signed-off-by: Andrea Bolognani
---
docs/{fonts/stylesheet.css => css/fonts.css} | 18 +-
docs/css/main.css| 2 +-
This unclutters the top-level docs directory.
Signed-off-by: Andrea Bolognani
---
docs/{ => css}/generic.css | 0
docs/{ => css}/libvirt.css | 4 ++--
docs/{ => css}/main.css| 2 +-
docs/css/meson.build | 16
docs/{ => css}/mobile.css | 2 +-
docs/meson.build
Changes from [v2]:
* drop all changes related to icons.
Changes from [v1]:
* extend scope to include icons and CSS files.
[v2] https://listman.redhat.com/archives/libvir-list/2021-December/msg00828.html
[v1] https://listman.redhat.com/archives/libvir-list/2021-December/msg00818.html
It was introduced in ff4ede005567 but it doesn't seem to have
ever actually been used anywhere.
Signed-off-by: Andrea Bolognani
Reviewed-by: Ani Sinha
---
docs/structures.svg | 187
1 file changed, 187 deletions(-)
delete mode 100644
On Mon, Dec 20, 2021 at 08:23:20PM -0700, Jim Fehlig wrote:
> Signed-off-by: Jim Fehlig
> ---
> NEWS.rst | 6 ++
> 1 file changed, 6 insertions(+)
Reviewed-by: Daniel P. Berrangé
Regards,
Daniel
--
|: https://berrange.com -o-https://www.flickr.com/photos/dberrange :|
|:
On Tue, Jan 04, 2022 at 12:24:25 +, Daniel P. Berrangé wrote:
> On Mon, Jan 03, 2022 at 04:25:54PM +0100, Peter Krempa wrote:
> > When -device is configured via JSON a bug is triggered in qemu were the
> > DEVICE_DELETED event for the removal of the device frontend is no longer
> > delivered
On Tue, Jan 04, 2022 at 02:56:00PM +0100, Andrea Bolognani wrote:
> This unclutters the top-level docs directory.
>
> Signed-off-by: Andrea Bolognani
> Reviewed-by: Ani Sinha
> ---
> docs/api.rst | 6 ++---
> docs/goals.html.in|
On Tue, Jan 04, 2022 at 02:56:02PM +0100, Andrea Bolognani wrote:
> We have a subdirectory specifically for CSS files now, so it makes
> sense to have the stylesheet that defines fonts to be there too.
>
> Signed-off-by: Andrea Bolognani
> ---
> docs/{fonts/stylesheet.css => css/fonts.css} | 18
On Tue, Jan 04, 2022 at 02:56:01PM +0100, Andrea Bolognani wrote:
> This unclutters the top-level docs directory.
>
> Signed-off-by: Andrea Bolognani
> ---
> docs/{ => css}/generic.css | 0
> docs/{ => css}/libvirt.css | 4 ++--
> docs/{ => css}/main.css| 2 +-
> docs/css/meson.build
On Tue, 4 Jan 2022, Divya Garg wrote:
> On 04/01/22 5:47 pm, Ani Sinha wrote:
> >
> > On Mon, 3 Jan 2022, Divya Garg wrote:
> >
> > > Thankyou Ani for the review. I will be taking up the comments
> > > in next patchset along with other comments.
> > >
> > > On 03/01/22 1:44 pm, Ani Sinha
On Tue, Jan 04, 2022 at 09:14:26AM +0100, Michal Privoznik wrote:
> Please note that the test suite is temporarily broken after 2/4 but
> fixed in 3/4. This could be resolved be swapping those two patches, but
> I figured I keep the order to demonstrate the bug. However, I can do the
> swap if
On Tue, Jan 04, 2022 at 01:53:05PM +0100, Michal Privoznik wrote:
The burst attribute for bandwidth specifies how much bytes can be
transmitted in a single burst. Therefore, the unit is in
multiples of 1024 (thus kibibytes) not SI-like 1000. It always
has been like that.
s/always has/has
On Tue, Dec 14, 2021 at 03:01:18PM +0100, Michal Privoznik wrote:
> qemuxml2xmloutdata: Turn kvm-features.xml and kvm-features-off.xml into
> symlinks
Maybe change this to
qemuxml2xmloutdata: Turn kvm-features*.xml into symlinks
for brevity's sake.
Either way,
Reviewed-by: Andrea
On Mon, 3 Jan 2022, Divya Garg wrote:
> Thankyou Ani for the review. I will be taking up the comments
> in next patchset along with other comments.
>
> On 03/01/22 1:44 pm, Ani Sinha wrote:
>
> >
> > On Mon, 3 Jan 2022, Divya Garg wrote:
> >
> > > Hi all !
> > >
> > > Looking forward for the
Signed-off-by: Martin Kletzander
---
src/util/virlog.c | 4
1 file changed, 4 insertions(+)
diff --git a/src/util/virlog.c b/src/util/virlog.c
index ac35e36148c6..90d3d7c5cb53 100644
--- a/src/util/virlog.c
+++ b/src/util/virlog.c
@@ -1187,6 +1187,10 @@ virLogParseDefaultPriority(const
And make callers check the return value as well. This helps error out early for
invalid environment variables.
That is desirable because it could lead to deadlocks. This can happen when
resetting logging after fork() reports translated errors because gettext
functions are not reentrant. Well,
There is no need to do that since both fallible functions do that already.
Signed-off-by: Martin Kletzander
---
src/remote/remote_daemon.c | 4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/remote/remote_daemon.c b/src/remote/remote_daemon.c
index
Before this series:
# LIBVIRT_LOG_OUTPUTS=1:asdf:fdsa:meh libvirtd
After this series:
# LIBVIRT_LOG_OUTPUTS=1:asdf:fdsa:meh libvirtd
libvirt: error : invalid argument: Invalid destination 'asdf' for output
'1:asdf:fdsa:meh'
Martin Kletzander (8):
util: Report error in
Callers that already do this anyway can be cleaned up thanks to this and the one
that does not (daemon startup) gains the benefit of the error being printed to
standard error output changing:
LIBVIRT_LOG_OUTPUTS=1:invalid libvirtd
/home/nert/dev/libvirt/upstream/build/src/libvirtd: initialisation
On Sat, Dec 11, 2021 at 07:57:47PM -0800, “Divya wrote:
> From: Divya Garg
>
> VM XML accepts target.port but this does not get passed while building the
> qemu
> command line for this VM.
Apologies, I failed to notice this had been sent out to the list; Re-posting
my comments from an
On Thu, Dec 30, 2021 at 08:01:43AM +0200, Andrew Melnychenko wrote:
> Added "rss" and "rss_hash_report" configuration that should be used with
> qemu virtio RSS.
> Both options are triswitches. Used as "driver" options and affects only NIC
> with model type "virtio".
> In other patches - options
*** BLURB HERE ***
Michal Prívozník (2):
docs: Clarify 'burst' units for QoS
virnetdevopenvswitch: Fix 'burst' value passed to ovs-vsctl
docs/formatnetwork.html.in | 2 +-
src/util/virnetdevbandwidth.h | 8
src/util/virnetdevopenvswitch.c | 16 +---
3 files
As described in the previous commit, the units for 'burst' are
kibibytes and not kilobytes, i.e. multiples of 1024 not 1000.
Therefore, when constructing ovs-vsctl command the burst value
must be multiplied by 1024 and not just 1000. And because ovs
expects this size in bits the value has to be
The burst attribute for bandwidth specifies how much bytes can be
transmitted in a single burst. Therefore, the unit is in
multiples of 1024 (thus kibibytes) not SI-like 1000. It always
has been like that.
The 'tc' output is still confusing though, for instance:
# tc class add dev $DEV parent
On Tue, Jan 04, 2022 at 12:07:59PM +, Daniel P. Berrangé wrote:
> On Tue, Dec 21, 2021 at 04:30:31PM +0100, Andrea Bolognani wrote:
> > This unclutters the top-level docs directory.
> >
> > Signed-off-by: Andrea Bolognani
> > ---
> > docs/browserconfig.xml | 2 +-
> >
On 04/01/22 5:47 pm, Ani Sinha wrote:
On Mon, 3 Jan 2022, Divya Garg wrote:
Thankyou Ani for the review. I will be taking up the comments
in next patchset along with other comments.
On 03/01/22 1:44 pm, Ani Sinha wrote:
On Mon, 3 Jan 2022, Divya Garg wrote:
Hi all !
Looking forward for
On Tue, Jan 04, 2022 at 02:55:59PM +0100, Andrea Bolognani wrote:
> It was introduced in ff4ede005567 but it doesn't seem to have
> ever actually been used anywhere.
>
> Signed-off-by: Andrea Bolognani
> Reviewed-by: Ani Sinha
> ---
> docs/structures.svg | 187
On 12/28/21 19:40, Andrea Bolognani wrote:
> Test pipeline: https://gitlab.com/abologna/libvirt/-/pipelines/438075320
>
> Andrea Bolognani (3):
> storage: Use the FICLONE ioctl unconditionally on Linux
> meson: Don't look for btrfs and xfs headers
> spec: Drop BuildRequires on
On Tue, Jan 04, 2022 at 01:53:06PM +0100, Michal Privoznik wrote:
As described in the previous commit, the units for 'burst' are
kibibytes and not kilobytes, i.e. multiples of 1024 not 1000.
Therefore, when constructing ovs-vsctl command the burst value
must be multiplied by 1024 and not just
On Tue, Dec 14, 2021 at 03:01:19PM +0100, Michal Privoznik wrote:
> case VIR_TRISTATE_SWITCH_ON:
> - virBufferAsprintf(, "<%s state='on'/>\n", name);
> - break;
> -
> case VIR_TRISTATE_SWITCH_OFF:
> - virBufferAsprintf(, "<%s
On Tue, Dec 21, 2021 at 04:30:29PM +0100, Andrea Bolognani wrote:
> It's not strictly necessary when the icon lives in the top-level
> directory of the website, as browsers will fall back to that
> path when the element is absent, but it's still considered
> good practice to spell out the path
On 04/01/22 5:17 pm, John Levon wrote:
On Sat, Dec 11, 2021 at 07:57:47PM -0800, “Divya wrote:
From: Divya Garg
VM XML accepts target.port but this does not get passed while building the qemu
command line for this VM.
Apologies, I failed to notice this had been sent out to the list;
DEBUG_IO and DEBUG_RAW_IO are disabled and hence the code #defined under them
are useless. Remove them. Also remove similar useless code from
qemu_monitor_json.
See also 4aae00bf1287f ("qemu: monitor: Remove disabled debug infrastructure")
Signed-off-by: Ani Sinha
---
src/qemu/qemu_agent.c
On Tue, Jan 04, 2022 at 11:47:39AM +, John Levon wrote:
> On Sat, Dec 11, 2021 at 07:57:47PM -0800, “Divya wrote:
>
> > From: Divya Garg
> >
> > VM XML accepts target.port but this does not get passed while building the
> > qemu
> > command line for this VM.
>
> Apologies, I failed to
On Tue, Jan 04, 2022 at 02:55:58PM +0100, Andrea Bolognani wrote:
> Changes from [v2]:
>
> * drop all changes related to icons.
>
> Changes from [v1]:
>
> * extend scope to include icons and CSS files.
>
> [v2]
> https://listman.redhat.com/archives/libvir-list/2021-December/msg00828.html
This change adds the domain name in the error and debug logs during
monitor IO processing so that we may infer which VM experienced
errors such as IO or socket hangup. This may help in debugging
monitor IO errors.
Signed-off-by: Rohit Kumar
---
src/qemu/qemu_monitor.c | 36
On Sun, Jan 02, 2022 at 09:41:37PM -0500, Laine Stump wrote:
> I'm currently working on switching the backend of the network driver from
> using iptables to using nftables. Due to some functionality that is not
> available with nftables (the rule that fixes up the checksum of DHCP packets
> which,
On Tue, 4 Jan 2022, Rohit Kumar wrote:
> This change adds the domain name in the error and debug logs during
> monitor IO processing so that we may infer which VM experienced
> errors such as IO or socket hangup. This may help in debugging
> monitor IO errors.
LGTM. If you wish you can add a
On 1/4/22 12:59, Daniel P. Berrangé wrote:
> We shouldn't be introducing use of gnutls_sec_param_to_pk_bits at
> all IMHO, rather we should be removing use of gnutls_dh_params_generate2
> instead.
>
> The recommendation is to use pre-generated DH parameters from the
> the FFDHE set of RFC7919.
>
1 - 100 of 104 matches
Mail list logo
