[PATCH v5 0/3] Fix virtio console port assignment issue

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

 



Changelog:
---
v5:
 - Added xml tests to tests/qemuxmlconfdata
 - Fixed virito -> virtio typo in commit message
---
v4:
 - Update commit messages
---
v3:
 - Added Reviewed-By
 - Included CI Results Link
---
v2:
 - Split patch into two commits
 - Added fixes tag
---

This libvirt patch series does the following:
1. fixes an issue with virtio console device port auto assignment on vioserial buses
2. updates console port reservation comment and changes the allowZero variable to allowPortZero for clarity
3. Adds tests for virtio console on the vioserial bus

Currently in libvirt, a virtio console device cannot be auto assigned a port number greater than zero on a vioserial bus. This leads to port collision errors when adding more than 1 virtio console device on a single vioserial bus.

After applying this patch, one can add multiple console ports under a single vioserial bus.

Here is a link to CI results for this series: https://gitlab.com/aaronbmalik/libvirt/-/pipelines/1855918840

Aaron M. Brown (3):
  qemuxmlconftest: Add console-virtio-vioserial tests
  virDomainVirtioSerialAddrAssign: Fix virtio console port auto
    assignment on vioserial bus
  domain_addr.c: update console port reservation comment and allowZero
    var to allowPortZero for clarity

 src/conf/domain_addr.c                        | 27 +++++---
 ...onsole-virtio-vioserial.x86_64-latest.args | 44 +++++++++++++
 ...console-virtio-vioserial.x86_64-latest.xml | 63 +++++++++++++++++++
 .../console-virtio-vioserial.xml              | 48 ++++++++++++++
 tests/qemuxmlconftest.c                       |  1 +
 5 files changed, 173 insertions(+), 10 deletions(-)
 create mode 100644 tests/qemuxmlconfdata/console-virtio-vioserial.x86_64-latest.args
 create mode 100644 tests/qemuxmlconfdata/console-virtio-vioserial.x86_64-latest.xml
 create mode 100644 tests/qemuxmlconfdata/console-virtio-vioserial.xml

--
2.39.5 (Apple Git-154)



[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