Em Sat, Aug 31, 2019 at 12:20:04AM +0900, Masami Hiramatsu escreveu:
> On Thu, 29 Aug 2019 17:41:17 -0500
> Josh Poimboeuf <jpoim...@redhat.com> wrote:
>
> > It's kind of silly that we have *three* identical copies of the x86 insn
> > decoder in the kernel tree. Make it approximately 50% less silly by
> > reducing that number to two.
> >
>
> Sounds good to me ;)
>
> Reviewed-by: Masami Hiramatsu <mhira...@kernel.org>
Good, two already, Adrian?
- Arnaldo
> Thanks,
>
> > Josh Poimboeuf (4):
> > objtool: Move x86 insn decoder to a common location
> > perf: Update .gitignore file
> > perf intel-pt: Remove inat.c from build dependency list
> > perf intel-pt: Use shared x86 insn decoder
> >
> > .../{objtool => }/arch/x86/include/asm/inat.h | 0
> > .../arch/x86/include/asm/inat_types.h | 0
> > .../{objtool => }/arch/x86/include/asm/insn.h | 0
> > .../arch/x86/include/asm/orc_types.h | 0
> > tools/{objtool => }/arch/x86/lib/inat.c | 0
> > tools/{objtool => }/arch/x86/lib/insn.c | 0
> > .../arch/x86/lib/x86-opcode-map.txt | 0
> > .../arch/x86/tools/gen-insn-attr-x86.awk | 0
> > tools/objtool/Makefile | 4 +-
> > tools/objtool/arch/x86/Build | 4 +-
> > tools/objtool/arch/x86/decode.c | 4 +-
> > tools/objtool/sync-check.sh | 12 +-
> > tools/perf/.gitignore | 3 +
> > tools/perf/arch/x86/tests/insn-x86.c | 2 +-
> > tools/perf/arch/x86/util/archinsn.c | 2 +-
> > tools/perf/check-headers.sh | 11 +-
> > tools/perf/util/intel-pt-decoder/Build | 22 +-
> > .../intel-pt-decoder/gen-insn-attr-x86.awk | 392 ------
> > tools/perf/util/intel-pt-decoder/inat.c | 82 --
> > tools/perf/util/intel-pt-decoder/inat.h | 230 ----
> > tools/perf/util/intel-pt-decoder/inat_types.h | 15 -
> > tools/perf/util/intel-pt-decoder/insn.c | 593 ---------
> > tools/perf/util/intel-pt-decoder/insn.h | 216 ----
> > .../intel-pt-decoder/intel-pt-insn-decoder.c | 10 +-
> > .../util/intel-pt-decoder/x86-opcode-map.txt | 1072 -----------------
> > 25 files changed, 34 insertions(+), 2640 deletions(-)
> > rename tools/{objtool => }/arch/x86/include/asm/inat.h (100%)
> > rename tools/{objtool => }/arch/x86/include/asm/inat_types.h (100%)
> > rename tools/{objtool => }/arch/x86/include/asm/insn.h (100%)
> > rename tools/{objtool => }/arch/x86/include/asm/orc_types.h (100%)
> > rename tools/{objtool => }/arch/x86/lib/inat.c (100%)
> > rename tools/{objtool => }/arch/x86/lib/insn.c (100%)
> > rename tools/{objtool => }/arch/x86/lib/x86-opcode-map.txt (100%)
> > rename tools/{objtool => }/arch/x86/tools/gen-insn-attr-x86.awk (100%)
> > delete mode 100644 tools/perf/util/intel-pt-decoder/gen-insn-attr-x86.awk
> > delete mode 100644 tools/perf/util/intel-pt-decoder/inat.c
> > delete mode 100644 tools/perf/util/intel-pt-decoder/inat.h
> > delete mode 100644 tools/perf/util/intel-pt-decoder/inat_types.h
> > delete mode 100644 tools/perf/util/intel-pt-decoder/insn.c
> > delete mode 100644 tools/perf/util/intel-pt-decoder/insn.h
> > delete mode 100644 tools/perf/util/intel-pt-decoder/x86-opcode-map.txt
> >
> > --
> > 2.20.1
> >
>
>
> --
> Masami Hiramatsu <mhira...@kernel.org>
--
- Arnaldo
