Re: [PATCH 3/4] perf inject: add jitdump mmap injection support

Stephane Eranian
2015-02-11T13:13:26Z

On Wed, Feb 11, 2015 at 6:59 AM, Peter Zijlstra <peterz@infradead.org> wrote:
>
> On Wed, Feb 11, 2015 at 12:42:44AM +0100, Stephane Eranian wrote:
> > diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
> > index aa6a504..c86c412 100644
> > --- a/tools/perf/Makefile.perf
> > +++ b/tools/perf/Makefile.perf
>
> > @@ -496,7 +499,8 @@ BUILTIN_OBJS += $(OUTPUT)builtin-inject.o
> > BUILTIN_OBJS += $(OUTPUT)tests/builtin-test.o
> > BUILTIN_OBJS += $(OUTPUT)builtin-mem.o
> >
> > -PERFLIBS = $(LIB_FILE) $(LIBAPIKFS) $(LIBTRACEEVENT)
> > +PERFLIBS = $(LIB_FILE) $(LIBAPIKFS) $(LIBTRACEEVENT) -lcrypto
> > +
>
> Should we not add auto-detect magic for that? Otherwise perf will stop
> building on machines without libssl.

Good point.
The ssl support is needed to create the buildid for each generated ELF
image.