On 18/02/2013 09:47, Andreas Färber wrote:
Am 12.02.2013 18:00, schrieb fred.kon...@greensocs.com:
From: KONRAD Frederic <fred.kon...@greensocs.com>
These structures must be made public to avoid two memory allocations for
refactored virtio devices.
Changes V2 <- V1:
* Move the dataplane include into the header (virtio-blk).
Signed-off-by: KONRAD Frederic <fred.kon...@greensocs.com>
---
hw/virtio-balloon.c | 15 ---------------
hw/virtio-balloon.h | 14 ++++++++++++++
hw/virtio-blk.c | 19 -------------------
hw/virtio-blk.h | 18 ++++++++++++++++++
hw/virtio-net.c | 50 -------------------------------------------------
hw/virtio-net.h | 51 ++++++++++++++++++++++++++++++++++++++++++++++++++
hw/virtio-rng.c | 19 -------------------
hw/virtio-rng.h | 19 +++++++++++++++++++
hw/virtio-scsi.c | 15 ---------------
hw/virtio-scsi.h | 16 ++++++++++++++++
hw/virtio-serial-bus.c | 41 ----------------------------------------
hw/virtio-serial.h | 41 ++++++++++++++++++++++++++++++++++++++++
12 files changed, 159 insertions(+), 159 deletions(-)
[...]
diff --git a/hw/virtio-scsi.h b/hw/virtio-scsi.h
index 8d9d15f..613deb5 100644
--- a/hw/virtio-scsi.h
+++ b/hw/virtio-scsi.h
@@ -16,6 +16,7 @@
#include "virtio.h"
#include "pci/pci.h"
+#include <hw/scsi.h>
This should be "hw/scsi.h" and change log should not be in commit
message. Otherwise looks okay as a pure movement, not tested.
Regards,
Andreas
Ok,
I thought the change log was allowed as it is a one patch series.
Do you think I must remove it?
I tested it, it was breaking dataplane compilation, that's why I made
the second version.
As it is a pure movement it don't require heaving testing right?
Thanks,
Fred
