From: Zhuoying Cai <zycai@linux.ibm.com>
To: qemu-s390x@nongnu.org, qemu-devel@nongnu.org
Cc: jrossi@linux.ibm.com, cohuck@redhat.com, berrange@redhat.com,
richard.henderson@linaro.org, david@kernel.org,
walling@linux.ibm.com, jjherne@linux.ibm.com,
pasic@linux.ibm.com, borntraeger@linux.ibm.com,
farman@linux.ibm.com, mjrosato@linux.ibm.com, iii@linux.ibm.com,
eblake@redhat.com, armbru@redhat.com, zycai@linux.ibm.com,
alifm@linux.ibm.com, brueckner@linux.ibm.com,
pierrick.bouvier@oss.qualcomm.com, jdaley@linux.ibm.com
Subject: [PATCH v13 17/33] hw/s390x/ipl: Add IPIB flags to IPL Parameter Block
Date: Thu, 2 Jul 2026 22:29:15 -0400 [thread overview]
Message-ID: <20260703022933.1805010-18-zycai@linux.ibm.com> (raw)
In-Reply-To: <20260703022933.1805010-1-zycai@linux.ibm.com>
Add IPIB flags to IPL Parameter Block to determine if IPL needs to
perform securely and if IPL Information Report Block (IIRB) exists.
Move DIAG308 flags to a separated header file and add flags for secure IPL.
Move IPLB length related definitions to include/hw/s390x/ipl/qipl.h and
add a maximum length constant to support secure IPL.
Secure boot in audit mode will perform if certificate(s) exist in the
key store. IIRB will exist and results of verification will be stored in
IIRB.
To ensure proper alignment of the IIRB and prevent overlap, set
iplb->len to the maximum length of the IPLB, allowing alignment
constraints to be determined based on its size.
Signed-off-by: Zhuoying Cai <zycai@linux.ibm.com>
Reviewed-by: Thomas Huth <thuth@redhat.com>
Reviewed-by: Collin Walling <walling@linux.ibm.com>
Reviewed-by: Jared Rossi <jrossi@linux.ibm.com>
---
hw/s390x/ipl.c | 36 ++++++++++++++++++++++++++++++++++
hw/s390x/ipl.h | 24 -----------------------
include/hw/s390x/ipl/diag308.h | 34 ++++++++++++++++++++++++++++++++
include/hw/s390x/ipl/qipl.h | 13 +++++++++++-
4 files changed, 82 insertions(+), 25 deletions(-)
create mode 100644 include/hw/s390x/ipl/diag308.h
diff --git a/hw/s390x/ipl.c b/hw/s390x/ipl.c
index 09c24203c7..f9f14d7b67 100644
--- a/hw/s390x/ipl.c
+++ b/hw/s390x/ipl.c
@@ -461,6 +461,34 @@ S390IPLCertificateStore *s390_ipl_get_certificate_store(void)
return &ipl->cert_store;
}
+static bool s390_has_certificate(void)
+{
+ S390IPLState *ipl = get_ipl_device();
+
+ return ipl->cert_store.count > 0;
+}
+
+static void s390_set_secure_boot_flags(IplParameterBlock *iplb,
+ bool audit_mode)
+{
+ if (!audit_mode) {
+ return;
+ }
+
+ /*
+ * For audit mode, enable the IPL Information
+ * Report (IPLIR) flag so that the firmware generates an IPL
+ * Information Report Block (IIRB).
+ *
+ * Results of secure boot will be stored in IIRB.
+ *
+ * Extend the IPL parameter block to its maximum length to ensure
+ * sufficient space for the BIOS to populate the IIRB.
+ */
+ iplb->hdr_flags |= DIAG308_IPIB_FLAGS_IPLIR;
+ iplb->len = cpu_to_be32(S390_IPLB_MAX_LEN);
+}
+
static bool s390_build_iplb(DeviceState *dev_st, IplParameterBlock *iplb)
{
CcwDevice *ccw_dev = NULL;
@@ -517,6 +545,8 @@ static bool s390_build_iplb(DeviceState *dev_st, IplParameterBlock *iplb)
s390_ipl_convert_loadparm((char *)lp, iplb->loadparm);
iplb->flags |= DIAG308_FLAGS_LP_VALID;
+ s390_set_secure_boot_flags(iplb, s390_has_certificate());
+
return true;
}
@@ -653,6 +683,12 @@ void s390_ipl_update_diag308(IplParameterBlock *iplb)
} else {
ipl->iplb = *iplb;
ipl->iplb_valid = true;
+
+ /*
+ * The kernel does not preserve secure boot flags across a reboot.
+ * Re-apply them here based on the current machine configuration.
+ */
+ s390_set_secure_boot_flags(&ipl->iplb, s390_has_certificate());
}
update_machine_ipl_properties(iplb);
diff --git a/hw/s390x/ipl.h b/hw/s390x/ipl.h
index f5a49a4431..9807ef18f2 100644
--- a/hw/s390x/ipl.h
+++ b/hw/s390x/ipl.h
@@ -23,7 +23,6 @@
#include "qom/object.h"
#include "target/s390x/kvm/pv.h"
-#define DIAG308_FLAGS_LP_VALID 0x80
#define MAX_BOOT_DEVS 8 /* Max number of devices that may have a bootindex */
void s390_ipl_convert_loadparm(char *ascii_lp, uint8_t *ebcdic_lp);
@@ -90,29 +89,6 @@ struct S390IPLState {
};
QEMU_BUILD_BUG_MSG(offsetof(S390IPLState, iplb) & 3, "alignment of iplb wrong");
-#define DIAG_308_RC_OK 0x0001
-#define DIAG_308_RC_NO_CONF 0x0102
-#define DIAG_308_RC_INVALID 0x0402
-#define DIAG_308_RC_NO_PV_CONF 0x0902
-#define DIAG_308_RC_INVAL_FOR_PV 0x0a02
-
-#define DIAG308_RESET_MOD_CLR 0
-#define DIAG308_RESET_LOAD_NORM 1
-#define DIAG308_LOAD_CLEAR 3
-#define DIAG308_LOAD_NORMAL_DUMP 4
-#define DIAG308_SET 5
-#define DIAG308_STORE 6
-#define DIAG308_PV_SET 8
-#define DIAG308_PV_STORE 9
-#define DIAG308_PV_START 10
-
-#define S390_IPLB_HEADER_LEN 8
-#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_QEMU_SCSI_LEN 200
-
static inline bool iplb_valid_len(IplParameterBlock *iplb)
{
return be32_to_cpu(iplb->len) <= sizeof(IplParameterBlock);
diff --git a/include/hw/s390x/ipl/diag308.h b/include/hw/s390x/ipl/diag308.h
new file mode 100644
index 0000000000..6e62f29215
--- /dev/null
+++ b/include/hw/s390x/ipl/diag308.h
@@ -0,0 +1,34 @@
+/*
+ * S/390 DIAGNOSE 308 definitions and structures
+ *
+ * Copyright 2025 IBM Corp.
+ * Author(s): Zhuoying Cai <zycai@linux.ibm.com>
+ *
+ * SPDX-License-Identifier: GPL-2.0-or-later
+ */
+
+#ifndef S390X_DIAG308_H
+#define S390X_DIAG308_H
+
+#define DIAG_308_RC_OK 0x0001
+#define DIAG_308_RC_NO_CONF 0x0102
+#define DIAG_308_RC_INVALID 0x0402
+#define DIAG_308_RC_NO_PV_CONF 0x0902
+#define DIAG_308_RC_INVAL_FOR_PV 0x0a02
+
+#define DIAG308_RESET_MOD_CLR 0
+#define DIAG308_RESET_LOAD_NORM 1
+#define DIAG308_LOAD_CLEAR 3
+#define DIAG308_LOAD_NORMAL_DUMP 4
+#define DIAG308_SET 5
+#define DIAG308_STORE 6
+#define DIAG308_PV_SET 8
+#define DIAG308_PV_STORE 9
+#define DIAG308_PV_START 10
+
+#define DIAG308_FLAGS_LP_VALID 0x80
+
+#define DIAG308_IPIB_FLAGS_SIPL 0x40
+#define DIAG308_IPIB_FLAGS_IPLIR 0x20
+
+#endif
diff --git a/include/hw/s390x/ipl/qipl.h b/include/hw/s390x/ipl/qipl.h
index 9940f1457c..a2180719b1 100644
--- a/include/hw/s390x/ipl/qipl.h
+++ b/include/hw/s390x/ipl/qipl.h
@@ -12,6 +12,8 @@
#ifndef S390X_QIPL_H
#define S390X_QIPL_H
+#include "diag308.h"
+
/* Boot Menu flags */
#define QIPL_FLAG_BM_OPTS_CMD 0x80
#define QIPL_FLAG_BM_OPTS_ZIPL 0x40
@@ -31,6 +33,14 @@ typedef enum S390IplType S390IplType;
#define QEMU_DEFAULT_IPL S390_IPL_TYPE_CCW
+#define S390_IPLB_HEADER_LEN 8
+#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_QEMU_SCSI_LEN 200
+#define S390_IPLB_MAX_LEN 4096
+
#define MAX_CERTIFICATES 64
/* largest supported block size - same as VIRTIO_DASD_DEFAULT_BLOCK_SIZE */
#define VIRTIO_MAX_BLOCK_SIZE 4096
@@ -125,7 +135,8 @@ typedef struct IplBlockPci IplBlockPci;
union IplParameterBlock {
struct {
uint32_t len;
- uint8_t reserved0[3];
+ uint8_t hdr_flags;
+ uint8_t reserved0[2];
uint8_t version;
uint32_t blk0_len;
uint8_t pbt;
--
2.54.0
next prev parent reply other threads:[~2026-07-03 2:31 UTC|newest]
Thread overview: 45+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-03 2:28 [PATCH v13 00/33] Secure IPL Support for SCSI Scheme of virtio-blk/virtio-scsi Devices Zhuoying Cai
2026-07-03 2:28 ` [PATCH v13 01/33] Add boot-certs to s390-ccw-virtio machine type option Zhuoying Cai
2026-07-03 13:24 ` Matthew Rosato
2026-07-03 2:29 ` [PATCH v13 02/33] crypto/x509-utils: Refactor with GNUTLS fallback Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 03/33] crypto/x509-utils: Add helper functions for certificate store Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 04/33] hw/s390x/ipl: Create " Zhuoying Cai
2026-07-03 16:56 ` Matthew Rosato
2026-07-03 2:29 ` [PATCH v13 05/33] s390x/diag: Introduce DIAG 320 for Certificate Store Facility Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 06/33] s390x/diag: Refactor address validation check from diag308_parm_check Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 07/33] s390x/diag: Implement DIAG 320 subcode 1 Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 08/33] crypto/x509-utils: Add helper functions for DIAG 320 subcode 2 Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 09/33] s390x/diag: Implement " Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 10/33] hw/s390x: Define finite size for single entry VCEntry Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 11/33] s390x/diag: Introduce DIAG 508 for secure IPL operations Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 12/33] crypto/x509-utils: Add helper functions for DIAG 508 subcode 1 Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 13/33] s390x/diag: Generalize s390_ipl_read/write to accept void * Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 14/33] s390x/diag: Implement DIAG 508 subcode 1 for signature verification Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 15/33] s390x/ipl: Introduce IPL Information Report Block (IIRB) Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 16/33] pc-bios/s390-ccw: Define memory for IPLB and convert IPLB to pointers Zhuoying Cai
2026-07-03 2:29 ` Zhuoying Cai [this message]
2026-07-03 2:29 ` [PATCH v13 18/33] hw/s390x/ipl: Rework s390_ipl_map_iplb_chain for certificate storage Zhuoying Cai
2026-07-03 13:47 ` Eric Farman
2026-07-03 2:29 ` [PATCH v13 19/33] s390x: Guest support for Secure-IPL Facility Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 20/33] pc-bios/s390-ccw: Refactor zipl_run() Zhuoying Cai
2026-07-03 2:40 ` Philippe Mathieu-Daudé
2026-07-03 2:29 ` [PATCH v13 21/33] pc-bios/s390-ccw: Rework zipl_load_segment function Zhuoying Cai
2026-07-03 2:38 ` Philippe Mathieu-Daudé
2026-07-03 14:28 ` Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 22/33] pc-bios/s390-ccw: Introduce ZiplBootMode enum for IPL mode selection Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 23/33] pc-bios/s390-ccw: Add signature verification for secure IPL in audit mode Zhuoying Cai
2026-07-03 11:40 ` Eric Farman
2026-07-03 13:56 ` Zhuoying Cai
2026-07-04 0:43 ` Eric Farman
2026-07-03 13:58 ` Eric Farman
2026-07-03 2:29 ` [PATCH v13 24/33] pc-bios/s390-ccw: Add signed component address overlap checks Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 25/33] s390x: Guest support for Secure-IPL Code Loading Attributes Facility (SCLAF) Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 26/33] pc-bios/s390-ccw: Add additional security checks for secure boot Zhuoying Cai
2026-07-03 12:17 ` Eric Farman
2026-07-03 2:29 ` [PATCH v13 27/33] Add secure-boot to s390-ccw-virtio machine type option Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 28/33] hw/s390x/ipl: Set IPIB flags for secure IPL Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 29/33] pc-bios/s390-ccw: Handle true secure IPL mode Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 30/33] hw/s390x/ipl: Handle secure boot with multiple boot devices Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 31/33] tests/functional/s390x: Add secure IPL functional test Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 32/33] docs/specs: Add secure IPL documentation Zhuoying Cai
2026-07-03 2:29 ` [PATCH v13 33/33] docs/system/s390x: " Zhuoying Cai
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=20260703022933.1805010-18-zycai@linux.ibm.com \
--to=zycai@linux.ibm.com \
--cc=alifm@linux.ibm.com \
--cc=armbru@redhat.com \
--cc=berrange@redhat.com \
--cc=borntraeger@linux.ibm.com \
--cc=brueckner@linux.ibm.com \
--cc=cohuck@redhat.com \
--cc=david@kernel.org \
--cc=eblake@redhat.com \
--cc=farman@linux.ibm.com \
--cc=iii@linux.ibm.com \
--cc=jdaley@linux.ibm.com \
--cc=jjherne@linux.ibm.com \
--cc=jrossi@linux.ibm.com \
--cc=mjrosato@linux.ibm.com \
--cc=pasic@linux.ibm.com \
--cc=pierrick.bouvier@oss.qualcomm.com \
--cc=qemu-devel@nongnu.org \
--cc=qemu-s390x@nongnu.org \
--cc=richard.henderson@linaro.org \
--cc=walling@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.