* [PATCH v2 0/1] perf test: Fix attr tests for PERF_FORMAT_LOST
@ 2022-10-12 9:46 James Clark
2022-10-12 9:46 ` [PATCH v2 1/1] " James Clark
0 siblings, 1 reply; 4+ messages in thread
From: James Clark @ 2022-10-12 9:46 UTC (permalink / raw)
To: linux-perf-users, acme, namhyung
Cc: linux-kernel, James Clark, Peter Zijlstra, Ingo Molnar,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Adrian Hunter
Changes since v1:
* Add rather than replace the new value so that the test still passes
on older kernels
James Clark (1):
perf test: Fix attr tests for PERF_FORMAT_LOST
tools/perf/tests/attr/base-record | 2 +-
tools/perf/tests/attr/system-wide-dummy | 2 +-
tools/perf/tests/attr/test-record-group | 4 ++--
tools/perf/tests/attr/test-record-group-sampling | 6 +++---
tools/perf/tests/attr/test-record-group1 | 4 ++--
tools/perf/tests/attr/test-record-group2 | 4 ++--
6 files changed, 11 insertions(+), 11 deletions(-)
--
2.28.0
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH v2 1/1] perf test: Fix attr tests for PERF_FORMAT_LOST
2022-10-12 9:46 [PATCH v2 0/1] perf test: Fix attr tests for PERF_FORMAT_LOST James Clark
@ 2022-10-12 9:46 ` James Clark
2022-10-12 16:06 ` Namhyung Kim
0 siblings, 1 reply; 4+ messages in thread
From: James Clark @ 2022-10-12 9:46 UTC (permalink / raw)
To: linux-perf-users, acme, namhyung
Cc: linux-kernel, James Clark, Peter Zijlstra, Ingo Molnar,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Adrian Hunter
Since PERF_FORMAT_LOST was added, the default read format has that bit
set, so add it to the tests. Keep the old value as well so that the test
still passes on older kernels.
This fixes the following failure:
expected read_format=0|4, got 20
FAILED './tests/attr/test-record-C0' - match failure
Fixes: 85b425f31c88 ("perf record: Set PERF_FORMAT_LOST by default")
Cc: Namhyung Kim <namhyung@kernel.org>
Signed-off-by: James Clark <james.clark@arm.com>
---
tools/perf/tests/attr/base-record | 2 +-
tools/perf/tests/attr/system-wide-dummy | 2 +-
tools/perf/tests/attr/test-record-group | 4 ++--
tools/perf/tests/attr/test-record-group-sampling | 6 +++---
tools/perf/tests/attr/test-record-group1 | 4 ++--
tools/perf/tests/attr/test-record-group2 | 4 ++--
6 files changed, 11 insertions(+), 11 deletions(-)
diff --git a/tools/perf/tests/attr/base-record b/tools/perf/tests/attr/base-record
index 8c10955eff93..3ef07a12aa14 100644
--- a/tools/perf/tests/attr/base-record
+++ b/tools/perf/tests/attr/base-record
@@ -9,7 +9,7 @@ size=128
config=0
sample_period=*
sample_type=263
-read_format=0|4
+read_format=0|4|20
disabled=1
inherit=1
pinned=0
diff --git a/tools/perf/tests/attr/system-wide-dummy b/tools/perf/tests/attr/system-wide-dummy
index 86a15dd359d9..8fec06eda5f9 100644
--- a/tools/perf/tests/attr/system-wide-dummy
+++ b/tools/perf/tests/attr/system-wide-dummy
@@ -11,7 +11,7 @@ size=128
config=9
sample_period=4000
sample_type=455
-read_format=4
+read_format=4|20
# Event will be enabled right away.
disabled=0
inherit=1
diff --git a/tools/perf/tests/attr/test-record-group b/tools/perf/tests/attr/test-record-group
index 14ee60fd3f41..6c1cff8aae8b 100644
--- a/tools/perf/tests/attr/test-record-group
+++ b/tools/perf/tests/attr/test-record-group
@@ -7,14 +7,14 @@ ret = 1
fd=1
group_fd=-1
sample_type=327
-read_format=4
+read_format=4|20
[event-2:base-record]
fd=2
group_fd=1
config=1
sample_type=327
-read_format=4
+read_format=4|20
mmap=0
comm=0
task=0
diff --git a/tools/perf/tests/attr/test-record-group-sampling b/tools/perf/tests/attr/test-record-group-sampling
index 300b9f7e6d69..97e7e64a38f0 100644
--- a/tools/perf/tests/attr/test-record-group-sampling
+++ b/tools/perf/tests/attr/test-record-group-sampling
@@ -7,7 +7,7 @@ ret = 1
fd=1
group_fd=-1
sample_type=343
-read_format=12
+read_format=12|28
inherit=0
[event-2:base-record]
@@ -21,8 +21,8 @@ config=3
# default | PERF_SAMPLE_READ
sample_type=343
-# PERF_FORMAT_ID | PERF_FORMAT_GROUP
-read_format=12
+# PERF_FORMAT_ID | PERF_FORMAT_GROUP | PERF_FORMAT_LOST
+read_format=12|28
task=0
mmap=0
comm=0
diff --git a/tools/perf/tests/attr/test-record-group1 b/tools/perf/tests/attr/test-record-group1
index 3ffe246e0228..eeb1db392bc9 100644
--- a/tools/perf/tests/attr/test-record-group1
+++ b/tools/perf/tests/attr/test-record-group1
@@ -7,7 +7,7 @@ ret = 1
fd=1
group_fd=-1
sample_type=327
-read_format=4
+read_format=4|20
[event-2:base-record]
fd=2
@@ -15,7 +15,7 @@ group_fd=1
type=0
config=1
sample_type=327
-read_format=4
+read_format=4|20
mmap=0
comm=0
task=0
diff --git a/tools/perf/tests/attr/test-record-group2 b/tools/perf/tests/attr/test-record-group2
index 6b9f8d182ce1..cebdaa8e64e4 100644
--- a/tools/perf/tests/attr/test-record-group2
+++ b/tools/perf/tests/attr/test-record-group2
@@ -9,7 +9,7 @@ group_fd=-1
config=0|1
sample_period=1234000
sample_type=87
-read_format=12
+read_format=12|28
inherit=0
freq=0
@@ -19,7 +19,7 @@ group_fd=1
config=0|1
sample_period=6789000
sample_type=87
-read_format=12
+read_format=12|28
disabled=0
inherit=0
mmap=0
--
2.28.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/1] perf test: Fix attr tests for PERF_FORMAT_LOST
2022-10-12 9:46 ` [PATCH v2 1/1] " James Clark
@ 2022-10-12 16:06 ` Namhyung Kim
2022-10-14 20:07 ` Arnaldo Carvalho de Melo
0 siblings, 1 reply; 4+ messages in thread
From: Namhyung Kim @ 2022-10-12 16:06 UTC (permalink / raw)
To: James Clark
Cc: linux-perf-users, Arnaldo Carvalho de Melo, linux-kernel,
Peter Zijlstra, Ingo Molnar, Mark Rutland, Alexander Shishkin,
Jiri Olsa, Adrian Hunter
Hello,
On Wed, Oct 12, 2022 at 2:47 AM James Clark <james.clark@arm.com> wrote:
>
> Since PERF_FORMAT_LOST was added, the default read format has that bit
> set, so add it to the tests. Keep the old value as well so that the test
> still passes on older kernels.
>
> This fixes the following failure:
>
> expected read_format=0|4, got 20
> FAILED './tests/attr/test-record-C0' - match failure
>
> Fixes: 85b425f31c88 ("perf record: Set PERF_FORMAT_LOST by default")
> Signed-off-by: James Clark <james.clark@arm.com>
Acked-by: Namhyung Kim <namhyung@kernel.org>
Thanks,
Namhyung
> ---
> tools/perf/tests/attr/base-record | 2 +-
> tools/perf/tests/attr/system-wide-dummy | 2 +-
> tools/perf/tests/attr/test-record-group | 4 ++--
> tools/perf/tests/attr/test-record-group-sampling | 6 +++---
> tools/perf/tests/attr/test-record-group1 | 4 ++--
> tools/perf/tests/attr/test-record-group2 | 4 ++--
> 6 files changed, 11 insertions(+), 11 deletions(-)
>
> diff --git a/tools/perf/tests/attr/base-record b/tools/perf/tests/attr/base-record
> index 8c10955eff93..3ef07a12aa14 100644
> --- a/tools/perf/tests/attr/base-record
> +++ b/tools/perf/tests/attr/base-record
> @@ -9,7 +9,7 @@ size=128
> config=0
> sample_period=*
> sample_type=263
> -read_format=0|4
> +read_format=0|4|20
> disabled=1
> inherit=1
> pinned=0
> diff --git a/tools/perf/tests/attr/system-wide-dummy b/tools/perf/tests/attr/system-wide-dummy
> index 86a15dd359d9..8fec06eda5f9 100644
> --- a/tools/perf/tests/attr/system-wide-dummy
> +++ b/tools/perf/tests/attr/system-wide-dummy
> @@ -11,7 +11,7 @@ size=128
> config=9
> sample_period=4000
> sample_type=455
> -read_format=4
> +read_format=4|20
> # Event will be enabled right away.
> disabled=0
> inherit=1
> diff --git a/tools/perf/tests/attr/test-record-group b/tools/perf/tests/attr/test-record-group
> index 14ee60fd3f41..6c1cff8aae8b 100644
> --- a/tools/perf/tests/attr/test-record-group
> +++ b/tools/perf/tests/attr/test-record-group
> @@ -7,14 +7,14 @@ ret = 1
> fd=1
> group_fd=-1
> sample_type=327
> -read_format=4
> +read_format=4|20
>
> [event-2:base-record]
> fd=2
> group_fd=1
> config=1
> sample_type=327
> -read_format=4
> +read_format=4|20
> mmap=0
> comm=0
> task=0
> diff --git a/tools/perf/tests/attr/test-record-group-sampling b/tools/perf/tests/attr/test-record-group-sampling
> index 300b9f7e6d69..97e7e64a38f0 100644
> --- a/tools/perf/tests/attr/test-record-group-sampling
> +++ b/tools/perf/tests/attr/test-record-group-sampling
> @@ -7,7 +7,7 @@ ret = 1
> fd=1
> group_fd=-1
> sample_type=343
> -read_format=12
> +read_format=12|28
> inherit=0
>
> [event-2:base-record]
> @@ -21,8 +21,8 @@ config=3
> # default | PERF_SAMPLE_READ
> sample_type=343
>
> -# PERF_FORMAT_ID | PERF_FORMAT_GROUP
> -read_format=12
> +# PERF_FORMAT_ID | PERF_FORMAT_GROUP | PERF_FORMAT_LOST
> +read_format=12|28
> task=0
> mmap=0
> comm=0
> diff --git a/tools/perf/tests/attr/test-record-group1 b/tools/perf/tests/attr/test-record-group1
> index 3ffe246e0228..eeb1db392bc9 100644
> --- a/tools/perf/tests/attr/test-record-group1
> +++ b/tools/perf/tests/attr/test-record-group1
> @@ -7,7 +7,7 @@ ret = 1
> fd=1
> group_fd=-1
> sample_type=327
> -read_format=4
> +read_format=4|20
>
> [event-2:base-record]
> fd=2
> @@ -15,7 +15,7 @@ group_fd=1
> type=0
> config=1
> sample_type=327
> -read_format=4
> +read_format=4|20
> mmap=0
> comm=0
> task=0
> diff --git a/tools/perf/tests/attr/test-record-group2 b/tools/perf/tests/attr/test-record-group2
> index 6b9f8d182ce1..cebdaa8e64e4 100644
> --- a/tools/perf/tests/attr/test-record-group2
> +++ b/tools/perf/tests/attr/test-record-group2
> @@ -9,7 +9,7 @@ group_fd=-1
> config=0|1
> sample_period=1234000
> sample_type=87
> -read_format=12
> +read_format=12|28
> inherit=0
> freq=0
>
> @@ -19,7 +19,7 @@ group_fd=1
> config=0|1
> sample_period=6789000
> sample_type=87
> -read_format=12
> +read_format=12|28
> disabled=0
> inherit=0
> mmap=0
> --
> 2.28.0
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v2 1/1] perf test: Fix attr tests for PERF_FORMAT_LOST
2022-10-12 16:06 ` Namhyung Kim
@ 2022-10-14 20:07 ` Arnaldo Carvalho de Melo
0 siblings, 0 replies; 4+ messages in thread
From: Arnaldo Carvalho de Melo @ 2022-10-14 20:07 UTC (permalink / raw)
To: Namhyung Kim
Cc: James Clark, linux-perf-users, linux-kernel, Peter Zijlstra,
Ingo Molnar, Mark Rutland, Alexander Shishkin, Jiri Olsa,
Adrian Hunter
Em Wed, Oct 12, 2022 at 09:06:27AM -0700, Namhyung Kim escreveu:
> Hello,
>
> On Wed, Oct 12, 2022 at 2:47 AM James Clark <james.clark@arm.com> wrote:
> >
> > Since PERF_FORMAT_LOST was added, the default read format has that bit
> > set, so add it to the tests. Keep the old value as well so that the test
> > still passes on older kernels.
> >
> > This fixes the following failure:
> >
> > expected read_format=0|4, got 20
> > FAILED './tests/attr/test-record-C0' - match failure
> >
> > Fixes: 85b425f31c88 ("perf record: Set PERF_FORMAT_LOST by default")
> > Signed-off-by: James Clark <james.clark@arm.com>
>
> Acked-by: Namhyung Kim <namhyung@kernel.org>
Thanks, applied.
- Arnaldo
> Thanks,
> Namhyung
>
>
> > ---
> > tools/perf/tests/attr/base-record | 2 +-
> > tools/perf/tests/attr/system-wide-dummy | 2 +-
> > tools/perf/tests/attr/test-record-group | 4 ++--
> > tools/perf/tests/attr/test-record-group-sampling | 6 +++---
> > tools/perf/tests/attr/test-record-group1 | 4 ++--
> > tools/perf/tests/attr/test-record-group2 | 4 ++--
> > 6 files changed, 11 insertions(+), 11 deletions(-)
> >
> > diff --git a/tools/perf/tests/attr/base-record b/tools/perf/tests/attr/base-record
> > index 8c10955eff93..3ef07a12aa14 100644
> > --- a/tools/perf/tests/attr/base-record
> > +++ b/tools/perf/tests/attr/base-record
> > @@ -9,7 +9,7 @@ size=128
> > config=0
> > sample_period=*
> > sample_type=263
> > -read_format=0|4
> > +read_format=0|4|20
> > disabled=1
> > inherit=1
> > pinned=0
> > diff --git a/tools/perf/tests/attr/system-wide-dummy b/tools/perf/tests/attr/system-wide-dummy
> > index 86a15dd359d9..8fec06eda5f9 100644
> > --- a/tools/perf/tests/attr/system-wide-dummy
> > +++ b/tools/perf/tests/attr/system-wide-dummy
> > @@ -11,7 +11,7 @@ size=128
> > config=9
> > sample_period=4000
> > sample_type=455
> > -read_format=4
> > +read_format=4|20
> > # Event will be enabled right away.
> > disabled=0
> > inherit=1
> > diff --git a/tools/perf/tests/attr/test-record-group b/tools/perf/tests/attr/test-record-group
> > index 14ee60fd3f41..6c1cff8aae8b 100644
> > --- a/tools/perf/tests/attr/test-record-group
> > +++ b/tools/perf/tests/attr/test-record-group
> > @@ -7,14 +7,14 @@ ret = 1
> > fd=1
> > group_fd=-1
> > sample_type=327
> > -read_format=4
> > +read_format=4|20
> >
> > [event-2:base-record]
> > fd=2
> > group_fd=1
> > config=1
> > sample_type=327
> > -read_format=4
> > +read_format=4|20
> > mmap=0
> > comm=0
> > task=0
> > diff --git a/tools/perf/tests/attr/test-record-group-sampling b/tools/perf/tests/attr/test-record-group-sampling
> > index 300b9f7e6d69..97e7e64a38f0 100644
> > --- a/tools/perf/tests/attr/test-record-group-sampling
> > +++ b/tools/perf/tests/attr/test-record-group-sampling
> > @@ -7,7 +7,7 @@ ret = 1
> > fd=1
> > group_fd=-1
> > sample_type=343
> > -read_format=12
> > +read_format=12|28
> > inherit=0
> >
> > [event-2:base-record]
> > @@ -21,8 +21,8 @@ config=3
> > # default | PERF_SAMPLE_READ
> > sample_type=343
> >
> > -# PERF_FORMAT_ID | PERF_FORMAT_GROUP
> > -read_format=12
> > +# PERF_FORMAT_ID | PERF_FORMAT_GROUP | PERF_FORMAT_LOST
> > +read_format=12|28
> > task=0
> > mmap=0
> > comm=0
> > diff --git a/tools/perf/tests/attr/test-record-group1 b/tools/perf/tests/attr/test-record-group1
> > index 3ffe246e0228..eeb1db392bc9 100644
> > --- a/tools/perf/tests/attr/test-record-group1
> > +++ b/tools/perf/tests/attr/test-record-group1
> > @@ -7,7 +7,7 @@ ret = 1
> > fd=1
> > group_fd=-1
> > sample_type=327
> > -read_format=4
> > +read_format=4|20
> >
> > [event-2:base-record]
> > fd=2
> > @@ -15,7 +15,7 @@ group_fd=1
> > type=0
> > config=1
> > sample_type=327
> > -read_format=4
> > +read_format=4|20
> > mmap=0
> > comm=0
> > task=0
> > diff --git a/tools/perf/tests/attr/test-record-group2 b/tools/perf/tests/attr/test-record-group2
> > index 6b9f8d182ce1..cebdaa8e64e4 100644
> > --- a/tools/perf/tests/attr/test-record-group2
> > +++ b/tools/perf/tests/attr/test-record-group2
> > @@ -9,7 +9,7 @@ group_fd=-1
> > config=0|1
> > sample_period=1234000
> > sample_type=87
> > -read_format=12
> > +read_format=12|28
> > inherit=0
> > freq=0
> >
> > @@ -19,7 +19,7 @@ group_fd=1
> > config=0|1
> > sample_period=6789000
> > sample_type=87
> > -read_format=12
> > +read_format=12|28
> > disabled=0
> > inherit=0
> > mmap=0
> > --
> > 2.28.0
> >
--
- Arnaldo
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-10-14 20:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-12 9:46 [PATCH v2 0/1] perf test: Fix attr tests for PERF_FORMAT_LOST James Clark
2022-10-12 9:46 ` [PATCH v2 1/1] " James Clark
2022-10-12 16:06 ` Namhyung Kim
2022-10-14 20:07 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).