Re: [PATCH] [v3] x86/mm/mpx: Work around MPX erratum SKD046

2016-05-12 Thread Ingo Molnar
* Ingo Molnar wrote: > > Changes from v1: > > * Unconditionally enable workaround on all CPUs with MPX despite > >whether we know it to be affected or not > > * Add a pr_warn() when the workaround is active > > > > Changes from v2: > > * fix build breakage from #ifdefs in bug.h It now e

Re: [PATCH] [v3] x86/mm/mpx: Work around MPX erratum SKD046

2016-05-11 Thread Ingo Molnar
* Dave Hansen wrote: > > Changes from v1: > * Unconditionally enable workaround on all CPUs with MPX despite >whether we know it to be affected or not > * Add a pr_warn() when the workaround is active > > Changes from v2: > * fix build breakage from #ifdefs in bug.h > > -- > > From: D

Re: [PATCH v3] x86/mm/mpx: Work around MPX erratum SKD046

2016-05-05 Thread Dave Hansen
On 05/05/2016 12:01 AM, Ingo Molnar wrote: > I also slightly twiddled the wording, removed the linebreak from the > condition (it > was not over col80), removed whitespace noise and added your Signed-off-by, > which > I suppose you intended to add? I also tidied up the changelog a bit. > > Fin

Re: [PATCH v3] x86/mm/mpx: Work around MPX erratum SKD046

2016-05-05 Thread Ingo Molnar
* Ingo Molnar wrote: > > -#if defined(CONFIG_CPU_SUP_INTEL) && defined(CONFIG_X86_32) > > +#if defined(CONFIG_CPU_SUP_INTEL) > > +void check_mpx_erratum(struct cpuinfo_x86 *c); > > +#else > > +static inline void check_mpx_erratum(struct cpuinfo_x86 *c) {} > > +#if defined(CONFIG_X86_32) > > int