mbox series

[SRU,o/n:linux-azure,0/1,v2] Revert fix for "Kernel fails to enable XSAVE when running in a “v5” AMD SEV-SNP VM"

Message ID 20240906174326.773072-1-john.cabaj@canonical.com
Headers show
Series Revert fix for "Kernel fails to enable XSAVE when running in a “v5” AMD SEV-SNP VM" | expand

Message

John Cabaj Sept. 6, 2024, 5:43 p.m. UTC
BugLink: https://bugs.launchpad.net/bugs/2079841

SRU Justification

[Impact]

* LP #2069204 disabled CET Shadow Stack on SEV-SNP due to CPUID emulation bug not being fixed in VMs until October 2024.
* This issue is now fixed, so the patch can be reverted.

[Fix]

* Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on SEV-SNP due to a paravisor bug"

[Test Plan]

* Compile tested
* Boot tested

[Regression potential]

* Reverting back to previous state, so regression very unlikely.
* Changes isolated, minimal regression risk

[Other info]

* SF #00387447

John Cabaj (1):
  Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on
    SEV-SNP due to a paravisor bug"

 arch/x86/kernel/cpu/mshyperv.c | 21 ---------------------
 1 file changed, 21 deletions(-)

Comments

Thibault Ferrante Sept. 6, 2024, 8:04 p.m. UTC | #1
Acked-by: Thibault Ferrante <thibault.ferrante@canonical.com>


On 06-09-2024 19:43, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2079841
> 
> SRU Justification
> 
> [Impact]
> 
> * LP #2069204 disabled CET Shadow Stack on SEV-SNP due to CPUID emulation bug not being fixed in VMs until October 2024.
> * This issue is now fixed, so the patch can be reverted.
> 
> [Fix]
> 
> * Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on SEV-SNP due to a paravisor bug"
> 
> [Test Plan]
> 
> * Compile tested
> * Boot tested
> 
> [Regression potential]
> 
> * Reverting back to previous state, so regression very unlikely.
> * Changes isolated, minimal regression risk
> 
> [Other info]
> 
> * SF #00387447
> 
> John Cabaj (1):
>    Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on
>      SEV-SNP due to a paravisor bug"
> 
>   arch/x86/kernel/cpu/mshyperv.c | 21 ---------------------
>   1 file changed, 21 deletions(-)
>
Magali Lemes Sept. 12, 2024, 6:21 p.m. UTC | #2
On 06/09/2024 14:43, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2079841
> 
> SRU Justification
> 
> [Impact]
> 
> * LP #2069204 disabled CET Shadow Stack on SEV-SNP due to CPUID emulation bug not being fixed in VMs until October 2024.
> * This issue is now fixed, so the patch can be reverted.
> 
> [Fix]
> 
> * Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on SEV-SNP due to a paravisor bug"
> 
> [Test Plan]
> 
> * Compile tested
> * Boot tested
> 
> [Regression potential]
> 
> * Reverting back to previous state, so regression very unlikely.
> * Changes isolated, minimal regression risk
> 
> [Other info]
> 
> * SF #00387447
> 

Acked-by: Magali Lemes <magali.lemes@canonical.com>

> John Cabaj (1):
>    Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on
>      SEV-SNP due to a paravisor bug"
> 
>   arch/x86/kernel/cpu/mshyperv.c | 21 ---------------------
>   1 file changed, 21 deletions(-)
>
John Cabaj Sept. 12, 2024, 8:01 p.m. UTC | #3
On 9/6/24 12:43 PM, John Cabaj wrote:
> BugLink: https://bugs.launchpad.net/bugs/2079841
> 
> SRU Justification
> 
> [Impact]
> 
> * LP #2069204 disabled CET Shadow Stack on SEV-SNP due to CPUID emulation bug not being fixed in VMs until October 2024.
> * This issue is now fixed, so the patch can be reverted.
> 
> [Fix]
> 
> * Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on SEV-SNP due to a paravisor bug"
> 
> [Test Plan]
> 
> * Compile tested
> * Boot tested
> 
> [Regression potential]
> 
> * Reverting back to previous state, so regression very unlikely.
> * Changes isolated, minimal regression risk
> 
> [Other info]
> 
> * SF #00387447
> 
> John Cabaj (1):
>    Revert "UBUNTU: SAUCE: x86/hyperv: temporarily disable CET SS on
>      SEV-SNP due to a paravisor bug"
> 
>   arch/x86/kernel/cpu/mshyperv.c | 21 ---------------------
>   1 file changed, 21 deletions(-)
> 

Applied to noble:linux-azure and oracular:linux-azure master-next branches.


Thanks,
John