Re: [PATCH 0/7] turbostat, Fix AMD output by making turbostat aware of nodes

From: Prarit Bhargava
Date: Wed Jul 19 2017 - 08:36:49 EST


Forgot to cc Len at his kernel.org address ...

P.

On 07/16/2017 04:49 PM, Prarit Bhargava wrote:
> AMD family processors do not show all cores in the output of turbostat. This
> occurs because AMD has multiple nodes per socket and enumerates cores
> within each node from 0. For example, socket 0 may have two nodes (0 and 1)
> and those nodes both have cores enumerated from 0 through 7. turbostat cannot
> handle this configuration, and as a result only shows 1/2 the cores in its
> output.
>
> This patchset makes turbostate aware of nodes. It has been tested on
> various AMD and Intel systems and no issues have been found.
>
> Signed-off-by: Prarit Bhargava <prarit@xxxxxxxxxx>
> Cc: Len Brown <len.brown@xxxxxxxxx>
>
> Prarit Bhargava (7):
> turbostat: set max_num_cpus equal to the cpumask length
> turbostat: Fix node and siblings lookup data
> turbostat: Calculate additional node information for a
> turbostat: track thread ID in cpu_topology
> turbostat: rename num_cores_per_pkg to num_cores_per_node
> turbostat: remove num_ from cpu_topology struct
> turbostat: add node information into turbostat calculations
>
> tools/power/x86/turbostat/turbostat.c | 419 +++++++++++++++++++++-------------
> 1 file changed, 259 insertions(+), 160 deletions(-)
>