Hello,
On 12/10/24 04:04, Nicholas Piggin wrote:
> This series makes a bunch of fixes and improvements to the HOMER and
> OCC unit models for powernv. It gets OPAL OCC code happier again,
Nice. I had similar changes exposing pstates and sensors for QEMU.
I am glad this is now possible.
> but requires this series of skiboot fixes (which I will hope to get
> upstream and then into QEMU in the same release as this series is
> merged.
>
> https://lists.ozlabs.org/pipermail/skiboot/2024-November/019013.html
>
> Big changes is moving HOMER from mmio register implementation to a
> RAM memory region which better maches hardware and is much easier to
> work with; and the addition of some dynamic OCC behaviour to the
> device model.
>
> I wouldn't expect any review of OCC details. It's neglected and this
> series gets it in better shape than before. But anything on the
> basic structure and QEMU details is welcome. I will add some test
> cases to help ensure it doesn't regress in future, I will have to
> juggle the skiboot fixes as well though, so will do that later.
Have you considered modeling the OCC in QEMU ? I am asking because
I have a series removing all 405 support in QEMU 10.0.
Thanks,
C.
>
> Thanks,
> Nick
>
> Nicholas Piggin (9):
> ppc/pnv/homer: Fix OCC registers
> ppc/pnv/homer: Make dummy reads return 0
> ppc/pnv/occ: Fix common area sensor offsets
> ppc/pnv/homer: class-based base and size
> ppc/pnv/occ: Better document OCCMISC bits
> ppc/pnv: Make HOMER memory a RAM region
> ppc/pnv/occ: Update pstate frequency tables
> ppc/pnv/occ: Add POWER10 OCC-OPAL data format
> ppc/pnv/occ: Implement a basic dynamic OCC model
>
> include/hw/ppc/pnv.h | 6 +-
> include/hw/ppc/pnv_homer.h | 12 +-
> include/hw/ppc/pnv_occ.h | 9 +
> hw/ppc/pnv.c | 87 +++--
> hw/ppc/pnv_homer.c | 230 +++----------
> hw/ppc/pnv_occ.c | 668 +++++++++++++++++++++++++++++++++++--
> 6 files changed, 742 insertions(+), 270 deletions(-)
>