linux-kernel - Re: [RFC PATCH v4 03/29] perf tools: Move linux/{list.h,poison.h} to tools/include

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20150527153024.GA24859@kernel.org>
Date:	Wed, 27 May 2015 12:30:24 -0300
From:	Arnaldo Carvalho de Melo <acme@...nel.org>
To:	Wang Nan <wangnan0@...wei.com>
Cc:	paulus@...ba.org, a.p.zijlstra@...llo.nl, mingo@...hat.com,
	namhyung@...nel.org, jolsa@...nel.org, dsahern@...il.com,
	daniel@...earbox.net, brendan.d.gregg@...il.com,
	masami.hiramatsu.pt@...achi.com, lizefan@...wei.com,
	linux-kernel@...r.kernel.org, pi3orama@....com
Subject: Re: [RFC PATCH v4 03/29] perf tools: Move linux/{list.h,poison.h} to
 tools/include

Em Wed, May 27, 2015 at 10:21:09AM -0300, Arnaldo Carvalho de Melo escreveu:
> Em Wed, May 27, 2015 at 10:15:12AM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Wed, May 27, 2015 at 05:19:38AM +0000, Wang Nan escreveu:
> > > This patch moves list.h from tools/perf/util/include/linux/list.h
> > > to tools/include/linux/list.h to enable other libraries use macros in
> 
> > I'll apply this, but when doing these moves, one has to worry that these
> > build targets continue working:
>  
> > [acme@...andy linux]$ make help | grep perf
> >   perf-tar-src-pkg    - Build perf-4.1.0-rc2.tar source tarball
> >   perf-targz-src-pkg  - Build perf-4.1.0-rc2.tar.gz source tarball
> 
> <SNIP>
>  
> > This, BTW, would have been caught if you had run:
> > 
> >   make -C tools/perf build-test
> > 
> > One of the last tests will do this packaging and building, alerting you that
> > something is missing.
> > 
> > I am doing this for the two patches in this series that moves things to tools,
> 
> So, to summarize this further, and to run just this 'tarpkg' test, to speed up
> testing for this specific case:

And it explodes on RHEL6.6, I'll have to test this further...

  MKDIR    /tmp/build/perf/util/
  CC       /tmp/build/perf/util/values.o
In file included from
/home/acme/git/linux/include/uapi/linux/kernel.h:4,
                 from util/util.h:73,
                 from util/values.c:3:
/home/acme/git/linux/include/uapi/linux/sysinfo.h:8: error: expected
specifier-qualifier-list before ‘__kernel_long_t’
cc1: warnings being treated as errors
In file included from util/values.c:3:
util/util.h: In function ‘path__join’:
util/util.h:306: error: implicit declaration of function ‘scnprintf’
util/util.h:306: error: nested extern declaration of ‘scnprintf’
make[3]: *** [/tmp/build/perf/util/values.o] Error 1
make[2]: *** [util] Error 2
make[1]: *** [/tmp/build/perf/libperf-in.o] Error 2
make[1]: *** Waiting for unfinished jobs....
  PERF_VERSION = 4.1.rc2.ga82d24e
make: *** [install-bin] Error 2
make: Leaving directory `/home/acme/git/linux/tools/perf'
[acme@...dy linux]$ 


- Arnaldo
--
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/