Re: [PATCH 1/2] export.h: remove defined(__KERNEL__)

From: Masahiro Yamada
Date: Thu Sep 12 2019 - 07:49:20 EST


On Mon, Sep 9, 2019 at 7:53 PM Masahiro Yamada
<yamada.masahiro@xxxxxxxxxxxxx> wrote:
>
> This line was touched by commit f235541699bc ("export.h: allow for
> per-symbol configurable EXPORT_SYMBOL()"), but the commit log did
> not explain why.
>
> CONFIG_TRIM_UNUSED_KSYMS works for me without defined(__KERNEL__).
>
> Signed-off-by: Masahiro Yamada <yamada.masahiro@xxxxxxxxxxxxx>
> ---

Both applied to linux-kbuild.


> include/linux/export.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/export.h b/include/linux/export.h
> index fd8711ed9ac4..cdd98a0d918c 100644
> --- a/include/linux/export.h
> +++ b/include/linux/export.h
> @@ -20,7 +20,7 @@ extern struct module __this_module;
>
> #ifdef CONFIG_MODULES
>
> -#if defined(__KERNEL__) && !defined(__GENKSYMS__)
> +#if !defined(__GENKSYMS__)
> #ifdef CONFIG_MODVERSIONS
> /* Mark the CRC weak since genksyms apparently decides not to
> * generate a checksums for some symbols */
> --
> 2.17.1
>


--
Best Regards
Masahiro Yamada