libqos: Remove PCI assumptions in constants of virtio driver
Convert PCI-specific constants names of libqos virtio driver. Signed-off-by: Marc Marí <marc.mari.barcelo@gmail.com> Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com> Message-id: 1424812915-25728-4-git-send-email-marc.mari.barcelo@gmail.com Signed-off-by: Stefan Hajnoczi <stefanha@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
		
							parent
							
								
									38d8364f4f
								
							
						
					
					
						commit
						50311a81f8
					
				| @ -102,31 +102,31 @@ static uint64_t qvirtio_pci_config_readq(QVirtioDevice *d, uint64_t addr) | ||||
| static uint32_t qvirtio_pci_get_features(QVirtioDevice *d) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     return qpci_io_readl(dev->pdev, dev->addr + QVIRTIO_DEVICE_FEATURES); | ||||
|     return qpci_io_readl(dev->pdev, dev->addr + QVIRTIO_PCI_DEVICE_FEATURES); | ||||
| } | ||||
| 
 | ||||
| static void qvirtio_pci_set_features(QVirtioDevice *d, uint32_t features) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     qpci_io_writel(dev->pdev, dev->addr + QVIRTIO_GUEST_FEATURES, features); | ||||
|     qpci_io_writel(dev->pdev, dev->addr + QVIRTIO_PCI_GUEST_FEATURES, features); | ||||
| } | ||||
| 
 | ||||
| static uint32_t qvirtio_pci_get_guest_features(QVirtioDevice *d) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     return qpci_io_readl(dev->pdev, dev->addr + QVIRTIO_GUEST_FEATURES); | ||||
|     return qpci_io_readl(dev->pdev, dev->addr + QVIRTIO_PCI_GUEST_FEATURES); | ||||
| } | ||||
| 
 | ||||
| static uint8_t qvirtio_pci_get_status(QVirtioDevice *d) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_DEVICE_STATUS); | ||||
|     return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_PCI_DEVICE_STATUS); | ||||
| } | ||||
| 
 | ||||
| static void qvirtio_pci_set_status(QVirtioDevice *d, uint8_t status) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     qpci_io_writeb(dev->pdev, dev->addr + QVIRTIO_DEVICE_STATUS, status); | ||||
|     qpci_io_writeb(dev->pdev, dev->addr + QVIRTIO_PCI_DEVICE_STATUS, status); | ||||
| } | ||||
| 
 | ||||
| static bool qvirtio_pci_get_queue_isr_status(QVirtioDevice *d, QVirtQueue *vq) | ||||
| @ -146,7 +146,7 @@ static bool qvirtio_pci_get_queue_isr_status(QVirtioDevice *d, QVirtQueue *vq) | ||||
|             return data == vqpci->msix_data; | ||||
|         } | ||||
|     } else { | ||||
|         return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_ISR_STATUS) & 1; | ||||
|         return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_PCI_ISR_STATUS) & 1; | ||||
|     } | ||||
| } | ||||
| 
 | ||||
| @ -166,26 +166,26 @@ static bool qvirtio_pci_get_config_isr_status(QVirtioDevice *d) | ||||
|             return data == dev->config_msix_data; | ||||
|         } | ||||
|     } else { | ||||
|         return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_ISR_STATUS) & 2; | ||||
|         return qpci_io_readb(dev->pdev, dev->addr + QVIRTIO_PCI_ISR_STATUS) & 2; | ||||
|     } | ||||
| } | ||||
| 
 | ||||
| static void qvirtio_pci_queue_select(QVirtioDevice *d, uint16_t index) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     qpci_io_writeb(dev->pdev, dev->addr + QVIRTIO_QUEUE_SELECT, index); | ||||
|     qpci_io_writeb(dev->pdev, dev->addr + QVIRTIO_PCI_QUEUE_SELECT, index); | ||||
| } | ||||
| 
 | ||||
| static uint16_t qvirtio_pci_get_queue_size(QVirtioDevice *d) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     return qpci_io_readw(dev->pdev, dev->addr + QVIRTIO_QUEUE_SIZE); | ||||
|     return qpci_io_readw(dev->pdev, dev->addr + QVIRTIO_PCI_QUEUE_SIZE); | ||||
| } | ||||
| 
 | ||||
| static void qvirtio_pci_set_queue_address(QVirtioDevice *d, uint32_t pfn) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     qpci_io_writel(dev->pdev, dev->addr + QVIRTIO_QUEUE_ADDRESS, pfn); | ||||
|     qpci_io_writel(dev->pdev, dev->addr + QVIRTIO_PCI_QUEUE_ADDRESS, pfn); | ||||
| } | ||||
| 
 | ||||
