linux-kernel - Re: [PATCH] locking: remove three unused Kconfig symbols

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: <20130408151609.GA13640@gmail.com>
Date:	Mon, 8 Apr 2013 17:16:09 +0200
From:	Ingo Molnar <mingo@...nel.org>
To:	Paul Bolle <pebolle@...cali.nl>
Cc:	Thomas Gleixner <tglx@...utronix.de>,
	Martin Schwidefsky <schwidefsky@...ibm.com>,
	Heiko Carstens <heiko.carstens@...ibm.com>,
	Ingo Molnar <mingo@...e.hu>,
	Peter Zijlstra <a.p.zijlstra@...llo.nl>, linux390@...ibm.com,
	linux-s390@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] locking: remove three unused Kconfig symbols


* Paul Bolle <pebolle@...cali.nl> wrote:

> The Kconfig symbols ARCH_INLINE_READ_UNLOCK_IRQ,
> ARCH_INLINE_SPIN_UNLOCK_IRQ, and ARCH_INLINE_WRITE_UNLOCK_IRQ were added
> in v2.6.33, but have never actually been used. It is safe to remove
> these.
> 
> Signed-off-by: Paul Bolle <pebolle@...cali.nl>
> ---
> Untested. These three symbols were added in commit
> 6beb000923882f6204ea2cfcd932e568e900803f ("locking: Make inlining
> decision Kconfig based").
> 
>  arch/s390/Kconfig    |  3 ---
>  kernel/Kconfig.locks | 10 ----------
>  2 files changed, 13 deletions(-)
> 
> diff --git a/arch/s390/Kconfig b/arch/s390/Kconfig
> index eb8fb62..8c544d2 100644
> --- a/arch/s390/Kconfig
> +++ b/arch/s390/Kconfig
> @@ -70,7 +70,6 @@ config S390
>  	select ARCH_INLINE_READ_TRYLOCK
>  	select ARCH_INLINE_READ_UNLOCK
>  	select ARCH_INLINE_READ_UNLOCK_BH
> -	select ARCH_INLINE_READ_UNLOCK_IRQ
>  	select ARCH_INLINE_READ_UNLOCK_IRQRESTORE
>  	select ARCH_INLINE_SPIN_LOCK
>  	select ARCH_INLINE_SPIN_LOCK_BH
> @@ -80,7 +79,6 @@ config S390
>  	select ARCH_INLINE_SPIN_TRYLOCK_BH
>  	select ARCH_INLINE_SPIN_UNLOCK
>  	select ARCH_INLINE_SPIN_UNLOCK_BH
> -	select ARCH_INLINE_SPIN_UNLOCK_IRQ
>  	select ARCH_INLINE_SPIN_UNLOCK_IRQRESTORE
>  	select ARCH_INLINE_WRITE_LOCK
>  	select ARCH_INLINE_WRITE_LOCK_BH
> @@ -89,7 +87,6 @@ config S390
>  	select ARCH_INLINE_WRITE_TRYLOCK
>  	select ARCH_INLINE_WRITE_UNLOCK
>  	select ARCH_INLINE_WRITE_UNLOCK_BH
> -	select ARCH_INLINE_WRITE_UNLOCK_IRQ
>  	select ARCH_INLINE_WRITE_UNLOCK_IRQRESTORE
>  	select ARCH_SAVE_PAGE_KEYS if HIBERNATION
>  	select ARCH_WANT_IPC_PARSE_VERSION
> diff --git a/kernel/Kconfig.locks b/kernel/Kconfig.locks
> index 44511d1..c438507 100644
> --- a/kernel/Kconfig.locks
> +++ b/kernel/Kconfig.locks
> @@ -25,9 +25,6 @@ config ARCH_INLINE_SPIN_UNLOCK
>  config ARCH_INLINE_SPIN_UNLOCK_BH
>  	bool
>  
> -config ARCH_INLINE_SPIN_UNLOCK_IRQ
> -	bool
> -
>  config ARCH_INLINE_SPIN_UNLOCK_IRQRESTORE
>  	bool
>  
> @@ -53,13 +50,9 @@ config ARCH_INLINE_READ_UNLOCK
>  config ARCH_INLINE_READ_UNLOCK_BH
>  	bool
>  
> -config ARCH_INLINE_READ_UNLOCK_IRQ
> -	bool
> -
>  config ARCH_INLINE_READ_UNLOCK_IRQRESTORE
>  	bool
>  
> -
>  config ARCH_INLINE_WRITE_TRYLOCK
>  	bool
>  
> @@ -81,9 +74,6 @@ config ARCH_INLINE_WRITE_UNLOCK
>  config ARCH_INLINE_WRITE_UNLOCK_BH
>  	bool
>  
> -config ARCH_INLINE_WRITE_UNLOCK_IRQ
> -	bool
> -
>  config ARCH_INLINE_WRITE_UNLOCK_IRQRESTORE
>  	bool

Erm, if you look at the code in question you'll see that they are unused I think 
due to a bug:

 config INLINE_READ_UNLOCK_IRQ
        def_bool y
        depends on !PREEMPT || ARCH_INLINE_READ_UNLOCK_BH

Shouldn't that depend on ARCH_INLINE_READ_UNLOCK_IRQ?

Similarly for the others.

Thanks,

	Ingo
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/