2014-04-03 21:52:41

by Ramkumar Ramachandra

Subject: Re: [PATCH] perf tools: Pick up libdw without explicit LIBDW_DIR

Ramkumar Ramachandra wrote:
> tools/perf/config/Makefile | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)

Jiri, any comments?

Ram

2014-04-04 07:46:32

by Jean Pihet

Subject: Re: [PATCH] perf tools: Pick up libdw without explicit LIBDW_DIR

On 3 April 2014 23:51, Ramkumar Ramachandra <[email protected]> wrote:
> Ramkumar Ramachandra wrote:
>> tools/perf/config/Makefile | 5 ++---
>> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> Jiri, any comments?
Note: this patch is equivalent to '[PATCH] perf tools: Fix feature
check for libdw dwarf unwind' [1] from Josh Boyer. Jiri, your pick ;-)

[1] http://www.spinics.net/lists/kernel/msg1716729.html

FWIW:
Acked-by: Jean Pihet <[email protected]>

>
> Ram
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/

2014-04-06 12:28:31

by Jiri Olsa

Subject: Re: [PATCH] perf tools: Pick up libdw without explicit LIBDW_DIR

On Fri, Apr 04, 2014 at 09:46:28AM +0200, Jean Pihet wrote:
> On 3 April 2014 23:51, Ramkumar Ramachandra <[email protected]> wrote:
> > Ramkumar Ramachandra wrote:
> >> tools/perf/config/Makefile | 5 ++---
> >> 1 file changed, 2 insertions(+), 3 deletions(-)
> >
> > Jiri, any comments?

hi,
sorry for late reply, I was out last week

> Note: this patch is equivalent to '[PATCH] perf tools: Fix feature
> check for libdw dwarf unwind' [1] from Josh Boyer. Jiri, your pick ;-)
>
> [1] http://www.spinics.net/lists/kernel/msg1716729.html
>
> FWIW:
> Acked-by: Jean Pihet <[email protected]>

I'll take Ramkumar's change, because he sent it out first ;-)

thanks,
jirka