From: Huang Rui <ray.huang@amd.com>
To: "Su, Jinzhou (Joe)" <Jinzhou.Su@amd.com>
Cc: "rjw@rjwysocki.net" <rjw@rjwysocki.net>,
"linux-pm@vger.kernel.org" <linux-pm@vger.kernel.org>,
"srinivas.pandruvada@linux.intel.com"
<srinivas.pandruvada@linux.intel.com>,
"dsmythies@telus.net" <dsmythies@telus.net>,
"viresh.kumar@linaro.org" <viresh.kumar@linaro.org>,
"todd.e.brandt@linux.intel.com" <todd.e.brandt@linux.intel.com>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Sharma, Deepak" <Deepak.Sharma@amd.com>,
"Deucher, Alexander" <Alexander.Deucher@amd.com>,
"Du, Xiaojian" <Xiaojian.Du@amd.com>,
"Yuan, Perry" <Perry.Yuan@amd.com>,
"Meng, Li (Jassmine)" <Li.Meng@amd.com>
Subject: Re: [PATCH v3 0/4] Add tracer tool for AMD P-State driver
Date: Wed, 9 Mar 2022 21:31:49 +0800 [thread overview]
Message-ID: <YiisRXfrIco8hnXf@amd.com> (raw)
In-Reply-To: <20220309012351.616826-1-Jinzhou.Su@amd.com>
On Wed, Mar 09, 2022 at 09:23:47AM +0800, Su, Jinzhou (Joe) wrote:
> Hello,
>
> intel_pstate_tracer is a useful tool to analyze the performance of
> intel_pstate driver. We upstream out AMD P-state driver into Linux
> kernel recently and like to use similar tool to tune the performance
> of the driver.
>
> I modified intel_pstate_tracer.py then it could import as a module to
> analyze AMD P-State trace event. Other trace event also can benifit from
> this change once they need this tool.
>
> intel_pstate_tracer could be used as the same way as before and the
> original functionality isn't broken.
>
> Changes from V2->V3
>
> -fix typo in amd-pstate RST
>
> Changes from V1->V2
>
> -Add tracer documentation in amd-pstate RST
>
> -fix typo in amd_pstate_trace.py
>
> -add "Co-developed-by" in patch 1/4
>
> Thanks,
> Joe
>
> Jinzhou Su (4):
> cpufreq: amd-pstate: Add more tracepoint for AMD P-State module
> tools/power/x86/intel_pstate_tracer: make tracer as a module
> tools/power/x86/amd_pstate_tracer: Add tracer tool for AMD P-state
> Documentation: amd-pstate: add tracer tool introduction
Series are Reviewed-by: Huang Rui <ray.huang@amd.com>
>
> Documentation/admin-guide/pm/amd-pstate.rst | 26 ++
> MAINTAINERS | 1 +
> drivers/cpufreq/amd-pstate-trace.h | 22 +-
> drivers/cpufreq/amd-pstate.c | 59 ++-
> .../x86/amd_pstate_tracer/amd_pstate_trace.py | 354 ++++++++++++++++++
> .../intel_pstate_tracer.py | 260 +++++++------
> 6 files changed, 588 insertions(+), 134 deletions(-)
> create mode 100755 tools/power/x86/amd_pstate_tracer/amd_pstate_trace.py
>
> --
> 2.27.0
>
next prev parent reply other threads:[~2022-03-09 13:32 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-03-09 1:23 [PATCH v3 0/4] Add tracer tool for AMD P-State driver Jinzhou Su
2022-03-09 1:23 ` [PATCH v3 1/4] cpufreq: amd-pstate: Add more tracepoint for AMD P-State module Jinzhou Su
2022-03-09 1:23 ` [PATCH v3 2/4] tools/power/x86/intel_pstate_tracer: make tracer as a module Jinzhou Su
2022-03-09 1:23 ` [PATCH v3 3/4] tools/power/x86/amd_pstate_tracer: Add tracer tool for AMD P-state Jinzhou Su
2022-03-09 1:23 ` [PATCH v3 4/4] Documentation: amd-pstate: add tracer tool introduction Jinzhou Su
2022-03-09 13:31 ` Huang Rui [this message]
2022-03-09 18:54 ` [PATCH v3 0/4] Add tracer tool for AMD P-State driver Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=YiisRXfrIco8hnXf@amd.com \
--to=ray.huang@amd.com \
--cc=Alexander.Deucher@amd.com \
--cc=Deepak.Sharma@amd.com \
--cc=Jinzhou.Su@amd.com \
--cc=Li.Meng@amd.com \
--cc=Perry.Yuan@amd.com \
--cc=Xiaojian.Du@amd.com \
--cc=dsmythies@telus.net \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=rjw@rjwysocki.net \
--cc=srinivas.pandruvada@linux.intel.com \
--cc=todd.e.brandt@linux.intel.com \
--cc=viresh.kumar@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.