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:   Mon, 28 Dec 2020 09:54:43 -0300
From:   Arnaldo Carvalho de Melo <acme@kernel.org>
To:     Leo Yan <leo.yan@linaro.org>
Cc:     John Garry <john.garry@huawei.com>, Will Deacon <will@kernel.org>,
        Mathieu Poirier <mathieu.poirier@linaro.org>,
        Masami Hiramatsu <mhiramat@kernel.org>,
        Peter Zijlstra <peterz@infradead.org>,
        Ingo Molnar <mingo@redhat.com>,
        Mark Rutland <mark.rutland@arm.com>,
        Alexander Shishkin <alexander.shishkin@linux.intel.com>,
        Jiri Olsa <jolsa@redhat.com>,
        Namhyung Kim <namhyung@kernel.org>,
        Alexandre Truong <alexandre.truong@arm.com>,
        Ian Rogers <irogers@google.com>,
        Thomas Richter <tmricht@linux.ibm.com>,
        He Zhe <zhe.he@windriver.com>,
        Sumanth Korikkar <sumanthk@linux.ibm.com>,
        Alexis Berlemont <alexis.berlemont@gmail.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 0/3] perf arm64: Support SDT
Message-ID: <20201228125443.GA521329@kernel.org>
References: <20201225052751.24513-1-leo.yan@linaro.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20201225052751.24513-1-leo.yan@linaro.org>
Precedence: bulk

Em Fri, Dec 25, 2020 at 01:27:48PM +0800, Leo Yan escreveu:
> This patch is to enable SDT on Arm64.
> 
> Since Arm64 SDT marker in ELF file is different from other archs,
> especially for using stack pointer (sp) to retrieve data for local
> variables, patch 01 is used to fixup the arguments for this special
> case.  Patch 02 is to add argument support for Arm64 SDT.
> 
> This patch set has been verified on Arm64/x86_64 platforms with a
> testing program usdt_test.  The testing approach is described in the
> patch set v1 [1].
> 
> Changes from v1:
> * Added Arnaldo's patch for fixing memory leak (Arnaldo);
> * Refined patch "perf probe: Fixup Arm64 SDT arguments" to use
>   asprintf() and check pointer is valid or not (Arnaldo);
> * Minor changes in patch "perf arm64: Add argument support for SDT"
>   for the regular expression;
> * Added Masami's Ack tag for patch 03.
> 
> [1] https://lore.kernel.org/patchwork/cover/1356212/
> 

This one made it into 5.11
 
> Arnaldo Carvalho de Melo (1):
>   perf probe: Fix memory leak in synthesize_sdt_probe_command()
 
These were now merged in perf/core.

> Leo Yan (2):
>   perf probe: Fixup Arm64 SDT arguments
>   perf arm64: Add argument support for SDT

Thanks,

- Arnaldo