[v3] Reorg ppc64 pmu insn counting

[PATCH v3 0/4] Reorg ppc64 pmu insn counting

Daniel Henrique Barboza posted 4 patches 2 years, 3 months ago

Diff against v1 v2
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20220103224746.167831-1-danielhb413@gmail.com

Maintainers: Greg Kurz <groug@kaod.org>, "Cédric Le Goater" <clg@kaod.org>, Daniel Henrique Barboza <danielhb413@gmail.com>, David Gibson <david@gibson.dropbear.id.au>

target/ppc/cpu.h         |   3 +
target/ppc/cpu_init.c    |   1 +
target/ppc/helper_regs.c |   2 +-
target/ppc/machine.c     |   2 +
target/ppc/power8-pmu.c  | 238 +++++++++++++++++----------------------
target/ppc/power8-pmu.h  |  14 +--
6 files changed, 117 insertions(+), 143 deletions(-)

Expand all Fold all

[PATCH v3 0/4] Reorg ppc64 pmu insn counting

Posted by Daniel Henrique Barboza 2 years, 3 months ago

Hi,

This new version implements Richard's suggestions made in the
v2 review.

Changes from v2:
- Patch 1:
  * fixed "PMC[1-5]" comment in target/ppc/cpu.h
- Former patch 4: squashed into patch 1
- Patch 4 (former 5):
  * use boolean variables instead of uint32_t
  * added Richard's r-b
- v2 link: https://lists.gnu.org/archive/html/qemu-devel/2022-01/msg00117.html


Daniel Henrique Barboza (1):
  target/ppc: do not call hreg_compute_hflags() in helper_store_mmcr0()

Richard Henderson (3):
  target/ppc: Cache per-pmc insn and cycle count settings
  target/ppc: Rewrite pmu_increment_insns
  target/ppc: Use env->pnc_cyc_cnt

 target/ppc/cpu.h         |   3 +
 target/ppc/cpu_init.c    |   1 +
 target/ppc/helper_regs.c |   2 +-
 target/ppc/machine.c     |   2 +
 target/ppc/power8-pmu.c  | 238 +++++++++++++++++----------------------
 target/ppc/power8-pmu.h  |  14 +--
 6 files changed, 117 insertions(+), 143 deletions(-)

-- 
2.33.1

Re: [PATCH v3 0/4] Reorg ppc64 pmu insn counting

Posted by Cédric Le Goater 2 years, 3 months ago

On 1/3/22 23:47, Daniel Henrique Barboza wrote:
> Hi,
> 
> This new version implements Richard's suggestions made in the
> v2 review.
> 
> Changes from v2:
> - Patch 1:
>    * fixed "PMC[1-5]" comment in target/ppc/cpu.h
> - Former patch 4: squashed into patch 1
> - Patch 4 (former 5):
>    * use boolean variables instead of uint32_t
>    * added Richard's r-b
> - v2 link: https://lists.gnu.org/archive/html/qemu-devel/2022-01/msg00117.html

Applied in ppc-next.

Thanks,

C.