[PATCH v5 00/11] Support for launchSecurity type s390-pv

[Boris Fiuczynski]

On 7/21/21 12:45 PM, Boris Fiuczynski wrote:
> On 7/21/21 12:04 PM, Pavel Hrdina wrote:
>> On Fri, Jul 16, 2021 at 11:44:24AM +0200, Boris Fiuczynski wrote:
>>> This patch series introduces the launch security type s390-pv.
>>> Specifying s390-pv as launch security type in an s390 domain prepares 
>>> for
>>> running the guest in protected virtualization secure mode, also known as
>>> IBM Secure Execution.
>>>
>>> diff to v4:
>>>   - changed rng to do the verification for every launchSecurity type
>>>   - removed previously added XML fail tests
>>>   - added domain capability documentation
>>>
>>> diff to v3:
>>>   - rebased to current master
>>>   - moved virDomainSEVDef into a union
>>>   - improved XML formating for launchSecurity
>>>   - use a shared id on the qemu cmd line for confidential-guest-support
>>>   - added check for s390-pv host support into XML validation
>>>   - changed from ignoring to failing if launchSecuroty child elements 
>>> are provided for s390-pv
>>>   - reduced test to  a single failing test
>>>   - add availability of s390-pv in domain capabilities
>>>
>>> diff to v2:
>>>   - broke up previous patch one into three patches
>>>
>>> diff to v1:
>>>   - rebased to current master
>>>   - added verification check for confidential-guest-support capability
>>>
>>> Boris Fiuczynski (11):
>>>    schemas: Refactor launch security
>>>    conf: Rework SEV XML parse and format methods
>>>    qemu: Make KVMSupportsSecureGuest capability available
>>>    conf: Refactor launch security to allow more types
>>>    qemu: Add s390-pv-guest capability
>>>    conf: Add s390-pv as launch security type
>>>    docs: Add s390-pv documentation
>>>    conf: Add availability of s390-pv in domain capabilities
>>>    docs: Add s390-pv in domain capabilities documentation
>>>    qemu: Use common id lsec0 for launchSecurity
>>>    qemu: Fix error code for SEV launchSecurity unsupported
>>
>> I can make the changes before pushing if you are agree with the pointed
>> issues.
>>
>> Reviewed-by: Pavel Hrdina <phrdina at redhat.com>
>>
> Hi Pavel,
> thanks for your review.
> The agree with the changes you pointed out.
The => I
> Please also apply the fixup patch I sent for patch 9. Thanks
> 
> 


-- 
Mit freundlichen Grüßen/Kind regards
    Boris Fiuczynski

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Gregor Pillen
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294