[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <96ec5885-6f6a-288b-475d-99556113f95e@arm.com>
Date: Mon, 20 Apr 2020 12:19:36 +0100
From: Vincenzo Frascino <vincenzo.frascino@....com>
To: Arnaldo Carvalho de Melo <arnaldo.melo@...il.com>
Cc: Ingo Molnar <mingo@...nel.org>,
Thomas Gleixner <tglx@...utronix.de>,
Jiri Olsa <jolsa@...nel.org>,
Namhyung Kim <namhyung@...nel.org>,
Clark Williams <williams@...hat.com>,
linux-kernel@...r.kernel.org, linux-perf-users@...r.kernel.org,
Arnaldo Carvalho de Melo <acme@...hat.com>,
Adrian Hunter <adrian.hunter@...el.com>,
Rikard Falkeborn <rikard.falkeborn@...il.com>
Subject: Re: [PATCH 15/15] tools headers: Synchronize linux/bits.h with the
kernel sources
On 4/16/20 4:54 PM, Arnaldo Carvalho de Melo wrote:
> Em Wed, Apr 15, 2020 at 10:14:06AM +0100, Vincenzo Frascino escreveu:
>> On 4/14/20 5:48 PM, Arnaldo Carvalho de Melo wrote:
>>> From: Arnaldo Carvalho de Melo <acme@...hat.com>
>>>
>>> To pick up the changes in these csets:
>>>
>>> 295bcca84916 ("linux/bits.h: add compile time sanity check of GENMASK inputs")
>>> 3945ff37d2f4 ("linux/bits.h: Extract common header for vDSO")
>>>
>>> To address this tools/perf build warning:
>>>
>>> Warning: Kernel ABI header at 'tools/include/linux/bits.h' differs from latest version at 'include/linux/bits.h'
>>> diff -u tools/include/linux/bits.h include/linux/bits.h
>>>
>>> This clashes with usage of userspace's static_assert(), that, at least
>>> on glibc, is guarded by a ifnded/endif pair, do the same to our copy of
>>> build_bug.h and avoid that diff in check_headers.sh so that we continue
>>> checking for drifts with the kernel sources master copy.
>>>
>>> This will all be tested with the set of build containers that includes
>>> uCLibc, musl libc, lots of glibc versions in lots of distros and cross
>>> build environments.
>>>
>>> The tools/objtool, tools/bpf, etc were tested as well.
>>>
>>
>> Acked-by: Vincenzo Frascino <vincenzo.frascino@....com> # vDSO parts
>
> Hey, thanks a lot for checking it! :-)
>
No issue :-) Thank you for updating the tools headers!
[...]
--
Regards,
Vincenzo
Powered by blists - more mailing lists