| 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
| ||
|
Message-ID: <CAP-5=fUx-_AhaJaOTYRmxHWDqAX+R9U8rx6jZBHdDF+vdWSS0A@mail.gmail.com>
Date: Wed, 3 Feb 2021 17:21:35 -0800
From: Ian Rogers <irogers@...gle.com>
To: Arnaldo Carvalho de Melo <acme@...nel.org>
Cc: Peter Zijlstra <peterz@...radead.org>,
Ingo Molnar <mingo@...hat.com>,
Mark Rutland <mark.rutland@....com>,
Alexander Shishkin <alexander.shishkin@...ux.intel.com>,
Jiri Olsa <jolsa@...hat.com>,
Namhyung Kim <namhyung@...nel.org>,
LKML <linux-kernel@...r.kernel.org>,
Stephane Eranian <eranian@...gle.com>
Subject: Re: [PATCH 2/2] perf parse-events: Break out tracepoint and printing.
On Wed, Feb 3, 2021 at 4:58 PM Arnaldo Carvalho de Melo <acme@...nel.org> wrote:
>
> Em Wed, Feb 03, 2021 at 05:50:24PM -0300, Arnaldo Carvalho de Melo escreveu:
> > Em Wed, Feb 03, 2021 at 10:38:41AM -0800, Ian Rogers escreveu:
> > > On Wed, Feb 3, 2021 at 8:21 AM Arnaldo Carvalho de Melo <acme@...nel.org>
> > > wrote:
> > >
> > > > Em Tue, Feb 02, 2021 at 09:26:59PM -0800, Ian Rogers escreveu:
> > > > > Move print_*_events functions out of parse-events.c into a new
> > > > > print-events.c. Move tracepoint code into tracepoint.c or
> > > > > trace-event-info.c (sole user). This reduces the dependencies of
> > > > > parse-events.c and makes it more amenable to being a library in the
> > > > > future.
> > > > > Remove some unnecessary definitions from parse-events.h. Fix a
> > > > > checkpatch.pl warning on using unsigned rather than unsigned int.
> > > >
> > > > Thanks, applied.
> > > >
> > >
> > > Wow, cool :-) I wasn't sure about the macros in the headers. Are you
> > > staging these patches before pushing to your perf/core or using
> > > tmp.perf/core ? I know you mentioned a different approach. When I look at:
> > >
> > > https://git.kernel.org/pub/scm/linux/kernel/git/acme/linux.git/
> > >
> > > I don't see anything newer than 9 days.
> >
> > Right, I'm collecting some more patches, will do build tests and push,
> > till then, lemme push tmp.perf/core with what I have.
>
> Strange, some intermediary versions of Alpine Linux are complaining, I'll continue tomorrow morning :-\
>
> [perfbuilder@...e ~]$ time dm
> Wed Feb 3 09:32:47 PM -03 2021
> # export PERF_TARBALL=http://192.168.86.5/perf/perf-5.11.0-rc6.tar.xz
> # dm
> 1 72.90 alpine:3.4 : Ok gcc (Alpine 5.3.0) 5.3.0, clang version 3.8.0 (tags/RELEASE_380/final)
> 2 74.62 alpine:3.5 : Ok gcc (Alpine 6.2.1) 6.2.1 20160822, clang version 3.8.1 (tags/RELEASE_381/final)
> 3 76.69 alpine:3.6 : Ok gcc (Alpine 6.3.0) 6.3.0, clang version 4.0.0 (tags/RELEASE_400/final)
> 4 84.43 alpine:3.7 : Ok gcc (Alpine 6.4.0) 6.4.0, Alpine clang version 5.0.0 (tags/RELEASE_500/final) (based on LLVM 5.0.0)
> 5 84.17 alpine:3.8 : Ok gcc (Alpine 6.4.0) 6.4.0, Alpine clang version 5.0.1 (tags/RELEASE_501/final) (based on LLVM 5.0.1)
> 6 16.71 alpine:3.9 : FAIL gcc (Alpine 8.3.0) 8.3.0, Alpine clang version 5.0.1 (tags/RELEASE_502/final) (based on LLVM 5.0.1)
> from util/print-events.c:10:
> /usr/include/linux/swab.h:161:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __swab16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:161:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab16p'
> static __always_inline __u16 __swab16p(const __u16 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:174:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __swab32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:174:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab32p'
> static __always_inline __u32 __swab32p(const __u32 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:187:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __swab64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:187:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab64p'
> static __always_inline __u64 __swab64p(const __u64 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:242:23: error: expected ';' before 'void'
> static __always_inline void __swab32s(__u32 *p)
> ^~~~~
> ;
> /usr/include/linux/swab.h:255:23: error: expected ';' before 'void'
> static __always_inline void __swab64s(__u64 *p)
> ^~~~~
> ;
> In file included from /usr/include/asm/byteorder.h:5,
> from /git/linux/tools/include/uapi/linux/perf_event.h:20,
> from util/print-events.c:10:
> /usr/include/linux/byteorder/little_endian.h:44:8: error: unknown type name '__always_inline'
> static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:44:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le64p'
> static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le64_to_cpup'
> static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:8: error: unknown type name '__always_inline'
> static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le32p'
> static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le32_to_cpup'
> static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:8: error: unknown type name '__always_inline'
> static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le16p'
> static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le16_to_cpup'
> static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:8: error: unknown type name '__always_inline'
> static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be64p'
> static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be64_to_cpup'
> static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:8: error: unknown type name '__always_inline'
> static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be32p'
> static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be32_to_cpup'
> static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:8: error: unknown type name '__always_inline'
> static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be16p'
> static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be16_to_cpup'
> static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> ^~~~~~~~~~~~~~
> make[3]: *** [/git/linux/tools/build/Makefile.build:139: util] Error 2
> 7 16.31 alpine:3.10 : FAIL gcc (Alpine 8.3.0) 8.3.0, Alpine clang version 8.0.0 (tags/RELEASE_800/final) (based on LLVM 8.0.0)
> from util/print-events.c:10:
> /usr/include/linux/swab.h:161:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __swab16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:161:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab16p'
> static __always_inline __u16 __swab16p(const __u16 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:174:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __swab32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:174:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab32p'
> static __always_inline __u32 __swab32p(const __u32 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:187:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __swab64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:187:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab64p'
> static __always_inline __u64 __swab64p(const __u64 *p)
> ^~~~~~~~~
> /usr/include/linux/swab.h:242:23: error: expected ';' before 'void'
> static __always_inline void __swab32s(__u32 *p)
> ^~~~~
> ;
> /usr/include/linux/swab.h:255:23: error: expected ';' before 'void'
> static __always_inline void __swab64s(__u64 *p)
> ^~~~~
> ;
> In file included from /usr/include/asm/byteorder.h:5,
> from /git/linux/tools/include/uapi/linux/perf_event.h:20,
> from util/print-events.c:10:
> /usr/include/linux/byteorder/little_endian.h:44:8: error: unknown type name '__always_inline'
> static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:44:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le64p'
> static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le64_to_cpup'
> static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:8: error: unknown type name '__always_inline'
> static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le32p'
> static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le32_to_cpup'
> static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:8: error: unknown type name '__always_inline'
> static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le16p'
> static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le16_to_cpup'
> static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:8: error: unknown type name '__always_inline'
> static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be64p'
> static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:8: error: unknown type name '__always_inline'
> static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be64_to_cpup'
> static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:8: error: unknown type name '__always_inline'
> static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be32p'
> static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:8: error: unknown type name '__always_inline'
> static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be32_to_cpup'
> static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:8: error: unknown type name '__always_inline'
> static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be16p'
> static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:8: error: unknown type name '__always_inline'
> static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be16_to_cpup'
> static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> ^~~~~~~~~~~~~~
> make[3]: *** [/git/linux/tools/build/Makefile.build:139: util] Error 2
> 8 18.59 alpine:3.11 : FAIL gcc (Alpine 9.3.0) 9.3.0, Alpine clang version 9.0.0 (https://git.alpinelinux.org/aports f7f0d2c2b8bcd6a5843401a9a702029556492689) (based on LLVM 9.0.0)
> from util/print-events.c:10:
> /usr/include/linux/swab.h:161:8: error: unknown type name '__always_inline'
> 161 | static __always_inline __u16 __swab16p(const __u16 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:161:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab16p'
> 161 | static __always_inline __u16 __swab16p(const __u16 *p)
> | ^~~~~~~~~
> /usr/include/linux/swab.h:174:8: error: unknown type name '__always_inline'
> 174 | static __always_inline __u32 __swab32p(const __u32 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:174:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab32p'
> 174 | static __always_inline __u32 __swab32p(const __u32 *p)
> | ^~~~~~~~~
> /usr/include/linux/swab.h:187:8: error: unknown type name '__always_inline'
> 187 | static __always_inline __u64 __swab64p(const __u64 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/swab.h:187:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__swab64p'
> 187 | static __always_inline __u64 __swab64p(const __u64 *p)
> | ^~~~~~~~~
> /usr/include/linux/swab.h:242:23: error: expected ';' before 'void'
> 242 | static __always_inline void __swab32s(__u32 *p)
> | ^~~~~
> | ;
> /usr/include/linux/swab.h:255:23: error: expected ';' before 'void'
> 255 | static __always_inline void __swab64s(__u64 *p)
> | ^~~~~
> | ;
> In file included from /usr/include/asm/byteorder.h:5,
> from /git/linux/tools/include/uapi/linux/perf_event.h:20,
> from util/print-events.c:10:
> /usr/include/linux/byteorder/little_endian.h:44:8: error: unknown type name '__always_inline'
> 44 | static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:44:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le64p'
> 44 | static __always_inline __le64 __cpu_to_le64p(const __u64 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:8: error: unknown type name '__always_inline'
> 48 | static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:48:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le64_to_cpup'
> 48 | static __always_inline __u64 __le64_to_cpup(const __le64 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:8: error: unknown type name '__always_inline'
> 52 | static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:52:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le32p'
> 52 | static __always_inline __le32 __cpu_to_le32p(const __u32 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:8: error: unknown type name '__always_inline'
> 56 | static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:56:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le32_to_cpup'
> 56 | static __always_inline __u32 __le32_to_cpup(const __le32 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:8: error: unknown type name '__always_inline'
> 60 | static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:60:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_le16p'
> 60 | static __always_inline __le16 __cpu_to_le16p(const __u16 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:8: error: unknown type name '__always_inline'
> 64 | static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:64:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__le16_to_cpup'
> 64 | static __always_inline __u16 __le16_to_cpup(const __le16 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:8: error: unknown type name '__always_inline'
> 68 | static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:68:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be64p'
> 68 | static __always_inline __be64 __cpu_to_be64p(const __u64 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:8: error: unknown type name '__always_inline'
> 72 | static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:72:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be64_to_cpup'
> 72 | static __always_inline __u64 __be64_to_cpup(const __be64 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:8: error: unknown type name '__always_inline'
> 76 | static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:76:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be32p'
> 76 | static __always_inline __be32 __cpu_to_be32p(const __u32 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:8: error: unknown type name '__always_inline'
> 80 | static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:80:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be32_to_cpup'
> 80 | static __always_inline __u32 __be32_to_cpup(const __be32 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:8: error: unknown type name '__always_inline'
> 84 | static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:84:31: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__cpu_to_be16p'
> 84 | static __always_inline __be16 __cpu_to_be16p(const __u16 *p)
> | ^~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:8: error: unknown type name '__always_inline'
> 88 | static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> | ^~~~~~~~~~~~~~~
> /usr/include/linux/byteorder/little_endian.h:88:30: error: expected '=', ',', ';', 'asm' or '__attribute__' before '__be16_to_cpup'
> 88 | static __always_inline __u16 __be16_to_cpup(const __be16 *p)
> | ^~~~~~~~~~~~~~
> make[3]: *** [/git/linux/tools/build/Makefile.build:139: util] Error 2
> 9 117.31 alpine:3.12 : Ok gcc (Alpine 9.3.0) 9.3.0, Alpine clang version 10.0.0 (https://gitlab.alpinelinux.org/alpine/aports.git 7445adce501f8473efdb93b17b5eaf2f1445ed4c)
> 10 124.70 alpine:edge : Ok gcc (Alpine 10.2.0) 10.2.0, Alpine clang version 10.0.1
> 11 72.02 alt:p8 : Ok x86_64-alt-linux-gcc (GCC) 5.3.1 20151207 (ALT p8 5.3.1-alt3.M80P.1), clang version 3.8.0 (tags/RELEASE_380/final)
> 12 87.30 alt:p9 : Ok x86_64-alt-linux-gcc (GCC) 8.4.1 20200305 (ALT p9 8.4.1-alt0.p9.1), clang version 10.0.0
> 13 87.33 alt:sisyphus : Ok x86_64-alt-linux-gcc (GCC) 10.2.1 20201125 (ALT Sisyphus 10.2.1-alt2), clang version 10.0.1
> 14 70.23 amazonlinux:1 : Ok gcc (GCC) 7.2.1 20170915 (Red Hat 7.2.1-2), clang version 3.6.2 (tags/RELEASE_362/final)
> 15 104.56 amazonlinux:2 : Ok gcc (GCC) 7.3.1 20180712 (Red Hat 7.3.1-12), clang version 7.0.1 (Amazon Linux 2 7.0.1-1.amzn2.0.2)
> 16 23.52 android-ndk:r12b-arm : Ok arm-linux-androideabi-gcc (GCC) 4.9.x 20150123 (prerelease)
> 17 21.96 android-ndk:r15c-arm : Ok arm-linux-androideabi-gcc (GCC) 4.9.x 20150123 (prerelease)
> 18 27.58 centos:6 : Ok gcc (GCC) 4.4.7 20120313 (Red Hat 4.4.7-23)
> 19 33.35 centos:7 : Ok gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-44)
> 20 98.93 centos:8 : Ok gcc (GCC) 8.3.1 20191121 (Red Hat 8.3.1-5), clang version 10.0.1 (Red Hat 10.0.1-1.module_el8.3.0+467+cb298d5b)
> 21 69.01 clearlinux:latest : Ok gcc (Clear Linux OS for Intel Architecture) 10.2.1 20201217 releases/gcc-10.2.0-643-g7cbb07d2fc, clang version 10.0.1
> 22 79.89 debian:8 : Ok gcc (Debian 4.9.2-10+deb8u2) 4.9.2, Debian clang version 3.5.0-10 (tags/RELEASE_350/final) (based on LLVM 3.5.0)
>
Ugh, sorry for that. I'm reminded of:
https://lore.kernel.org/lkml/20210113223609.3358812-1-irogers@google.com/
On a similar subject, a header file weirdness/rant I have is:
tools/include/uapi/linux/bpf.h
It includes:
#include <linux/types.h>
#include <linux/bpf_common.h>
The include paths for perf are:
-I${SRC_DIR}/tools/lib/perf/include
-I${SRC_DIR}/tools/perf/util/include
-I${SRC_DIR}/tools/perf/arch/x86/include
-I${SRC_DIR}/tools/include/
-I${SRC_DIR}/tools/arch/x86/include/uapi
-I${SRC_DIR}/tools/include/uapi
-I${SRC_DIR}/tools/arch/x86/include/
-I${SRC_DIR}/tools/arch/x86/
-I${DST_DIR}/util -I${DST_DIR}
-I${SRC_DIR}/tools/perf/util
-I${SRC_DIR}/tools/perf
-I${SRC_DIR}/tools/lib/
Note that tools/include is before tools/include/uapi and that
linux/types.h is included with the directory name. From this the
search order is given in:
https://gcc.gnu.org/onlinedocs/gcc/Directory-Options.html
"Directories specified with -I options are scanned in left-to-right order."
So when perf includes linux/bpf.h the types.h is coming from
tools/include/linux/types.h rather than
tools/include/uapi/linux/types.h that normal logic would expect
(prefer headers in the same directory). bpf_common.h is coming from
the uapi directory.
Thanks,
Ian
Powered by blists - more mailing lists