mbox series

[0/2] tools/power/turbostat: Fix long time duration RAPL calculation

Message ID cover.1619600637.git.yu.c.chen@intel.com (mailing list archive)
Headers show
Series tools/power/turbostat: Fix long time duration RAPL calculation | expand

Message

Chen Yu April 28, 2021, 9:08 a.m. UTC 
A combination of patch set to fix the recently introduced long time
duration RAPL calculation issue found on AMD CPUs.

Youling, Kurt, Bingsong, Artem, could you please confirm again
if this patch set works for you on your platform?

Bas Nieuwenhuizen (1):
  tools/power/turbostat: Fix turbostat for AMD Zen CPUs

Calvin Walton (1):
  tools/power turbostat: Fix offset overflow issue in index converting

 tools/power/x86/turbostat/turbostat.c | 19 ++++++++++++-------
 1 file changed, 12 insertions(+), 7 deletions(-)

Comments

Bowman, Terry April 29, 2021, 7:47 p.m. UTC | #1 
Both patches look good to me. I tested both on AMD Epyc Rome. Iverified
the missing output is fixed and used instrumentationto verify the
sign-extension issues are resolved.

Please add:
Signed-off-by: Terry Bowman <Terry.Bowman@amd.com>

Regards,
Terry

On 4/28/21 4:08 AM, Chen Yu wrote:
> A combination of patch set to fix the recently introduced long time
> duration RAPL calculation issue found on AMD CPUs.
>
> Youling, Kurt, Bingsong, Artem, could you please confirm again
> if this patch set works for you on your platform?
>
> Bas Nieuwenhuizen (1):
>    tools/power/turbostat: Fix turbostat for AMD Zen CPUs
>
> Calvin Walton (1):
>    tools/power turbostat: Fix offset overflow issue in index converting
>
>   tools/power/x86/turbostat/turbostat.c | 19 ++++++++++++-------
>   1 file changed, 12 insertions(+), 7 deletions(-)
>