Skip to content

Commit

Permalink
libqos: use virtio_ids.h for device ID definitions
Browse files Browse the repository at this point in the history
Avoid redefining device IDs.  Use the standard Linux headers that are
already in the source tree.

Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com>
Message-id: 1462798061-30382-2-git-send-email-stefanha@redhat.com
  • Loading branch information
stefanhaRH committed Jun 20, 2016
1 parent 482b618 commit 8ac9e20
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 16 deletions.
9 changes: 0 additions & 9 deletions tests/libqos/virtio.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,15 +19,6 @@
#define QVIRTIO_DRIVER 0x2
#define QVIRTIO_DRIVER_OK 0x4

#define QVIRTIO_NET_DEVICE_ID 0x1
#define QVIRTIO_BLK_DEVICE_ID 0x2
#define QVIRTIO_CONSOLE_DEVICE_ID 0x3
#define QVIRTIO_RNG_DEVICE_ID 0x4
#define QVIRTIO_BALLOON_DEVICE_ID 0x5
#define QVIRTIO_RPMSG_DEVICE_ID 0x7
#define QVIRTIO_SCSI_DEVICE_ID 0x8
#define QVIRTIO_9P_DEVICE_ID 0x9

#define QVIRTIO_F_NOTIFY_ON_EMPTY 0x01000000
#define QVIRTIO_F_ANY_LAYOUT 0x08000000
#define QVIRTIO_F_RING_INDIRECT_DESC 0x10000000
Expand Down
7 changes: 4 additions & 3 deletions tests/virtio-blk-test.c
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@
#include "libqos/malloc-pc.h"
#include "libqos/malloc-generic.h"
#include "qemu/bswap.h"
#include "standard-headers/linux/virtio_ids.h"

#define QVIRTIO_BLK_F_BARRIER 0x00000001
#define QVIRTIO_BLK_F_SIZE_MAX 0x00000002
Expand Down Expand Up @@ -118,9 +119,9 @@ static QVirtioPCIDevice *virtio_blk_pci_init(QPCIBus *bus, int slot)
{
QVirtioPCIDevice *dev;

dev = qvirtio_pci_device_find(bus, QVIRTIO_BLK_DEVICE_ID);
dev = qvirtio_pci_device_find(bus, VIRTIO_ID_BLOCK);
g_assert(dev != NULL);
g_assert_cmphex(dev->vdev.device_type, ==, QVIRTIO_BLK_DEVICE_ID);
g_assert_cmphex(dev->vdev.device_type, ==, VIRTIO_ID_BLOCK);
g_assert_cmphex(dev->pdev->devfn, ==, ((slot << 3) | PCI_FN));

qvirtio_pci_device_enable(dev);
Expand Down Expand Up @@ -732,7 +733,7 @@ static void mmio_basic(void)

dev = qvirtio_mmio_init_device(MMIO_DEV_BASE_ADDR, MMIO_PAGE_SIZE);
g_assert(dev != NULL);
g_assert_cmphex(dev->vdev.device_type, ==, QVIRTIO_BLK_DEVICE_ID);
g_assert_cmphex(dev->vdev.device_type, ==, VIRTIO_ID_BLOCK);

qvirtio_reset(&qvirtio_mmio, &dev->vdev);
qvirtio_set_acknowledge(&qvirtio_mmio, &dev->vdev);
Expand Down
5 changes: 3 additions & 2 deletions tests/virtio-net-test.c
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,7 @@
#include "libqos/malloc-generic.h"
#include "qemu/bswap.h"
#include "hw/virtio/virtio-net.h"
#include "standard-headers/linux/virtio_ids.h"

#define PCI_SLOT_HP 0x06
#define PCI_SLOT 0x04
Expand All @@ -39,9 +40,9 @@ static QVirtioPCIDevice *virtio_net_pci_init(QPCIBus *bus, int slot)
{
QVirtioPCIDevice *dev;

dev = qvirtio_pci_device_find(bus, QVIRTIO_NET_DEVICE_ID);
dev = qvirtio_pci_device_find(bus, VIRTIO_ID_NET);
g_assert(dev != NULL);
g_assert_cmphex(dev->vdev.device_type, ==, QVIRTIO_NET_DEVICE_ID);
g_assert_cmphex(dev->vdev.device_type, ==, VIRTIO_ID_NET);

qvirtio_pci_device_enable(dev);
qvirtio_reset(&qvirtio_pci, &dev->vdev);
Expand Down
5 changes: 3 additions & 2 deletions tests/virtio-scsi-test.c
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@
#include "libqos/malloc.h"
#include "libqos/malloc-pc.h"
#include "libqos/malloc-generic.h"
#include "standard-headers/linux/virtio_ids.h"

#define PCI_SLOT 0x02
#define PCI_FN 0x00
Expand Down Expand Up @@ -163,10 +164,10 @@ static QVirtIOSCSI *qvirtio_scsi_pci_init(int slot)
vs->alloc = pc_alloc_init();
vs->bus = qpci_init_pc();

dev = qvirtio_pci_device_find(vs->bus, QVIRTIO_SCSI_DEVICE_ID);
dev = qvirtio_pci_device_find(vs->bus, VIRTIO_ID_SCSI);
vs->dev = (QVirtioDevice *)dev;
g_assert(dev != NULL);
g_assert_cmphex(vs->dev->device_type, ==, QVIRTIO_SCSI_DEVICE_ID);
g_assert_cmphex(vs->dev->device_type, ==, VIRTIO_ID_SCSI);

qvirtio_pci_device_enable(dev);
qvirtio_reset(&qvirtio_pci, vs->dev);
Expand Down

0 comments on commit 8ac9e20

Please sign in to comment.