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

2022-01-04 Thread Daniel Henrique Barboza
On 1/4/22 07:32, Alex Bennée wrote: Daniel Henrique Barboza writes: On 1/3/22 12:07, Alex Bennée wrote: Daniel Henrique Barboza writes: On 12/23/21 00:01, Richard Henderson wrote: In contrast to Daniel's version, the code stays in power8-pmu.c, but is better organized to not take so

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

2022-01-04 Thread Alex Bennée
Daniel Henrique Barboza writes: > On 1/3/22 12:07, Alex Bennée wrote: >> Daniel Henrique Barboza writes: >> >>> On 12/23/21 00:01, Richard Henderson wrote: In contrast to Daniel's version, the code stays in power8-pmu.c, but is better organized to not take so much overhead.

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

2022-01-03 Thread Daniel Henrique Barboza
On 1/3/22 12:07, Alex Bennée wrote: Daniel Henrique Barboza writes: On 12/23/21 00:01, Richard Henderson wrote: In contrast to Daniel's version, the code stays in power8-pmu.c, but is better organized to not take so much overhead. Before: 32.97% qemu-system-ppc qemu-system-ppc64

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

2022-01-03 Thread Alex Bennée
Daniel Henrique Barboza writes: > On 12/23/21 00:01, Richard Henderson wrote: >> In contrast to Daniel's version, the code stays in power8-pmu.c, >> but is better organized to not take so much overhead. >> Before: >> 32.97% qemu-system-ppc qemu-system-ppc64 [.] pmc_get_event >>

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

2022-01-03 Thread Daniel Henrique Barboza
On 1/3/22 03:48, Cédric Le Goater wrote: On 12/30/21 23:12, Daniel Henrique Barboza wrote: On 12/23/21 18:19, Richard Henderson wrote: On 12/23/21 12:36 PM, Daniel Henrique Barboza wrote: This reorg is breaking PMU-EBB tests, unfortunately. These tests are run from the kernel tree [1]

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

2022-01-02 Thread Cédric Le Goater
On 12/30/21 23:12, Daniel Henrique Barboza wrote: On 12/23/21 18:19, Richard Henderson wrote: On 12/23/21 12:36 PM, Daniel Henrique Barboza wrote: This reorg is breaking PMU-EBB tests, unfortunately. These tests are run from the kernel tree [1] and I test them inside a pSeries TCG guest.

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

2021-12-30 Thread Daniel Henrique Barboza
On 12/23/21 18:19, Richard Henderson wrote: On 12/23/21 12:36 PM, Daniel Henrique Barboza wrote: This reorg is breaking PMU-EBB tests, unfortunately. These tests are run from the kernel tree [1] and I test them inside a pSeries TCG guest. You'll need to apply patches 9 and 10 of [2]

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

2021-12-23 Thread Daniel Henrique Barboza
On 12/23/21 18:19, Richard Henderson wrote: On 12/23/21 12:36 PM, Daniel Henrique Barboza wrote: This reorg is breaking PMU-EBB tests, unfortunately. These tests are run from the kernel tree [1] and I test them inside a pSeries TCG guest. You'll need to apply patches 9 and 10 of [2]

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

2021-12-23 Thread Richard Henderson
On 12/23/21 12:36 PM, Daniel Henrique Barboza wrote: This reorg is breaking PMU-EBB tests, unfortunately. These tests are run from the kernel tree [1] and I test them inside a pSeries TCG guest. You'll need to apply patches 9 and 10 of [2] beforehand (they apply cleanly in current master)

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

2021-12-23 Thread Daniel Henrique Barboza
On 12/23/21 00:01, Richard Henderson wrote: In contrast to Daniel's version, the code stays in power8-pmu.c, but is better organized to not take so much overhead. Before: 32.97% qemu-system-ppc qemu-system-ppc64 [.] pmc_get_event 20.22% qemu-system-ppc qemu-system-ppc64

[PATCH 0/3] Reorg ppc64 pmu insn counting

2021-12-22 Thread Richard Henderson
In contrast to Daniel's version, the code stays in power8-pmu.c, but is better organized to not take so much overhead. Before: 32.97% qemu-system-ppc qemu-system-ppc64 [.] pmc_get_event 20.22% qemu-system-ppc qemu-system-ppc64 [.] helper_insns_inc 4.52% qemu-system-ppc