[Qemu-devel] buildbot failure in qemu on default_i386_debian_5_0

[Qemu-devel] buildbot failure in qemu on default_i386_debian_5_0

[qemu]

The Buildbot has detected a new failure on builder default_i386_debian_5_0 while building qemu.
Full details are available at:
 http://buildbot.b1-systems.de/qemu/builders/default_i386_debian_5_0/builds/163

Buildbot URL: http://buildbot.b1-systems.de/qemu/

Buildslave for this Build: yuzuki

Build Reason: The Nightly scheduler named 'nightly_default' triggered this build
Build Source Stamp: [branch master] HEAD
Blamelist: 

BUILD FAILED: failed compile

sincerely,
 -The Buildbot

Re: [Qemu-devel] buildbot failure in qemu on default_i386_debian_5_0

[Stefan Hajnoczi]

On Fri, Jul 22, 2011 at 12:05 AM,  <qemu@buildbot.b1-systems.de> wrote:
> The Buildbot has detected a new failure on builder default_i386_debian_5_0 while building qemu.
> Full details are available at:
>  http://buildbot.b1-systems.de/qemu/builders/default_i386_debian_5_0/builds/163

Does anyone have a patch for this?  Yesterday someone mentioned this
known build issue on IRC, I think.

Stefan

[Qemu-devel] [PATCH] target-sparc: Fix compiler errors (format strings)

[Stefan Weil]

This change is needed because commit 06e12b65
now uses an unsigned long long value
(uint64_t && unsigned long long => unsigned long long).

Cc: Tsuneo Saito <tsnsaito@gmail.com>
Cc: Blue Swirl <blauwirbel@gmail.com>
Signed-off-by: Stefan Weil <weil@mail.berlios.de>
---
 target-sparc/helper.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/target-sparc/helper.c b/target-sparc/helper.c
index acc07f5..efab885 100644
--- a/target-sparc/helper.c
+++ b/target-sparc/helper.c
@@ -700,7 +700,7 @@ void dump_mmu(FILE *f, fprintf_function cpu_fprintf, CPUState *env)
                 break;
             }
             if (TTE_IS_VALID(env->dtlb[i].tte)) {
-                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %" PRIx64
+                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %llx"
                                ", %s, %s, %s, %s, ctx %" PRId64 " %s\n",
                                i,
                                env->dtlb[i].tag & (uint64_t)~0x1fffULL,
@@ -737,7 +737,7 @@ void dump_mmu(FILE *f, fprintf_function cpu_fprintf, CPUState *env)
                 break;
             }
             if (TTE_IS_VALID(env->itlb[i].tte)) {
-                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %" PRIx64
+                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %llx"
                                ", %s, %s, %s, ctx %" PRId64 " %s\n",
                                i,
                                env->itlb[i].tag & (uint64_t)~0x1fffULL,
-- 
1.7.2.5

Re: [Qemu-devel] [PATCH] target-sparc: Fix compiler errors (format strings)

[tsnsaito]

At Fri, 22 Jul 2011 07:44:27 +0200,
Stefan Weil wrote:

> This change is needed because commit 06e12b65
> now uses an unsigned long long value
> (uint64_t && unsigned long long => unsigned long long).
> 
> Cc: Tsuneo Saito <tsnsaito@gmail.com>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Signed-off-by: Stefan Weil <weil@mail.berlios.de>

I'm sorry, my patch has broken the build.
Your fix looks good to me.

---- 
Tsuneo Saito <tsnsaito@gmail.com>

Re: [Qemu-devel] [PATCH] target-sparc: Fix compiler errors (format strings)

[Blue Swirl]

Thanks, applied.

I also didn't notice the build breakage but it looks like I also ran
my tests with some old version, sorry.

On Fri, Jul 22, 2011 at 8:44 AM, Stefan Weil <weil@mail.berlios.de> wrote:
> This change is needed because commit 06e12b65
> now uses an unsigned long long value
> (uint64_t && unsigned long long => unsigned long long).
>
> Cc: Tsuneo Saito <tsnsaito@gmail.com>
> Cc: Blue Swirl <blauwirbel@gmail.com>
> Signed-off-by: Stefan Weil <weil@mail.berlios.de>
> ---
>  target-sparc/helper.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/target-sparc/helper.c b/target-sparc/helper.c
> index acc07f5..efab885 100644
> --- a/target-sparc/helper.c
> +++ b/target-sparc/helper.c
> @@ -700,7 +700,7 @@ void dump_mmu(FILE *f, fprintf_function cpu_fprintf, CPUState *env)
>                 break;
>             }
>             if (TTE_IS_VALID(env->dtlb[i].tte)) {
> -                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %" PRIx64
> +                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %llx"
>                                ", %s, %s, %s, %s, ctx %" PRId64 " %s\n",
>                                i,
>                                env->dtlb[i].tag & (uint64_t)~0x1fffULL,
> @@ -737,7 +737,7 @@ void dump_mmu(FILE *f, fprintf_function cpu_fprintf, CPUState *env)
>                 break;
>             }
>             if (TTE_IS_VALID(env->itlb[i].tte)) {
> -                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %" PRIx64
> +                (*cpu_fprintf)(f, "[%02u] VA: %" PRIx64 ", PA: %llx"
>                                ", %s, %s, %s, ctx %" PRId64 " %s\n",
>                                i,
>                                env->itlb[i].tag & (uint64_t)~0x1fffULL,
> --
> 1.7.2.5
>
>