Re: [PATCH v5 01/13] KVM: s390: Interface to test whether APXA installed

[Harald Freudenberger]

On 16.05.2018 12:45, Tony Krowiak wrote:
> On 05/16/2018 06:21 AM, Cornelia Huck wrote:
>> On Mon, 7 May 2018 11:11:40 -0400
>> Tony Krowiak <akrowiak@xxxxxxxxxxxxxxxxxx> wrote:
>>
>>> Relocates an existing static function that tests whether
>>> the AP extended addressing facility (APXA) is installed on
>>> the linux host. The primary reason for relocating this
>>> function is because a new compilation unit (arch/s390/kvm/kvm-ap.c)
>>> is being created to contain all of the interfaces and logic
>>> for configuring an AP matrix for a KVM guest. Some of its
>>> functions will also need to determine whether APXA is installed,
>>> so, let's go ahead and relocate this static function as a
>>> public interface in kvm-ap.c.
>>>
>>> Notes:
>>> ----
>>> 1. The interface to determine whether APXA is installed on the linux
>>> ÂÂÂ host the information returned from the AP Query Configuration
>>> ÂÂÂ Information (QCI) function. This function will not be available
>>> ÂÂÂ if the AP instructions are not installed on the linux host, so a check
>>> ÂÂÂ will be included to verify that.
>>>
>>> 2. Currently, the AP bus interfaces accessing the AP instructions will
>>> ÂÂÂ not be accessible if CONFIG_ZCRYPT=n, so the relevant code will be
>>> ÂÂÂ temporarily contained in the new arch/s390/kvm/kvm-ap.c file until
>>> ÂÂÂ the patch(es) to statically build the required AP bus interfaces are
>>> ÂÂÂ available.
>> Any ETA for those interfaces? Would be nice if we could avoid
>> introducing temporary interfaces (but I'm certainly not opposing this
>> patch).
>
> I'll check with the developer.
The proposal is out on the internal mailing list. I'd like to release it (internal) tomorrow or start
next week. Already talked with Christian about this because we need to align it somehow with
kvm and s390 kernel development.

regards
Harald Freudenberger
>
>>
>>> Signed-off-by: Tony Krowiak <akrowiak@xxxxxxxxxxxxxxxxxx>
>>> ---
>>> Â MAINTAINERSÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ |ÂÂÂ 1 +
>>> Â arch/s390/include/asm/kvm-ap.h |ÂÂ 60 +++++++++++++++++++++++++++++
>>> Â arch/s390/kvm/MakefileÂÂÂÂÂÂÂÂ |ÂÂÂ 2 +-
>>> Â arch/s390/kvm/kvm-ap.cÂÂÂÂÂÂÂÂ |ÂÂ 83 ++++++++++++++++++++++++++++++++++++++++
>>> Â arch/s390/kvm/kvm-s390.cÂÂÂÂÂÂ |ÂÂ 42 +-------------------
>>> Â 5 files changed, 147 insertions(+), 41 deletions(-)
>>> Â create mode 100644 arch/s390/include/asm/kvm-ap.h
>>> Â create mode 100644 arch/s390/kvm/kvm-ap.c
>>>
>>> diff --git a/MAINTAINERS b/MAINTAINERS
>>> index eab763f..224e97b 100644
>>> --- a/MAINTAINERS
>>> +++ b/MAINTAINERS
>>> @@ -7792,6 +7792,7 @@ M:ÂÂÂ Christian Borntraeger <borntraeger@xxxxxxxxxx>
>>> Â M:ÂÂÂ Janosch Frank <frankja@xxxxxxxxxxxxx>
>>> Â R:ÂÂÂ David Hildenbrand <david@xxxxxxxxxx>
>>> Â R:ÂÂÂ Cornelia Huck <cohuck@xxxxxxxxxx>
>>> +R:ÂÂÂ Tony Krowiak <akrowiak@xxxxxxxxxxxxxxxxxx>
>> Don't you want to drop the 'vnet' from your address, as the vnet-less
>> form seems to be the one that will continue working from what I've
>> heard?
>>
>>> Â L:ÂÂÂ linux-s390@xxxxxxxxxxxxxxx
>>> Â W:ÂÂÂ http://www.ibm.com/developerworks/linux/linux390/
>>> Â T:ÂÂÂ git git://git.kernel.org/pub/scm/linux/kernel/git/kvms390/linux.git
>> No objection against this patch, although I still hope it will not be
>> needed :)
>>
>
> -- 
> To unsubscribe from this list: send the line "unsubscribe linux-s390" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>