linux-kernel - Re: [PATCH 04/15] tools headers: Update linux/vdso.h and grab a copy of vdso/const.h

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]

Date:   Wed, 15 Apr 2020 10:10:58 +0100
From:   Vincenzo Frascino <vincenzo.frascino@....com>
To:     Arnaldo Carvalho de Melo <acme@...nel.org>,
        Ingo Molnar <mingo@...nel.org>,
        Thomas Gleixner <tglx@...utronix.de>
Cc:     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>
Subject: Re: [PATCH 04/15] tools headers: Update linux/vdso.h and grab a copy
 of vdso/const.h

On 4/14/20 5:48 PM, Arnaldo Carvalho de Melo wrote:
> From: Arnaldo Carvalho de Melo <acme@...hat.com>
> 
> To get in line with:
> 
>   8165b57bca21 ("linux/const.h: Extract common header for vDSO")
> 
> And silence this tools/perf/ build warning:
> 
>   Warning: Kernel ABI header at 'tools/include/linux/const.h' differs from latest version at 'include/linux/const.h'
>   diff -u tools/include/linux/const.h include/linux/const.h
> 
> Cc: Adrian Hunter <adrian.hunter@...el.com>
> Cc: Jiri Olsa <jolsa@...nel.org>
> Cc: Namhyung Kim <namhyung@...nel.org>
> Cc: Thomas Gleixner <tglx@...utronix.de>
> Cc: Vincenzo Frascino <vincenzo.frascino@....com>
> Signed-off-by: Arnaldo Carvalho de Melo <acme@...hat.com>

Acked-by: Vincenzo Frascino <vincenzo.frascino@....com>

> ---
>  tools/include/linux/const.h |  5 +----
>  tools/include/vdso/const.h  | 10 ++++++++++
>  tools/perf/check-headers.sh |  1 +
>  3 files changed, 12 insertions(+), 4 deletions(-)
>  create mode 100644 tools/include/vdso/const.h
> 
> diff --git a/tools/include/linux/const.h b/tools/include/linux/const.h
> index 7b55a55f5911..81b8aae5a855 100644
> --- a/tools/include/linux/const.h
> +++ b/tools/include/linux/const.h
> @@ -1,9 +1,6 @@
>  #ifndef _LINUX_CONST_H
>  #define _LINUX_CONST_H
>  
> -#include <uapi/linux/const.h>
> -
> -#define UL(x)		(_UL(x))
> -#define ULL(x)		(_ULL(x))
> +#include <vdso/const.h>
>  
>  #endif /* _LINUX_CONST_H */
> diff --git a/tools/include/vdso/const.h b/tools/include/vdso/const.h
> new file mode 100644
> index 000000000000..94b385ad438d
> --- /dev/null
> +++ b/tools/include/vdso/const.h
> @@ -0,0 +1,10 @@
> +/* SPDX-License-Identifier: GPL-2.0 */
> +#ifndef __VDSO_CONST_H
> +#define __VDSO_CONST_H
> +
> +#include <uapi/linux/const.h>
> +
> +#define UL(x)		(_UL(x))
> +#define ULL(x)		(_ULL(x))
> +
> +#endif /* __VDSO_CONST_H */
> diff --git a/tools/perf/check-headers.sh b/tools/perf/check-headers.sh
> index bfb21d049e6c..c905c683606a 100755
> --- a/tools/perf/check-headers.sh
> +++ b/tools/perf/check-headers.sh
> @@ -23,6 +23,7 @@ include/uapi/linux/vhost.h
>  include/uapi/sound/asound.h
>  include/linux/bits.h
>  include/linux/const.h
> +include/vdso/const.h
>  include/linux/hash.h
>  include/uapi/linux/hw_breakpoint.h
>  arch/x86/include/asm/disabled-features.h
> 

-- 
Regards,
Vincenzo