On Thu, 31 Aug 2017 13:01:39 +0200
Christian Borntraeger <borntraeger@xxxxxxxxxx> wrote:
> Reviewed-by: Christian Borntraeger <borntraeger@xxxxxxxxxx>
>
> I will add some patch description like:
>
> We already know that the machine check information was part of the vsie_page.
Shorten that to "The machine check information is part of the
vsie_page."?
In any case,
Reviewed-by: Cornelia Huck <cohuck@xxxxxxxxxx>
>
> applied.
>
> On 08/30/2017 06:06 PM, David Hildenbrand wrote:
> > Signed-off-by: David Hildenbrand <david@xxxxxxxxxx>
> > ---
> > arch/s390/kvm/vsie.c | 6 +-----
> > 1 file changed, 1 insertion(+), 5 deletions(-)
> >
> > diff --git a/arch/s390/kvm/vsie.c b/arch/s390/kvm/vsie.c
> > index 715c19c45d9a..b5eec30eb37d 100644
> > --- a/arch/s390/kvm/vsie.c
> > +++ b/arch/s390/kvm/vsie.c
> > @@ -806,8 +806,6 @@ static int do_vsie_run(struct kvm_vcpu *vcpu, struct vsie_page *vsie_page)
> > {
> > struct kvm_s390_sie_block *scb_s = &vsie_page->scb_s;
> > struct kvm_s390_sie_block *scb_o = vsie_page->scb_o;
> > - struct mcck_volatile_info *mcck_info;
> > - struct sie_page *sie_page;
> > int rc;
> >
> > handle_last_fault(vcpu, vsie_page);
> > @@ -831,9 +829,7 @@ static int do_vsie_run(struct kvm_vcpu *vcpu, struct vsie_page *vsie_page)
> >
> > if (rc == -EINTR) {
> > VCPU_EVENT(vcpu, 3, "%s", "machine check");
> > - sie_page = container_of(scb_s, struct sie_page, sie_block);
> > - mcck_info = &sie_page->mcck_info;
> > - kvm_s390_reinject_machine_check(vcpu, mcck_info);
> > + kvm_s390_reinject_machine_check(vcpu, &vsie_page->mcck_info);
> > return 0;
> > }
> >
>
