Em Wed, Oct 25, 2023 at 10:26:32PM -0700, Namhyung Kim escreveu: > On Tue, Oct 24, 2023 at 12:10 PM Namhyung Kim <namhyung@xxxxxxxxxx> wrote: > > On Mon, Oct 23, 2023 at 9:38 AM Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote: > > > Em Wed, Oct 11, 2023 at 08:50:36PM -0700, Namhyung Kim escreveu: > > > > The annotate_get_insn_location() is to get the detailed information of > > > > instruction locations like registers and offset. It has source and > > > > target operands locations in an array. Each operand can have a > > > > register and an offset. The offset is meaningful when mem_ref flag is > > > > set. > > > This needs to be enclosed in HAVE_DWARF_SUPPORT, as the build is failing > > > in systems where this isn't available, see patch below. > > Thanks for the test and the patch, will add it to v2. > Hmm.. I think this function can work without DWARF. > If the only problem is get_dwarf_regnum() probably > I can add a dummy function when libdw is not found. > Maybe I need to use arch reg number here and > convert to DWARF later. That can be a possibility, but the interesting thing is to try to run 'make -C tools/perf build-test' regularly to catch these builds without some of the components, I bet this one: make_no_libelf := NO_LIBELF=1 As my container builds did, as that is one of the things built there. One other suggestion would be to add some warning when the required components to build the data profiling feature are not present. - Arnaldo
|