Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Tue, 1 Jun 2021 00:49:34 +0200
From:   Jiri Olsa <jolsa@redhat.com>
To:     Jin Yao <yao.jin@linux.intel.com>
Cc:     acme@kernel.org, jolsa@kernel.org, peterz@infradead.org,
        mingo@redhat.com, alexander.shishkin@linux.intel.com,
        Linux-kernel@vger.kernel.org, ak@linux.intel.com,
        kan.liang@intel.com, yao.jin@intel.com
Subject: Re: [PATCH v2 0/8] perf: Support perf-mem/perf-c2c for AlderLake
Message-ID: <YLVn/jh/8H5fZ/QJ@krava>
References: <20210527001610.10553-1-yao.jin@linux.intel.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20210527001610.10553-1-yao.jin@linux.intel.com>
Precedence: bulk

On Thu, May 27, 2021 at 08:16:02AM +0800, Jin Yao wrote:
> AlderLake uses a hybrid architecture utilizing Golden Cove cores
> (core CPU) and Gracemont cores (atom CPU). This patchset supports
> perf-mem and perf-c2c for AlderLake.
> 
> v2:
> ---
> - Use mem_loads_name__init to keep original behavior for non-hybrid platform.
> - Move x86 specific perf_mem_events[] to arch/x86/util/mem-events.c.
> - Move mem-store event to a new patch.
> - Add a new patch to fix wrong verbose output for recording events
> - Add a new patch to disable 'mem-loads-aux' group before reporting

Acked-by: Jiri Olsa <jolsa@redhat.com>

thanks,
jirka

> 
> Jin Yao (8):
>   perf tools: Check mem-loads auxiliary event
>   perf tools: Support pmu prefix for mem-load event
>   perf tools: Support pmu prefix for mem-store event
>   perf tools: Check if mem_events is supported for hybrid platform
>   perf mem: Support record for hybrid platform
>   perf mem: Fix wrong verbose output for recording events
>   perf mem: Disable 'mem-loads-aux' group before reporting
>   perf c2c: Support record for hybrid platform
> 
>  tools/perf/arch/arm64/util/mem-events.c   |   2 +-
>  tools/perf/arch/powerpc/util/mem-events.c |   2 +-
>  tools/perf/arch/x86/util/mem-events.c     |  54 ++++++++++--
>  tools/perf/builtin-c2c.c                  |  40 +++++----
>  tools/perf/builtin-mem.c                  |  51 ++++++-----
>  tools/perf/builtin-report.c               |   2 +
>  tools/perf/util/evlist.c                  |  25 ++++++
>  tools/perf/util/evlist.h                  |   1 +
>  tools/perf/util/mem-events.c              | 101 ++++++++++++++++++++--
>  tools/perf/util/mem-events.h              |   4 +-
>  10 files changed, 225 insertions(+), 57 deletions(-)
> 
> -- 
> 2.17.1
>