* [PATCH] Fix %#08 misuses
@ 2021-10-14 18:52 Dr. David Alan Gilbert (git)
2021-10-14 19:33 ` Richard Henderson
0 siblings, 1 reply; 4+ messages in thread
From: Dr. David Alan Gilbert (git) @ 2021-10-14 18:52 UTC (permalink / raw)
To: qemu-devel, eblake, peter.maydell
Cc: mark.cave-ayland, alistair.francis, richard.henderson, shorne, atar4qemu
From: "Dr. David Alan Gilbert" <dgilbert@redhat.com>
Joe Perches pointed out on lkml [1] that the format special %# (which
adds 0x) is included in the character count, i.e.:
printf("0: %#08x\n0: %#08x\n", 0xabcdef01,1);
gives:
0: 0xabcdef01
0: 0x000001
rather than padding to the expected 8 data characters.
It looks like we use '%#.8x' in some places and that
seems like it works:
printf(".: %#.8x\n.: %#.8x\n", 0xabcdef01,1);
.: 0xabcdef01
.: 0x00000001
Replace all the '%#08' cases by '%#.8' ; I've not
attacked the other lengths yet.
Although I'm tempted to think that perhaps we should just
prefer '0x%08' which seems clearer.
(Note I've not managed to test most of these)
Signed-off-by: Dr. David Alan Gilbert <dgilbert@redhat.com>
[1] https://lore.kernel.org/lkml/9499203f1e993872b384aabdec59ac223a8ab931.camel@perches.com/
---
disas/alpha.c | 2 +-
disas/sparc.c | 2 +-
hw/arm/omap1.c | 2 +-
hw/timer/a9gtimer.c | 4 ++--
include/hw/arm/omap.h | 4 ++--
softmmu/device_tree.c | 2 +-
target/openrisc/disas.c | 2 +-
7 files changed, 9 insertions(+), 9 deletions(-)
diff --git a/disas/alpha.c b/disas/alpha.c
index 3db90fa665..6da5c473d8 100644
--- a/disas/alpha.c
+++ b/disas/alpha.c
@@ -1851,7 +1851,7 @@ print_insn_alpha (bfd_vma memaddr, struct disassemble_info *info)
}
/* No instruction found */
- (*info->fprintf_func) (info->stream, ".long %#08x", insn);
+ (*info->fprintf_func) (info->stream, ".long %#.8x", insn);
return 4;
diff --git a/disas/sparc.c b/disas/sparc.c
index 5689533ce1..9f1160ab1c 100644
--- a/disas/sparc.c
+++ b/disas/sparc.c
@@ -3231,6 +3231,6 @@ print_insn_sparc (bfd_vma memaddr, disassemble_info *info)
}
info->insn_type = dis_noninsn; /* Mark as non-valid instruction. */
- (*info->fprintf_func) (stream, ".long %#08lx", insn);
+ (*info->fprintf_func) (stream, ".long %#.8lx", insn);
return sizeof (buffer);
}
diff --git a/hw/arm/omap1.c b/hw/arm/omap1.c
index 180d3788f8..cff8ecb7d6 100644
--- a/hw/arm/omap1.c
+++ b/hw/arm/omap1.c
@@ -42,7 +42,7 @@
static inline void omap_log_badwidth(const char *funcname, hwaddr addr, int sz)
{
- qemu_log_mask(LOG_GUEST_ERROR, "%s: %d-bit register %#08" HWADDR_PRIx "\n",
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: %d-bit register %#.8" HWADDR_PRIx "\n",
funcname, 8 * sz, addr);
}
diff --git a/hw/timer/a9gtimer.c b/hw/timer/a9gtimer.c
index 7233068a37..c53edd8b44 100644
--- a/hw/timer/a9gtimer.c
+++ b/hw/timer/a9gtimer.c
@@ -169,7 +169,7 @@ static uint64_t a9_gtimer_read(void *opaque, hwaddr addr, unsigned size)
return 0;
}
- DB_PRINT("addr:%#x data:%#08" PRIx64 "\n", (unsigned)addr, ret);
+ DB_PRINT("addr:%#x data:%#.8" PRIx64 "\n", (unsigned)addr, ret);
return ret;
}
@@ -180,7 +180,7 @@ static void a9_gtimer_write(void *opaque, hwaddr addr, uint64_t value,
A9GTimerState *s = gtb->parent;
int shift = 0;
- DB_PRINT("addr:%#x data:%#08" PRIx64 "\n", (unsigned)addr, value);
+ DB_PRINT("addr:%#x data:%#.8" PRIx64 "\n", (unsigned)addr, value);
switch (addr) {
case R_COUNTER_HI:
diff --git a/include/hw/arm/omap.h b/include/hw/arm/omap.h
index ff6a173f8a..47616bbba3 100644
--- a/include/hw/arm/omap.h
+++ b/include/hw/arm/omap.h
@@ -1003,10 +1003,10 @@ void omap_badwidth_write32(void *opaque, hwaddr addr,
void omap_mpu_wakeup(void *opaque, int irq, int req);
# define OMAP_BAD_REG(paddr) \
- qemu_log_mask(LOG_GUEST_ERROR, "%s: Bad register %#08"HWADDR_PRIx"\n", \
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: Bad register %#.8"HWADDR_PRIx"\n", \
__func__, paddr)
# define OMAP_RO_REG(paddr) \
- qemu_log_mask(LOG_GUEST_ERROR, "%s: Read-only register %#08" \
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: Read-only register %#.8" \
HWADDR_PRIx "\n", \
__func__, paddr)
diff --git a/softmmu/device_tree.c b/softmmu/device_tree.c
index b621f63fba..bd271ab66e 100644
--- a/softmmu/device_tree.c
+++ b/softmmu/device_tree.c
@@ -367,7 +367,7 @@ int qemu_fdt_setprop_cell(void *fdt, const char *node_path,
r = fdt_setprop_cell(fdt, findnode_nofail(fdt, node_path), property, val);
if (r < 0) {
- error_report("%s: Couldn't set %s/%s = %#08x: %s", __func__,
+ error_report("%s: Couldn't set %s/%s = %#.8x: %s", __func__,
node_path, property, val, fdt_strerror(r));
exit(1);
}
diff --git a/target/openrisc/disas.c b/target/openrisc/disas.c
index dc025bd64d..024f243a45 100644
--- a/target/openrisc/disas.c
+++ b/target/openrisc/disas.c
@@ -45,7 +45,7 @@ int print_insn_or1k(bfd_vma addr, disassemble_info *info)
insn = bfd_getb32(buffer);
if (!decode(info, insn)) {
- output(".long", "%#08x", insn);
+ output(".long", "%#.8x", insn);
}
return 4;
}
--
2.31.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix %#08 misuses
2021-10-14 18:52 [PATCH] Fix %#08 misuses Dr. David Alan Gilbert (git)
@ 2021-10-14 19:33 ` Richard Henderson
2021-10-14 21:56 ` Stafford Horne
0 siblings, 1 reply; 4+ messages in thread
From: Richard Henderson @ 2021-10-14 19:33 UTC (permalink / raw)
To: Dr. David Alan Gilbert (git), qemu-devel, eblake, peter.maydell
Cc: alistair.francis, shorne, mark.cave-ayland, atar4qemu
On 10/14/21 11:52 AM, Dr. David Alan Gilbert (git) wrote:
> Although I'm tempted to think that perhaps we should just
> prefer '0x%08' which seems clearer.
That's what I think we should prefer.
r~
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix %#08 misuses
2021-10-14 19:33 ` Richard Henderson
@ 2021-10-14 21:56 ` Stafford Horne
2021-10-18 9:47 ` Dr. David Alan Gilbert
0 siblings, 1 reply; 4+ messages in thread
From: Stafford Horne @ 2021-10-14 21:56 UTC (permalink / raw)
To: Richard Henderson
Cc: peter.maydell, mark.cave-ayland, Dr. David Alan Gilbert (git),
qemu-devel, alistair.francis, eblake, atar4qemu
On Thu, Oct 14, 2021 at 12:33:06PM -0700, Richard Henderson wrote:
> On 10/14/21 11:52 AM, Dr. David Alan Gilbert (git) wrote:
> > Although I'm tempted to think that perhaps we should just
> > prefer '0x%08' which seems clearer.
>
> That's what I think we should prefer.
I agree, it's always more clear to me too.
-Stafford
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH] Fix %#08 misuses
2021-10-14 21:56 ` Stafford Horne
@ 2021-10-18 9:47 ` Dr. David Alan Gilbert
0 siblings, 0 replies; 4+ messages in thread
From: Dr. David Alan Gilbert @ 2021-10-18 9:47 UTC (permalink / raw)
To: Stafford Horne
Cc: peter.maydell, Richard Henderson, mark.cave-ayland, qemu-devel,
alistair.francis, eblake, atar4qemu
* Stafford Horne (shorne@gmail.com) wrote:
> On Thu, Oct 14, 2021 at 12:33:06PM -0700, Richard Henderson wrote:
> > On 10/14/21 11:52 AM, Dr. David Alan Gilbert (git) wrote:
> > > Although I'm tempted to think that perhaps we should just
> > > prefer '0x%08' which seems clearer.
> >
> > That's what I think we should prefer.
>
> I agree, it's always more clear to me too.
OK, I'll rework it.
Dave
> -Stafford
>
--
Dr. David Alan Gilbert / dgilbert@redhat.com / Manchester, UK
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2021-10-18 9:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-14 18:52 [PATCH] Fix %#08 misuses Dr. David Alan Gilbert (git)
2021-10-14 19:33 ` Richard Henderson
2021-10-14 21:56 ` Stafford Horne
2021-10-18 9:47 ` Dr. David Alan Gilbert
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.