Messages in this thread | | | Date | Thu, 11 Dec 2014 07:19:35 +0100 | From | Ingo Molnar <> | Subject | Re: [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
| |