On Wed, 18 May 2016 13:11:08 +0200 Paolo Bonzini <pbonz...@redhat.com> wrote:
> On 18/05/2016 10:59, Cornelia Huck wrote: > >> That's on one of my s390x systems; on another one (where I don't test), > >> it builds. But something's wrong :) > > > > Ah, that's because the functioning one used --disable-kvm. > > > > Should the call to kvm_check_extension() be moved to target-s390x/kvm.c? > > It can also be moved to machine.c too... Your current branch now works fine. > Since I don't know exactly > which patch causes it, I can put this at the beginning: > > From 2f86d8c15e0b0514116d36727ecaf3f479f2af33 Mon Sep 17 00:00:00 2001 > From: Paolo Bonzini <pbonz...@redhat.com> > Date: Wed, 18 May 2016 13:08:02 +0200 > Subject: [PATCH] s390x: move vregs_needed to machine.c > > It is only needed in one file, move it there. > > Signed-off-by: Paolo Bonzini <pbonz...@redhat.com> > --- > target-s390x/cpu.h | 15 --------------- > target-s390x/machine.c | 10 ++++++++++ > 2 files changed, 10 insertions(+), 15 deletions(-) Acked-by: Cornelia Huck <cornelia.h...@de.ibm.com> Should I look at the other patches touching s390x as well, or will you resend?
