71aed0d95c
- kvm-virtio-net-Add-queues-before-loading-them.patch [RHEL-69477] - kvm-docs-system-s390x-bootdevices-Update-loadparm-docume.patch [RHEL-68440] - kvm-docs-system-bootindex-Make-it-clear-that-s390x-can-a.patch [RHEL-68440] - kvm-hw-s390x-Restrict-loadparm-property-to-devices-that-.patch [RHEL-68440] - kvm-hw-Add-loadparm-property-to-scsi-disk-devices-for-bo.patch [RHEL-68440] - kvm-scsi-fix-allocation-for-s390x-loadparm.patch [RHEL-68440] - kvm-pc-bios-s390x-Initialize-cdrom-type-to-false-for-eac.patch [RHEL-68440] - kvm-pc-bios-s390x-Initialize-machine-loadparm-before-pro.patch [RHEL-68440] - kvm-pc-bios-s390-ccw-Re-initialize-receive-queue-index-b.patch [RHEL-68440] - kvm-vnc-fix-crash-when-no-console-attached.patch [RHEL-61633] - Resolves: RHEL-69477 (qemu crashed when migrate vm with multiqueue from rhel9.4 to rhel9.6) - Resolves: RHEL-68440 (The new "boot order" feature is sometimes not working as expected [RHEL 9]) - Resolves: RHEL-61633 (Qemu-kvm crashed if no display device setting and switching display by remote-viewer [rhel-9])
114 lines
4.3 KiB
Diff
114 lines
4.3 KiB
Diff
From ffffba85b269096d25fef307b342b8f576610d34 Mon Sep 17 00:00:00 2001
|
|
From: Thomas Huth <thuth@redhat.com>
|
|
Date: Wed, 13 Nov 2024 12:47:41 +0100
|
|
Subject: [PATCH 04/10] hw/s390x: Restrict "loadparm" property to devices that
|
|
can be used for booting
|
|
MIME-Version: 1.0
|
|
Content-Type: text/plain; charset=UTF-8
|
|
Content-Transfer-Encoding: 8bit
|
|
|
|
RH-Author: Thomas Huth <thuth@redhat.com>
|
|
RH-MergeRequest: 298: [c9s] Fixes for the new s390x "boot order" feature
|
|
RH-Jira: RHEL-68440
|
|
RH-Acked-by: Cédric Le Goater <clg@redhat.com>
|
|
RH-Acked-by: Miroslav Rezanina <mrezanin@redhat.com>
|
|
RH-Commit: [3/8] e381997b44f5b8c80a90ae99f5ae5d863f6d5aa5 (thuth/qemu-kvm-cs9)
|
|
|
|
Commit bb185de423 ("s390x: Add individual loadparm assignment to
|
|
CCW device") added a "loadparm" property to all CCW devices. This
|
|
was a little bit unfortunate, since this property is only useful
|
|
for devices that can be used for booting, but certainly it is not
|
|
useful for devices like virtio-gpu or virtio-tablet.
|
|
|
|
Thus let's restrict the property to CCW devices that we can boot from
|
|
(i.e. virtio-block, virtio-net and vfio-ccw devices).
|
|
|
|
Message-ID: <20241113114741.681096-1-thuth@redhat.com>
|
|
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
|
|
Reviewed-by: Jared Rossi <jrossi@linux.ibm.com>
|
|
Signed-off-by: Thomas Huth <thuth@redhat.com>
|
|
(cherry picked from commit 6e7c96ae61e0542e97d385084f1f2281a0331054)
|
|
---
|
|
hw/s390x/ccw-device.c | 4 +---
|
|
hw/s390x/ccw-device.h | 5 +++++
|
|
hw/s390x/virtio-ccw-blk.c | 1 +
|
|
hw/s390x/virtio-ccw-net.c | 1 +
|
|
hw/vfio/ccw.c | 1 +
|
|
5 files changed, 9 insertions(+), 3 deletions(-)
|
|
|
|
diff --git a/hw/s390x/ccw-device.c b/hw/s390x/ccw-device.c
|
|
index 4e54f34b1c..d7bb364579 100644
|
|
--- a/hw/s390x/ccw-device.c
|
|
+++ b/hw/s390x/ccw-device.c
|
|
@@ -73,7 +73,7 @@ static void ccw_device_set_loadparm(Object *obj, Visitor *v,
|
|
s390_ipl_fmt_loadparm(dev->loadparm, val, errp);
|
|
}
|
|
|
|
-static const PropertyInfo ccw_loadparm = {
|
|
+const PropertyInfo ccw_loadparm = {
|
|
.name = "ccw_loadparm",
|
|
.description = "Up to 8 chars in set of [A-Za-z0-9. ] to pass"
|
|
" to the guest loader/kernel",
|
|
@@ -85,8 +85,6 @@ static Property ccw_device_properties[] = {
|
|
DEFINE_PROP_CSS_DEV_ID("devno", CcwDevice, devno),
|
|
DEFINE_PROP_CSS_DEV_ID_RO("dev_id", CcwDevice, dev_id),
|
|
DEFINE_PROP_CSS_DEV_ID_RO("subch_id", CcwDevice, subch_id),
|
|
- DEFINE_PROP("loadparm", CcwDevice, loadparm, ccw_loadparm,
|
|
- typeof(uint8_t[8])),
|
|
DEFINE_PROP_END_OF_LIST(),
|
|
};
|
|
|
|
diff --git a/hw/s390x/ccw-device.h b/hw/s390x/ccw-device.h
|
|
index 1e1737c0f3..4439feb140 100644
|
|
--- a/hw/s390x/ccw-device.h
|
|
+++ b/hw/s390x/ccw-device.h
|
|
@@ -51,4 +51,9 @@ static inline CcwDevice *to_ccw_dev_fast(DeviceState *d)
|
|
|
|
OBJECT_DECLARE_TYPE(CcwDevice, CCWDeviceClass, CCW_DEVICE)
|
|
|
|
+extern const PropertyInfo ccw_loadparm;
|
|
+
|
|
+#define DEFINE_PROP_CCW_LOADPARM(_n, _s, _f) \
|
|
+ DEFINE_PROP(_n, _s, _f, ccw_loadparm, typeof(uint8_t[8]))
|
|
+
|
|
#endif
|
|
diff --git a/hw/s390x/virtio-ccw-blk.c b/hw/s390x/virtio-ccw-blk.c
|
|
index 8e0e58b77d..2364432c6e 100644
|
|
--- a/hw/s390x/virtio-ccw-blk.c
|
|
+++ b/hw/s390x/virtio-ccw-blk.c
|
|
@@ -48,6 +48,7 @@ static Property virtio_ccw_blk_properties[] = {
|
|
VIRTIO_CCW_FLAG_USE_IOEVENTFD_BIT, true),
|
|
DEFINE_PROP_UINT32("max_revision", VirtioCcwDevice, max_rev,
|
|
VIRTIO_CCW_MAX_REV),
|
|
+ DEFINE_PROP_CCW_LOADPARM("loadparm", CcwDevice, loadparm),
|
|
DEFINE_PROP_END_OF_LIST(),
|
|
};
|
|
|
|
diff --git a/hw/s390x/virtio-ccw-net.c b/hw/s390x/virtio-ccw-net.c
|
|
index 484e617659..a4a3f65c7e 100644
|
|
--- a/hw/s390x/virtio-ccw-net.c
|
|
+++ b/hw/s390x/virtio-ccw-net.c
|
|
@@ -51,6 +51,7 @@ static Property virtio_ccw_net_properties[] = {
|
|
VIRTIO_CCW_FLAG_USE_IOEVENTFD_BIT, true),
|
|
DEFINE_PROP_UINT32("max_revision", VirtioCcwDevice, max_rev,
|
|
VIRTIO_CCW_MAX_REV),
|
|
+ DEFINE_PROP_CCW_LOADPARM("loadparm", CcwDevice, loadparm),
|
|
DEFINE_PROP_END_OF_LIST(),
|
|
};
|
|
|
|
diff --git a/hw/vfio/ccw.c b/hw/vfio/ccw.c
|
|
index 115862f430..99f16614ad 100644
|
|
--- a/hw/vfio/ccw.c
|
|
+++ b/hw/vfio/ccw.c
|
|
@@ -662,6 +662,7 @@ static Property vfio_ccw_properties[] = {
|
|
DEFINE_PROP_LINK("iommufd", VFIOCCWDevice, vdev.iommufd,
|
|
TYPE_IOMMUFD_BACKEND, IOMMUFDBackend *),
|
|
#endif
|
|
+ DEFINE_PROP_CCW_LOADPARM("loadparm", CcwDevice, loadparm),
|
|
DEFINE_PROP_END_OF_LIST(),
|
|
};
|
|
|
|
--
|
|
2.39.3
|
|
|