ACK/cmnt: [SRU][FOCAL][PATCH 1/1] kvm: svm: Update svm_xsaves_supported

[Kleber Souza]

I should have added "/cmnt" to the first "ACK" reply...

On 16.09.20 11:19, Kleber Souza wrote:
> On 15.09.20 22:53, William Breathitt Gray wrote:
>> From: Aaron Lewis <aaronlewis at google.co>
> 
> The BugLink generally is added here as the first line of the commit message:
> 
> BugLink: https://bugs.launchpad.net/bugs/1887490
>  
>> AMD CPUs now support XSAVES in a limited fashion (they require IA32_XSS
>> to be zero).
>>
>> AMD has no equivalent of Intel's "Enable XSAVES/XRSTORS" VM-execution
>> control. Instead, XSAVES is always available to the guest when supported
>> on the host.
>>
>> Reviewed-by: Jim Mattson <jmattson at google.com>
>> Signed-off-by: Aaron Lewis <aaronlewis at google.com>
>> Change-Id: I40dc2c682eb0d38c2208d95d5eb7bbb6c47f6317
>> Signed-off-by: Paolo Bonzini <pbonzini at redhat.com>
>> BugLink: https://bugs.launchpad.net/bugs/1887490
> 
> We should move the BugLink to the top ^.
> 
>> (cherry-picked from commit 52297436199dde85be557ee6bc779f5b96082f74)
> 
> The above line should follow the pattern added by the '-x' flag from
> 'git cherry-pick':
> 
> (cherry picked from commit ...)
> 
> This can all be fixed when applying the patch.
> 
> Otherwise the change looks good to me, clean cherry-pick:
> 
> Acked-by: Kleber Sacilotto de Souza <kleber.souza at canonical.com>
> 
> 
> Thanks. 
> 
>> Signed-off-by: William Breathitt Gray <william.gray at canonical.com>
>> ---
>>  arch/x86/kvm/svm.c | 2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/arch/x86/kvm/svm.c b/arch/x86/kvm/svm.c
>> index 3243a80ea32c..027004daec56 100644
>> --- a/arch/x86/kvm/svm.c
>> +++ b/arch/x86/kvm/svm.c
>> @@ -6019,7 +6019,7 @@ static bool svm_mpx_supported(void)
>>  
>>  static bool svm_xsaves_supported(void)
>>  {
>> -	return false;
>> +	return boot_cpu_has(X86_FEATURE_XSAVES);
>>  }
>>  
>>  static bool svm_umip_emulated(void)
>>
>