Re: [PATCH 1/3] conf: Store the iothread 'poll' settings in the XML

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 31, 2023 at 02:00:32PM +0200, Peter Krempa wrote:
Currently we allow configuring the 'poll-max-ns', 'poll-grow', and
'poll-shrink' parameters of qemu iothreads only during runtime and they
are not persisted. Add XML machinery to persist them.

Signed-off-by: Peter Krempa <pkrempa@xxxxxxxxxx>
---
docs/formatdomain.rst                         | 11 ++++-
src/conf/domain_conf.c                        | 41 ++++++++++++++++++-
src/conf/domain_conf.h                        |  7 ++++
src/conf/schemas/domaincommon.rng             | 19 +++++++++
.../iothreads-ids-pool-sizes.xml              | 12 ++++--
5 files changed, 85 insertions(+), 5 deletions(-)


[...]

diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 5a2c70f012..8ac76f25d3 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -2713,6 +2713,13 @@ struct _virDomainIOThreadIDDef {

    virDomainThreadSchedParam sched;

+    unsigned long long poll_max_ns;
+    bool set_poll_max_ns;
+    unsigned int poll_grow;
+    bool set_poll_grow;
+    unsigned int poll_shrink;
+    bool set_poll_shrink;
+

All these are int64_t in QEMU, although I understand we don't need to
represent all the possible values, but it is weird when each one is
different here.

I think you can't go wrong here if you make all of them unsigned long
long.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux