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

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

[Darren Kenny]

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@oracle.com>
Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Acked-by: David Woodhouse <dwmw@amazon.co.uk>
---
 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

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@oracle.com>
>Reviewed-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
>Acked-by: David Woodhouse <dwmw@amazon.co.uk>
>---
>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
>