Re: [PATCH v2] Correct use of IBRS_ATT name, should be IBRS_ALL

[Darren Kenny]

Please ignore this, didn't realise it had been already applied - guess I
looked in the wrong place.

Thanks,

Darren.

On Mon, Feb 05, 2018 at 11:02:20AM +0000, Darren Kenny wrote:
> Fixes a comment in commit 117cc7a908c83697b0b737d15ae1eb5943afe35b
> ("x86/retpoline: Fill return stack buffer on vmexit")
>
> While the original name was "IBRS All The Time" (IBRS_ATT),
> the publicly documented name is IBRS_ALL, we should be using that.
>
> Signed-off-by: Darren Kenny <darren.kenny@xxxxxxxxxx>
> Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@xxxxxxxxxx>
> Acked-by: David Woodhouse <dwmw@xxxxxxxxxxxx>
> ---
> arch/x86/include/asm/nospec-branch.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/x86/include/asm/nospec-branch.h b/arch/x86/include/asm/nospec-branch.h
> index 4ad4108..5b778d2 100644
> --- a/arch/x86/include/asm/nospec-branch.h
> +++ b/arch/x86/include/asm/nospec-branch.h
> @@ -201,7 +201,7 @@ extern char __indirect_thunk_end[];
> * On VMEXIT we must ensure that no RSB predictions learned in the guest
> * can be followed in the host, by overwriting the RSB completely. Both
> * retpoline and IBRS mitigations for Spectre v2 need this; only on future
> - * CPUs with IBRS_ATT *might* it be avoided.
> + * CPUs with IBRS_ALL *might* it be avoided.
> */
> static inline void vmexit_fill_RSB(void)
> {
> --
> 2.9.5