[PATCH v2 0/7] ppc: pnv ChipTOD and various timebase fixes

Nicholas Piggin posted 7 patches 1 year ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20231124064001.198572-1-npiggin@gmail.com
Maintainers: Nicholas Piggin <npiggin@gmail.com>, Daniel Henrique Barboza <danielhb413@gmail.com>, "Cédric Le Goater" <clg@kaod.org>, "Frédéric Barrat" <fbarrat@linux.ibm.com>
There is a newer version of this series
include/hw/ppc/pnv.h         |   2 +
include/hw/ppc/pnv_chip.h    |   3 +
include/hw/ppc/pnv_chiptod.h |  54 ++++
include/hw/ppc/pnv_xscom.h   |   9 +
target/ppc/cpu.h             |  50 ++-
hw/ppc/pnv.c                 |  45 +++
hw/ppc/pnv_chiptod.c         | 611 +++++++++++++++++++++++++++++++++++
target/ppc/helper_regs.c     |  39 ++-
target/ppc/ppc-qmp-cmds.c    |   4 +
target/ppc/timebase_helper.c | 309 +++++++++++++++++-
target/ppc/translate.c       |  42 ++-
hw/ppc/meson.build           |   1 +
hw/ppc/trace-events          |   4 +
13 files changed, 1148 insertions(+), 25 deletions(-)
create mode 100644 include/hw/ppc/pnv_chiptod.h
create mode 100644 hw/ppc/pnv_chiptod.c
[PATCH v2 0/7] ppc: pnv ChipTOD and various timebase fixes
Posted by Nicholas Piggin 1 year ago
The chiptod/TFMR/state machine is not really tied to the other
time register fixes, but they touch some of the same code, and
logically same facility.

Changes since v1 of chiptod patches:
- Split hackish ChipTOD<->TFMR/TBST interface into its own patch
- Fix multi-socket addressing on P9 / chip ID mode (P10 works)
- Change chiptod primary/secondary setting to use class properties
- Add more comments to explain TOD overview and timebase state
  machine.
- SMT support for TFMR, some functionality is limited to thread 0.
- FIRMWARE_CONTROL_ERROR bit implemented in TFMR.
- Misc cleanups and bug fixes.

Since v1 of combined chiptod+timebase fixes series (most improvements
thanks to Cedric):
- Cut down unused ChipTOD FSM states, documented it, and added more
  state/transition checking.
- Only return running status if TOD FSM is RUNNING.
- Add reset function that sets TOD to ERROR state as per workbook.
- Put "TTYPE" broadcasting to other chip TODs into a class function.
- Put TOD-to-core TTYPE addressing into a class function.
- Move hacky addressing helpers private to pnv_chiptod.c
- Small changelog improvements.
- Minor changelog and comment typos and improvements, variable and
  function name improvements, format string fixes.

The hacky part, addressing core from chiptod, is still hacky. Is
there strong objection to it?

This successfully runs skiboot chiptod initialisation code with
POWER9 and POWER10 multi-socket, multi-core, SMT. That requires
skiboot 7.1 (not in-tree), otherwise chiptod init is skipped on
QEMU machines.

Thanks,
Nick

Nicholas Piggin (7):
  target/ppc: Rename TBL to TB on 64-bit
  target/ppc: Improve timebase register defines naming
  target/ppc: Fix move-to timebase SPR access permissions
  pnv/chiptod: Add POWER9/10 chiptod model
  pnv/chiptod: Implement the ChipTOD to Core transfer
  target/ppc: Implement core timebase state machine and TFMR
  target/ppc: Add SMT support to time facilities

 include/hw/ppc/pnv.h         |   2 +
 include/hw/ppc/pnv_chip.h    |   3 +
 include/hw/ppc/pnv_chiptod.h |  54 ++++
 include/hw/ppc/pnv_xscom.h   |   9 +
 target/ppc/cpu.h             |  50 ++-
 hw/ppc/pnv.c                 |  45 +++
 hw/ppc/pnv_chiptod.c         | 611 +++++++++++++++++++++++++++++++++++
 target/ppc/helper_regs.c     |  39 ++-
 target/ppc/ppc-qmp-cmds.c    |   4 +
 target/ppc/timebase_helper.c | 309 +++++++++++++++++-
 target/ppc/translate.c       |  42 ++-
 hw/ppc/meson.build           |   1 +
 hw/ppc/trace-events          |   4 +
 13 files changed, 1148 insertions(+), 25 deletions(-)
 create mode 100644 include/hw/ppc/pnv_chiptod.h
 create mode 100644 hw/ppc/pnv_chiptod.c

-- 
2.42.0