The firmware descriptors have 'amd-sev-snp` feature which
describes whether firmware is suitable for SEV-SNP guests.
Provide necessary implementation to detect the feature and pick
the right firmware if guest is SEV-SNP enabled.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_firmware.c
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_driver.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/qemu/qemu_driver.c b/src/qemu/qemu_driver.c
index fc1704f4fc..3a76df8ddb 100644
--- a/src/qemu/qemu_driver.c
+++ b/src/qemu/qemu_driver.c
@@ -19185,9 +19185,10
SEV-SNP is an enhancement of SEV/SEV-ES and thus it shares some
fields with it. Nevertheless, on XML level, it's yet another type
of .
Signed-off-by: Michal Privoznik
---
docs/formatdomain.rst | 108 ++
src/conf/domain_conf.c| 73
Pretty straightforward as qemu has 'sev-snp-guest' object which
attributes maps pretty much 1:1 to our XML model. Except for
@vcek where QEMU has 'vcek-disabled`, an inverted boolean, while
we model it as virTristateBool. But that's easy to map too.
Signed-off-by: Michal Privoznik
---
src/qemu
This capability tracks sev-snp-guest object availability.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_capabilities.c | 4
src/qemu/qemu_capabilities.h | 3 +++
tests/qemucapabilitiesdata/caps_9.1.0_x86_64.xml | 1 +
3 files changed, 8
Signed-off-by: Michal Privoznik
---
include/libvirt/libvirt-domain.h | 10 ++
src/qemu/qemu_driver.c | 6 ++
2 files changed, 16 insertions(+)
diff --git a/include/libvirt/libvirt-domain.h b/include/libvirt/libvirt-domain.h
index 2f5b01bbfe..8f00e9e959 100644
In a few instances there is a plain if() check for
_virDomainSecDef::sectype. While this works perfectly for now,
soon there'll be another type and we can utilize compiler to
identify all the places that need adaptation. Switch those if()
statements to switch().
Signed-off-by: Michal Privoznik
In QEMU commit v9.0.0-1155-g59d3740cb4 the return type of
'query-sev' monitor command changed to accommodate SEV-SNP. Even
though we currently support launching plain SNP guests, this will
soon change.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_driver.c | 32 ++
src
The sectype member of _virDomainSecDef struct is already declared
as of virDomainLaunchSecurity type. There's no need to typecast
it to the very same type when passing it to switch().
Signed-off-by: Michal Privoznik
---
src/conf/domain_conf.c| 6 +++---
src/qemu/qemu_command.c | 4
To avoid convolution of switch() inside of virDomainSecDefFormat() even
more (as new sectypes are added), move formatting into a separate
function.
Signed-off-by: Michal Privoznik
---
src/conf/domain_conf.c | 26 +++---
1 file changed, 15 insertions(+), 11 deletions(-)
diff
Some parts of SEV are to be shared with SEV SNP. In order to
reuse XML parsing / formatting code cleanly, let's move those
common bits into a new struct (virDomainSEVCommonDef) and adjust
rest of the code.
Signed-off-by: Michal Privoznik
---
src/conf/domain_conf.c| 55
While working on qemuMonitorJSONGetSEVMeasurement() and
qemuMonitorJSONGetSEVInfo() I've noticed that if these functions
fail, they do so without appropriate error set. Fill in error
reporting.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_monitor_json.c | 10 --
1 file changed, 8
SEV-SNP support just landed in QEMU. Here is the first round of patches
to incorporate support into libvirt.
TODOs (aka problems of future me):
- Teach tools/virt-qemu-sev-validate how to deal with SEV-SNP
- Try to find a SEV-SNP machine a test these patches in real worl
- Write a kbase article
-by: Michal Privoznik
---
src/conf/capabilities.c| 6 ++--
src/conf/domain_conf.c | 57 +++---
src/conf/network_conf.c| 6 ++--
src/conf/node_device_conf.c| 57 --
src/conf/snapshot_conf.c
per
(remote_daemon_dispatch_stubs.h:74)
by 0x4B0FA5E: virNetServerProgramDispatchCall (virnetserverprogram.c:423)
by 0x4B0F591: virNetServerProgramDispatch (virnetserverprogram.c:299)
by 0x4B18AE3: virNetServerProcessMsg (virnetserver.c:135)
Signed-off-by: Michal Privoznik
---
(libvirt.c:665)
by 0x15A6C6: daemonRunStateInit (remote_daemon.c:611)
by 0x49E69F0: virThreadHelper (virthread.c:256)
by 0x532B428: start_thread (in /lib64/libc.so.6)
by 0x5397373: clone (in /lib64/libc.so.6)
Signed-off-by: Michal Privoznik
---
src/conf/virnetworkobj.c | 1 +
1 file
)
by 0x4CB1FA6: virStateInitialize (libvirt.c:665)
by 0x15A6C6: daemonRunStateInit (remote_daemon.c:611)
by 0x49E69F0: virThreadHelper (virthread.c:256)
by 0x532B428: start_thread (in /lib64/libc.so.6)
Signed-off-by: Michal Privoznik
---
src/util/virfirewall.c | 2 +-
1 file changed, 1
The third patch MIGHT fix the following issue:
https://issues.redhat.com/browse/RHEL-22574
but at this point it's still unclear. I'll append appropriate
'Resolves:' line when I learn more.
Michal Prívozník (3):
virfirewall: Fir a memleak in virFirewallParseXML()
virnetworkobj: Free
Signed-off-by: Michal Privoznik
---
NEWS.rst | 7 +++
1 file changed, 7 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index f3ca29443a..2e735f69ce 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -17,6 +17,13 @@ v10.5.0 (unreleased)
* **New features**
+ * Introduce pstore device
The acpi-erst backend for pstore device exposes a path in the
host accessible to the guest and as such we must set seclabels on
it to grant QEMU RW access.
Signed-off-by: Michal Privoznik
---
src/security/security_dac.c | 10 ++
src/security/security_selinux.c | 9 +
src
Nothing special going on here.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_alias.c | 10
src/qemu/qemu_command.c | 51 +++
.../pstore-acpi-erst.x86_64-latest.args | 2 +
3 files changed, 63 insertions(+)
diff --git
any additional RAM to the
guest and thus I've decided to expose it as another device type
instead of memory model.
Signed-off-by: Michal Privoznik
---
docs/formatdomain.rst | 32
src/ch/ch_domain.c| 1 +
src/conf/domain_conf.c
This capability tracks whether QEMU has acpi-erst device.
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_capabilities.c | 4
src/qemu/qemu_capabilities.h | 3 +++
tests/qemucapabilitiesdata/caps_7.0.0_aarch64+hvf.xml | 1 +
tests
*** BLURB HERE ***
Michal Prívozník (5):
qemu_capabilities: Introduce QEMU_CAPS_DEVICE_ACPI_ERST
conf: Introduce pstore device
qemu: Build cmd line for pstore device
security: Set seclabels for pstore device
NEWS: Document pstore device addition
NEWS.rst
I'm working on something that's calling qemuMonitorCreateObjectProps()
and was getting random errors only to find out I was missing NULL
sentinel. This sparked me to look at other functions that might be
missing the G_GNUC_NULL_TERMINATED attribute too and found some.
Michal Prívozník (2):
lib:
in.
After this, there are still some functions left untouched because
they expect a different sentinel than NULL. Unfortunately, glib
does not provide macro for different sentinels. We may come up
with our own, but let's save that for future work.
Signed-off-by: Michal Privoznik
---
src/qemu
The qemuMonitorTestAddErrorResponse() function is a printf-like
function. But the annotation was mistakenly done in .c file
instead of corresponding .h file rendering the annotation
ineffective. Move the annotation to the header file.
Signed-off-by: Michal Privoznik
---
tests
These are either features/bugfixes I've worked on or
participated in.
Signed-off-by: Michal Privoznik
---
NEWS.rst | 23 +++
1 file changed, 23 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index 14505116b1..57e8f0d0c3 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -45,6 +45,16
its removal) and thus instead of removing the old QoS
setting nothing is done.
Fortunately, the fix is simple - pass olddev which contains the
old QoS setting.
Fixes: 812a146dfe784315edece43d09f8d9e432f8230e
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_hotplug.c | 2 +-
1 file changed, 1
an empty
string instead of NULL which then makes str2int parsing fail.
Just check for this case before parsing the string.
Based on the original patch sent by David.
Reported-by: David Negreira
Signed-off-by: Michal Privoznik
---
The original patch was posted here:
https://lists.libvirt.org
progpath.startswith(builddir).
While builddir is an absolute path, progpath isn't necessarily.
And while looking into the code, I've noticed chcon() function
accessing variable outside its scope when printing out the path
it's working on.
Signed-off-by: Michal Privoznik
---
v2 of:
https
just right but is then
subsequently passed to vshCompleterFilter() which isn't prepared
for this case.
Signed-off-by: Michal Privoznik
---
tools/vsh.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/vsh.c b/tools/vsh.c
index c91d756885..6cc1f60d87 100644
--- a/tools/vsh.c
: 41400ac1dda55b817388a4050aa823051bda2e05
Fixes: a0e1ada63c0afdc2af3b9405cbf637d8bd28700c
Signed-off-by: Michal Privoznik
---
tools/vsh.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/tools/vsh.c b/tools/vsh.c
index de869248b4..c91d756885 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -3511,6 +3511,7
no daemon is running. Suppress extra prints by passing '-q'
in the bash-completion script.
Signed-off-by: Michal Privoznik
---
tools/bash-completion/vsh.in | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/bash-completion/vsh.in b/tools/bash-completion/vsh.in
index
stdin.
Signed-off-by: Michal Privoznik
---
tools/vsh.c | 11 +++
1 file changed, 7 insertions(+), 4 deletions(-)
diff --git a/tools/vsh.c b/tools/vsh.c
index 61a3066f49..de869248b4 100644
--- a/tools/vsh.c
+++ b/tools/vsh.c
@@ -3488,17 +3488,20 @@ const vshCmdInfo info_complete
We currently require full argument specification:
virt-admin daemon-timeout --timeout X
Well, the '--timeout' feels a bit redundant. Turn the argument
into a positional so that the following works too:
virt-admin daemon-timeout X
Signed-off-by: Michal Privoznik
---
docs/manpages/virt
It started simple - I pointed somebody to debug logs kbase article only
to realize our own examples don't work. And ended up fixing bash
completion. Fans of 'Malcolm in the Middle' know the feeling.
Michal Prívozník (6):
kbase: Use virt-admin daemon-timeout correctly
virt-admin: Make
-by: Michal Privoznik
---
docs/kbase/debuglogs.rst | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/docs/kbase/debuglogs.rst b/docs/kbase/debuglogs.rst
index f08132d099..dff2cfd144 100644
--- a/docs/kbase/debuglogs.rst
+++ b/docs/kbase/debuglogs.rst
@@ -25,7 +25,7 @@ the system
progpath.startswith(builddir).
While builddir is an absolute path, progpath isn't necessarily.
And while looking into the code, I've noticed chcon() function
accessing variable outside its scope when printing out the path
it's working on.
Signed-off-by: Michal Privoznik
---
run.in | 4 ++--
1
The virStateDriver struct has .stateInitialize callback which is
declared to return virDrvStateInitResult enum. But some drivers
return a plain int in their implementation which is UB.
Signed-off-by: Michal Privoznik
---
src/bhyve/bhyve_driver.c| 2 +-
src/ch/ch_driver.c
*** BLURB HERE ***
Michal Prívozník (2):
qemu_process: Issue an info message when subtracting isolcpus
virprocess: Debug affinity map in virProcessSetAffinity()
src/qemu/qemu_process.c | 6 ++
src/util/virprocess.c | 6 --
2 files changed, 10 insertions(+), 2 deletions(-)
--
that to a regular WARN if users
complain.
Fixes: da95bcb6b2d9b04958e0f2603202801dd29debb8
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_process.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index 7ef7040a85..2c33ac3bf7 100644
--- a/src/qemu
The aim of virProcessSetAffinity() is to set affinity of given
process to given CPUs. While we currently print the PID into
logs, the CPU map is not printed. It may help when debugging
weird scenarios.
Signed-off-by: Michal Privoznik
---
src/util/virprocess.c | 6 --
1 file changed, 4
It's currently running against AlmaLinux 8 which went out of
support.
Signed-off-by: Michal Privoznik
---
.gitlab-ci.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 3114c7410e..7f80896e6e 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab
It's currently running against AlmaLinux 8 which went out of
support.
Signed-off-by: Michal Privoznik
---
.gitlab-ci.yml | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index 81ac2ecddd..3114c7410e 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab
I've switched our CI jobs from AlmaLinux 8 to AlmaLinux 9, but
apparently forgot about two.
Michal Prívozník (2):
gitlab-ci: Switch potfile job to AlmaLinux 9
gitlab-ci: Switch coverity job to AlmaLinux 9
.gitlab-ci.yml | 8
1 file changed, 4 insertions(+), 4 deletions(-)
--
ver with the QEMU driver, well, its test suite
suitable version (libtest_qemu_driver.so).
Signed-off-by: Michal Privoznik
---
This obsoletes some patches I've sent earlier:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/message/RM56P5RDBYCVYQVKHOFFKUZ2B276NQD3/
https://lists.l
The message that's thrown at users when they try to open a pull
request on github suggests opening the MR on gitlab instead.
While this works for other libvirt subprojects, for the main
libvirt.git we still use e-mail workflow. Update the message to
reflect this fact.
Signed-off-by: Michal
Signed-off-by: Michal Privoznik
---
NEWS.rst | 5 +
1 file changed, 5 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index d72c15bf10..ef0e4f8b51 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -22,6 +22,11 @@ v10.4.0 (unreleased)
It is now possible to set on/off ``ras`` feature in the domain
Signed-off-by: Michal Privoznik
---
docs/docs.rst | 3 ++
docs/meson.build | 1 +
docs/nss.rst | 7
docs/ssh-proxy.rst | 85 ++
4 files changed, 96 insertions(+)
create mode 100644 docs/ssh-proxy.rst
diff --git a/docs/docs.rst b
d/blob/main/src/ssh-generator/ssh-proxy.c
2:
https://github.com/systemd/systemd/blob/main/src/ssh-generator/20-systemd-ssh-proxy.conf.in
Resolves: https://gitlab.com/libvirt/libvirt/-/issues/579
Signed-off-by: Michal Privoznik
---
libvirt.spec.in | 33 +++
meson
v3 of:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/EXJPQJSZEOTXAJRKPDAMCWRC4S2QDQFF/
diff to v2:
- New naming scheme: user@qemu:system/VM and user@qemu:session/VM for
choosing explicit connection URI for fetching VM.
Old user@qemu/VM is kept for convenience.
It's now more than two years since Ubuntu 22.04 was released and
per our support policy, Ubuntu 20.04 (the previous major release)
is now not supported. Remove it from our CI testing.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
ci/buildenv/ubuntu-2004.sh | 103
Ubuntu 24.04 was released recently. Add it to our CI. Also, to be
able to run ASAN/UBSAN builds on Ubuntu 24.04 libclang-rt-dev
needs to be installed (because clang's runtime was moved into a
separate package). Hence so many seemingly unrelated changes.
Signed-off-by: Michal Privoznik
---
ci
-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
meson.build | 2 +-
src/libvirt_private.syms | 1 -
src/util/glibcompat.c| 125 ---
src/util/glibcompat.h| 10
4 files changed, 1 insertion(+), 137 deletions(-)
diff --git
Since Fedora 40 was released recently, Fedora 38 is now
unsupported. Drop Fedora 38 and introduce Fedora 40 to our CI.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
...-mingw32.sh => fedora-40-cross-mingw32.sh} | 0
...-mingw64.sh => fedora-40-cross-mingw64.sh} |
v2 of:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/MMENBGZ3HYYKUVPY6FJ443SYCBNFTDHH/
diff to v1:
- Juggled some patches around, namely: squashed Fedora and AlmaLinux
drop & reintroduction patches, reordered some for easier review.
- Switched website job to AlmaLinux
to be moved to newer AlmaLinux.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
.gitlab-ci.yml | 6 +++---
ci/buildenv/{almalinux-8.sh => almalinux-9.sh} | 8 +++-
...linux-8.Dockerfile => almalinux-9.Dockerfile} | 10 --
ci/
behavior.
Ideally, we would fix our rpcgen to generate proper function
headers, but: a) my brain is too small to do that, and b) we
would lose compiler protection if an xdr_*() function is called
directly but argument of a wrong type is passed.
Silence UBSAN for now.
Signed-off-by: Michal
strictly bigger space of possible values than the enum.
Defined those aforementioned callbacks so that they return the
correct enum instead of int.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
src/security/security_apparmor.c | 2 +-
src/security/security_selinux.c | 2
Allocated in testQemuInfoSetArgs(), the vdpafds member of
testQemuArgs is never freed.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
---
tests/testutilsqemu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c
index 9c12a165b1
While Linux linker has no trouble resolving the symbols, valgrind
does. It has probably something to do with the fact that we don't
tell what symbols to export from mock libraries. Anyway, just
resolve the symbol at runtime.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
While Linux linker has no trouble resolving the symbols, valgrind
does. It has probably something to do with the fact that we don't
tell what symbols to export from mock libraries. Anyway, just
resolve the symbol at runtime.
Signed-off-by: Michal Privoznik
Reviewed-by: Daniel P. Berrangé
Signed-off-by: Michal Privoznik
---
docs/docs.rst | 3 ++
docs/meson.build | 1 +
docs/nss.rst | 7 +
docs/ssh-proxy.rst | 68 ++
4 files changed, 79 insertions(+)
create mode 100644 docs/ssh-proxy.rst
diff --git a/docs/docs.rst b
Signed-off-by: Michal Privoznik
---
NEWS.rst | 5 +
1 file changed, 5 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index d72c15bf10..ef0e4f8b51 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -22,6 +22,11 @@ v10.4.0 (unreleased)
It is now possible to set on/off ``ras`` feature in the domain
v2 of:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/QYFHZFXUVKPM3JCPUKFVPBRZOIG2R2RK/
diff to v1:
- rebased onto current master
- Worked in Dan's review
Michal Prívozník (3):
tools: Introduce SSH proxy
docs: Document SSH proxy
NEWS: Document SSH proxy feature
d/blob/main/src/ssh-generator/ssh-proxy.c
2:
https://github.com/systemd/systemd/blob/main/src/ssh-generator/20-systemd-ssh-proxy.conf.in
Resolves: https://gitlab.com/libvirt/libvirt/-/issues/579
Signed-off-by: Michal Privoznik
---
libvirt.spec.in | 33 +++
meson
strictly bigger space of possible values than the enum.
Defined those aforementioned callbacks so that they return the
correct enum instead of int.
Signed-off-by: Michal Privoznik
---
src/security/security_apparmor.c | 2 +-
src/security/security_selinux.c | 2 +-
2 files changed, 2 insertions
AlmaLinux 9 was released a while ago, but for some reason it's
missing in our CI. Add it there.
Signed-off-by: Michal Privoznik
---
ci/buildenv/almalinux-9.sh | 101 ++
ci/containers/almalinux-9.Dockerfile | 104 +++
ci/gitlab/builds.yml
Ubuntu 24.04 was released recently. Add it to our CI.
Signed-off-by: Michal Privoznik
---
ci/buildenv/centos-stream-9.sh| 1 +
ci/buildenv/debian-12-cross-aarch64.sh| 1 +
ci/buildenv/debian-12-cross-armv6l.sh | 1 +
ci/buildenv/debian-12-cross-armv7l.sh
Fedora 40 was released recently. Add it to our CI.
Signed-off-by: Michal Privoznik
---
ci/buildenv/fedora-40-cross-mingw32.sh| 77
ci/buildenv/fedora-40-cross-mingw64.sh| 77
ci/buildenv/fedora-40.sh | 98
-by: Michal Privoznik
---
meson.build | 2 +-
src/libvirt_private.syms | 1 -
src/util/glibcompat.c| 125 ---
src/util/glibcompat.h| 10
4 files changed, 1 insertion(+), 137 deletions(-)
diff --git a/meson.build b/meson.build
index
By the time of release, it's going to be more than two years
since AlmaLinux 9 was released and per our support policy,
AlmaLinux 8 (the previous major release) will be not supported.
Remove it from our CI testing.
Signed-off-by: Michal Privoznik
---
ci/buildenv/almalinux-8.sh | 103
It's now more than two years since Ubuntu 22.04 was released and
per our support policy, Ubuntu 20.04 (the previous major release)
is now not supported. Remove it from our CI testing.
Signed-off-by: Michal Privoznik
---
ci/buildenv/ubuntu-2004.sh | 103 --
ci
Since Fedora 40 was released recently, Fedora 38 is now
unsupported. Remove it from our CI.
Signed-off-by: Michal Privoznik
---
ci/buildenv/fedora-38-cross-mingw32.sh| 77
ci/buildenv/fedora-38-cross-mingw64.sh| 77
ci/buildenv/fedora-38.sh
behavior.
Ideally, we would fix our rpcgen to generate proper function
headers, but: a) my brain is too small to do that, and b) we
would lose compiler protection if an xdr_*() function is called
directly but argument of a wrong type is passed.
Silence UBSAN for now.
Signed-off-by: Michal
Currently, our website job depends on almalinux-8 container.
Well, AlmaLinux 8 is going to be dropped soon. Therefore, switch
the job to something newer. Fedora 39 was chosen by a roll of
dice.
Signed-off-by: Michal Privoznik
---
.gitlab-ci.yml | 6 +++---
1 file changed, 3 insertions(+), 3
Allocated in testQemuInfoSetArgs(), the vdpafds member of
testQemuArgs is never freed.
Signed-off-by: Michal Privoznik
---
tests/testutilsqemu.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/tests/testutilsqemu.c b/tests/testutilsqemu.c
index 9c12a165b1..d70850cb5d 100644
--- a/tests
While Linux linker has no trouble resolving the symbols, valgrind
does. It has probably something to do with the fact that we don't
tell what symbols to export from mock libraries. Anyway, just
resolve the symbol at runtime.
Signed-off-by: Michal Privoznik
---
tests/qemuxml2argvmock.c | 8
While Linux linker has no trouble resolving the symbols, valgrind
does. It has probably something to do with the fact that we don't
tell what symbols to export from mock libraries. Anyway, just
resolve the symbol at runtime.
Signed-off-by: Michal Privoznik
---
tests/domaincapsmock.c | 13
There's a lot happening here, but I did not find a way to split this any
better. It all started with me wanting to switch from Ubuntu 20.04 to
24.04 and resulted in this.
Firstly, Ubuntu 24.04 started to complain about a few things:
1) ASAN wasn't working (some missing .a archive). Fix is
t.c this means some test cases might
create virsh history file which breaks our promise of not
touching user's data in test suite.
Resolves: https://bugs.gentoo.org/931109
Signed-off-by: Michal Privoznik
---
tools/vsh.c | 11 ---
1 file changed, 8 insertions(+), 3 deletions(-)
diff --
Signed-off-by: Michal Privoznik
---
Rebased version of the patch sent earlier, because the file was changed
meanwhile.
NEWS.rst | 22 ++
1 file changed, 22 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index 7ced82e3da..554ca559a2 100644
--- a/NEWS.rst
+++ b/NEWS.rst
Signed-off-by: Michal Privoznik
---
NEWS.rst | 22 ++
1 file changed, 22 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index 852dadf532..4958326875 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -17,8 +17,20 @@ v10.3.0 (unreleased)
* **New features**
+ * conf: Introduce
-off-by: Michal Privoznik
---
Another option is to pass both directories as arguments. But this
inconsistent solution bothers me. Especially since I want to copy the
script somewhere else (stay tuned to learn more).
meson.build | 4 ++--
scripts/meson-dist.py | 4 ++--
2 files changed
TC.
Fixes: 065054daa71f645fc83aff0271f194d326208616
Resolves: https://issues.redhat.com/browse/RHEL-34112
Signed-off-by: Michal Privoznik
---
src/util/virnetdevbandwidth.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/src/util/virnetdevbandwidth.c b/src/util/virnetdevbandwidth.c
index
Resolves: https://issues.redhat.com/browse/RHEL-33082
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_process.c | 9 +
1 file changed, 9 insertions(+)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index da2b024f92..521598471f 100644
--- a/src/qemu/qemu_process.c
+++ b/src
v2 of:
https://lists.libvirt.org/archives/list/devel@lists.libvirt.org/thread/4V7OI5AEGYRN4GFQMQPIN4MYPJNK3NYJ/
diff to v1:
- Don't error out on systems where /sys/devices/system/cpu/isolated is
unavailable.
- Don't error out on systems where /sys/devices/system/cpu/isolated is
empty.
Both
This is a helper that parses /sys/devices/system/cpu/isolated
into a virBitmap. It's going to be needed soon.
Signed-off-by: Michal Privoznik
---
src/libvirt_private.syms | 1 +
src/util/virhostcpu.c| 31 +++
src/util/virhostcpu.h| 1 +
3 files changed, 33
that accepts
empty files.
Signed-off-by: Michal Privoznik
---
src/libvirt_private.syms | 1 +
src/util/virbitmap.c | 40 +++-
src/util/virbitmap.h | 3 +++
tests/virbitmaptest.c| 40
4 files changed, 79
that accepts empty files.
Signed-off-by: Michal Privoznik
---
src/libvirt_private.syms | 1 +
src/util/virfile.c | 81 ++--
src/util/virfile.h | 2 +
3 files changed, 65 insertions(+), 19 deletions(-)
diff --git a/src/libvirt_private.syms b/src
Resolves: https://issues.redhat.com/browse/RHEL-33082
Signed-off-by: Michal Privoznik
---
src/qemu/qemu_process.c | 7 +++
1 file changed, 7 insertions(+)
diff --git a/src/qemu/qemu_process.c b/src/qemu/qemu_process.c
index da2b024f92..ef5338eda3 100644
--- a/src/qemu/qemu_process.c
+++ b/src/qemu
This is a helper that parses /sys/devices/system/cpu/isolated
into a virBitmap. It's going to be needed soon.
Signed-off-by: Michal Privoznik
---
src/libvirt_private.syms | 1 +
src/util/virhostcpu.c| 21 +
src/util/virhostcpu.h| 1 +
3 files changed, 23 insertions
*** BLURB HERE ***
Michal Prívozník (2):
virhostcpu: Introduce virHostCPUGetIsolated()
qemu: Substract isolcpus from all online affinity
src/libvirt_private.syms | 1 +
src/qemu/qemu_process.c | 7 +++
src/util/virhostcpu.c| 21 +
src/util/virhostcpu.h| 1
Signed-off-by: Michal Privoznik
---
src/util/virnetdevopenvswitch.c | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/src/util/virnetdevopenvswitch.c b/src/util/virnetdevopenvswitch.c
index f1765ae1c8..e23f4c83b6 100644
--- a/src/util/virnetdevopenvswitch.c
+++ b/src/util/virnetdevope
Signed-off-by: Michal Privoznik
---
docs/docs.rst | 3 +++
docs/meson.build | 1 +
docs/nss.rst | 7 ++
docs/ssh-proxy.rst | 60 ++
4 files changed, 71 insertions(+)
create mode 100644 docs/ssh-proxy.rst
diff --git a/docs/docs.rst
Signed-off-by: Michal Privoznik
---
NEWS.rst | 5 +
1 file changed, 5 insertions(+)
diff --git a/NEWS.rst b/NEWS.rst
index 852dadf532..3bfd6d6919 100644
--- a/NEWS.rst
+++ b/NEWS.rst
@@ -17,6 +17,11 @@ v10.3.0 (unreleased)
* **New features**
+ * SSH proxy for VM
+
+Libvirt now
d/blob/main/src/ssh-generator/ssh-proxy.c
2:
https://github.com/systemd/systemd/blob/main/src/ssh-generator/20-systemd-ssh-proxy.conf.in
Resolves: https://gitlab.com/libvirt/libvirt/-/issues/579
Signed-off-by: Michal Privoznik
---
libvirt.spec.in | 27 +++
meson
Some public objects (like virDomain, virInterface, and so on) are
missing g_autoptr() cleanup functions. Provide missing
declarations. Note, this is only for our internal use - hence
datatypes.h.
Signed-off-by: Michal Privoznik
---
src/datatypes.h | 16
1 file changed, 16
*** BLURB HERE ***
Michal Prívozník (4):
datatypes: Declare g_autoptr cleanup functions for more public objects
tools: Introduce SSH proxy
docs: Document SSH proxy
NEWS: Document SSH proxy feature
NEWS.rst | 5 +
docs/docs.rst
1 - 100 of 235 matches
Mail list logo