diff mbox

ARM: make OABI default to off

Message ID 20131106211352.GA24006@www.outflux.net (mailing list archive)
State New, archived
Headers show

Commit Message

Kees Cook Nov. 6, 2013, 9:13 p.m. UTC
Given recent discussions about the lack of OABI in the wild, switch
CONFIG_OABI_COMPAT to off-by-default to encourage more system builders
to avoid it.

Signed-off-by: Kees Cook <keescook@chromium.org>
---
 arch/arm/Kconfig |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Olof Johansson Nov. 6, 2013, 9:23 p.m. UTC | #1
On Wed, Nov 6, 2013 at 1:13 PM, Kees Cook <keescook@chromium.org> wrote:
> Given recent discussions about the lack of OABI in the wild, switch
> CONFIG_OABI_COMPAT to off-by-default to encourage more system builders
> to avoid it.
>
> Signed-off-by: Kees Cook <keescook@chromium.org>
> ---
>  arch/arm/Kconfig |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 1ad6fb6c094d..4ace7f33e8a1 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1728,7 +1728,7 @@ config AEABI
>  config OABI_COMPAT
>         bool "Allow old ABI binaries to run with this kernel (EXPERIMENTAL)"
>         depends on AEABI && !THUMB2_KERNEL
> -       default y
> +       default n

Nit: "default n" is the default default. So you can just delete the
line instead.


-Olof
Nicolas Pitre Nov. 8, 2013, 11:13 p.m. UTC | #2
On Wed, 6 Nov 2013, Kees Cook wrote:

> Given recent discussions about the lack of OABI in the wild, switch
> CONFIG_OABI_COMPAT to off-by-default to encourage more system builders
> to avoid it.
> 
> Signed-off-by: Kees Cook <keescook@chromium.org>

Acked-by: Nicolas Pitre <nico@linaro.org>

> ---
>  arch/arm/Kconfig |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index 1ad6fb6c094d..4ace7f33e8a1 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1728,7 +1728,7 @@ config AEABI
>  config OABI_COMPAT
>  	bool "Allow old ABI binaries to run with this kernel (EXPERIMENTAL)"
>  	depends on AEABI && !THUMB2_KERNEL
> -	default y
> +	default n
>  	help
>  	  This option preserves the old syscall interface along with the
>  	  new (ARM EABI) one. It also provides a compatibility layer to
> @@ -1740,7 +1740,7 @@ config OABI_COMPAT
>  	  can say N here. If this option is not selected and you attempt
>  	  to execute a legacy ABI binary then the result will be
>  	  UNPREDICTABLE (in fact it can be predicted that it won't work
> -	  at all). If in doubt say Y.
> +	  at all). If in doubt say N.
>  
>  config ARCH_HAS_HOLES_MEMORYMODEL
>  	bool
> -- 
> 1.7.9.5
> 
> 
> -- 
> Kees Cook
> Chrome OS Security
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>
Olof Johansson Nov. 8, 2013, 11:17 p.m. UTC | #3
On Fri, Nov 8, 2013 at 3:13 PM, Nicolas Pitre <nico@fluxnic.net> wrote:
> On Wed, 6 Nov 2013, Kees Cook wrote:
>
>> Given recent discussions about the lack of OABI in the wild, switch
>> CONFIG_OABI_COMPAT to off-by-default to encourage more system builders
>> to avoid it.
>>
>> Signed-off-by: Kees Cook <keescook@chromium.org>
>
> Acked-by: Nicolas Pitre <nico@linaro.org>

Yeah, sorry for the delay:

Acked-by: Olof Johansson <olof@lixom.net

(send to the patch tracker :)

-Olof
diff mbox

Patch

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index 1ad6fb6c094d..4ace7f33e8a1 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1728,7 +1728,7 @@  config AEABI
 config OABI_COMPAT
 	bool "Allow old ABI binaries to run with this kernel (EXPERIMENTAL)"
 	depends on AEABI && !THUMB2_KERNEL
-	default y
+	default n
 	help
 	  This option preserves the old syscall interface along with the
 	  new (ARM EABI) one. It also provides a compatibility layer to
@@ -1740,7 +1740,7 @@  config OABI_COMPAT
 	  can say N here. If this option is not selected and you attempt
 	  to execute a legacy ABI binary then the result will be
 	  UNPREDICTABLE (in fact it can be predicted that it won't work
-	  at all). If in doubt say Y.
+	  at all). If in doubt say N.
 
 config ARCH_HAS_HOLES_MEMORYMODEL
 	bool