From: jrossi@linux.ibm.com
To: qemu-devel@nongnu.org, qemu-s390x@nongnu.org, cohuck@redhat.com,
mjrosato@linux.ibm.com
Cc: farman@linux.ibm.com, jjherne@linux.ibm.com,
jrossi@linux.ibm.com, zycai@linux.ibm.com,
qemu-stable@nongnu.org
Subject: [PATCH v3 2/8] hw/s390x/ipl: Fix incorrect PCI IPL block lengths
Date: Mon, 31 Aug 2026 15:49:13 -0400 [thread overview]
Message-ID: <20260831194919.3830270-3-jrossi@linux.ibm.com> (raw)
In-Reply-To: <20260831194919.3830270-1-jrossi@linux.ibm.com>
From: Jared Rossi <jrossi@linux.ibm.com>
The length of the PCI IPLB is calculated by size of the IplBlockPci struct
plus the number of bytes used for common IPLB fields.
Fix the S390_IPLB_MIN_PCI_LEN to reflect this, and assign a valid blk0_len
while we are at it.
Cc: qemu-stable@nongnu.org
Fixes: 395116781df5 ("s390x: Build IPLB for virtio-pci devices")
Signed-off-by: Jared Rossi <jrossi@linux.ibm.com>
---
hw/s390x/ipl.c | 2 ++
include/hw/s390x/ipl/qipl.h | 2 +-
2 files changed, 3 insertions(+), 1 deletion(-)
diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c
index d59ed36c78..ba69ee89bb 100644
--- a/hw/s390x/ipl.c
+++ b/hw/s390x/ipl.c
@@ -615,6 +615,8 @@ static bool s390_build_iplb(DeviceState *dev_st, IplParameterBlock *iplb)
switch (devtype) {
case PCI_DEVTYPE_VIRTIO:
iplb->len = cpu_to_be32(S390_IPLB_MIN_PCI_LEN);
+ iplb->blk0_len =
+ cpu_to_be32(S390_IPLB_MIN_PCI_LEN - S390_IPLB_HEADER_LEN);
iplb->pbt = S390_IPL_TYPE_PCI;
iplb->pci.fid = cpu_to_be32(pbdev->fid);
break;
diff --git a/include/hw/s390x/ipl/qipl.h b/include/hw/s390x/ipl/qipl.h
index 3fa9649112..69b0023ea3 100644
--- a/include/hw/s390x/ipl/qipl.h
+++ b/include/hw/s390x/ipl/qipl.h
@@ -39,7 +39,7 @@ typedef enum S390IplType S390IplType;
#define S390_IPLB_MIN_PV_LEN 148
#define S390_IPLB_MIN_CCW_LEN 200
#define S390_IPLB_MIN_FCP_LEN 384
-#define S390_IPLB_MIN_PCI_LEN 376
+#define S390_IPLB_MIN_PCI_LEN (offsetof(IplParameterBlock, pci) + sizeof(IplBlockPci))
#define S390_IPLB_MIN_QEMU_SCSI_LEN 200
#define S390_IPLB_MAX_LEN 4096
--
2.54.0
next prev parent reply other threads:[~2026-08-31 19:50 UTC|newest]
Thread overview: 23+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-08-31 19:49 [PATCH v3 0/8] s390x: Add support for virtio-scsi-pci boot device jrossi
2026-08-31 19:49 ` [PATCH v3 1/8] pc-bios/s390-ccw: Check if a PCI function is already enabled before trying to enable it jrossi
2026-09-03 12:44 ` Jason J. Herne
2026-08-31 19:49 ` jrossi [this message]
2026-09-02 9:53 ` [PATCH v3 2/8] hw/s390x/ipl: Fix incorrect PCI IPL block lengths Eric Farman
2026-09-03 12:56 ` Jason J. Herne
2026-09-09 15:03 ` Matthew Rosato
2026-08-31 19:49 ` [PATCH v3 3/8] pc-bios/s390-ccw: Use bus specific IPL_TYPE directly for scsi IPL jrossi
2026-09-03 13:15 ` Jason J. Herne
2026-08-31 19:49 ` [PATCH v3 4/8] s390x/ipl: Add PCI and bus fields to IplBlockQemuScsi jrossi
2026-09-03 13:53 ` Jason J. Herne
2026-08-31 19:49 ` [PATCH v3 5/8] pc-bios/s390-ccw: Abstract virtio_run() for generic use jrossi
2026-09-03 14:55 ` Jason J. Herne
2026-08-31 19:49 ` [PATCH v3 6/8] pc-bios/s390-ccw: Add support for virtio-scsi-pci IPL jrossi
2026-09-02 10:18 ` Eric Farman
2026-09-03 15:04 ` Jason J. Herne
2026-08-31 19:49 ` [PATCH v3 7/8] s390x: Find scsi-pci boot device and build IPLB jrossi
2026-09-01 2:54 ` Joshua Daley
2026-09-02 20:19 ` Eric Farman
2026-09-03 19:07 ` Jason J. Herne
2026-08-31 19:49 ` [PATCH v3 8/8] tests/qtest: Add s390x PCI SCSI fallback test to cdrom-test.c jrossi
2026-09-01 2:06 ` Joshua Daley
2026-09-02 20:44 ` Eric Farman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20260831194919.3830270-3-jrossi@linux.ibm.com \
--to=jrossi@linux.ibm.com \
--cc=cohuck@redhat.com \
--cc=farman@linux.ibm.com \
--cc=jjherne@linux.ibm.com \
--cc=mjrosato@linux.ibm.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=qemu-stable@nongnu.org \
--cc=zycai@linux.ibm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.