is check.
To address this issue in "perf report", exploring an option of
having stubs range as part of the /proc/kallsyms, so that perf
report can resolve addresses in stubs range
However this patch uses text_end to skip the stub range for
Object code reading testcase.
Reported-by: Dish
7: Simple expression parser:
--- start ---
test child forked, pid 87922
Using CPUID 0x00800200
division by zero
syntax error
test child finished with 0
end
Simple expression parser: Ok
Tested-by: Disha Goel
---
tools/perf/arch/powerpc/util/header.c
r fail for the
testcase.
Signed-off-by: Athira Rajeev
With this patch applied(on power) perf metricgroups test works correctly when
perf_event access is restricted.
# ./perf test "perf all metricgroups test"
96: perf all metricgroups test : Ok
T
ch changes:
[command]# ./perf bench breakpoint thread
Skipping perf bench breakpoint thread: No hardware support
Reported-by: Disha Goel
Signed-off-by: Kajol Jain
With this patch applied perf bench breakpoint test works correctly on Power9
and Power10.
Result on Power9 system with patch chan
core=1/ event is not supported by the kernel.
failed to read counter hv_24x7/CPM_ADJUNCT_INST,domain=0,core=1/
Performance counter stats for 'system wide':
hv_24x7/CPM_ADJUNCT_INST,domain=0,core=1/
1.001352771 seconds time elapsed
Tested-by: Disha Goel
---
arch/pow
Link to linux-ci job result:
https://github.com/disgoel/linux-ci/actions
Disha Goel (3):
powerpc/asm-offset: Remove unused KVM_TLB_SETS macros
powerpc/asm-offset: Remove unused KVM_RADIX macros
powerpc/kvm: Remove unused macros from asm-offset
Kajol Jain (1):
powerpc/kvm: Remove unused
this is un-used.
Patch fixes by removing this.
Fixes: 2e1ae9cd56f8 ("KVM: PPC: Book3S HV: Implement radix prefetch workaround
by disabling MMU")
Signed-off-by: Disha Goel
---
arch/powerpc/kernel/asm-offsets.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/powerpc/kernel/a
it.
Fixes: 9769a7fd79b6 ("KVM: PPC: Book3S HV: Remove radix guest support from P7/8
path")
Signed-off-by: Disha Goel
---
arch/powerpc/kernel/asm-offsets.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/powerpc/kernel/asm-offsets.c
b/arch/powerpc/kernel/asm-offsets.c
index d5d9c1dcef1c.
re un-used. Patch fixes by removing those.
Fixes: fae5c9f3664b ("KVM: PPC: Book3S HV: remove ISA v3.0 and v3.1 support
from P7/8 path")
Signed-off-by: Disha Goel
---
arch/powerpc/kernel/asm-offsets.c | 20
1 file changed, 20 deletions(-)
diff --git a/arch/powerpc/ker
From: Kajol Jain
Commit 57dc0eed73ca ("KVM: PPC: Book3S HV P9: Implement PMU save/restore
in C") removed the PMU save/restore functions from assembly code and
implemented these functions in C, for power9 and later platforms.
After the code refactoring, Performance Monitoring Unit (PMU) registers
On 9/13/22 1:45 PM, Christophe Leroy wrote:
Le 13/09/2022 à 09:40, Disha Goel a écrit :
There were commits which did code refactoring and converting some of kvm
assembly routines to C. When doing it, many of the asm-offset macro
definitions were missed to remove. Patchset here removes those
xppc-dev/patch/20220913074025.132160-2-disg...@linux.vnet.ibm.com/
Link to the script used to get unused macro:
https://github.com/maddy-kerneldev/scripts/blob/master/check_asm-offset.sh
Link to linux-ci job result:
https://github.com/disgoel/linux-ci/actions?query=branch%3Akvmmacro.v2
Disha Go
and v3.1 support
from P7/8 path")
Commit 57dc0eed73ca ("KVM: PPC: Book3S HV P9: Implement PMU save/restore in C")
Many of the asm-offset macro definitions were missed to remove. Patch
fixes by removing the unused macros.
Signed-off-by: Disha Goel
---
arch/powerpc/kernel/asm-off
From: Kajol Jain"
Commit 57dc0eed73ca ("KVM: PPC: Book3S HV P9: Implement PMU save/restore
in C") removed the PMU save/restore functions from assembly code and
implemented these functions in C, for power9 and later platforms.
After the code refactoring, Performance Monitoring Unit (PMU) register
e above
error, to skip the test in these scenarios too.
Reported-by: Disha Goel
Signed-off-by: Athira Rajeev
Tested-by: Disha Goel
---
changelog:
v1 -> v2:
Corrected formatting of spaces in error log.
With spaces in v1 of the patch, the egrep search was
considering spaces also
instructions
CPU-1 59,418 instructions
CPU-1 70,478 instructions
CPU-1 65,201 instructions
CPU-1 63,704 instructions
<<>>
This is caught while running "perf test" for
"stat+json_
n't support sampling/overflow-interrupts.
Try 'perf stat'
Reported-by: Disha Goel
Signed-off-by: Athira Rajeev
Reviewed-by: Madhavan Srinivasan
Tested-by: Disha Goel
---
arch/powerpc/perf/core-book3s.c | 15 +++
1 file changed, 15 insertions(+)
diff --git a/a
56dbc9 ("perf test: Add checking for perf stat CSV output.")
Reported-by: Disha Goel
Signed-off-by: Athira Rajeev
Suggested-by: James Clark
Suggested-by: Ian Rogers
For the patchset,
Tested-by: Disha Goel
---
tools/perf/tests/shell/stat+csv_output.sh | 43
s correctly without
any segfault now.
# ./perf list
List of pre-defined events (to be used in -e or -M):
branch-instructions OR branches[Hardware event]
branch-misses [Hardware event]
Tested-by: Dis
r/lib64/glibc-hwcaps/power10/libc.so.6)
11ef03534 [unknown] (/usr/bin/ping)
test child finished with 0
end
probe libc's inet_pton & backtrace it with ping: Ok
Signed-off-by: Likhitha Korrapati
Reported-by: Disha Goel
Thanks for the fix patch.
I have tested on a Power10 machine, &
ests/shell/buildid.sh: Bad substitution
test child finished with -2
end
After applying the patch in same environment, error is not seen and test runs
fine.
Tested the same patch in bash as well.
# readlink -f $(which sh)
/usr/bin/bash
The test works fine with the changes.
Tested-by: Disha Goe
t
-a option.
Fix this issue by adding system wide check for perf bench workload.
Result with the patch changes in powerpc:
92: perf all metrics test : Ok
Signed-off-by: Kajol Jain
Tested the patch on powerpc machine, perf metrics test works fine.
91: perf all metrics test
: Skip
(not compiled in or missing libtraceevent support)
Tested-by: Disha Goel
---
tools/perf/tests/bpf.c | 22 +++---
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/tools/perf/tests/bpf.c b/tools/perf/tests/bpf.c
index 17c023823713..4af3
,backend cycles
idle
S0,1,88766,,instructions,2030942,100.00,0.35,insn per cycle
S0,1,,,1.27,stalled cycles per insn
Fixes: 92a61f6412d3 ("perf stat: Implement CSV metrics output")
Reported-by: Disha Goel
Signed-off-by: Athira Rajeev
perf stat csv test passed after appl
t; 1K and other configurations as
well, verified the perf benchmark numa, futex and epoll for whole
series with the patch set.Tested-by: Disha Goel <
disg...@linux.vnet.ibm.com>
tools/perf/bench/epoll-ctl.c | 25 --
tools/perf/bench/epoll-wait.c | 25 --
t
fs file and fixing the compilation issues observed in
debian
Tesed the patches on powerpc with CPU > 1K and other configurations as
well, verified the perf bench numa with the patch set.Tested-by: Disha
Goel
tools/perf/bench/numa.c | 136 +--
tools/perf/util/
rf test session
topology test with the patch set.Tested-by: Disha Goel <
disg...@linux.vnet.ibm.com>
tools/perf/tests/topology.c | 17
tools/perf/util/header.c| 54 +
tools/perf/util/header.h| 1 + 3 files changed, 72 insertions(+)
Kajol. Use "strim"
to remove space from string. Also use "feof" to check for EOF instead
of using new variable "ret".
Tested the patches on powervm and powernv, verified perf test session
topology test with the patch set.Tested-by: Disha Goel <
disg...@linux.vnet.
;>
Fixes: ba1fae431e74 ("perf test: Add 'perf test BPF'")Signed-off-by:
Athira Rajeev ---Changelog: Addressed
review comments from Arnaldo by adding reason for skipping of testcase.
tools/perf/tests/bpf.c | 10 ++ 1 file changed, 6
insertions(+), 4 deletions(-)
Tested the pat
d, 11
insertions(+)
Tested the patch on powerpc and powernv, session topology test works
fine with the patch.Tested-by: Disha Goel
diff --git a/tools/perf/tests/topology.c
b/tools/perf/tests/topology.cindex ee1e3dcbc0bd..d23a9e322ff5 100644---
a/tools/perf/tests/topology.c+++ b/tools/perf/
pus and not the number
of online cpus.
Signed-off-by: Athira Rajeev
Thanks for the fix patches, Athira.
I have tested all three patches on a power machine (both small and max config),
and the perf bench futex and epoll tests run fine.
For the series:
Tested-by: Disha Goel
---
tools/p
On 23/07/24 10:51 am, Kajol Jain wrote:
Update JSON/events for power10 platform with additional events.
Also move PM_VECTOR_LD_CMPL event from others.json to
frontend.json file.
Signed-off-by: Kajol Jain
I have tested the patch on power10 machine. Looks good to me.
Tested-by: Disha Goel
e patch on PowerPC by booting in different compat modes and
at base also, and it's working as expected.
For the series:
Tested-by: Disha Goel
---
.../arch/powerpc/compat/generic-events.json | 117 ++
.../perf/pmu-events/arch/powerpc/mapfile.csv | 1 +
2 files changed,
:
Tested-by: Disha Goel
---
Changelog:
v1 -> v2
No code changes. Rebased to latest upstream
.../pmu/event_code_tests/event_alternatives_tests_p10.c| 3 ++-
.../pmu/event_code_tests/generic_events_valid_test.c | 3 ++-
.../reserved_bits_mmcra_sample_elig_mode_test.c
34 matches
Mail list logo
