2020-07-07 17:18:24

by Sven Schnelle

[permalink] [raw]
Subject: [PATCH] perf: add psw_idle and psw_idle_exit to list of idle symbols

Add the s390 idle functions so they don't show up in top when
using software sampling.

Signed-off-by: Sven Schnelle <[email protected]>
---
tools/perf/util/symbol.c | 2 ++
1 file changed, 2 insertions(+)

diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
index 5ddf84dcbae7..d33d24c61d24 100644
--- a/tools/perf/util/symbol.c
+++ b/tools/perf/util/symbol.c
@@ -666,6 +666,8 @@ static bool symbol__is_idle(const char *name)
"poll_idle",
"ppc64_runlatch_off",
"pseries_dedicated_idle_sleep",
+ "psw_idle",
+ "psw_idle_exit",
NULL
};
int i;
--
2.17.1


2020-08-03 11:05:52

by Sven Schnelle

[permalink] [raw]
Subject: Re: [PATCH] perf: add psw_idle and psw_idle_exit to list of idle symbols

Sven Schnelle <[email protected]> writes:

> Add the s390 idle functions so they don't show up in top when
> using software sampling.
>
> Signed-off-by: Sven Schnelle <[email protected]>
> ---
> tools/perf/util/symbol.c | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
> index 5ddf84dcbae7..d33d24c61d24 100644
> --- a/tools/perf/util/symbol.c
> +++ b/tools/perf/util/symbol.c
> @@ -666,6 +666,8 @@ static bool symbol__is_idle(const char *name)
> "poll_idle",
> "ppc64_runlatch_off",
> "pseries_dedicated_idle_sleep",
> + "psw_idle",
> + "psw_idle_exit",
> NULL
> };
> int i;

gentle ping?

Regards
Sven

2020-08-04 12:35:12

by Arnaldo Carvalho de Melo

[permalink] [raw]
Subject: Re: [PATCH] perf: add psw_idle and psw_idle_exit to list of idle symbols

Em Mon, Aug 03, 2020 at 01:02:06PM +0200, Sven Schnelle escreveu:
> Sven Schnelle <[email protected]> writes:
>
> > Add the s390 idle functions so they don't show up in top when
> > using software sampling.
> >
> > Signed-off-by: Sven Schnelle <[email protected]>
> > ---
> > tools/perf/util/symbol.c | 2 ++
> > 1 file changed, 2 insertions(+)
> >
> > diff --git a/tools/perf/util/symbol.c b/tools/perf/util/symbol.c
> > index 5ddf84dcbae7..d33d24c61d24 100644
> > --- a/tools/perf/util/symbol.c
> > +++ b/tools/perf/util/symbol.c
> > @@ -666,6 +666,8 @@ static bool symbol__is_idle(const char *name)
> > "poll_idle",
> > "ppc64_runlatch_off",
> > "pseries_dedicated_idle_sleep",
> > + "psw_idle",
> > + "psw_idle_exit",
> > NULL
> > };
> > int i;
>
> gentle ping?

This was merged already:


commit 19bf119ccfa071b73a54465e8082ab2c8f8c30eb
Author: Sven Schnelle <[email protected]>
AuthorDate: Tue Jul 7 19:14:55 2020 +0200
Commit: Arnaldo Carvalho de Melo <[email protected]>
CommitDate: Tue Jul 7 16:44:57 2020 -0300

perf symbols: Add s390 idle functions 'psw_idle' and 'psw_idle_exit' to list of idle symbols