s390: compile fixes

Submitted by Alexander Graf on Nov. 17, 2010, 12:01 p.m.

Details

Message ID 1289995264-4948-1-git-send-email-agraf@suse.de
State New
Headers show

Commit Message

Alexander Graf Nov. 17, 2010, 12:01 p.m.
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 <agraf@suse.de>
---
 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(-)

Comments

Stefan Weil Nov. 17, 2010, 5:07 p.m.
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 <agraf@suse.de>
> ---
> 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.
Alexander Graf Nov. 17, 2010, 5:10 p.m.
On 17.11.2010, at 18:07, Stefan Weil wrote:

> 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 <agraf@suse.de>
>> ---
>> 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.

It doesn't matter really, as the code only works on 64bit anyways.


Alex
Aurelien Jarno Dec. 25, 2010, 11:33 p.m.
On Wed, Nov 17, 2010 at 01:01:04PM +0100, Alexander Graf wrote:
> 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 <agraf@suse.de>

Thanks, applied.

> ---
>  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);
>          if ((i % 4) == 3) {
>              cpu_fprintf(f, "\n");
>          } else {
> -- 
> 1.6.0.2
> 
> 
>

Patch hide | download patch | download mbox

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);
         if ((i % 4) == 3) {
             cpu_fprintf(f, "\n");
         } else {