lkml.org 
[lkml]   [2014]   [Dec]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [GIT pull] x86 mpx support for 3.19

* Dave Hansen <dave.hansen@linux.intel.com> wrote:

> @@ -1575,6 +1571,27 @@ config X86_SMAP
>
> If unsure, say Y.
>
> +config X86_INTEL_MPX
> + prompt "Intel MPX (Memory Protection Extensions)" if EXPERT

I think the 'if EXPERT' needs to be dropped.

> + def_bool y
> + depends on CPU_SUP_INTEL

On the one hand, the 'def_bool y' might be acceptable, if we
think of MPX as X32 or SECCOMP: ABI extensions that are only
really useful if all distros enable it.

On the other hand, unlike x32 and seccomp, MPX increases data
structure size and adds a few instructions to common, non-MPX
code paths, so the cost isn't just kernel image size.

Linus, what's your preference?

> + Enabling this option will make the kernel larger and
> + slightly increase the size of some kernel data
> + structures.

And will add a few branches to critical code paths, right?

It would be nice to give some numeric data in such cases, by what
percentage does MPX support increase the x86_64 defconfig kernel
for example? By how much does it increase data structure size?
Make costs and benefits transparent and most people will chose
wisely, or at least well informed.

Thanks,

Ingo


\
 
 \ /
  Last update: 2014-12-11 08:01    [W:0.144 / U:0.108 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site