From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1754571AbdBGNnX (ORCPT <rfc822;w@1wt.eu>);
        Tue, 7 Feb 2017 08:43:23 -0500
Received: from mail.kernel.org ([198.145.29.136]:40428 "EHLO mail.kernel.org"
        rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP
        id S1753850AbdBGNnV (ORCPT <rfc822;linux-kernel@vger.kernel.org>);
        Tue, 7 Feb 2017 08:43:21 -0500
Date: Tue, 7 Feb 2017 22:43:09 +0900
From: Masami Hiramatsu <mhiramat@kernel.org>
To: He Kuang <hekuang@huawei.com>
Cc: <peterz@infradead.org>, <will.deacon@arm.com>, <mingo@redhat.com>,
        <acme@kernel.org>, <alexander.shishkin@linux.intel.com>,
        <jolsa@redhat.com>, <wangnan0@huawei.com>, <bintian.wang@huawei.com>,
        <linux-kernel@vger.kernel.org>, <linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v4 1/2] perf tools arm64: Add support for generating bpf
 prologue
Message-Id: <20170207224309.9260a3c718911393eda7a4bf@kernel.org>
In-Reply-To: <20170207073412.26983-1-hekuang@huawei.com>
References: <20170207073412.26983-1-hekuang@huawei.com>
X-Mailer: Sylpheed 3.5.0 (GTK+ 2.24.31; x86_64-redhat-linux-gnu)
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Tue, 7 Feb 2017 07:34:11 +0000
He Kuang <hekuang@huawei.com> wrote:

> Since HAVE_KPROBES can be enabled in arm64, this patch introduces
> regs_query_register_offset() to convert register name to offset for
> arm64, so the BPF prologue feature is ready to use.
> 

Ok, looks good to me.

Acked-by: Masami Hiramatsu <mhiramat@kernel.org>

Thanks!

> Signed-off-by: He Kuang <hekuang@huawei.com>
> ---
>  tools/perf/arch/arm64/Makefile          |  1 +
>  tools/perf/arch/arm64/util/dwarf-regs.c | 15 ++++++++++++++-
>  2 files changed, 15 insertions(+), 1 deletion(-)
> 
> diff --git a/tools/perf/arch/arm64/Makefile b/tools/perf/arch/arm64/Makefile
> index 18b1351..eebe1ec 100644
> --- a/tools/perf/arch/arm64/Makefile
> +++ b/tools/perf/arch/arm64/Makefile
> @@ -2,3 +2,4 @@ ifndef NO_DWARF
>  PERF_HAVE_DWARF_REGS := 1
>  endif
>  PERF_HAVE_JITDUMP := 1
> +PERF_HAVE_ARCH_REGS_QUERY_REGISTER_OFFSET := 1
> diff --git a/tools/perf/arch/arm64/util/dwarf-regs.c b/tools/perf/arch/arm64/util/dwarf-regs.c
> index d49efeb..068b618 100644
> --- a/tools/perf/arch/arm64/util/dwarf-regs.c
> +++ b/tools/perf/arch/arm64/util/dwarf-regs.c
> @@ -10,17 +10,20 @@
>  
>  #include <stddef.h>
>  #include <dwarf-regs.h>
> +#include <linux/ptrace.h> /* for struct user_pt_regs */
> +#include "util.h"
>  
>  struct pt_regs_dwarfnum {
>  	const char *name;
>  	unsigned int dwarfnum;
>  };
>  
> -#define STR(s) #s
>  #define REG_DWARFNUM_NAME(r, num) {.name = r, .dwarfnum = num}
>  #define GPR_DWARFNUM_NAME(num) \
>  	{.name = STR(%x##num), .dwarfnum = num}
>  #define REG_DWARFNUM_END {.name = NULL, .dwarfnum = 0}
> +#define DWARFNUM2OFFSET(index) \
> +	(index * sizeof((struct user_pt_regs *)0)->regs[0])
>  
>  /*
>   * Reference:
> @@ -78,3 +81,13 @@ const char *get_arch_regstr(unsigned int n)
>  			return roff->name;
>  	return NULL;
>  }
> +
> +int regs_query_register_offset(const char *name)
> +{
> +	const struct pt_regs_dwarfnum *roff;
> +
> +	for (roff = regdwarfnum_table; roff->name != NULL; roff++)
> +		if (!strcmp(roff->name, name))
> +			return DWARFNUM2OFFSET(roff->dwarfnum);
> +	return -EINVAL;
> +}
> -- 
> 1.8.5.2
> 


-- 
Masami Hiramatsu <mhiramat@kernel.org>