[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20150914213753.GA2834@redhat.com>
Date: Mon, 14 Sep 2015 18:37:53 -0300
From: Arnaldo Carvalho de Melo <acme@...hat.com>
To: Wang Nan <wangnan0@...wei.com>
Cc: ast@...mgrid.com, masami.hiramatsu.pt@...achi.com,
namhyung@...nel.org, a.p.zijlstra@...llo.nl,
brendan.d.gregg@...il.com, acme@...nel.org, daniel@...earbox.net,
dsahern@...il.com, hekuang@...wei.com, jolsa@...nel.org,
lizefan@...wei.com, paulus@...ba.org, xiakaixu@...wei.com,
pi3orama@....com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 19/27] perf tools: Introduce regs_query_register_offset()
for x86
Em Sun, Sep 06, 2015 at 07:13:35AM +0000, Wang Nan escreveu:
> regs_query_register_offset() is a helper function which converts
> register name like "%rax" to offset of a register in 'struct pt_regs',
> which is required by BPF prologue generator. Since the function is
> identical, try to reuse the code in arch/x86/kernel/ptrace.c.
>
> Comment inside dwarf-regs.c list the differences between this
> implementation and kernel code.
>
> get_arch_regstr() switches to regoffset_table and the old string table
> is dropped.
Trying to cherry pick this one, but found this problem, trying to fix by
adding the prototype somewhere...
[acme@zoo linux]$ m
make: Entering directory '/home/git/linux/tools/perf'
BUILD: Doing 'make -j4' parallel build
CC /tmp/build/perf/arch/x86/util/dwarf-regs.o
CC /tmp/build/perf/arch/x86/util/intel-pt.o
arch/x86/util/dwarf-regs.c:122:5: error: no previous prototype for
‘regs_query_register_offset’ [-Werror=missing-prototypes]
int regs_query_register_offset(const char *name)
^
cc1: all warnings being treated as errors
/home/git/linux/tools/build/Makefile.build:70: recipe for target
'/tmp/build/perf/arch/x86/util/dwarf-regs.o' failed
make[5]: *** [/tmp/build/perf/arch/x86/util/dwarf-regs.o] Error 1
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists