qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH CFT v3 00/50] NEED_CPU_H / cpu.h / hw/hw.h clean


From: Cornelia Huck
Subject: Re: [Qemu-devel] [PATCH CFT v3 00/50] NEED_CPU_H / cpu.h / hw/hw.h cleanups
Date: Wed, 18 May 2016 15:46:05 +0200

On Wed, 18 May 2016 13:11:08 +0200
Paolo Bonzini <address@hidden> 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 <address@hidden>
> 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 <address@hidden>
> ---
>  target-s390x/cpu.h     | 15 ---------------
>  target-s390x/machine.c | 10 ++++++++++
>  2 files changed, 10 insertions(+), 15 deletions(-)

Acked-by: Cornelia Huck <address@hidden>

Should I look at the other patches touching s390x as well, or will you
resend?




reply via email to

[Prev in Thread] Current Thread [Next in Thread]