linux-kernel - Re: [PATCH 2/5] tools: Honour the O= flag when tool build called from a higher Makefile

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <87haph94os.fsf@sejong.aot.lge.com>
Date:	Fri, 26 Oct 2012 14:34:59 +0900
From:	Namhyung Kim <namhyung@...nel.org>
To:	David Howells <dhowells@...hat.com>
Cc:	mingo@...nel.org, tglx@...utronix.de, acme@...stprotocols.net,
	davem@...emloft.net, torvalds@...ux-foundation.org,
	paulus@...ba.org, linux-arch@...r.kernel.org,
	linux-kernel@...r.kernel.org, x86@...nel.org
Subject: Re: [PATCH 2/5] tools: Honour the O= flag when tool build called from a higher Makefile

On Fri, 19 Oct 2012 17:56:16 +0100, David Howells wrote:
> Honour the O= flag that was passed to a higher level Makefile and then passed
> down as part of a tool build.
>
> To make this work, the top-level Makefile passes the original O= flag and
> subdir=tools to the tools/Makefile, and that in turn passes
> subdir=$(O)/$(subdir)/foodir when building tool foo in directory
> $(O)/$(subdir)/foodir (where the intervening slashes aren't added if an
> element is missing).
>
> For example, take perf.  This is found in tools/perf/.  Assume we're building
> into directory ~/zebra/, so we pass O=~/zebra to make.  Dependening on where
> we run the build from, we see:
>
> 	make run in dir		$(OUTPUT) dir
> 	=======================	==================
> 	linux			~/zebra/tools/perf/
> 	linux/tools		~/zebra/perf/
> 	linux/tools/perf	~/zebra/
>
> and if O= is not set, we get:
>
> 	make run in dir		$(OUTPUT) dir
> 	=======================	==================
> 	linux			linux/tools/perf/
> 	linux/tools		linux/tools/perf/
> 	linux/tools/perf	linux/tools/perf/
>
> The output directories are created by the descend function if they don't
> already exist.

This is my test:

  namhyung@...ong:~$ cd project/linux
  namhyung@...ong:linux$ make O=~/build/zebra tools/perf
  /bin/sh: line 0: cd: /home/namhyung/build/zebra: No such file or directory
  Makefile:121: *** output directory "/home/namhyung/build/zebra" does not exist.  Stop.
  
  namhyung@...ong:tools$ mkdir ~/build/zebra
  namhyung@...ong:linux$ make O=~/build/zebra tools/perf
    HOSTCC  scripts/basic/fixdep
    GEN     /home/namhyung/build/zebra/Makefile
    HOSTCC  scripts/kconfig/conf.o
    HOSTCC  scripts/kconfig/zconf.tab.o
    HOSTLD  scripts/kconfig/conf
  scripts/kconfig/conf --silentoldconfig Kconfig
  ***
  *** Configuration file ".config" not found!
  ***
  *** Please run some configurator (e.g. "make oldconfig" or
  *** "make menuconfig" or "make xconfig").
  ***
  make[3]: *** [silentoldconfig] Error 1
  make[2]: *** [silentoldconfig] Error 2
      DESCEND perf
      MKDIR /home/namhyung/build/zebra/tools/perf/arch/
      MKDIR /home/namhyung/build/zebra/tools/perf/arch/x86/util/
      MKDIR /home/namhyung/build/zebra/tools/perf/bench/
      MKDIR /home/namhyung/build/zebra/tools/perf/scripts/perl/Perf-Trace-Util/
      MKDIR /home/namhyung/build/zebra/tools/perf/scripts/python/Perf-Trace-Util/
      MKDIR /home/namhyung/build/zebra/tools/perf/ui/
      MKDIR /home/namhyung/build/zebra/tools/perf/ui/browsers/
      MKDIR /home/namhyung/build/zebra/tools/perf/ui/gtk/
      MKDIR /home/namhyung/build/zebra/tools/perf/ui/stdio/
      MKDIR /home/namhyung/build/zebra/tools/perf/ui/tui/
      MKDIR /home/namhyung/build/zebra/tools/perf/util/
      MKDIR /home/namhyung/build/zebra/tools/perf/util/scripting-engines/
  PERF_VERSION = 3.7.rc2.1655.g54fa2b.dirty
      GEN /home/namhyung/build/zebra/tools/perf/common-cmds.h
      * new build flags or prefix
      CC /home/namhyung/build/zebra/tools/perf/perf.o
      ...

This looks ok but it'd be better if we can skip the config check when
building tools IMHO.


  namhyung@...ong:linux cd tools
  
  namhyung@...ong:tools$ make O=~/build/zebra perf
      DESCEND perf
      ...
      * new build flags or prefix
      CC /home/namhyung/build/zebra/perf.o
      ...

This looks not good as it doesn't build perf into
~/build/zebra/perf/perf.o.

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