On Tue, 09 Jun 2026 13:26:09 +0200, Philippe Mathieu-Daudé <[email protected]> wrote: > > Hi Kirill, > > On 9/6/26 04:30, Kirill A. Korinsky wrote: > > On Tue, 09 Jun 2026 04:19:45 +0200, > > "Kirill A. Korinsky" <[email protected]> wrote: > >> > >> OpenBSD/powernv programs PTCR for LPID 0 with a partition > >> table size exponent one smaller than QEMU's existing ISA v3.0 > >> interpretation. Try QEMU's existing PATS interpretation first, > >> then fall back to the OpenBSD LPID 0 form; nonzero LPIDs keep > >> the old behaviour. > >> > >> The PSI model now exposes POWER9 IRQ level and pending status > >> registers, and keeps both updated while delivering through the > >> existing XIVE LSI source. This lets guests that select the > >> POWER9 PSI LSI IRQ method continue to receive LPC interrupts. > >> > >> The blast radius is probably minimal: the partition table > >> fallback is limited to bare metal LPID 0 after the original > >> lookup fails, while the PSI change only touches POWER9 PSI > >> state and reuses the existing delivery path. > >> > >> Signed-off-by: Kirill A. Korinsky <[email protected]> > > > > > > It was tested as: > > > > qemu-system-ppc64 \ > > -M powernv9 \ > > -cpu power9 \ > > -m 2G \ > > -nographic \ > > -kernel pnor.BOOTKERNEL \ > > -device ich9-ahci,id=sata0,bus=pcie.0,addr=0x0 \ > > -drive file=miniroot79.img,format=raw,if=none,id=bootdisk,snapshot=on \ > > -device ide-hd,bus=sata0.0,unit=0,drive=bootdisk,bootindex=1 > > Cool. Could you add as a functional test in tests/functional/ppc64/? See > examples around this directory. >
Sure, here I added functional test and I confirm that all old one passed. I host artifacts for test at my host because I have no idea where to host them, but feel free to move to the right place. Miniroot from 7.9 release which is available at: https://cdn.openbsd.org/pub/OpenBSD/7.9/powerpc64/miniroot79.img but in few years can be gone. pnor.BOOTKERNEL from talos-ii-v2.10.pnor which was extracted by my script. -- wbr, Kirill
>From f3641d1db5e6a2ec78e89f333b5957064db9bdc6 Mon Sep 17 00:00:00 2001 From: "Kirill A. Korinsky" <[email protected]> Date: Tue, 9 Jun 2026 18:29:44 +0200 Subject: [PATCH] powernv: boot OpenBSD on POWER9 OpenBSD/powernv programs PTCR for LPID 0 with a partition table size exponent one smaller than QEMU's existing ISA v3.0 interpretation. Try QEMU's existing PATS interpretation first, then fall back to the OpenBSD LPID 0 form; nonzero LPIDs keep the old behaviour. The PSI model now exposes POWER9 IRQ level and pending status registers, and keeps both updated while delivering through the existing XIVE LSI source. This lets guests that select the POWER9 PSI LSI IRQ method continue to receive LPC interrupts. The blast radius is probably minimal: the partition table fallback is limited to bare metal LPID 0 after the original lookup fails, while the PSI change only touches POWER9 PSI state and reuses the existing delivery path. Signed-off-by: Kirill A. Korinsky <[email protected]> --- hw/ppc/pnv_psi.c | 17 ++++----- target/ppc/mmu-book3s-v3.c | 33 +++++++++++++--- tests/functional/ppc64/meson.build | 2 + tests/functional/ppc64/test_openbsd.py | 52 ++++++++++++++++++++++++++ 4 files changed, 90 insertions(+), 14 deletions(-) create mode 100755 tests/functional/ppc64/test_openbsd.py diff --git hw/ppc/pnv_psi.c hw/ppc/pnv_psi.c index e8701c6100..941540df2d 100644 --- hw/ppc/pnv_psi.c +++ hw/ppc/pnv_psi.c @@ -688,6 +688,8 @@ static uint64_t pnv_psi_p9_mmio_read(void *opaque, hwaddr addr, unsigned size) case PSIHB9_ESB_CI_BASE: case PSIHB9_ESB_NOTIF_ADDR: case PSIHB9_IVT_OFFSET: + case PSIHB9_IRQ_LEVEL: + case PSIHB9_IRQ_STAT: val = psi->regs[reg]; break; default: @@ -817,18 +819,15 @@ static const MemoryRegionOps pnv_psi_p9_xscom_ops = { static void pnv_psi_power9_set_irq(void *opaque, int irq, int state) { PnvPsi *psi = opaque; - uint64_t irq_method = psi->regs[PSIHB_REG(PSIHB9_INTERRUPT_CONTROL)]; + uint64_t irq_bit = PPC_BIT(irq); - if (irq_method & PSIHB9_IRQ_METHOD) { - qemu_log_mask(LOG_GUEST_ERROR, "PSI: LSI IRQ method no supported\n"); - return; - } - - /* Update LSI levels */ + /* Update LSI levels and pending status */ if (state) { - psi->regs[PSIHB_REG(PSIHB9_IRQ_LEVEL)] |= PPC_BIT(irq); + psi->regs[PSIHB_REG(PSIHB9_IRQ_LEVEL)] |= irq_bit; + psi->regs[PSIHB_REG(PSIHB9_IRQ_STAT)] |= irq_bit; } else { - psi->regs[PSIHB_REG(PSIHB9_IRQ_LEVEL)] &= ~PPC_BIT(irq); + psi->regs[PSIHB_REG(PSIHB9_IRQ_LEVEL)] &= ~irq_bit; + psi->regs[PSIHB_REG(PSIHB9_IRQ_STAT)] &= ~irq_bit; } qemu_set_irq(psi->qirqs[irq], state); diff --git target/ppc/mmu-book3s-v3.c target/ppc/mmu-book3s-v3.c index 3865556310..4babe4c536 100644 --- target/ppc/mmu-book3s-v3.c +++ target/ppc/mmu-book3s-v3.c @@ -23,19 +23,21 @@ #include "mmu-hash64.h" #include "mmu-book3s-v3.h" -bool ppc64_v3_get_pate(PowerPCCPU *cpu, target_ulong lpid, ppc_v3_pate_t *entry) +static bool ppc64_v3_get_pate_from_size(PowerPCCPU *cpu, target_ulong lpid, + ppc_v3_pate_t *entry, + uint64_t table_size) { uint64_t patb = cpu->env.spr[SPR_PTCR] & PTCR_PATB; - uint64_t pats = cpu->env.spr[SPR_PTCR] & PTCR_PATS; + uint64_t entries; /* Check if partition table is properly aligned */ - if (patb & MAKE_64BIT_MASK(0, pats + 12)) { + if (patb & (table_size - 1)) { return false; } /* Calculate number of entries */ - pats = 1ull << (pats + 12 - 4); - if (pats <= lpid) { + entries = table_size / sizeof(*entry); + if (entries <= lpid) { return false; } @@ -45,3 +47,24 @@ bool ppc64_v3_get_pate(PowerPCCPU *cpu, target_ulong lpid, ppc_v3_pate_t *entry) entry->dw1 = ldq_phys(CPU(cpu)->as, patb + 8); return true; } + +bool ppc64_v3_get_pate(PowerPCCPU *cpu, target_ulong lpid, ppc_v3_pate_t *entry) +{ + uint64_t pats = cpu->env.spr[SPR_PTCR] & PTCR_PATS; + + /* + * Keep the existing ISA v3.0 PATS interpretation first. OpenBSD/powernv + * uses the PATSIZE value it writes to PTCR as one exponent smaller, and it + * only needs that interpretation for the bare metal LPID 0 table. + */ + if (ppc64_v3_get_pate_from_size(cpu, lpid, entry, 1ull << (pats + 12))) { + return true; + } + + if (lpid == 0) { + return ppc64_v3_get_pate_from_size(cpu, lpid, entry, + 1ull << (pats + 11)); + } + + return false; +} diff --git tests/functional/ppc64/meson.build tests/functional/ppc64/meson.build index f0f8ab8f61..cb3c745624 100644 --- tests/functional/ppc64/meson.build +++ tests/functional/ppc64/meson.build @@ -4,6 +4,7 @@ test_ppc64_timeouts = { 'fadump' : 480, 'hv' : 1000, 'mac99' : 120, + 'openbsd' : 240, 'powernv' : 480, 'pseries' : 480, 'replay' : 210, @@ -20,6 +21,7 @@ tests_ppc64_system_thorough = [ 'fadump', 'hv', 'mac99', + 'openbsd', 'powernv', 'pseries', 'replay', diff --git tests/functional/ppc64/test_openbsd.py tests/functional/ppc64/test_openbsd.py new file mode 100755 index 0000000000..bdbef6bf82 --- /dev/null +++ tests/functional/ppc64/test_openbsd.py @@ -0,0 +1,52 @@ +#!/usr/bin/env python3 +# +# Test that OpenBSD boots on a ppc powernv machine and reaches the installer. +# +# SPDX-License-Identifier: GPL-2.0-or-later + +from qemu_test import QemuSystemTest, Asset +from qemu_test import wait_for_console_pattern + + +class OpenBSDPowerNV(QemuSystemTest): + + ASSET_MINIROOT = Asset( + 'https://kirill.korins.ky/pub/qemu-powerpc64-openbsd/miniroot79.img', + '7829e42b75d81cafd732038b9d63228b79c1f5828d8375872a4bb655e1d6b13c') + + ASSET_BOOTKERNEL = Asset( + 'https://kirill.korins.ky/pub/qemu-powerpc64-openbsd/pnor.BOOTKERNEL', + '397ce43ce61910e1a2c4f13d301f957e61513a9ec5371bc3e87d3095411fae7b') + + def test_powernv9_openbsd_installer(self): + self.set_machine('powernv9') + self.require_accelerator('tcg') + + miniroot_path = self.ASSET_MINIROOT.fetch() + bootkernel_path = self.ASSET_BOOTKERNEL.fetch() + + self.vm.set_console() + self.vm.add_args('-cpu', 'power9', + '-accel', 'tcg,thread=single', + '-smp', '1,cores=1,threads=1', + '-m', '2g', + '-kernel', bootkernel_path, + '-device', + 'ich9-ahci,id=sata0,bus=pcie.0,addr=0x0', + '-drive', + f'file={miniroot_path},format=raw,if=none,' + 'id=bootdisk,snapshot=on', + '-device', + 'ide-hd,bus=sata0.0,unit=0,drive=bootdisk,' + 'bootindex=1') + self.vm.launch() + + wait_for_console_pattern(self, 'OpenBSD 7.9 (RAMDISK)', 'panic:') + wait_for_console_pattern( + self, + '(I)nstall, (U)pgrade, (A)utoinstall or (S)hell?', + 'panic:') + + +if __name__ == '__main__': + QemuSystemTest.main() -- 2.54.0
