[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL v3 03/14] virtio-net: remove function calls from asse
From: |
Michael S. Tsirkin |
Subject: |
[Qemu-devel] [PULL v3 03/14] virtio-net: remove function calls from assert |
Date: |
Sun, 9 Mar 2014 21:19:42 +0200 |
From: Joel Stanley <address@hidden>
peer_{de,at}tach were called from inside assert().
We don't support building without NDEBUG but it's not tidy.
Rearrange to attach peer outside assert calls.
Signed-off-by: Joel Stanley <address@hidden>
Reviewed-by: Michael S. Tsirkin <address@hidden>
Signed-off-by: Michael S. Tsirkin <address@hidden>
---
hw/net/virtio-net.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/hw/net/virtio-net.c b/hw/net/virtio-net.c
index 3c0342e..fd23c46 100644
--- a/hw/net/virtio-net.c
+++ b/hw/net/virtio-net.c
@@ -397,12 +397,15 @@ static int peer_detach(VirtIONet *n, int index)
static void virtio_net_set_queues(VirtIONet *n)
{
int i;
+ int r;
for (i = 0; i < n->max_queues; i++) {
if (i < n->curr_queues) {
- assert(!peer_attach(n, i));
+ r = peer_attach(n, i);
+ assert(!r);
} else {
- assert(!peer_detach(n, i));
+ r = peer_detach(n, i);
+ assert(!r);
}
}
}
--
MST
- [Qemu-devel] [PULL v3 00/14] acpi,pc,pci,virtio,memory bug fixes, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 01/14] acpi-build: append description for non-hotplug, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 02/14] acpi-test-data: update expected files, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 03/14] virtio-net: remove function calls from assert,
Michael S. Tsirkin <=
- [Qemu-devel] [PULL v3 04/14] memory_region_present: return false if address is not found in child MemoryRegion, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 06/14] Rework --name to use QemuOpts, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 05/14] PCIE: fix regression with coldplugged multifunction device, Michael S. Tsirkin, 2014/03/09
- [Qemu-devel] [PULL v3 08/14] Add a 'name' parameter to qemu_thread_create, Michael S. Tsirkin, 2014/03/09
- Re: [Qemu-devel] [PULL v3 08/14] Add a 'name' parameter to qemu_thread_create, Jan Kiszka, 2014/03/11
- [Qemu-devel] [PATCH] qemu-thread-posix: Fix build against older glibc version, Jan Kiszka, 2014/03/11
- Re: [Qemu-devel] [PATCH] qemu-thread-posix: Fix build against older glibc version, Peter Maydell, 2014/03/11
- Re: [Qemu-devel] [PATCH] qemu-thread-posix: Fix build against older glibc version, Dr. David Alan Gilbert, 2014/03/11
- Re: [Qemu-devel] [PATCH] qemu-thread-posix: Fix build against older glibc version, Peter Maydell, 2014/03/11
- Re: [Qemu-devel] [PATCH] qemu-thread-posix: Fix build against older glibc version, Dr. David Alan Gilbert, 2014/03/11