* [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 15:44 ` Boxuan Li
0 siblings, 0 replies; 15+ messages in thread
From: Boxuan Li @ 2019-05-03 15:44 UTC (permalink / raw)
To: qemu-devel
Cc: qemu-trivial, eblake, philmd, yuval.shaia, alex.bennee,
Boxuan Li, Michael S. Tsirkin
Use traces for debug message and qemu_log_mask for errors.
Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
---
v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
v4: Fix indentation and do not convert uint64_t to int
---
hw/virtio/trace-events | 7 +++++++
hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
2 files changed, 28 insertions(+), 23 deletions(-)
diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
index 60c649c4bc..e28ba48da6 100644
--- a/hw/virtio/trace-events
+++ b/hw/virtio/trace-events
@@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
+
+# virtio-mmio.c
+virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
+virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
+virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
+virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
+virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
index 5807aa87fe..96c762f0bf 100644
--- a/hw/virtio/virtio-mmio.c
+++ b/hw/virtio/virtio-mmio.c
@@ -27,16 +27,8 @@
#include "sysemu/kvm.h"
#include "hw/virtio/virtio-bus.h"
#include "qemu/error-report.h"
-
-/* #define DEBUG_VIRTIO_MMIO */
-
-#ifdef DEBUG_VIRTIO_MMIO
-
-#define DPRINTF(fmt, ...) \
-do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
-#else
-#define DPRINTF(fmt, ...) do {} while (0)
-#endif
+#include "qemu/log.h"
+#include "trace.h"
/* QOM macros */
/* virtio-mmio-bus */
@@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
- DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
+ trace_virtio_mmio_read(offset);
if (!vdev) {
/* If no backend is present, we treat most registers as
@@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
}
}
if (size != 4) {
- DPRINTF("wrong size access to register!\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: wrong size access to register!\n",
+ __func__);
return 0;
}
switch (offset) {
@@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
case VIRTIO_MMIO_QUEUE_ALIGN:
case VIRTIO_MMIO_QUEUE_NOTIFY:
case VIRTIO_MMIO_INTERRUPT_ACK:
- DPRINTF("read of write-only register\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: read of write-only register\n",
+ __func__);
return 0;
default:
- DPRINTF("bad register offset\n");
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
return 0;
}
return 0;
@@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
- DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
- (int)offset, value);
+ trace_virtio_mmio_write_offset(offset, value);
if (!vdev) {
/* If no backend is present, we just make all registers
@@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
return;
}
if (size != 4) {
- DPRINTF("wrong size access to register!\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: wrong size access to register!\n",
+ __func__);
return;
}
switch (offset) {
@@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
if (proxy->guest_page_shift > 31) {
proxy->guest_page_shift = 0;
}
- DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
- proxy->guest_page_shift);
+ trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
break;
case VIRTIO_MMIO_QUEUE_SEL:
if (value < VIRTIO_QUEUE_MAX) {
@@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
}
break;
case VIRTIO_MMIO_QUEUE_NUM:
- DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
+ trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
virtio_queue_set_num(vdev, vdev->queue_sel, value);
/* Note: only call this function for legacy devices */
virtio_queue_update_rings(vdev, vdev->queue_sel);
@@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
case VIRTIO_MMIO_DEVICE_FEATURES:
case VIRTIO_MMIO_QUEUE_NUM_MAX:
case VIRTIO_MMIO_INTERRUPT_STATUS:
- DPRINTF("write to readonly register\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: write to readonly register\n",
+ __func__);
break;
default:
- DPRINTF("bad register offset\n");
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
}
}
@@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
return;
}
level = (atomic_read(&vdev->isr) != 0);
- DPRINTF("virtio_mmio setting IRQ %d\n", level);
+ trace_virtio_mmio_setting_irq(level);
qemu_set_irq(proxy->irq, level);
}
--
2.13.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 15:44 ` Boxuan Li
0 siblings, 0 replies; 15+ messages in thread
From: Boxuan Li @ 2019-05-03 15:44 UTC (permalink / raw)
To: qemu-devel
Cc: Michael S. Tsirkin, qemu-trivial, yuval.shaia, alex.bennee,
Boxuan Li, philmd
Use traces for debug message and qemu_log_mask for errors.
Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
---
v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
v4: Fix indentation and do not convert uint64_t to int
---
hw/virtio/trace-events | 7 +++++++
hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
2 files changed, 28 insertions(+), 23 deletions(-)
diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
index 60c649c4bc..e28ba48da6 100644
--- a/hw/virtio/trace-events
+++ b/hw/virtio/trace-events
@@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
+
+# virtio-mmio.c
+virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
+virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
+virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
+virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
+virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
index 5807aa87fe..96c762f0bf 100644
--- a/hw/virtio/virtio-mmio.c
+++ b/hw/virtio/virtio-mmio.c
@@ -27,16 +27,8 @@
#include "sysemu/kvm.h"
#include "hw/virtio/virtio-bus.h"
#include "qemu/error-report.h"
-
-/* #define DEBUG_VIRTIO_MMIO */
-
-#ifdef DEBUG_VIRTIO_MMIO
-
-#define DPRINTF(fmt, ...) \
-do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
-#else
-#define DPRINTF(fmt, ...) do {} while (0)
-#endif
+#include "qemu/log.h"
+#include "trace.h"
/* QOM macros */
/* virtio-mmio-bus */
@@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
- DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
+ trace_virtio_mmio_read(offset);
if (!vdev) {
/* If no backend is present, we treat most registers as
@@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
}
}
if (size != 4) {
- DPRINTF("wrong size access to register!\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: wrong size access to register!\n",
+ __func__);
return 0;
}
switch (offset) {
@@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
case VIRTIO_MMIO_QUEUE_ALIGN:
case VIRTIO_MMIO_QUEUE_NOTIFY:
case VIRTIO_MMIO_INTERRUPT_ACK:
- DPRINTF("read of write-only register\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: read of write-only register\n",
+ __func__);
return 0;
default:
- DPRINTF("bad register offset\n");
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
return 0;
}
return 0;
@@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
- DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
- (int)offset, value);
+ trace_virtio_mmio_write_offset(offset, value);
if (!vdev) {
/* If no backend is present, we just make all registers
@@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
return;
}
if (size != 4) {
- DPRINTF("wrong size access to register!\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: wrong size access to register!\n",
+ __func__);
return;
}
switch (offset) {
@@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
if (proxy->guest_page_shift > 31) {
proxy->guest_page_shift = 0;
}
- DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
- proxy->guest_page_shift);
+ trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
break;
case VIRTIO_MMIO_QUEUE_SEL:
if (value < VIRTIO_QUEUE_MAX) {
@@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
}
break;
case VIRTIO_MMIO_QUEUE_NUM:
- DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
+ trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
virtio_queue_set_num(vdev, vdev->queue_sel, value);
/* Note: only call this function for legacy devices */
virtio_queue_update_rings(vdev, vdev->queue_sel);
@@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
case VIRTIO_MMIO_DEVICE_FEATURES:
case VIRTIO_MMIO_QUEUE_NUM_MAX:
case VIRTIO_MMIO_INTERRUPT_STATUS:
- DPRINTF("write to readonly register\n");
+ qemu_log_mask(LOG_GUEST_ERROR,
+ "%s: write to readonly register\n",
+ __func__);
break;
default:
- DPRINTF("bad register offset\n");
+ qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
}
}
@@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
return;
}
level = (atomic_read(&vdev->isr) != 0);
- DPRINTF("virtio_mmio setting IRQ %d\n", level);
+ trace_virtio_mmio_setting_irq(level);
qemu_set_irq(proxy->irq, level);
}
--
2.13.2
^ permalink raw reply related [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 15:58 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 15+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-03 15:58 UTC (permalink / raw)
To: Boxuan Li, qemu-devel
Cc: qemu-trivial, eblake, yuval.shaia, alex.bennee, Michael S. Tsirkin
On 5/3/19 5:44 PM, Boxuan Li wrote:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Thanks for all your iterations!
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 15:58 ` Philippe Mathieu-Daudé
0 siblings, 0 replies; 15+ messages in thread
From: Philippe Mathieu-Daudé @ 2019-05-03 15:58 UTC (permalink / raw)
To: Boxuan Li, qemu-devel
Cc: qemu-trivial, alex.bennee, yuval.shaia, Michael S. Tsirkin
On 5/3/19 5:44 PM, Boxuan Li wrote:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Tested-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Thanks for all your iterations!
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
>
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 16:18 ` Alex Bennée
0 siblings, 0 replies; 15+ messages in thread
From: Alex Bennée @ 2019-05-03 16:18 UTC (permalink / raw)
To: Boxuan Li
Cc: qemu-devel, qemu-trivial, eblake, philmd, yuval.shaia,
Michael S. Tsirkin
Boxuan Li <liboxuan@connect.hku.hk> writes:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
You didn't add my r-b tags from last time. Anyway:
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
--
Alex Bennée
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 16:18 ` Alex Bennée
0 siblings, 0 replies; 15+ messages in thread
From: Alex Bennée @ 2019-05-03 16:18 UTC (permalink / raw)
To: Boxuan Li
Cc: Michael S. Tsirkin, qemu-trivial, qemu-devel, yuval.shaia, philmd
Boxuan Li <liboxuan@connect.hku.hk> writes:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
You didn't add my r-b tags from last time. Anyway:
Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
--
Alex Bennée
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 16:23 ` LI, BO XUAN
0 siblings, 0 replies; 15+ messages in thread
From: LI, BO XUAN @ 2019-05-03 16:23 UTC (permalink / raw)
To: Alex Bennée
Cc: qemu-devel, qemu-trivial, Eric Blake, Philippe Mathieu-Daudé,
Yuval Shaia, Michael S. Tsirkin
Hi Alex,
Sorry about that, I am still trying to get familiar with the patch
submission process. Since my patch has been changed from your last review,
I thought it would be safe to not include the r-b tag from last time. Will
take care next time!
Best regards,
Boxuan Li
On Sat, May 4, 2019 at 12:18 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Boxuan Li <liboxuan@connect.hku.hk> writes:
>
> > Use traces for debug message and qemu_log_mask for errors.
> >
> > Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
>
> You didn't add my r-b tags from last time. Anyway:
>
> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>
> > ---
> > v1:
> https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> > v2:
> https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> > v3:
> https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> > v4: Fix indentation and do not convert uint64_t to int
> > ---
> > hw/virtio/trace-events | 7 +++++++
> > hw/virtio/virtio-mmio.c | 44
> +++++++++++++++++++++-----------------------
> > 2 files changed, 28 insertions(+), 23 deletions(-)
> >
> > diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> > index 60c649c4bc..e28ba48da6 100644
> > --- a/hw/virtio/trace-events
> > +++ b/hw/virtio/trace-events
> > @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name,
> uint64_t gpa) "section name: %s g
> > virtio_balloon_get_config(uint32_t num_pages, uint32_t actual)
> "num_pages: %d actual: %d"
> > virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual:
> %d oldactual: %d"
> > virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon
> target: 0x%"PRIx64" num_pages: %d"
> > +
> > +# virtio-mmio.c
> > +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> > +virtio_mmio_write_offset(uint64_t offset, uint64_t value)
> "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> > +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%"
> PRIx64 " shift %d"
> > +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write
> 0x%" PRIx64 " max %d"
> > +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> > diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> > index 5807aa87fe..96c762f0bf 100644
> > --- a/hw/virtio/virtio-mmio.c
> > +++ b/hw/virtio/virtio-mmio.c
> > @@ -27,16 +27,8 @@
> > #include "sysemu/kvm.h"
> > #include "hw/virtio/virtio-bus.h"
> > #include "qemu/error-report.h"
> > -
> > -/* #define DEBUG_VIRTIO_MMIO */
> > -
> > -#ifdef DEBUG_VIRTIO_MMIO
> > -
> > -#define DPRINTF(fmt, ...) \
> > -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> > -#else
> > -#define DPRINTF(fmt, ...) do {} while (0)
> > -#endif
> > +#include "qemu/log.h"
> > +#include "trace.h"
> >
> > /* QOM macros */
> > /* virtio-mmio-bus */
> > @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr
> offset, unsigned size)
> > VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> > VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
> >
> > - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> > + trace_virtio_mmio_read(offset);
> >
> > if (!vdev) {
> > /* If no backend is present, we treat most registers as
> > @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque,
> hwaddr offset, unsigned size)
> > }
> > }
> > if (size != 4) {
> > - DPRINTF("wrong size access to register!\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: wrong size access to register!\n",
> > + __func__);
> > return 0;
> > }
> > switch (offset) {
> > @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque,
> hwaddr offset, unsigned size)
> > case VIRTIO_MMIO_QUEUE_ALIGN:
> > case VIRTIO_MMIO_QUEUE_NOTIFY:
> > case VIRTIO_MMIO_INTERRUPT_ACK:
> > - DPRINTF("read of write-only register\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: read of write-only register\n",
> > + __func__);
> > return 0;
> > default:
> > - DPRINTF("bad register offset\n");
> > + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n",
> __func__);
> > return 0;
> > }
> > return 0;
> > @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> > VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
> >
> > - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> > - (int)offset, value);
> > + trace_virtio_mmio_write_offset(offset, value);
> >
> > if (!vdev) {
> > /* If no backend is present, we just make all registers
> > @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > return;
> > }
> > if (size != 4) {
> > - DPRINTF("wrong size access to register!\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: wrong size access to register!\n",
> > + __func__);
> > return;
> > }
> > switch (offset) {
> > @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > if (proxy->guest_page_shift > 31) {
> > proxy->guest_page_shift = 0;
> > }
> > - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> > - proxy->guest_page_shift);
> > + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> > break;
> > case VIRTIO_MMIO_QUEUE_SEL:
> > if (value < VIRTIO_QUEUE_MAX) {
> > @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > }
> > break;
> > case VIRTIO_MMIO_QUEUE_NUM:
> > - DPRINTF("mmio_queue write %d max %d\n", (int)value,
> VIRTQUEUE_MAX_SIZE);
> > + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> > virtio_queue_set_num(vdev, vdev->queue_sel, value);
> > /* Note: only call this function for legacy devices */
> > virtio_queue_update_rings(vdev, vdev->queue_sel);
> > @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > case VIRTIO_MMIO_DEVICE_FEATURES:
> > case VIRTIO_MMIO_QUEUE_NUM_MAX:
> > case VIRTIO_MMIO_INTERRUPT_STATUS:
> > - DPRINTF("write to readonly register\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: write to readonly register\n",
> > + __func__);
> > break;
> >
> > default:
> > - DPRINTF("bad register offset\n");
> > + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n",
> __func__);
> > }
> > }
> >
> > @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState
> *opaque, uint16_t vector)
> > return;
> > }
> > level = (atomic_read(&vdev->isr) != 0);
> > - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> > + trace_virtio_mmio_setting_irq(level);
> > qemu_set_irq(proxy->irq, level);
> > }
>
>
> --
> Alex Bennée
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 16:23 ` LI, BO XUAN
0 siblings, 0 replies; 15+ messages in thread
From: LI, BO XUAN @ 2019-05-03 16:23 UTC (permalink / raw)
To: Alex Bennée
Cc: Michael S. Tsirkin, qemu-trivial, qemu-devel, Yuval Shaia,
Philippe Mathieu-Daudé
Hi Alex,
Sorry about that, I am still trying to get familiar with the patch
submission process. Since my patch has been changed from your last review,
I thought it would be safe to not include the r-b tag from last time. Will
take care next time!
Best regards,
Boxuan Li
On Sat, May 4, 2019 at 12:18 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> Boxuan Li <liboxuan@connect.hku.hk> writes:
>
> > Use traces for debug message and qemu_log_mask for errors.
> >
> > Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
>
> You didn't add my r-b tags from last time. Anyway:
>
> Reviewed-by: Alex Bennée <alex.bennee@linaro.org>
>
> > ---
> > v1:
> https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> > v2:
> https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> > v3:
> https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> > v4: Fix indentation and do not convert uint64_t to int
> > ---
> > hw/virtio/trace-events | 7 +++++++
> > hw/virtio/virtio-mmio.c | 44
> +++++++++++++++++++++-----------------------
> > 2 files changed, 28 insertions(+), 23 deletions(-)
> >
> > diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> > index 60c649c4bc..e28ba48da6 100644
> > --- a/hw/virtio/trace-events
> > +++ b/hw/virtio/trace-events
> > @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name,
> uint64_t gpa) "section name: %s g
> > virtio_balloon_get_config(uint32_t num_pages, uint32_t actual)
> "num_pages: %d actual: %d"
> > virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual:
> %d oldactual: %d"
> > virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon
> target: 0x%"PRIx64" num_pages: %d"
> > +
> > +# virtio-mmio.c
> > +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> > +virtio_mmio_write_offset(uint64_t offset, uint64_t value)
> "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> > +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%"
> PRIx64 " shift %d"
> > +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write
> 0x%" PRIx64 " max %d"
> > +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> > diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> > index 5807aa87fe..96c762f0bf 100644
> > --- a/hw/virtio/virtio-mmio.c
> > +++ b/hw/virtio/virtio-mmio.c
> > @@ -27,16 +27,8 @@
> > #include "sysemu/kvm.h"
> > #include "hw/virtio/virtio-bus.h"
> > #include "qemu/error-report.h"
> > -
> > -/* #define DEBUG_VIRTIO_MMIO */
> > -
> > -#ifdef DEBUG_VIRTIO_MMIO
> > -
> > -#define DPRINTF(fmt, ...) \
> > -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> > -#else
> > -#define DPRINTF(fmt, ...) do {} while (0)
> > -#endif
> > +#include "qemu/log.h"
> > +#include "trace.h"
> >
> > /* QOM macros */
> > /* virtio-mmio-bus */
> > @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr
> offset, unsigned size)
> > VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> > VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
> >
> > - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> > + trace_virtio_mmio_read(offset);
> >
> > if (!vdev) {
> > /* If no backend is present, we treat most registers as
> > @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque,
> hwaddr offset, unsigned size)
> > }
> > }
> > if (size != 4) {
> > - DPRINTF("wrong size access to register!\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: wrong size access to register!\n",
> > + __func__);
> > return 0;
> > }
> > switch (offset) {
> > @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque,
> hwaddr offset, unsigned size)
> > case VIRTIO_MMIO_QUEUE_ALIGN:
> > case VIRTIO_MMIO_QUEUE_NOTIFY:
> > case VIRTIO_MMIO_INTERRUPT_ACK:
> > - DPRINTF("read of write-only register\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: read of write-only register\n",
> > + __func__);
> > return 0;
> > default:
> > - DPRINTF("bad register offset\n");
> > + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n",
> __func__);
> > return 0;
> > }
> > return 0;
> > @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> > VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
> >
> > - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> > - (int)offset, value);
> > + trace_virtio_mmio_write_offset(offset, value);
> >
> > if (!vdev) {
> > /* If no backend is present, we just make all registers
> > @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > return;
> > }
> > if (size != 4) {
> > - DPRINTF("wrong size access to register!\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: wrong size access to register!\n",
> > + __func__);
> > return;
> > }
> > switch (offset) {
> > @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > if (proxy->guest_page_shift > 31) {
> > proxy->guest_page_shift = 0;
> > }
> > - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> > - proxy->guest_page_shift);
> > + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> > break;
> > case VIRTIO_MMIO_QUEUE_SEL:
> > if (value < VIRTIO_QUEUE_MAX) {
> > @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > }
> > break;
> > case VIRTIO_MMIO_QUEUE_NUM:
> > - DPRINTF("mmio_queue write %d max %d\n", (int)value,
> VIRTQUEUE_MAX_SIZE);
> > + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> > virtio_queue_set_num(vdev, vdev->queue_sel, value);
> > /* Note: only call this function for legacy devices */
> > virtio_queue_update_rings(vdev, vdev->queue_sel);
> > @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr
> offset, uint64_t value,
> > case VIRTIO_MMIO_DEVICE_FEATURES:
> > case VIRTIO_MMIO_QUEUE_NUM_MAX:
> > case VIRTIO_MMIO_INTERRUPT_STATUS:
> > - DPRINTF("write to readonly register\n");
> > + qemu_log_mask(LOG_GUEST_ERROR,
> > + "%s: write to readonly register\n",
> > + __func__);
> > break;
> >
> > default:
> > - DPRINTF("bad register offset\n");
> > + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n",
> __func__);
> > }
> > }
> >
> > @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState
> *opaque, uint16_t vector)
> > return;
> > }
> > level = (atomic_read(&vdev->isr) != 0);
> > - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> > + trace_virtio_mmio_setting_irq(level);
> > qemu_set_irq(proxy->irq, level);
> > }
>
>
> --
> Alex Bennée
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 17:00 ` Alex Bennée
0 siblings, 0 replies; 15+ messages in thread
From: Alex Bennée @ 2019-05-03 17:00 UTC (permalink / raw)
To: LI, BO XUAN
Cc: qemu-devel, qemu-trivial, Eric Blake, Philippe Mathieu-Daudé,
Yuval Shaia, Michael S. Tsirkin
LI, BO XUAN <liboxuan@connect.hku.hk> writes:
> Hi Alex,
>
> Sorry about that, I am still trying to get familiar with the patch
> submission process. Since my patch has been changed from your last review,
> I thought it would be safe to not include the r-b tag from last time. Will
> take care next time!
That's ok. As a general rule as long as you haven't substantially
changed a patch it's safe to keep previous r-b tags. You can always
mention it in your cover letter if you are unsure.
--
Alex Bennée
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 17:00 ` Alex Bennée
0 siblings, 0 replies; 15+ messages in thread
From: Alex Bennée @ 2019-05-03 17:00 UTC (permalink / raw)
To: LI, BO XUAN
Cc: Michael S. Tsirkin, qemu-trivial, qemu-devel, Yuval Shaia,
Philippe Mathieu-Daudé
LI, BO XUAN <liboxuan@connect.hku.hk> writes:
> Hi Alex,
>
> Sorry about that, I am still trying to get familiar with the patch
> submission process. Since my patch has been changed from your last review,
> I thought it would be safe to not include the r-b tag from last time. Will
> take care next time!
That's ok. As a general rule as long as you haven't substantially
changed a patch it's safe to keep previous r-b tags. You can always
mention it in your cover letter if you are unsure.
--
Alex Bennée
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 17:13 ` LI, BO XUAN
0 siblings, 0 replies; 15+ messages in thread
From: LI, BO XUAN @ 2019-05-03 17:13 UTC (permalink / raw)
To: Alex Bennée
Cc: qemu-devel, qemu-trivial, Eric Blake, Philippe Mathieu-Daudé,
Yuval Shaia, Michael S. Tsirkin
Gotcha, thanks for the tip!
Best regards,
Boxuan Li
On Sat, May 4, 2019 at 1:00 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> LI, BO XUAN <liboxuan@connect.hku.hk> writes:
>
> > Hi Alex,
> >
> > Sorry about that, I am still trying to get familiar with the patch
> > submission process. Since my patch has been changed from your last
> review,
> > I thought it would be safe to not include the r-b tag from last time.
> Will
> > take care next time!
>
> That's ok. As a general rule as long as you haven't substantially
> changed a patch it's safe to keep previous r-b tags. You can always
> mention it in your cover letter if you are unsure.
>
> --
> Alex Bennée
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-03 17:13 ` LI, BO XUAN
0 siblings, 0 replies; 15+ messages in thread
From: LI, BO XUAN @ 2019-05-03 17:13 UTC (permalink / raw)
To: Alex Bennée
Cc: Michael S. Tsirkin, qemu-trivial, qemu-devel, Yuval Shaia,
Philippe Mathieu-Daudé
Gotcha, thanks for the tip!
Best regards,
Boxuan Li
On Sat, May 4, 2019 at 1:00 AM Alex Bennée <alex.bennee@linaro.org> wrote:
>
> LI, BO XUAN <liboxuan@connect.hku.hk> writes:
>
> > Hi Alex,
> >
> > Sorry about that, I am still trying to get familiar with the patch
> > submission process. Since my patch has been changed from your last
> review,
> > I thought it would be safe to not include the r-b tag from last time.
> Will
> > take care next time!
>
> That's ok. As a general rule as long as you haven't substantially
> changed a patch it's safe to keep previous r-b tags. You can always
> mention it in your cover letter if you are unsure.
>
> --
> Alex Bennée
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-05 10:57 ` Yuval Shaia
0 siblings, 0 replies; 15+ messages in thread
From: Yuval Shaia @ 2019-05-05 10:57 UTC (permalink / raw)
To: Boxuan Li
Cc: qemu-devel, qemu-trivial, eblake, philmd, alex.bennee,
Michael S. Tsirkin
On Fri, May 03, 2019 at 11:44:24PM +0800, Boxuan Li wrote:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
>
> --
> 2.13.2
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
@ 2019-05-05 10:57 ` Yuval Shaia
0 siblings, 0 replies; 15+ messages in thread
From: Yuval Shaia @ 2019-05-05 10:57 UTC (permalink / raw)
To: Boxuan Li
Cc: Michael S. Tsirkin, qemu-trivial, qemu-devel, alex.bennee, philmd
On Fri, May 03, 2019 at 11:44:24PM +0800, Boxuan Li wrote:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
Reviewed-by: Yuval Shaia <yuval.shaia@oracle.com>
>
> --
> 2.13.2
>
^ permalink raw reply [flat|nested] 15+ messages in thread
* Re: [Qemu-devel] [Qemu-trivial] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log
2019-05-03 15:44 ` Boxuan Li
` (3 preceding siblings ...)
(?)
@ 2019-05-22 13:33 ` Laurent Vivier
-1 siblings, 0 replies; 15+ messages in thread
From: Laurent Vivier @ 2019-05-22 13:33 UTC (permalink / raw)
To: Boxuan Li, qemu-devel
Cc: Michael S. Tsirkin, qemu-trivial, philmd, yuval.shaia, alex.bennee
On 03/05/2019 17:44, Boxuan Li wrote:
> Use traces for debug message and qemu_log_mask for errors.
>
> Signed-off-by: Boxuan Li <liboxuan@connect.hku.hk>
> ---
> v1: https://patchew.org/QEMU/20190428110258.86681-1-liboxuan@connect.hku.hk/
> v2: https://patchew.org/QEMU/20190501081039.58938-1-liboxuan@connect.hku.hk/
> v3: https://patchew.org/QEMU/20190503084654.18413-1-liboxuan@connect.hku.hk/
> v4: Fix indentation and do not convert uint64_t to int
> ---
> hw/virtio/trace-events | 7 +++++++
> hw/virtio/virtio-mmio.c | 44 +++++++++++++++++++++-----------------------
> 2 files changed, 28 insertions(+), 23 deletions(-)
>
> diff --git a/hw/virtio/trace-events b/hw/virtio/trace-events
> index 60c649c4bc..e28ba48da6 100644
> --- a/hw/virtio/trace-events
> +++ b/hw/virtio/trace-events
> @@ -46,3 +46,10 @@ virtio_balloon_handle_output(const char *name, uint64_t gpa) "section name: %s g
> virtio_balloon_get_config(uint32_t num_pages, uint32_t actual) "num_pages: %d actual: %d"
> virtio_balloon_set_config(uint32_t actual, uint32_t oldactual) "actual: %d oldactual: %d"
> virtio_balloon_to_target(uint64_t target, uint32_t num_pages) "balloon target: 0x%"PRIx64" num_pages: %d"
> +
> +# virtio-mmio.c
> +virtio_mmio_read(uint64_t offset) "virtio_mmio_read offset 0x%" PRIx64
> +virtio_mmio_write_offset(uint64_t offset, uint64_t value) "virtio_mmio_write offset 0x%" PRIx64 " value 0x%" PRIx64
> +virtio_mmio_guest_page(uint64_t size, int shift) "guest page size 0x%" PRIx64 " shift %d"
> +virtio_mmio_queue_write(uint64_t value, int max_size) "mmio_queue write 0x%" PRIx64 " max %d"
> +virtio_mmio_setting_irq(int level) "virtio_mmio setting IRQ %d"
> diff --git a/hw/virtio/virtio-mmio.c b/hw/virtio/virtio-mmio.c
> index 5807aa87fe..96c762f0bf 100644
> --- a/hw/virtio/virtio-mmio.c
> +++ b/hw/virtio/virtio-mmio.c
> @@ -27,16 +27,8 @@
> #include "sysemu/kvm.h"
> #include "hw/virtio/virtio-bus.h"
> #include "qemu/error-report.h"
> -
> -/* #define DEBUG_VIRTIO_MMIO */
> -
> -#ifdef DEBUG_VIRTIO_MMIO
> -
> -#define DPRINTF(fmt, ...) \
> -do { printf("virtio_mmio: " fmt , ## __VA_ARGS__); } while (0)
> -#else
> -#define DPRINTF(fmt, ...) do {} while (0)
> -#endif
> +#include "qemu/log.h"
> +#include "trace.h"
>
> /* QOM macros */
> /* virtio-mmio-bus */
> @@ -107,7 +99,7 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_read offset 0x%x\n", (int)offset);
> + trace_virtio_mmio_read(offset);
>
> if (!vdev) {
> /* If no backend is present, we treat most registers as
> @@ -144,7 +136,9 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> }
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return 0;
> }
> switch (offset) {
> @@ -182,10 +176,12 @@ static uint64_t virtio_mmio_read(void *opaque, hwaddr offset, unsigned size)
> case VIRTIO_MMIO_QUEUE_ALIGN:
> case VIRTIO_MMIO_QUEUE_NOTIFY:
> case VIRTIO_MMIO_INTERRUPT_ACK:
> - DPRINTF("read of write-only register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: read of write-only register\n",
> + __func__);
> return 0;
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> return 0;
> }
> return 0;
> @@ -197,8 +193,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> VirtIOMMIOProxy *proxy = (VirtIOMMIOProxy *)opaque;
> VirtIODevice *vdev = virtio_bus_get_device(&proxy->bus);
>
> - DPRINTF("virtio_mmio_write offset 0x%x value 0x%" PRIx64 "\n",
> - (int)offset, value);
> + trace_virtio_mmio_write_offset(offset, value);
>
> if (!vdev) {
> /* If no backend is present, we just make all registers
> @@ -226,7 +221,9 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> return;
> }
> if (size != 4) {
> - DPRINTF("wrong size access to register!\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: wrong size access to register!\n",
> + __func__);
> return;
> }
> switch (offset) {
> @@ -246,8 +243,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> if (proxy->guest_page_shift > 31) {
> proxy->guest_page_shift = 0;
> }
> - DPRINTF("guest page size %" PRIx64 " shift %d\n", value,
> - proxy->guest_page_shift);
> + trace_virtio_mmio_guest_page(value, proxy->guest_page_shift);
> break;
> case VIRTIO_MMIO_QUEUE_SEL:
> if (value < VIRTIO_QUEUE_MAX) {
> @@ -255,7 +251,7 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> }
> break;
> case VIRTIO_MMIO_QUEUE_NUM:
> - DPRINTF("mmio_queue write %d max %d\n", (int)value, VIRTQUEUE_MAX_SIZE);
> + trace_virtio_mmio_queue_write(value, VIRTQUEUE_MAX_SIZE);
> virtio_queue_set_num(vdev, vdev->queue_sel, value);
> /* Note: only call this function for legacy devices */
> virtio_queue_update_rings(vdev, vdev->queue_sel);
> @@ -303,11 +299,13 @@ static void virtio_mmio_write(void *opaque, hwaddr offset, uint64_t value,
> case VIRTIO_MMIO_DEVICE_FEATURES:
> case VIRTIO_MMIO_QUEUE_NUM_MAX:
> case VIRTIO_MMIO_INTERRUPT_STATUS:
> - DPRINTF("write to readonly register\n");
> + qemu_log_mask(LOG_GUEST_ERROR,
> + "%s: write to readonly register\n",
> + __func__);
> break;
>
> default:
> - DPRINTF("bad register offset\n");
> + qemu_log_mask(LOG_GUEST_ERROR, "%s: bad register offset\n", __func__);
> }
> }
>
> @@ -327,7 +325,7 @@ static void virtio_mmio_update_irq(DeviceState *opaque, uint16_t vector)
> return;
> }
> level = (atomic_read(&vdev->isr) != 0);
> - DPRINTF("virtio_mmio setting IRQ %d\n", level);
> + trace_virtio_mmio_setting_irq(level);
> qemu_set_irq(proxy->irq, level);
> }
>
>
Applied to my trivial-patches branch.
Thanks,
Laurent
^ permalink raw reply [flat|nested] 15+ messages in thread
end of thread, other threads:[~2019-05-22 13:38 UTC | newest]
Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-05-03 15:44 [Qemu-devel] [PATCH v4] hw/virtio/virtio-mmio: Convert DPRINTF to trace and log Boxuan Li
2019-05-03 15:44 ` Boxuan Li
2019-05-03 15:58 ` Philippe Mathieu-Daudé
2019-05-03 15:58 ` Philippe Mathieu-Daudé
2019-05-03 16:18 ` Alex Bennée
2019-05-03 16:18 ` Alex Bennée
2019-05-03 16:23 ` LI, BO XUAN
2019-05-03 16:23 ` LI, BO XUAN
2019-05-03 17:00 ` Alex Bennée
2019-05-03 17:00 ` Alex Bennée
2019-05-03 17:13 ` LI, BO XUAN
2019-05-03 17:13 ` LI, BO XUAN
2019-05-05 10:57 ` Yuval Shaia
2019-05-05 10:57 ` Yuval Shaia
2019-05-22 13:33 ` [Qemu-devel] [Qemu-trivial] " Laurent Vivier
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.