Extends the existing watchpoint facility from TCG DAWR0 emulation to DAWR1 on
Power10 for powernv in the first patch, and for pseries in the second patch
with both TCG and KVM.
---
Changelog:
v7: https://lore.kernel.org/qemu-devel/170063834599.621665.9541440879278084501.stgit@ltcd48-lp2.aus.stglab.ibm.com/
v7->v8:
- Fixed the missed out ppc_store_dawr* calls.
- Removed the macros and split the patch into 2 one just enabling the
facility for powernv and the next one doing the same for pseries guest.
- The macro removal barely increased the number of lines by 12 as against
the previous version.
v6: https://lore.kernel.org/qemu-devel/168871963321.58984.15628382614621248470.stgit@ltcd89-lp2/
v6->v7:
- Sorry about the delay in sending out this version, I have dropped the
Reviewed-bys as suggested and converted the patch to RFC back again.
- Added the TCG support. Basically, converted the existing DAWR0 support
routines into macros for reuse by the DAWR1. Let me know if the macro
conversions should be moved to a separate independent patch.
- As the dawr1 works on TCG, the checks in cap_dawr1_apply() report a warning
now only for P9 or P9 compat modes for both KVM and TCG use cases.
- 'make test' passes for caps checks. Also, as suggested by Greg Kurz, the
'make test' after making the DAWR1 default 'on' and updating defaut cpu
to Power10, shows no failures.
v5: https://lore.kernel.org/all/20210412114433.129702-1-ravi.bangoria@linux.ibm.com/
v5->v6:
- The other patches in the original series already merged.
- Rebased to the top of the tree. So, the gen_spr_book3s_310_dbg() is renamed
to register_book3s_310_dbg_sprs() and moved to cpu_init.c accordingly.
- No functional changes.
v4: https://lore.kernel.org/r/20210406053833.282907-1-ravi.bangoria@linux.ibm.com
v3->v4:
- Make error message more proper.
v3: https://lore.kernel.org/r/20210330095350.36309-1-ravi.bangoria@linux.ibm.com
v3->v4:
- spapr_dt_pa_features(): POWER10 processor is compatible with 3.0
(PCR_COMPAT_3_00). No need to ppc_check_compat(3_10) for now as
ppc_check_compati(3_00) will also be true. ppc_check_compat(3_10)
can be added while introducing pa_features_310 in future.
- Use error_append_hint() for hints. Also add ERRP_GUARD().
- Add kvmppc_set_cap_dawr1() stub function for CONFIG_KVM=n.
v2: https://lore.kernel.org/r/20210329041906.213991-1-ravi.bangoria@linux.ibm.com
v2->v3:
- Don't introduce pa_features_310[], instead, reuse pa_features_300[]
for 3.1 guests, as there is no difference between initial values of
them atm.
- Call gen_spr_book3s_310_dbg() from init_proc_POWER10() instead of
init_proc_POWER8(). Also, Don't call gen_spr_book3s_207_dbg() from
gen_spr_book3s_310_dbg() as init_proc_POWER10() already calls it.
v1: https://lore.kernel.org/r/20200723104220.314671-1-ravi.bangoria@linux.ibm.com
v1->v2:
- Introduce machine capability cap-dawr1 to enable/disable
the feature. By default, 2nd DAWR is OFF for guests even
when host kvm supports it. User has to manually enable it
with -machine cap-dawr1=on if he wishes to use it.
- Split the header file changes into separate patch. (Sync
headers from v5.12-rc3)
Shivaprasad G Bhat (2):
ppc: Enable 2nd DAWR support on Power10 PowerNV machine
ppc: spapr: Enable 2nd DAWR on Power10 pSeries machine
hw/ppc/spapr.c | 7 ++++-
hw/ppc/spapr_caps.c | 36 ++++++++++++++++++++++++
hw/ppc/spapr_hcall.c | 25 ++++++++++------
include/hw/ppc/spapr.h | 6 +++-
target/ppc/cpu.c | 45 ++++++++++++++++++++---------
target/ppc/cpu.h | 8 ++++--
target/ppc/cpu_init.c | 15 ++++++++++
target/ppc/excp_helper.c | 61 ++++++++++++++++++++++------------------
target/ppc/helper.h | 2 ++
target/ppc/kvm.c | 12 ++++++++
target/ppc/kvm_ppc.h | 12 ++++++++
target/ppc/machine.c | 3 +-
target/ppc/misc_helper.c | 10 +++++++
target/ppc/spr_common.h | 2 ++
target/ppc/translate.c | 12 ++++++++
15 files changed, 202 insertions(+), 54 deletions(-)
--
Signature