From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752962AbcANJ4i (ORCPT ); Thu, 14 Jan 2016 04:56:38 -0500 Received: from szxga01-in.huawei.com ([58.251.152.64]:18054 "EHLO szxga01-in.huawei.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751360AbcANJ4f (ORCPT ); Thu, 14 Jan 2016 04:56:35 -0500 Message-ID: <569770B6.4030803@huawei.com> Date: Thu, 14 Jan 2016 17:56:06 +0800 From: "Wangnan (F)" User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0 MIME-Version: 1.0 To: Jiri Olsa CC: , , , , Arnaldo Carvalho de Melo , Jiri Olsa , Namhyung Kim Subject: Re: [PATCH 2/9] perf build: Pass O option to Makefile.perf in build-test References: <1452687442-6186-1-git-send-email-wangnan0@huawei.com> <1452687442-6186-3-git-send-email-wangnan0@huawei.com> <20160114093258.GA8715@krava.brq.redhat.com> In-Reply-To: <20160114093258.GA8715@krava.brq.redhat.com> Content-Type: text/plain; charset="utf-8"; format=flowed Content-Transfer-Encoding: 7bit X-Originating-IP: [10.111.66.109] X-CFilter-Loop: Reflected X-Mirapoint-Virus-RAPID-Raw: score=unknown(0), refid=str=0001.0A020205.569770C1.0173,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0, ip=0.0.0.0, so=2013-06-18 04:22:30, dmn=2013-03-21 17:37:32 X-Mirapoint-Loop-Id: 8ff24f9df7dc47ca61240847c20e2464 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 2016/1/14 17:32, Jiri Olsa wrote: > On Wed, Jan 13, 2016 at 12:17:15PM +0000, Wang Nan wrote: >> Doesn't like tools/perf/Makefile, tools/perf/Makefile.perf obey 'O' >> option when it is passed through cmdline only, because of code in >> tools/scripts/Makefile.include: >> >> ifneq ($(O),) >> ifeq ($(origin O), command line) >> ... >> ABSOLUTE_O := $(shell cd $(O) ; pwd) >> OUTPUT := $(ABSOLUTE_O)/$(if $(subdir),$(subdir)/) >> endif >> endif >> >> This patch passes 'O' to Makefile.perf through cmdline explicitly >> to make it follow O variable during build-test. >> >> Signed-off-by: Wang Nan >> Cc: Arnaldo Carvalho de Melo >> Cc: Jiri Olsa >> Cc: Namhyung Kim >> --- >> tools/perf/tests/make | 12 ++++++++++-- >> 1 file changed, 10 insertions(+), 2 deletions(-) >> >> diff --git a/tools/perf/tests/make b/tools/perf/tests/make >> index c0ee679..14d7b8d 100644 >> --- a/tools/perf/tests/make >> +++ b/tools/perf/tests/make >> @@ -5,7 +5,7 @@ ifeq ($(MAKECMDGOALS),) >> # no target specified, trigger the whole suite >> all: >> @echo "Testing Makefile"; $(MAKE) -sf tests/make MK=Makefile >> - @echo "Testing Makefile.perf"; $(MAKE) -sf tests/make MK=Makefile.perf SET_PARALLEL=1 >> + @echo "Testing Makefile.perf"; $(MAKE) -sf tests/make MK=Makefile.perf SET_PARALLEL=1 SET_O=1 >> else >> # run only specific test over 'Makefile' >> %: >> @@ -13,6 +13,14 @@ else >> endif >> else >> PERF := . >> +O_OPT := >> + >> +ifneq ($(O),) >> + FULL_O := $(shell readlink -f $(O) || echo $(O)) >> + ifeq ($(SET_O),1) >> + O_OPT := 'O=$(FULL_O)' >> + endif >> +endif >> >> PARALLEL_OPT= >> ifeq ($(SET_PARALLEL),1) >> @@ -261,7 +269,7 @@ clean := @(cd $(PERF); make -s -f $(MK) clean >/dev/null) >> $(run): >> $(call clean) >> @TMP_DEST=$$(mktemp -d); \ >> - cmd="cd $(PERF) && make -f $(MK) $(PARALLEL_OPT) DESTDIR=$$TMP_DEST $($@)"; \ >> + cmd="cd $(PERF) && make -f $(MK) $(PARALLEL_OPT) $(O_OPT) DESTDIR=$$TMP_DEST $($@)"; \ > hum, but this set is to test make without O=... so why would you set it? > run_O is the target for O=... tests I have strong motivation to avoid polluting source directory. I have different platforms and sometime I want to build them (cross-compile) parallelly. I use yocto for this. Yocto appends 'O' automatically. This is good for building, but when I use the framework for build-test I have to manually adjust the generated scripts (and it would be regenerated again in next run) to make it work, and also I'm unable to test them in parallel. For people who don't care about this, $(O_OPT) is empty so he/she still tests without 'O=...' case. Thank you.