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]
Message-ID: <87ee4w1x95.fsf@email.froward.int.ebiederm.org>
Date:   Mon, 24 Jan 2022 18:20:54 -0600
From:   "Eric W. Biederman" <ebiederm@...ssion.com>
To:     Randy Dunlap <rdunlap@...radead.org>
Cc:     linux-kernel@...r.kernel.org,
        Andrew Morton <akpm@...ux-foundation.org>,
        linux-ia64@...r.kernel.org, Tony Luck <tony.luck@...el.com>,
        kernel test robot <lkp@...el.com>,
        Christoph Hellwig <hch@...radead.org>
Subject: Re: [PATCH -next v2] ia64: make IA64_MCA_RECOVERY bool instead of
 tristate

Randy Dunlap <rdunlap@...radead.org> writes:

> In linux-next, IA64_MCA_RECOVERY uses the (new) function make_task_dead(),
> which is not exported for use by modules. Instead of exporting it for
> one user, convert IA64_MCA_RECOVERY to be a bool Kconfig symbol.
>
> In a config file from "kernel test robot <lkp@...el.com>" for a
> different problem, this linker error was exposed when
> CONFIG_IA64_MCA_RECOVERY=m.
>
> Fixes this build error:
>
> ERROR: modpost: "make_task_dead" [arch/ia64/kernel/mca_recovery.ko] undefined!

Reviewed-by: "Eric W. Biederman" <ebiederm@...ssion.com>

I looked and ia64_unreg_MCA_extension has no synchronization so I don't
think it has ever been safe to this code when built as a module.


> Fixes: 0e25498f8cd4 ("exit: Add and use make_task_dead.")
> Signed-off-by: Randy Dunlap <rdunlap@...radead.org>
> Cc: "Eric W. Biederman" <ebiederm@...ssion.com>
> Cc: Andrew Morton <akpm@...ux-foundation.org>
> Cc: linux-ia64@...r.kernel.org
> Cc: Tony Luck <tony.luck@...el.com>
> Cc: kernel test robot <lkp@...el.com>
> Suggested-by: Christoph Hellwig <hch@...radead.org>
> ---
> v1 was [PATCH -next] exit: export make_task_dead().
> Christoph suggests and prefers that IA64 MCA recovery code just be
> forced to be builtin if it is enabled.
>
>  arch/ia64/Kconfig |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> --- linux-next-20220121.orig/arch/ia64/Kconfig
> +++ linux-next-20220121/arch/ia64/Kconfig
> @@ -318,7 +318,7 @@ config ARCH_PROC_KCORE_TEXT
>  	depends on PROC_KCORE
>  
>  config IA64_MCA_RECOVERY
> -	tristate "MCA recovery from errors other than TLB."
> +	bool "MCA recovery from errors other than TLB."
>  
>  config IA64_PALINFO
>  	tristate "/proc/pal support"

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