Messages in this thread | | | Date | Tue, 15 Aug 2023 15:26:47 -0300 | From | Arnaldo Carvalho de Melo <> | Subject | Re: [PATCH v6 10/10] perf: tests: Adapt mmap-basic.c for riscv |
| |
Em Mon, Aug 14, 2023 at 09:44:58AM -0700, Ian Rogers escreveu: > On Wed, Aug 2, 2023 at 1:14 AM Alexandre Ghiti <alexghiti@rivosinc.com> wrote: > > > > riscv now supports mmaping hardware counters to userspace so adapt the test > > to run on this architecture. > > > > Signed-off-by: Alexandre Ghiti <alexghiti@rivosinc.com> > > Reviewed-by: Andrew Jones <ajones@ventanamicro.com> > > Reviewed-by: Atish Patra <atishp@rivosinc.com> > > Reviewed-by: Ian Rogers <irogers@google.com>
Thanks, applied.
- Arnaldo
> Thanks, > Ian > > > --- > > tools/perf/tests/mmap-basic.c | 6 ++++-- > > 1 file changed, 4 insertions(+), 2 deletions(-) > > > > diff --git a/tools/perf/tests/mmap-basic.c b/tools/perf/tests/mmap-basic.c > > index e68ca6229756..886a13a77a16 100644 > > --- a/tools/perf/tests/mmap-basic.c > > +++ b/tools/perf/tests/mmap-basic.c > > @@ -284,7 +284,8 @@ static struct test_case tests__basic_mmap[] = { > > "permissions"), > > TEST_CASE_REASON("User space counter reading of instructions", > > mmap_user_read_instr, > > -#if defined(__i386__) || defined(__x86_64__) || defined(__aarch64__) > > +#if defined(__i386__) || defined(__x86_64__) || defined(__aarch64__) || \ > > + (defined(__riscv) && __riscv_xlen == 64) > > "permissions" > > #else > > "unsupported" > > @@ -292,7 +293,8 @@ static struct test_case tests__basic_mmap[] = { > > ), > > TEST_CASE_REASON("User space counter reading of cycles", > > mmap_user_read_cycles, > > -#if defined(__i386__) || defined(__x86_64__) || defined(__aarch64__) > > +#if defined(__i386__) || defined(__x86_64__) || defined(__aarch64__) || \ > > + (defined(__riscv) && __riscv_xlen == 64) > > "permissions" > > #else > > "unsupported" > > -- > > 2.39.2 > >
--
- Arnaldo
| |