linux-kernel - Re: [PATCH] apm: Remove CONFIG_APM_REAL_MODE_POWER

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] [day] [month] [year] [list]

Message-ID: <20081120085043.GK21785@elte.hu>
Date:	Thu, 20 Nov 2008 09:50:43 +0100
From:	Ingo Molnar <mingo@...e.hu>
To:	Niels de Vos <niels.devos@...cor-nixdorf.com>
Cc:	sfr@...b.auug.org.au, linux-laptop@...r.kernel.org,
	linux-kernel@...r.kernel.org, Alan Cox <alan@...rguk.ukuu.org.uk>,
	Thomas Gleixner <tglx@...utronix.de>,
	"H. Peter Anvin" <hpa@...or.com>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] apm: Remove CONFIG_APM_REAL_MODE_POWER_OFF in favor of
	kernel parameter

* Niels de Vos <niels.devos@...cor-nixdorf.com> wrote:

> Remove CONFIG_APM_REAL_MODE_POWER_OFF like CONFIG_APM_POWER_OFF 
> which has been done for linux-2.2.14pre8 
> (http://lkml.org/lkml/1999/11/23/3).
> 
> Re-introducing CONFIG_APM_POWER_OFF got nack-ed. Stephen didn't bother
> to remove CONFIG_APM_REAL_MODE_POWER_OFF, let's get rid of it now.
> 	Reference: http://lkml.org/lkml/2008/5/7/97
> 
> Signed-off-by: Niels de Vos <niels.devos@...cor-nixdorf.com>
> 
> ---
>  arch/x86/Kconfig         |    7 -------
>  arch/x86/kernel/apm_32.c |    4 ----
>  2 files changed, 0 insertions(+), 11 deletions(-)
> 
> diff --git a/arch/x86/Kconfig b/arch/x86/Kconfig
> index 93224b5..d0b9ef7 100644
> --- a/arch/x86/Kconfig
> +++ b/arch/x86/Kconfig
> @@ -1629,13 +1629,6 @@ config APM_ALLOW_INTS
>  	  many of the newer IBM Thinkpads.  If you experience hangs when you
>  	  suspend, try setting this to Y.  Otherwise, say N.
>  
> -config APM_REAL_MODE_POWER_OFF
> -	bool "Use real mode APM BIOS call to power off"
> -	help
> -	  Use real mode APM BIOS calls to switch off the computer. This is
> -	  a work-around for a number of buggy BIOSes. Switch this option on if
> -	  your computer crashes instead of powering off properly.
> -

> -#ifdef CONFIG_APM_REAL_MODE_POWER_OFF
> -static int realmode_power_off = 1;
> -#else
>  static int realmode_power_off;
> -#endif

this might be an option for obsolete hardware, but still it has the 
potential to help people - and they could depend on it. The proper way 
to phase out the Kconfig feature would be to insert an obsoletion 
warning that triggers in some form when 
CONFIG_APM_REAL_MODE_POWER_OFF=y is enabled, and then remove the 
config option in 1-2 years. (and keep the boot option of course)

even better we should have a transparent kernel-parameters framework 
that would be a one-stop-shop for _both_ .config and boot-time flags. 
Something like:

  define_bootparam("realmode_power_off");

plus a way to define the help text - all in a single file and in a 
single place.

Which would automatically be turned into a kconfig option, a boot 
option and (optionally) a sysctl as well, during the kernel build.

We do have the CONFIG_CMDLINE facility in the latest kernels to inject 
arbitrary boot parameters during the kernel build, but that does not 
have any semantic dimensions and its opacity makes it fragile: there's 
no facility that warns if an option is typoed, if an option goes away 
or gets changed, there's no help text like we have when there's a 
Kconfig entry, etc.

	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/