| static QVirtQueue *qvirtio_pci_virtqueue_setup(QVirtioDevice *d, | ||||
| @ -227,7 +227,7 @@ static QVirtQueue *qvirtio_pci_virtqueue_setup(QVirtioDevice *d, | ||||
| static void qvirtio_pci_virtqueue_kick(QVirtioDevice *d, QVirtQueue *vq) | ||||
| { | ||||
|     QVirtioPCIDevice *dev = (QVirtioPCIDevice *)d; | ||||
|     qpci_io_writew(dev->pdev, dev->addr + QVIRTIO_QUEUE_NOTIFY, vq->index); | ||||
|     qpci_io_writew(dev->pdev, dev->addr + QVIRTIO_PCI_QUEUE_NOTIFY, vq->index); | ||||
| } | ||||
| 
 | ||||
| const QVirtioBus qvirtio_pci = { | ||||
| @ -307,8 +307,8 @@ void qvirtqueue_pci_msix_setup(QVirtioPCIDevice *d, QVirtQueuePCI *vqpci, | ||||
|                                         control & ~PCI_MSIX_ENTRY_CTRL_MASKBIT); | ||||
| 
 | ||||
|     qvirtio_pci_queue_select(&d->vdev, vqpci->vq.index); | ||||
|     qpci_io_writew(d->pdev, d->addr + QVIRTIO_MSIX_QUEUE_VECTOR, entry); | ||||
|     vector = qpci_io_readw(d->pdev, d->addr + QVIRTIO_MSIX_QUEUE_VECTOR); | ||||
|     qpci_io_writew(d->pdev, d->addr + QVIRTIO_PCI_MSIX_QUEUE_VECTOR, entry); | ||||
|     vector = qpci_io_readw(d->pdev, d->addr + QVIRTIO_PCI_MSIX_QUEUE_VECTOR); | ||||
|     g_assert_cmphex(vector, !=, QVIRTIO_MSI_NO_VECTOR); | ||||
| } | ||||
| 
 | ||||
| @ -339,7 +339,7 @@ void qvirtio_pci_set_msix_configuration_vector(QVirtioPCIDevice *d, | ||||
|     qpci_io_writel(d->pdev, addr + PCI_MSIX_ENTRY_VECTOR_CTRL, | ||||
|                                         control & ~PCI_MSIX_ENTRY_CTRL_MASKBIT); | ||||
| 
 | ||||
|     qpci_io_writew(d->pdev, d->addr + QVIRTIO_MSIX_CONF_VECTOR, entry); | ||||
|     vector = qpci_io_readw(d->pdev, d->addr + QVIRTIO_MSIX_CONF_VECTOR); | ||||
|     qpci_io_writew(d->pdev, d->addr + QVIRTIO_PCI_MSIX_CONF_VECTOR, entry); | ||||
|     vector = qpci_io_readw(d->pdev, d->addr + QVIRTIO_PCI_MSIX_CONF_VECTOR); | ||||
|     g_assert_cmphex(vector, !=, QVIRTIO_MSI_NO_VECTOR); | ||||
| } | ||||
|  | ||||
| @ -13,18 +13,18 @@ | ||||
| #include "libqos/virtio.h" | ||||
| #include "libqos/pci.h" | ||||
| 
 | ||||
| #define QVIRTIO_DEVICE_FEATURES         0x00 | ||||
| #define QVIRTIO_GUEST_FEATURES          0x04 | ||||
| #define QVIRTIO_QUEUE_ADDRESS           0x08 | ||||
| #define QVIRTIO_QUEUE_SIZE              0x0C | ||||
| #define QVIRTIO_QUEUE_SELECT            0x0E | ||||
| #define QVIRTIO_QUEUE_NOTIFY            0x10 | ||||
| #define QVIRTIO_DEVICE_STATUS           0x12 | ||||
| #define QVIRTIO_ISR_STATUS              0x13 | ||||
| #define QVIRTIO_MSIX_CONF_VECTOR        0x14 | ||||
| #define QVIRTIO_MSIX_QUEUE_VECTOR       0x16 | ||||
| #define QVIRTIO_DEVICE_SPECIFIC_MSIX    0x18 | ||||
| #define QVIRTIO_DEVICE_SPECIFIC_NO_MSIX 0x14 | ||||
| #define QVIRTIO_PCI_DEVICE_FEATURES         0x00 | ||||
| #define QVIRTIO_PCI_GUEST_FEATURES          0x04 | ||||
| #define QVIRTIO_PCI_QUEUE_ADDRESS           0x08 | ||||
| #define QVIRTIO_PCI_QUEUE_SIZE              0x0C | ||||
| #define QVIRTIO_PCI_QUEUE_SELECT            0x0E | ||||
| #define QVIRTIO_PCI_QUEUE_NOTIFY            0x10 | ||||
| #define QVIRTIO_PCI_DEVICE_STATUS           0x12 | ||||
| #define QVIRTIO_PCI_ISR_STATUS              0x13 | ||||
| #define QVIRTIO_PCI_MSIX_CONF_VECTOR        0x14 | ||||
| #define QVIRTIO_PCI_MSIX_QUEUE_VECTOR       0x16 | ||||
| #define QVIRTIO_PCI_DEVICE_SPECIFIC_MSIX    0x18 | ||||
| #define QVIRTIO_PCI_DEVICE_SPECIFIC_NO_MSIX 0x14 | ||||
| 
 | ||||
| #define QVIRTIO_PCI_ALIGN   4096 | ||||
| 
 | ||||
|  | ||||
| @ -157,6 +157,7 @@ static void test_basic(const QVirtioBus *bus, QVirtioDevice *dev, | ||||
|     char *data; | ||||
| 
 | ||||
|     capacity = qvirtio_config_readq(bus, dev, device_specific); | ||||
| 
 | ||||
|     g_assert_cmpint(capacity, ==, TEST_IMAGE_SIZE / 512); | ||||
| 
 | ||||
|     features = qvirtio_get_features(bus, dev); | ||||
| @ -283,7 +284,7 @@ static void pci_basic(void) | ||||
|                                                                     alloc, 0); | ||||
| 
 | ||||
|     /* MSI-X is not enabled */ | ||||
|     addr = dev->addr + QVIRTIO_DEVICE_SPECIFIC_NO_MSIX; | ||||
|     addr = dev->addr + QVIRTIO_PCI_DEVICE_SPECIFIC_NO_MSIX; | ||||
| 
 | ||||
|     test_basic(&qvirtio_pci, &dev->vdev, alloc, &vqpci->vq, | ||||
|                                                     (uint64_t)(uintptr_t)addr); | ||||
| @ -318,7 +319,7 @@ static void pci_indirect(void) | ||||
|     dev = virtio_blk_pci_init(bus, PCI_SLOT); | ||||
| 
 | ||||
|     /* MSI-X is not enabled */ | ||||
|     addr = dev->addr + QVIRTIO_DEVICE_SPECIFIC_NO_MSIX; | ||||
|     addr = dev->addr + QVIRTIO_PCI_DEVICE_SPECIFIC_NO_MSIX; | ||||
| 
 | ||||
|     capacity = qvirtio_config_readq(&qvirtio_pci, &dev->vdev, | ||||
|                                                     (uint64_t)(uintptr_t)addr); | ||||
| @ -412,7 +413,7 @@ static void pci_config(void) | ||||
|     dev = virtio_blk_pci_init(bus, PCI_SLOT); | ||||
| 
 | ||||
|     /* MSI-X is not enabled */ | ||||
|     addr = dev->addr + QVIRTIO_DEVICE_SPECIFIC_NO_MSIX; | ||||
|     addr = dev->addr + QVIRTIO_PCI_DEVICE_SPECIFIC_NO_MSIX; | ||||
| 
 | ||||
|     capacity = qvirtio_config_readq(&qvirtio_pci, &dev->vdev, | ||||
|                                                     (uint64_t)(uintptr_t)addr); | ||||
| @ -459,7 +460,7 @@ static void pci_msix(void) | ||||
|     qvirtio_pci_set_msix_configuration_vector(dev, alloc, 0); | ||||
| 
 | ||||
|     /* MSI-X is enabled */ | ||||
|     addr = dev->addr + QVIRTIO_DEVICE_SPECIFIC_MSIX; | ||||
|     addr = dev->addr + QVIRTIO_PCI_DEVICE_SPECIFIC_MSIX; | ||||
| 
 | ||||
|     capacity = qvirtio_config_readq(&qvirtio_pci, &dev->vdev, | ||||
|                                                     (uint64_t)(uintptr_t)addr); | ||||
| @ -572,7 +573,7 @@ static void pci_idx(void) | ||||
|     qvirtio_pci_set_msix_configuration_vector(dev, alloc, 0); | ||||
| 
 | ||||
|     /* MSI-X is enabled */ | ||||
|     addr = dev->addr + QVIRTIO_DEVICE_SPECIFIC_MSIX; | ||||
|     addr = dev->addr + QVIRTIO_PCI_DEVICE_SPECIFIC_MSIX; | ||||
| 
 | ||||
|     capacity = qvirtio_config_readq(&qvirtio_pci, &dev->vdev, | ||||
|                                                     (uint64_t)(uintptr_t)addr); | ||||
|  | ||||
		Loading…
	
	
			
			x
			
			
		
	
		Reference in New Issue
	
	Block a user
	 Marc Marí
						Marc Marí