Started as a part of the fix for
ae596615d93d ("virt: sev-guest: Reduce the scope of SNP command mutex")

these are the remainder of followup fixes/improvements as
the SNP Guest Request code evolved enough to make it a bit hard to
follow.


This is based on sha1
6c201ee95a30 Ingo Molnar "Merge branch into tip/master: 'x86/sgx'".

Please comment. Thanks.



Alexey Kardashevskiy (4):
  virt: sev-guest: Contain snp_guest_request_ioctl in sev-guest
  x86/sev: Allocate request in TSC_INFO_REQ on stack
  x86/sev: Document requirement for linear mapping of Guest Request
    buffers
  x86/sev: Drop unnecessary parameter in snp_issue_guest_request

 arch/x86/include/asm/sev.h              | 10 ++-
 arch/x86/coco/sev/core.c                | 73 +++++++++-----------
 drivers/virt/coco/sev-guest/sev-guest.c |  9 ++-
 3 files changed, 44 insertions(+), 48 deletions(-)

-- 
2.49.0

On 5/5/25 09:12, Alexey Kardashevskiy wrote:
> Started as a part of the fix for
> ae596615d93d ("virt: sev-guest: Reduce the scope of SNP command mutex")
> 
> these are the remainder of followup fixes/improvements as
> the SNP Guest Request code evolved enough to make it a bit hard to
> follow.
> 
> 
> This is based on sha1
> 6c201ee95a30 Ingo Molnar "Merge branch into tip/master: 'x86/sgx'".
> 
> Please comment. Thanks.
> 
> 
> 
> Alexey Kardashevskiy (4):
>   virt: sev-guest: Contain snp_guest_request_ioctl in sev-guest
>   x86/sev: Allocate request in TSC_INFO_REQ on stack
>   x86/sev: Document requirement for linear mapping of Guest Request
>     buffers
>   x86/sev: Drop unnecessary parameter in snp_issue_guest_request
> 
>  arch/x86/include/asm/sev.h              | 10 ++-
>  arch/x86/coco/sev/core.c                | 73 +++++++++-----------
>  drivers/virt/coco/sev-guest/sev-guest.c |  9 ++-
>  3 files changed, 44 insertions(+), 48 deletions(-)

For the series:

Reviewed-by: Tom Lendacky <thomas.lendacky@amd.com>

>

On 7/5/25 04:55, Tom Lendacky wrote:
> On 5/5/25 09:12, Alexey Kardashevskiy wrote:
>> Started as a part of the fix for
>> ae596615d93d ("virt: sev-guest: Reduce the scope of SNP command mutex")
>>
>> these are the remainder of followup fixes/improvements as
>> the SNP Guest Request code evolved enough to make it a bit hard to
>> follow.
>>
>>
>> This is based on sha1
>> 6c201ee95a30 Ingo Molnar "Merge branch into tip/master: 'x86/sgx'".
>>
>> Please comment. Thanks.
>>
>>
>>
>> Alexey Kardashevskiy (4):
>>    virt: sev-guest: Contain snp_guest_request_ioctl in sev-guest
>>    x86/sev: Allocate request in TSC_INFO_REQ on stack
>>    x86/sev: Document requirement for linear mapping of Guest Request
>>      buffers
>>    x86/sev: Drop unnecessary parameter in snp_issue_guest_request
>>
>>   arch/x86/include/asm/sev.h              | 10 ++-
>>   arch/x86/coco/sev/core.c                | 73 +++++++++-----------
>>   drivers/virt/coco/sev-guest/sev-guest.c |  9 ++-
>>   3 files changed, 44 insertions(+), 48 deletions(-)
> 
> For the series:
> 
> Reviewed-by: Tom Lendacky <thomas.lendacky@amd.com>


Ping?

> 
>>

-- 
Alexey

On Thu, Jun 05, 2025 at 12:40:15PM +1000, Alexey Kardashevskiy wrote:
> Ping?

Yes, please send a new version collecting all feedback and rebased on
6.16-rc1 after it releases next week.

Thx.