From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-6.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id E6A34C43331 for ; Mon, 28 Dec 2020 16:48:02 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id B218D20659 for ; Mon, 28 Dec 2020 16:48:02 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2441747AbgL1Qrz (ORCPT ); Mon, 28 Dec 2020 11:47:55 -0500 Received: from mail.kernel.org ([198.145.29.99]:51758 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728474AbgL1MzJ (ORCPT ); Mon, 28 Dec 2020 07:55:09 -0500 Received: by mail.kernel.org (Postfix) with ESMTPSA id 0FCD322573; Mon, 28 Dec 2020 12:54:29 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1609160069; bh=cCr9rjr5oy6TVkR4t8ptkBnMacdfaREGpVlXYRjz16w=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=ZAmbgUMKHomsZQXSf3pdqO6VeQKjIo9oLoo18Su6+qJ/3zKyVt8fWKNLlGC0szWCM DHuYxhQmmrzC2SdHCVroQuT3ZdDdP79cKz6ovN2D7hfgwvvz3B6+nhmpqu42f1k885 +5Ra2LQB2HP4G1VK5eGwMljUUNcvrj29aISIZFcgQG0u4sbM4MhAOWDxsQO8exrtLS efYmHWW6W1NTDZRkfNv8PUFifn0rVJ9F5dw6dBwxd6HWYRDWPYLI3pijXo46QzsRob 43uJKcUbeK2YWdQOALsGaj8dNaEuuGSyPcx9+9+AJiyrRDfTosOGB22bx9TUXAlVGX noM9Uhp8ofsOA== Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 5AA3B411E9; Mon, 28 Dec 2020 09:54:43 -0300 (-03) Date: Mon, 28 Dec 2020 09:54:43 -0300 From: Arnaldo Carvalho de Melo To: Leo Yan Cc: John Garry , Will Deacon , Mathieu Poirier , Masami Hiramatsu , Peter Zijlstra , Ingo Molnar , Mark Rutland , Alexander Shishkin , Jiri Olsa , Namhyung Kim , Alexandre Truong , Ian Rogers , Thomas Richter , He Zhe , Sumanth Korikkar , Alexis Berlemont , 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> X-Url: http://acmel.wordpress.com Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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 From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 4B12EC433E0 for ; Mon, 28 Dec 2020 12:55:51 +0000 (UTC) Received: from merlin.infradead.org (merlin.infradead.org [205.233.59.134]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 08FD521D94 for ; Mon, 28 Dec 2020 12:55:51 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 08FD521D94 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=merlin.20170209; h=Sender:Content-Transfer-Encoding: Content-Type:Cc:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References:Message-ID: Subject:To:From:Date:Reply-To:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=8BGemiPduIKtQ5H5t9Aw5lL//jGtKGQZvsQs94Cx414=; b=MrrRlpkmxRDznmfKJJ8et/1rE PPhPmdffoSOxe9hTUCXYSzcuP53oxQdZmxxqt0wgzxQyEMPoa1Tbpo7CvPXjRPLlWUju7wHjoAqGb k5lokuj+LEVWfWOkJcF+/bzyAD57Rvj7EGFdpYzJL/PgX71StAb7tm5MytSNCWgforuJ5zrabCpHX 90ykA36L+ylK1RovcvJn/KD9TTqYHZynkuNrwixYABVvxncZ/T748+njbcJbxSY1TyOqQE48d22BV twuCeSalO2khxhcb+s4gfBTb7d/Y/VNkYpXFynqIZH9tHNeM/JnzVAV877WKSvlwmyYIQvvQQUApL 94sPX+CZg==; Received: from localhost ([::1] helo=merlin.infradead.org) by merlin.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1kts2d-0003VJ-CE; Mon, 28 Dec 2020 12:54:27 +0000 Received: from [179.97.37.151] (helo=quaco.ghostprotocols.net) by merlin.infradead.org with esmtpsa (Exim 4.92.3 #3 (Red Hat Linux)) id 1kts2c-0003V6-DM; Mon, 28 Dec 2020 12:54:26 +0000 Received: by quaco.ghostprotocols.net (Postfix, from userid 1000) id 5AA3B411E9; Mon, 28 Dec 2020 09:54:43 -0300 (-03) Date: Mon, 28 Dec 2020 09:54:43 -0300 From: Arnaldo Carvalho de Melo To: Leo Yan 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-Disposition: inline In-Reply-To: <20201225052751.24513-1-leo.yan@linaro.org> X-Url: http://acmel.wordpress.com X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Mark Rutland , Ian Rogers , Thomas Richter , Mathieu Poirier , Peter Zijlstra , Jiri Olsa , John Garry , linux-kernel@vger.kernel.org, Alexander Shishkin , Alexandre Truong , Ingo Molnar , Masami Hiramatsu , Namhyung Kim , Sumanth Korikkar , Alexis Berlemont , Will Deacon , He Zhe , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org 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 _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel