Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-12-10 Thread Pavel Machek
Hi!

>  Documentation/admin-guide/kernel-parameters.txt |   56 ++
>  Documentation/userspace-api/spec_ctrl.rst   |9 

Could we name this speculation.rst instead? _ is inconsistent, and
spec could be shorthand for other stuff, too...
Pavel

-- 
(english) http://www.livejournal.com/~pavelmachek
(cesky, pictures) 
http://atrey.karlin.mff.cuni.cz/~pavel/picture/horses/blog.html


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-29 Thread Tim Chen
On 11/28/2018 06:24 AM, Thomas Gleixner wrote:

> 
> I've integrated the latest review feedback and the change which plugs the
> TIF async update issue and pushed all of it out to:
> 
>git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/pti
> 
> For the stable 4.14.y and 4.19.y trees, I've collected the missing bits and
> pieces and uploaded tarballs which contain everything ready for consumption:
> 
>https://tglx.de/~tglx/patches-spec-4.14.y.tar.xz
> 
>   sha256 of patches-spec-4.14.y.tar:
>   3d2976ef06ab5556c1c6cba975b0c9390eb57f43c506fb7f8834bb484feb9b17
> 
>https://tglx.de/~tglx/patches-spec-4.19.y.tar.xz
> 
>   sha256 of patches-spec-4.19.y.tar:
>   b7666cf378ad63810a17e98a471aae81a49738c552dbe912aea49de83f8145cc
> 
> Thanks everyone for review, discussion, testing ... !

Big thanks to Thomas for getting all these changes merged.

Tim


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-29 Thread Tim Chen
On 11/28/2018 06:24 AM, Thomas Gleixner wrote:

> 
> I've integrated the latest review feedback and the change which plugs the
> TIF async update issue and pushed all of it out to:
> 
>git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/pti
> 
> For the stable 4.14.y and 4.19.y trees, I've collected the missing bits and
> pieces and uploaded tarballs which contain everything ready for consumption:
> 
>https://tglx.de/~tglx/patches-spec-4.14.y.tar.xz
> 
>   sha256 of patches-spec-4.14.y.tar:
>   3d2976ef06ab5556c1c6cba975b0c9390eb57f43c506fb7f8834bb484feb9b17
> 
>https://tglx.de/~tglx/patches-spec-4.19.y.tar.xz
> 
>   sha256 of patches-spec-4.19.y.tar:
>   b7666cf378ad63810a17e98a471aae81a49738c552dbe912aea49de83f8145cc
> 
> Thanks everyone for review, discussion, testing ... !

Big thanks to Thomas for getting all these changes merged.

Tim


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-28 Thread Thomas Gleixner
On Sun, 25 Nov 2018, Thomas Gleixner wrote:

> Thats hopefully the final version of this. Changes since V1:
> 
>   - Renamed the command line option and related code to spectre_v2_user= as
> suggested by Josh.
> 
>   - Thought more about the back to back optimization and finally left the
> IBPB code in switch_mm().
> 
> It still removes the ptrace check for the always IBPB case. That's
> substantial overhead for dubious value now that the default is
> conditional (prctl/seccomp) IBPB.
> 
>   - Added two options which allow conditional STIBP and IBPB always mode.
> 
>   - Addressed the review comments
> 
> Documentation is still work in progress. Thanks Andi for providing the
> first draft for it.
> 
> Still based on tip.git x86/pti as it has been discussed to remove the
> minimal RETPOLINE bandaid from stable kernels as well.

I've integrated the latest review feedback and the change which plugs the
TIF async update issue and pushed all of it out to:

   git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/pti

For the stable 4.14.y and 4.19.y trees, I've collected the missing bits and
pieces and uploaded tarballs which contain everything ready for consumption:

   https://tglx.de/~tglx/patches-spec-4.14.y.tar.xz

  sha256 of patches-spec-4.14.y.tar:
  3d2976ef06ab5556c1c6cba975b0c9390eb57f43c506fb7f8834bb484feb9b17

   https://tglx.de/~tglx/patches-spec-4.19.y.tar.xz

  sha256 of patches-spec-4.19.y.tar:
  b7666cf378ad63810a17e98a471aae81a49738c552dbe912aea49de83f8145cc

Thanks everyone for review, discussion, testing ... !

Thanks,

tglx


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-28 Thread Thomas Gleixner
On Sun, 25 Nov 2018, Thomas Gleixner wrote:

> Thats hopefully the final version of this. Changes since V1:
> 
>   - Renamed the command line option and related code to spectre_v2_user= as
> suggested by Josh.
> 
>   - Thought more about the back to back optimization and finally left the
> IBPB code in switch_mm().
> 
> It still removes the ptrace check for the always IBPB case. That's
> substantial overhead for dubious value now that the default is
> conditional (prctl/seccomp) IBPB.
> 
>   - Added two options which allow conditional STIBP and IBPB always mode.
> 
>   - Addressed the review comments
> 
> Documentation is still work in progress. Thanks Andi for providing the
> first draft for it.
> 
> Still based on tip.git x86/pti as it has been discussed to remove the
> minimal RETPOLINE bandaid from stable kernels as well.

I've integrated the latest review feedback and the change which plugs the
TIF async update issue and pushed all of it out to:

   git://git.kernel.org/pub/scm/linux/kernel/git/tip/tip.git x86/pti

For the stable 4.14.y and 4.19.y trees, I've collected the missing bits and
pieces and uploaded tarballs which contain everything ready for consumption:

   https://tglx.de/~tglx/patches-spec-4.14.y.tar.xz

  sha256 of patches-spec-4.14.y.tar:
  3d2976ef06ab5556c1c6cba975b0c9390eb57f43c506fb7f8834bb484feb9b17

   https://tglx.de/~tglx/patches-spec-4.19.y.tar.xz

  sha256 of patches-spec-4.19.y.tar:
  b7666cf378ad63810a17e98a471aae81a49738c552dbe912aea49de83f8145cc

Thanks everyone for review, discussion, testing ... !

Thanks,

tglx


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-26 Thread Ingo Molnar


* Thomas Gleixner  wrote:

> Thats hopefully the final version of this. Changes since V1:
> 
>   - Renamed the command line option and related code to spectre_v2_user= as
> suggested by Josh.
> 
>   - Thought more about the back to back optimization and finally left the
> IBPB code in switch_mm().
> 
> It still removes the ptrace check for the always IBPB case. That's
> substantial overhead for dubious value now that the default is
> conditional (prctl/seccomp) IBPB.
> 
>   - Added two options which allow conditional STIBP and IBPB always mode.
> 
>   - Addressed the review comments

With the typo review feedback outlined in the discussions:

Reviewed-by: Ingo Molnar 

Thanks,

Ingo


Re: [patch V2 00/28] x86/speculation: Remedy the STIBP/IBPB overhead

2018-11-26 Thread Ingo Molnar


* Thomas Gleixner  wrote:

> Thats hopefully the final version of this. Changes since V1:
> 
>   - Renamed the command line option and related code to spectre_v2_user= as
> suggested by Josh.
> 
>   - Thought more about the back to back optimization and finally left the
> IBPB code in switch_mm().
> 
> It still removes the ptrace check for the always IBPB case. That's
> substantial overhead for dubious value now that the default is
> conditional (prctl/seccomp) IBPB.
> 
>   - Added two options which allow conditional STIBP and IBPB always mode.
> 
>   - Addressed the review comments

With the typo review feedback outlined in the discussions:

Reviewed-by: Ingo Molnar 

Thanks,

Ingo