Re: [PATCH] perf: fix building error in x86_64 when dwarf unwind is on

From: Jiri Olsa
Date: Fri Jan 09 2015 - 09:44:01 EST


On Thu, Jan 08, 2015 at 02:09:20PM +0900, Namhyung Kim wrote:

SNIP

> > ---
> > diff --git a/tools/perf/Makefile.perf b/tools/perf/Makefile.perf
> > index 67a03a82..1f71a32 100644
> > --- a/tools/perf/Makefile.perf
> > +++ b/tools/perf/Makefile.perf
> > @@ -462,10 +462,12 @@ BUILTIN_OBJS += $(OUTPUT)builtin-bench.o
> > # Benchmark modules
> > BUILTIN_OBJS += $(OUTPUT)bench/sched-messaging.o
> > BUILTIN_OBJS += $(OUTPUT)bench/sched-pipe.o
> > -ifeq ($(RAW_ARCH),x86_64)
> > +ifeq ($(ARCH), x86)
> > +ifeq ($(IS_64_BIT), 1)
> > BUILTIN_OBJS += $(OUTPUT)bench/mem-memcpy-x86-64-asm.o
> > BUILTIN_OBJS += $(OUTPUT)bench/mem-memset-x86-64-asm.o
> > endif
> > +endif
> > BUILTIN_OBJS += $(OUTPUT)bench/mem-memcpy.o
> > BUILTIN_OBJS += $(OUTPUT)bench/futex-hash.o
> > BUILTIN_OBJS += $(OUTPUT)bench/futex-wake.o
>
> Looks good to me!

I'm little confused now.. is this on top of your earlier
patch or this change stands by itself..?

could any of you please resend v2 ? ;-)

thanks,
jirka
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/