diff mbox

target-sparc: Fix compiler errors (format strings)

Message ID 1311313467-27740-1-git-send-email-weil@mail.berlios.de
State Accepted
Headers show

Commit Message

Stefan Weil July 22, 2011, 5:44 a.m. UTC
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(-)

Comments

Tsuneo Saito July 22, 2011, 8:17 a.m. UTC | #1
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>
Blue Swirl July 22, 2011, 3:37 p.m. UTC | #2
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
>
>
diff mbox

Patch

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,