Re: [Qemu-devel] [PATCH] s390: compile fixes

[Stefan Weil]

Am 17.11.2010 13:01, schrieb Alexander Graf:
> The s390 target doesn't compile out of the box anymore. This patch 
> fixes all
> the obvious glitches that got introduced in the last few weeks.
>
> Signed-off-by: Alexander Graf <address@hidden>
> ---
> hw/s390-virtio-bus.h | 2 ++
> hw/s390-virtio.c | 1 +
> target-s390x/kvm.c | 2 +-
> target-s390x/translate.c | 2 +-
> 4 files changed, 5 insertions(+), 2 deletions(-)
>
> diff --git a/hw/s390-virtio-bus.h b/hw/s390-virtio-bus.h
> index 41558c9..669b610 100644
> --- a/hw/s390-virtio-bus.h
> +++ b/hw/s390-virtio-bus.h
> @@ -17,6 +17,8 @@
> * License along with this library; if not, see 
> <http://www.gnu.org/licenses/>.
> */
>
> +#include "virtio-net.h"
> +
> #define VIRTIO_DEV_OFFS_TYPE 0 /* 8 bits */
> #define VIRTIO_DEV_OFFS_NUM_VQ 1 /* 8 bits */
> #define VIRTIO_DEV_OFFS_FEATURE_LEN 2 /* 8 bits */
> diff --git a/hw/s390-virtio.c b/hw/s390-virtio.c
> index e7aec14..f29b624 100644
> --- a/hw/s390-virtio.c
> +++ b/hw/s390-virtio.c
> @@ -19,6 +19,7 @@
>
> #include "hw.h"
> #include "block.h"
> +#include "blockdev.h"
> #include "sysemu.h"
> #include "net.h"
> #include "boards.h"
> diff --git a/target-s390x/kvm.c b/target-s390x/kvm.c
> index 9bf6abb..adf4a9e 100644
> --- a/target-s390x/kvm.c
> +++ b/target-s390x/kvm.c
> @@ -119,7 +119,7 @@ int kvm_arch_put_registers(CPUState *env, int level)
>
> int kvm_arch_get_registers(CPUState *env)
> {
> - uint32_t ret;
> + int ret;
> struct kvm_regs regs;
> int i;
>
> diff --git a/target-s390x/translate.c b/target-s390x/translate.c
> index 881d8c4..d33bfb1 100644
> --- a/target-s390x/translate.c
> +++ b/target-s390x/translate.c
> @@ -36,7 +36,7 @@ void cpu_dump_state(CPUState *env, FILE *f, 
> fprintf_function cpu_fprintf,
> }
> }
> for (i = 0; i < 16; i++) {
> - cpu_fprintf(f, "F%02d=%016lx", i, env->fregs[i]);
> + cpu_fprintf(f, "F%02d=%016lx", i, (long)env->fregs[i].i);

I have no s390 build environment, but from the code I assume
that using PRIx64 might be better than using a type cast here.

Regards,

Stefan

PS. Alexander, there are more patches with format checking
in the queue, perhaps you can test them on s390, too.
You can pull them from git://git.weilnetz.de/git/qemu for-blueswirl.