On 25.01.2018 18:06, Christian Borntraeger wrote:
> From: Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx>
>
> If the AIV facility is available, a GISA will be used to manage emulated
> adapter interrupts.
>
> Signed-off-by: Michael Mueller <mimu@xxxxxxxxxxxxxxxxxx>
> Reviewed-by: Halil Pasic <pasic@xxxxxxxxxxxxxxxxxx>
> Reviewed-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
> Signed-off-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
> ---
> arch/s390/kvm/interrupt.c | 8 +++++++-
> 1 file changed, 7 insertions(+), 1 deletion(-)
>
> diff --git a/arch/s390/kvm/interrupt.c b/arch/s390/kvm/interrupt.c
> index 1a561ad..ec4b7bf 100644
> --- a/arch/s390/kvm/interrupt.c
> +++ b/arch/s390/kvm/interrupt.c
> @@ -2840,7 +2840,13 @@ void kvm_s390_gisa_clear(struct kvm *kvm)
>
> void kvm_s390_gisa_init(struct kvm *kvm)
> {
> - /* not implemented yet */
> + if (!css_general_characteristics.aiv)
> + kvm->arch.gisa = NULL;
> + else {
Hmm, isn't it NULL as default?
if (css_general_characteristics.aiv) {
...
Anyhow,
Reviewed-by: David Hildenbrand <david@xxxxxxxxxx>
> + kvm->arch.gisa = &kvm->arch.sie_page2->gisa;
> + VM_EVENT(kvm, 3, "gisa 0x%pK initialized", kvm->arch.gisa);
> + kvm_s390_gisa_clear(kvm);
> + }
> }
>
> void kvm_s390_gisa_destroy(struct kvm *kvm)
>
--
Thanks,
David / dhildenb
