All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH] Show length mismatch error is hex
@ 2014-07-19  9:09 Alex Bligh
  2014-08-02 13:24 ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev
  0 siblings, 1 reply; 2+ messages in thread
From: Alex Bligh @ 2014-07-19  9:09 UTC (permalink / raw)
  To: qemu-devel, qemu-trivial; +Cc: peter.maydell, Alex Bligh, pbonzini

When live migrate fails due to a section length mismatch we currently
see an error message like:

Length mismatch: 0000:00:03.0/virtio-net-pci.rom: 10000 in != 20000

The section lengths are in fact in hex, so this should read

Length mismatch: 0000:00:03.0/virtio-net-pci.rom: 0x10000 in != 0x20000

Correct the error string to reflect this.

Signed-off-by: Alex Bligh <alex@alex.org.uk>
---
 arch_init.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch_init.c b/arch_init.c
index 8ddaf35..28ece76 100644
--- a/arch_init.c
+++ b/arch_init.c
@@ -1072,8 +1072,8 @@ static int ram_load(QEMUFile *f, void *opaque, int version_id)
                 QTAILQ_FOREACH(block, &ram_list.blocks, next) {
                     if (!strncmp(id, block->idstr, sizeof(id))) {
                         if (block->length != length) {
-                            error_report("Length mismatch: %s: " RAM_ADDR_FMT
-                                         " in != " RAM_ADDR_FMT, id, length,
+                            error_report("Length mismatch: %s: 0x" RAM_ADDR_FMT
+                                         " in != 0x" RAM_ADDR_FMT, id, length,
                                          block->length);
                             ret =  -EINVAL;
                         }
-- 
1.7.9.5

^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [Qemu-devel] [Qemu-trivial] [PATCH] Show length mismatch error is hex
  2014-07-19  9:09 [Qemu-devel] [PATCH] Show length mismatch error is hex Alex Bligh
@ 2014-08-02 13:24 ` Michael Tokarev
  0 siblings, 0 replies; 2+ messages in thread
From: Michael Tokarev @ 2014-08-02 13:24 UTC (permalink / raw)
  To: Alex Bligh, qemu-devel, qemu-trivial; +Cc: peter.maydell, pbonzini

19.07.2014 13:09, Alex Bligh wrote:
> When live migrate fails due to a section length mismatch we currently
> see an error message like:
> 
> Length mismatch: 0000:00:03.0/virtio-net-pci.rom: 10000 in != 20000
> 
> The section lengths are in fact in hex, so this should read
> 
> Length mismatch: 0000:00:03.0/virtio-net-pci.rom: 0x10000 in != 0x20000
> 
> Correct the error string to reflect this.

Applied to -trivial, thank you!

/mjt

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2014-08-02 13:24 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-07-19  9:09 [Qemu-devel] [PATCH] Show length mismatch error is hex Alex Bligh
2014-08-02 13:24 ` [Qemu-devel] [Qemu-trivial] " Michael Tokarev

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.