From: Cornelia Huck <cohuck@redhat.com>
To: Christian Borntraeger <borntraeger@de.ibm.com>
Cc: KVM <kvm@vger.kernel.org>,
linux-s390 <linux-s390@vger.kernel.org>,
David Hildenbrand <david@redhat.com>
Subject: Re: [PATCH 3/5] KVM: s390: vsie: use common code functions for pinning
Date: Wed, 8 Nov 2017 10:07:53 +0100 [thread overview]
Message-ID: <20171108100753.7afc9368.cohuck@redhat.com> (raw)
In-Reply-To: <20171108084143.78654-4-borntraeger@de.ibm.com>
On Wed, 8 Nov 2017 09:41:41 +0100
Christian Borntraeger <borntraeger@de.ibm.com> wrote:
> From: David Hildenbrand <david@redhat.com>
>
> We will not see -ENOMEM (gfn_to_hva() will return KVM_ERR_PTR_BAD_PAGE
> for all errors). So we can also get rid of special handling in the
> callers of pin_guest_page() and always assume that it is a g2 error.
>
> As also kvm_s390_inject_program_int() should never fail, we can
> simplify pin_scb(), too.
>
> Signed-off-by: David Hildenbrand <david@redhat.com>
> Message-Id: <20170901151143.22714-1-david@redhat.com>
> Signed-off-by: Christian Borntraeger <borntraeger@de.ibm.com>
> ---
> arch/s390/kvm/vsie.c | 50 +++++++++++++++++-------------------------------
> include/linux/kvm_host.h | 1 +
> virt/kvm/kvm_main.c | 4 ++--
> 3 files changed, 21 insertions(+), 34 deletions(-)
Acked-by: Cornelia Huck <cohuck@redhat.com>
next prev parent reply other threads:[~2017-11-08 9:07 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-08 8:41 [PATCH 0/5] KVM: s390: patches for 4.15 Christian Borntraeger
2017-11-08 8:41 ` [PATCH 1/5] KVM: s390: document memory ordering for kvm_s390_vcpu_wakeup Christian Borntraeger
2017-11-08 9:05 ` Cornelia Huck
2017-11-08 9:37 ` David Hildenbrand
2017-11-08 8:41 ` [PATCH 2/5] KVM: s390: SIE considerations for AP Queue virtualization Christian Borntraeger
2017-11-08 9:06 ` Cornelia Huck
2017-11-08 9:29 ` David Hildenbrand
2017-11-09 8:59 ` Christian Borntraeger
2017-11-08 8:41 ` [PATCH 3/5] KVM: s390: vsie: use common code functions for pinning Christian Borntraeger
2017-11-08 9:07 ` Cornelia Huck [this message]
2017-11-08 8:41 ` [PATCH 4/5] KVM: s390: abstract conversion between isc and enum irq_types Christian Borntraeger
2017-11-08 9:11 ` Cornelia Huck
2017-11-08 9:41 ` David Hildenbrand
2017-11-08 8:41 ` [PATCH 5/5] KVM: s390: clear_io_irq() requests are not expected for adapter interrupts Christian Borntraeger
2017-11-08 9:19 ` Cornelia Huck
2017-11-08 11:04 ` Christian Borntraeger
2017-11-08 11:09 ` Cornelia Huck
2017-11-08 12:14 ` Michael Mueller
2017-11-13 12:25 ` Cornelia Huck
2017-11-08 11:10 ` [PATCH 0/5] KVM: s390: patches for 4.15 Cornelia Huck
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20171108100753.7afc9368.cohuck@redhat.com \
--to=cohuck@redhat.com \
--cc=borntraeger@de.ibm.com \
--cc=david@redhat.com \
--cc=kvm@vger.kernel.org \
--cc=linux-s390@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.