lkml.org 
[lkml]   [2022]   [Jan]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    Date
    Subject[PATCH v4 29/48] perf counts: Switch name cpu to cpu_map_idx
    From
    Try to reduce confusion in particular when the cpu map doesn't contain
    an entry for every CPU.

    Signed-off-by: Ian Rogers <irogers@google.com>
    ---
    tools/perf/util/counts.h | 12 ++++++------
    1 file changed, 6 insertions(+), 6 deletions(-)

    diff --git a/tools/perf/util/counts.h b/tools/perf/util/counts.h
    index 3e275e9c60d1..5de275194f2b 100644
    --- a/tools/perf/util/counts.h
    +++ b/tools/perf/util/counts.h
    @@ -18,21 +18,21 @@ struct perf_counts {


    static inline struct perf_counts_values*
    -perf_counts(struct perf_counts *counts, int cpu, int thread)
    +perf_counts(struct perf_counts *counts, int cpu_map_idx, int thread)
    {
    - return xyarray__entry(counts->values, cpu, thread);
    + return xyarray__entry(counts->values, cpu_map_idx, thread);
    }

    static inline bool
    -perf_counts__is_loaded(struct perf_counts *counts, int cpu, int thread)
    +perf_counts__is_loaded(struct perf_counts *counts, int cpu_map_idx, int thread)
    {
    - return *((bool *) xyarray__entry(counts->loaded, cpu, thread));
    + return *((bool *) xyarray__entry(counts->loaded, cpu_map_idx, thread));
    }

    static inline void
    -perf_counts__set_loaded(struct perf_counts *counts, int cpu, int thread, bool loaded)
    +perf_counts__set_loaded(struct perf_counts *counts, int cpu_map_idx, int thread, bool loaded)
    {
    - *((bool *) xyarray__entry(counts->loaded, cpu, thread)) = loaded;
    + *((bool *) xyarray__entry(counts->loaded, cpu_map_idx, thread)) = loaded;
    }

    struct perf_counts *perf_counts__new(int ncpus, int nthreads);
    --
    2.34.1.448.ga2b2bfdf31-goog
    \
     
     \ /
      Last update: 2022-01-05 07:16    [W:2.561 / U:0.028 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site