[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PATCH] milkymist-minimac2: Use qemu_log_mask(GUEST_ERROR)
From: |
Philippe Mathieu-Daudé |
Subject: |
[Qemu-devel] [PATCH] milkymist-minimac2: Use qemu_log_mask(GUEST_ERROR) instead of error_report |
Date: |
Thu, 28 Jun 2018 13:52:54 -0300 |
qemu_log_mask(GUEST_ERROR) is more appropriate:
$ qemu -d help
Log items (comma separated):
guest_errors log when the guest OS does something invalid (eg accessing a
non-existent register)
Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
---
hw/net/milkymist-minimac2.c | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/hw/net/milkymist-minimac2.c b/hw/net/milkymist-minimac2.c
index 3eaa19dfde..94c3bc6595 100644
--- a/hw/net/milkymist-minimac2.c
+++ b/hw/net/milkymist-minimac2.c
@@ -30,7 +30,7 @@
#include "hw/sysbus.h"
#include "trace.h"
#include "net/net.h"
-#include "qemu/error-report.h"
+#include "qemu/log.h"
#include <zlib.h>
@@ -214,7 +214,8 @@ static size_t assemble_frame(uint8_t *buf, size_t size,
uint32_t crc;
if (size < payload_size + 12) {
- error_report("milkymist_minimac2: received too big ethernet frame");
+ qemu_log_mask(LOG_GUEST_ERROR, "milkymist_minimac2: frame too big "
+ "(%zd bytes)\n", payload_size);
return 0;
}
@@ -347,8 +348,9 @@ minimac2_read(void *opaque, hwaddr addr, unsigned size)
break;
default:
- error_report("milkymist_minimac2: read access to unknown register 0x"
- TARGET_FMT_plx, addr << 2);
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "milkymist_minimac2_rd%d: 0x%" HWADDR_PRIx "\n",
+ size, addr << 2);
break;
}
@@ -413,8 +415,9 @@ minimac2_write(void *opaque, hwaddr addr, uint64_t value,
break;
default:
- error_report("milkymist_minimac2: write access to unknown register 0x"
- TARGET_FMT_plx, addr << 2);
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "milkymist_minimac2_wr%d: 0x%" HWADDR_PRIx " = 0x%lx\n",
+ size, addr << 2, value);
break;
}
}
--
2.18.0
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- [Qemu-devel] [PATCH] milkymist-minimac2: Use qemu_log_mask(GUEST_ERROR) instead of error_report,
Philippe Mathieu-Daudé <=