Add support for customizable grabToggle key combinations with
.
Signed-off-by: Justin Gatzen
---
docs/formatdomain.rst | 12 +++-
docs/schemas/domaincommon.rng | 12
src/conf/domain_conf.c| 19 ++
> -Original Message-
> From: Daniel P. Berrangé
> Sent: Tuesday, July 20, 2021 5:08 PM
> To: Huang, Haibin
> Cc: libvir-list@redhat.com; Ding, Jian-feng ; Yang,
> Lin A ; Lu, Lianhao
> Subject: Re: [libvirt][PATCH v4 3/4] qemu: Add command-line to enable SGX
>
> On Thu, Jul 01, 2021
Daniel P. Berrangé writes:
> On Thu, Jul 01, 2021 at 06:09:47PM -0300, Beraldo Leal wrote:
>>
>> This is supported already:
>>
>> $ avocado run --tap - --test-runner='nrunner' tests/domain/transient.py
>> 1..3
>> ok 1 tests/domain/transient.py:TransientDomain.test_autostart
>> ok 2 tests/domai
On Wed, Jul 21, 2021 at 06:50:03PM +0100, Daniel P. Berrangé wrote:
> On Thu, Jul 01, 2021 at 06:09:47PM -0300, Beraldo Leal wrote:
> > On Thu, Jul 01, 2021 at 07:04:32PM +0100, Daniel P. Berrangé wrote:
> > > On Wed, Jun 30, 2021 at 01:36:30PM -0300, Beraldo Leal wrote:
> > > > I'm adding more inf
On Tue, Jul 20, 2021 at 02:19:25PM +0200, Erik Skultety wrote:
> On Thu, Jul 01, 2021 at 07:04:32PM +0100, Daniel P. Berrangé wrote:
> > Libvirt has consumers writing applications in a variety of
> > languages, and periodically reporting bugs. My general wish
> > for a test harness would be for so
On Thu, Jul 01, 2021 at 06:09:47PM -0300, Beraldo Leal wrote:
> On Thu, Jul 01, 2021 at 07:04:32PM +0100, Daniel P. Berrangé wrote:
> > On Wed, Jun 30, 2021 at 01:36:30PM -0300, Beraldo Leal wrote:
> > > I'm adding more information with some details inside the README file.
> >
> > Overall, I'm mor
On Wed, Jul 21, 2021 at 2:29 PM Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:05:05 +0200, Kristina Hanicova wrote:
> > We didn't always save status xml after generating new taint
> > message, which resulted in it being deleted in case of a libvirtd
> > restart. Some taint messages were prese
On 7/21/21 3:02 PM, Daniel P. Berrangé wrote:
> On Wed, Jul 21, 2021 at 02:29:34PM +0200, Peter Krempa wrote:
>> On Wed, Jul 21, 2021 at 14:05:05 +0200, Kristina Hanicova wrote:
>>> We didn't always save status xml after generating new taint
>>> message, which resulted in it being deleted in case o
On Fri, Jul 16, 2021 at 11:10:28AM +0800, Zhenzhong Duan wrote:
> Thanks Peter, Pavel and Daniel's comments on v1 version, now the v2 comes.
>
> * What's TDX?
> TDX stands for Trust Domain Extensions which isolates VMs from
> the virtual-machine manager (VMM)/hypervisor and any other software on
>
On Wed, Jul 21, 2021 at 10:13:32AM -0400, Andrea Bolognani wrote:
> On Wed, Jul 21, 2021 at 02:07:58PM +0100, Daniel P. Berrangé wrote:
> > On Wed, Jul 21, 2021 at 03:16:39AM -0700, Andrea Bolognani wrote:
> > > Also pre-existing: do we even care about handling upgrades from
> > > versions of the d
On Wed, Jul 21, 2021 at 02:07:58PM +0100, Daniel P. Berrangé wrote:
> On Wed, Jul 21, 2021 at 03:16:39AM -0700, Andrea Bolognani wrote:
> > Also pre-existing: do we even care about handling upgrades from
> > versions of the daemon that didn't have support for systemd socket
> > passing at this poin
On Wed, Jul 21, 2021 at 03:08:02PM +0200, Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:46:43 +0200, Tim Wiederhake wrote:
> > The undefined behaviour sanitizer (UBSAN) defaults to merely printing an
> > error message if it detects undefined behaviour. These error messages often
> > end up in ca
On Wed, Jul 21, 2021 at 03:08:02PM +0200, Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:46:43 +0200, Tim Wiederhake wrote:
> > +++ b/.gitlab-ci.yml
> > @@ -89,6 +89,8 @@ stages:
> > - meson build --werror -Ddocs=disabled -Db_lundef=false
> > -Db_sanitize="$SANITIZER"
> > - ninja -C bu
Am Wed, 21 Jul 2021 06:00:12 -0700
schrieb Andrea Bolognani :
> For most software, the default configuration files consist of mostly
> comments and act more like documentation for what the local
> configuration might look like. Claiming that the defaults are
> actually defined in the source code i
ping
On Wed, 2021-07-14 at 11:44 +0200, Tim Wiederhake wrote:
> "virHashNew" cannot return NULL, yet we check for NULL in various
> places.
>
> See
> https://listman.redhat.com/archives/libvir-list/2021-July/msg00074.html
> .
>
> Tim Wiederhake (10):
> conf: Add AUTOPTR_CLEANUP_FUNC for virNWF
On Wed, Jul 21, 2021 at 03:02:59PM +0200, Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:46:42 +0200, Tim Wiederhake wrote:
> > virFileReadLimFD always returns null-terminated data. To that end, it has to
> > add one to the maximum file size. If the maxium file size is INT_MAX, this
> > triggers
On Wed, Jul 21, 2021 at 03:16:39AM -0700, Andrea Bolognani wrote:
> On Tue, Jul 20, 2021 at 07:00:20PM +0200, Olaf Hering wrote:
> > if [ -f %{_localstatedir}/lib/rpm-state/libvirt/restart ]; then
> > # See if user has previously modified their install to
> > # tell libvirtd to use --lis
On Wed, Jul 21, 2021 at 14:46:43 +0200, Tim Wiederhake wrote:
> The undefined behaviour sanitizer (UBSAN) defaults to merely printing an
> error message if it detects undefined behaviour. These error messages often
> end up in captured output and do not fail the tests, effectively hiding
> the warn
On a %A in %Y, Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:46:40 +0200, Tim Wiederhake wrote:
>> The next commit will use this macro outside the apparmor context.
>>
>> Signed-off-by: Tim Wiederhake
>> ---
>> src/security/security_apparmor.c | 4 ++--
>> src/security/security_apparmor.h |
On Wed, Jul 21, 2021 at 14:46:42 +0200, Tim Wiederhake wrote:
> virFileReadLimFD always returns null-terminated data. To that end, it has to
> add one to the maximum file size. If the maxium file size is INT_MAX, this
> triggers a signed integer overflow.
>
> There is no instance left where a call
On Wed, Jul 21, 2021 at 02:29:34PM +0200, Peter Krempa wrote:
> On Wed, Jul 21, 2021 at 14:05:05 +0200, Kristina Hanicova wrote:
> > We didn't always save status xml after generating new taint
> > message, which resulted in it being deleted in case of a libvirtd
> > restart. Some taint messages we
On Wed, Jul 21, 2021 at 14:46:40 +0200, Tim Wiederhake wrote:
> The next commit will use this macro outside the apparmor context.
>
> Signed-off-by: Tim Wiederhake
> ---
> src/security/security_apparmor.c | 4 ++--
> src/security/security_apparmor.h | 1 -
> src/security/virt-aa-helper.c|
On Wed, Jul 21, 2021 at 01:23:56PM +0200, Olaf Hering wrote:
> Am Wed, 21 Jul 2021 03:16:39 -0700 schrieb Andrea Bolognani
> :
> > On Tue, Jul 20, 2021 at 07:00:20PM +0200, Olaf Hering wrote:
> > > sysconfig files are owned by the admin of the host. He has the liberty
> > > to put anything he want
On Wed, Jul 21, 2021 at 14:46:41 +0200, Tim Wiederhake wrote:
> The use of INT_MAX as maximum file length is problematic for reasons
> discussed in the next commit.
>
> Signed-off-by: Tim Wiederhake
> ---
> tests/networkxml2firewalltest.c | 2 +-
> tests/testutils.c | 5 +++--
> 2
virFileReadLimFD always returns null-terminated data. To that end, it has to
add one to the maximum file size. If the maxium file size is INT_MAX, this
triggers a signed integer overflow.
There is no instance left where a caller would call virFileReadLimFD with a
maxium file size of INT_MAX. Make
The undefined behaviour sanitizer (UBSAN) defaults to merely printing an
error message if it detects undefined behaviour. These error messages often
end up in captured output and do not fail the tests, effectively hiding
the warning. Make the test cases fail to make the issues visible.
Signed-off-
The use of INT_MAX as maximum file length is problematic for reasons
discussed in the next commit.
Signed-off-by: Tim Wiederhake
---
tests/networkxml2firewalltest.c | 2 +-
tests/testutils.c | 5 +++--
2 files changed, 4 insertions(+), 3 deletions(-)
diff --git a/tests/networkxml2
The next commit will use this macro outside the apparmor context.
Signed-off-by: Tim Wiederhake
---
src/security/security_apparmor.c | 4 ++--
src/security/security_apparmor.h | 1 -
src/security/virt-aa-helper.c| 10 +-
src/util/virfile.h | 3 +++
4 files changed, 1
The undefined behaviour sanitizer (UBSAN) defaults to merely printing an
error message if it detects undefined behaviour. These error messages often
end up in captured output and do not fail the tests, effectively hiding
the warning.
This series fixes all previously hidden issues and makes UBSAN f
On Wed, Jul 21, 2021 at 14:05:05 +0200, Kristina Hanicova wrote:
> We didn't always save status xml after generating new taint
> message, which resulted in it being deleted in case of a libvirtd
> restart. Some taint messages were preserved thanks to saving
> status xml separately at the end of th
On 7/20/21 4:44 PM, Kristina Hanicova wrote:
>
> Kristina Hanicova (2):
> genericxml2xmltest: Substitute 'inactive' variable with 'active'
> genericxml2xmltest: Refactor DO_TEST_FULL() calls
>
> tests/genericxml2xmltest.c | 71 --
> 1 file changed, 29 inse
On Wed, Jul 21, 2021 at 12:06:35PM +0100, Daniel P. Berrangé wrote:
> On Wed, Jul 21, 2021 at 11:48:17AM +0100, Daniel P. Berrangé wrote:
> > Thus I'm really puzzelled why libvirt didn't see a real benefit
> > when I tested it. As you say though, FreeBSD builds are no worse
> > than Linux builds,
On Wed, Jul 21, 2021 at 12:36:40PM +0100, Daniel P. Berrangé wrote:
> On Wed, Jul 21, 2021 at 03:16:39AM -0700, Andrea Bolognani wrote:
> > In particular, I worry about changes in defaults being more difficult
> > for users to detect: in Debian at least, changes to the default
> > sysconfig files r
On a %A in %Y, Peter Krempa wrote:
> Peter Krempa (33):
> qemu: monitor: Remove handlers for the 'POWERDOWN' event
> qemu: monitor: Remove return value from qemuMonitorEmit* functions
> qemu: Remove return value from qemuMonitorDomainEventCallback
> qemu: Remove return value from qemuMonito
On a %A in %Y, Peter Krempa wrote:
> The submission of the event to the helper thread has a verbose cleanup
> path which was duplicated in all the event handlers. Simplify it by
> extracting the code into a helper named 'qemuProcessEventSubmit' and
> reuse it where appropriate.
>
> Signed-off-by:
On a %A in %Y, Peter Krempa wrote:
> The callers in the monitor code invoking the callbacks after events are
> received don't actually check the return value from the callbacks and
> there isn't really anything we could do on failure.
>
> Remove the return value from the intermediary functions so
We didn't always save status xml after generating new taint
message, which resulted in it being deleted in case of a libvirtd
restart. Some taint messages were preserved thanks to saving
status xml separately at the end of the calling functions (which
makes sense, because qemuDomainObjTaint was us
On Wed, Jul 21, 2021 at 01:51:01PM +0200, Olaf Hering wrote:
> Am Wed, 21 Jul 2021 12:36:40 +0100
> schrieb Daniel P. Berrangé :
>
> > If we remove the sysconfig files, we're not expecting users to
> > modify the .service files. Instead they will be using the systemd
> > overrides in /etc/systemd/
Am Wed, 21 Jul 2021 12:36:40 +0100
schrieb Daniel P. Berrangé :
> If we remove the sysconfig files, we're not expecting users to
> modify the .service files. Instead they will be using the systemd
> overrides in /etc/systemd/system/libvirtd.service.d/ to
> customize.
Why not /etc/sysconfig/file i
On Wed, Jul 21, 2021 at 03:16:39AM -0700, Andrea Bolognani wrote:
> On Tue, Jul 20, 2021 at 07:00:20PM +0200, Olaf Hering wrote:
> In particular, I worry about changes in defaults being more difficult
> for users to detect: in Debian at least, changes to the default
> sysconfig files result in the
Commit e9b534905f4 introduced an error when parsing an empty list
returned from mdevctl.
This occurs e.g. if nodedev-undefine is used to undefine the last
defined mdev which causes the following error messages
libvirtd[33143]: internal error: Unexpected format for mdevctl response
libvirtd[3314
Am Wed, 21 Jul 2021 03:16:39 -0700
schrieb Andrea Bolognani :
> On Tue, Jul 20, 2021 at 07:00:20PM +0200, Olaf Hering wrote:
> > sysconfig files are owned by the admin of the host. He has the liberty
> > to put anything he wants into these files. This makes it difficult to
> > provide different bu
On Wed, Jul 21, 2021 at 01:07:04PM +0200, Michal Prívozník wrote:
> On 7/20/21 5:05 PM, Daniel P. Berrangé wrote:
> > For macOS you always get the maximum configuration by default (12 CPUs,
> > 24 GB RAM), but for FreeBSD you get 2 CPUs, 4 GBs by default. This
> > change increases the allocation to
On 7/20/21 5:05 PM, Daniel P. Berrangé wrote:
> For macOS you always get the maximum configuration by default (12 CPUs,
> 24 GB RAM), but for FreeBSD you get 2 CPUs, 4 GBs by default. This
> change increases the allocation to 8 CPUs, 8 GBs for FreeBSD.
>
> Signed-off-by: Daniel P. Berrangé
> ---
On Wed, Jul 21, 2021 at 11:48:17AM +0100, Daniel P. Berrangé wrote:
> On Wed, Jul 21, 2021 at 06:39:07AM -0400, Andrea Bolognani wrote:
> > On Tue, Jul 20, 2021 at 04:05:22PM +0100, Daniel P. Berrangé wrote:
> > > For macOS you always get the maximum configuration by default (12 CPUs,
> > > 24 GB R
On Mon, Jul 19, 2021 at 14:36:59 +0200, Tim Wiederhake wrote:
> On Mon, 2021-07-19 at 14:08 +0200, Peter Krempa wrote:
> > On Mon, Jul 19, 2021 at 11:55:44 +0200, Tim Wiederhake wrote:
> > > "virHashNew" cannot return NULL, yet we check for NULL in various
> > > places.
> > >
> > > See
> > > https
On 7/21/21 12:45 PM, Boris Fiuczynski wrote:
On 7/21/21 12:04 PM, Pavel Hrdina wrote:
On Fri, Jul 16, 2021 at 11:44:24AM +0200, Boris Fiuczynski wrote:
This patch series introduces the launch security type s390-pv.
Specifying s390-pv as launch security type in an s390 domain prepares
for
runn
On Wed, Jul 21, 2021 at 06:39:07AM -0400, Andrea Bolognani wrote:
> On Tue, Jul 20, 2021 at 04:05:22PM +0100, Daniel P. Berrangé wrote:
> > For macOS you always get the maximum configuration by default (12 CPUs,
> > 24 GB RAM), but for FreeBSD you get 2 CPUs, 4 GBs by default. This
> > change incre
The submission of the event to the helper thread has a verbose cleanup
path which was duplicated in all the event handlers. Simplify it by
extracting the code into a helper named 'qemuProcessEventSubmit' and
reuse it where appropriate.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_process.c | 11
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 12 ++--
src/qemu/qemu_process.c | 4 +---
2 files changed, 7 insertions(+), 9 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 4 +---
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monito
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 4 ++--
2 files changed, 6 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monito
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 14 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu
On 7/21/21 12:04 PM, Pavel Hrdina wrote:
On Fri, Jul 16, 2021 at 11:44:24AM +0200, Boris Fiuczynski wrote:
This patch series introduces the launch security type s390-pv.
Specifying s390-pv as launch security type in an s390 domain prepares for
running the guest in protected virtualization secure
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 4 ++--
src/qemu/qemu_processpriv.h | 8
3 files changed, 10 insertions(+), 10 deletions(-)
diff --
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
Change the callback prototype and fix the callback registered in the
process code.
The removed error messages are impossible as the enum values are
converted via VIR_ENUM helpers and guarded by compiler checks.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 10 +-
src/qemu/qemu_process.c | 13 -
2 files changed, 9 insertions(+), 14 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qe
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 3 +--
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 4 +---
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 12 ++--
src/qemu/qemu_process.c | 3 +--
2 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_m
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 16
src/qemu/qemu_process.c | 4 +---
2 files changed, 9 insertions(+), 11 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 4 insertions(+), 5 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.h
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 3 +--
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 26 +-
src/qemu/qemu_process.c | 4 +---
2 files changed, 14 insertions(+), 16 deletions(-)
diff --git a/src/qemu/qemu_monitor.h
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 3 +--
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 4 +---
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monito
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 5 +
2 files changed, 4 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 4 +---
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.
On Tue, Jul 20, 2021 at 04:05:22PM +0100, Daniel P. Berrangé wrote:
> For macOS you always get the maximum configuration by default (12 CPUs,
> 24 GB RAM), but for FreeBSD you get 2 CPUs, 4 GBs by default. This
> change increases the allocation to 8 CPUs, 8 GBs for FreeBSD.
>
> ---
> In theory this
Change the callback prototype and fix the callback registered in the
process code.
It is also impossible for @info to be non-NULL in the cleanup section so
the cleanup can be completely removed.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 14 +++---
src/qemu/qemu_process.c
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 12 ++--
src/qemu/qemu_process.c | 3 +--
2 files changed, 7 insertions(+), 8 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_m
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 3 +--
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 6 +++---
src/qemu/qemu_process.c | 4 +---
2 files changed, 4 insertions(+), 6 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monitor.
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 10 +-
src/qemu/qemu_process.c | 4 ++--
2 files changed, 7 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_mo
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 10 +-
src/qemu/qemu_process.c | 3 +--
2 files changed, 6 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_mon
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 14 +++---
src/qemu/qemu_process.c | 3 +--
2 files changed, 8 insertions(+), 9 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 8
src/qemu/qemu_process.c | 4 +---
2 files changed, 5 insertions(+), 7 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qemu_monito
Change the callback prototype and fix the callback registered in the
process code.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.h | 14 +++---
src/qemu/qemu_process.c | 4 +---
2 files changed, 8 insertions(+), 10 deletions(-)
diff --git a/src/qemu/qemu_monitor.h b/src/qemu/qe
The callers in the monitor code invoking the callbacks after events are
received don't actually check the return value from the callbacks and
there isn't really anything we could do on failure.
Remove the return value from the intermediary functions so we can later
remove them from the callback pr
The qemu process code doesn't register a callback for it so we don't
need to be handling it at all.
Signed-off-by: Peter Krempa
---
src/qemu/qemu_monitor.c | 11 ---
src/qemu/qemu_monitor.h | 5 -
src/qemu/qemu_monitor_json.c | 7 ---
3 files changed, 23 deletions(-)
Peter Krempa (33):
qemu: monitor: Remove handlers for the 'POWERDOWN' event
qemu: monitor: Remove return value from qemuMonitorEmit* functions
qemu: Remove return value from qemuMonitorDomainEventCallback
qemu: Remove return value from qemuMonitorDomainShutdownCallback
qemu: Remove return
On Tue, Jul 20, 2021 at 07:00:20PM +0200, Olaf Hering wrote:
> sysconfig files are owned by the admin of the host. He has the liberty
> to put anything he wants into these files. This makes it difficult to
> provide different built-in defaults.
s/He has/They have/
s/he wants/they want/
> +++ b/NE
On Fri, Jul 16, 2021 at 11:44:24AM +0200, Boris Fiuczynski wrote:
> This patch series introduces the launch security type s390-pv.
> Specifying s390-pv as launch security type in an s390 domain prepares for
> running the guest in protected virtualization secure mode, also known as
> IBM Secure Exec
On Fri, Jul 16, 2021 at 11:44:28AM +0200, Boris Fiuczynski wrote:
> Adding virDomainSecDef for general launch security data
> and moving virDomainSEVDef as an element for SEV data.
>
> Signed-off-by: Boris Fiuczynski
> Reviewed-by: Daniel Henrique Barboza
> ---
> src/conf/domain_conf.c | 1
Adding libvirt-security since I forgot when sending the patch.
On Wed, Jul 21, 2021 at 11:27:41 +0200, Peter Krempa wrote:
> 'virStoragePoolObjListSearch' returns a locked and refed object, thus we
> must release it on ACL permission failure.
>
> Fixes: 7aa0e8c0cb8
> Resolves: https://bugzilla.re
'virStoragePoolObjListSearch' returns a locked and refed object, thus we
must release it on ACL permission failure.
Fixes: 7aa0e8c0cb8
Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1984318
Signed-off-by: Peter Krempa
---
Technically a security issue since it DoS-es the objects a user does
I removed negation from the name of a variable to make the code
more readable.
Signed-off-by: Kristina Hanicova
---
tests/lxcxml2xmltest.c | 12 ++--
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/tests/lxcxml2xmltest.c b/tests/lxcxml2xmltest.c
index 00341ad695..69c333b9b5
Having negation in a name of a bool variable seems a bit
confusing to me. I think the substitution makes the code much
more readable.
Signed-off-by: Kristina Hanicova
---
diff to v1:
* I forgot to change inactive_only variable in struct testInfo
tests/genericxml2xmltest.c | 50 +
91 matches
Mail list logo