xref: /kvmtool/virtio/pci.c (revision 1382aba018748f7d2494ab2e22d51ff91939e3e4)
136f5dc91SSasha Levin #include "kvm/virtio-pci.h"
236f5dc91SSasha Levin 
336f5dc91SSasha Levin #include "kvm/ioport.h"
436f5dc91SSasha Levin #include "kvm/kvm.h"
536f5dc91SSasha Levin #include "kvm/virtio-pci-dev.h"
636f5dc91SSasha Levin #include "kvm/irq.h"
736f5dc91SSasha Levin #include "kvm/virtio.h"
81599d724SSasha Levin #include "kvm/ioeventfd.h"
91c47ce69SSasha Levin #include "kvm/virtio-trans.h"
1036f5dc91SSasha Levin 
1136f5dc91SSasha Levin #include <linux/virtio_pci.h>
1236f5dc91SSasha Levin #include <string.h>
1336f5dc91SSasha Levin 
141c47ce69SSasha Levin struct virtio_trans_ops *virtio_pci__get_trans_ops(void)
151c47ce69SSasha Levin {
161c47ce69SSasha Levin 	static struct virtio_trans_ops virtio_pci_trans = (struct virtio_trans_ops) {
171c47ce69SSasha Levin 		.signal_vq	= virtio_pci__signal_vq,
181c47ce69SSasha Levin 		.signal_config	= virtio_pci__signal_config,
191c47ce69SSasha Levin 		.init		= virtio_pci__init,
201c47ce69SSasha Levin 	};
211c47ce69SSasha Levin 	return &virtio_pci_trans;
221c47ce69SSasha Levin };
231c47ce69SSasha Levin 
241599d724SSasha Levin static void virtio_pci__ioevent_callback(struct kvm *kvm, void *param)
251599d724SSasha Levin {
261599d724SSasha Levin 	struct virtio_pci_ioevent_param *ioeventfd = param;
271c47ce69SSasha Levin 	struct virtio_pci *vpci = ioeventfd->vtrans->virtio;
281599d724SSasha Levin 
291c47ce69SSasha Levin 	ioeventfd->vtrans->virtio_ops->notify_vq(kvm, vpci->dev, ioeventfd->vq);
301599d724SSasha Levin }
311599d724SSasha Levin 
321c47ce69SSasha Levin static int virtio_pci__init_ioeventfd(struct kvm *kvm, struct virtio_trans *vtrans, u32 vq)
331599d724SSasha Levin {
341599d724SSasha Levin 	struct ioevent ioevent;
351c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
361599d724SSasha Levin 
371599d724SSasha Levin 	vpci->ioeventfds[vq] = (struct virtio_pci_ioevent_param) {
381c47ce69SSasha Levin 		.vtrans		= vtrans,
391599d724SSasha Levin 		.vq		= vq,
401599d724SSasha Levin 	};
411599d724SSasha Levin 
421599d724SSasha Levin 	ioevent = (struct ioevent) {
431599d724SSasha Levin 		.io_addr	= vpci->base_addr + VIRTIO_PCI_QUEUE_NOTIFY,
441599d724SSasha Levin 		.io_len		= sizeof(u16),
451599d724SSasha Levin 		.fn		= virtio_pci__ioevent_callback,
461599d724SSasha Levin 		.fn_ptr		= &vpci->ioeventfds[vq],
471599d724SSasha Levin 		.datamatch	= vq,
481599d724SSasha Levin 		.fn_kvm		= kvm,
491599d724SSasha Levin 		.fd		= eventfd(0, 0),
501599d724SSasha Levin 	};
511599d724SSasha Levin 
521599d724SSasha Levin 	ioeventfd__add_event(&ioevent);
531599d724SSasha Levin 
54263b80e8SSasha Levin 	if (vtrans->virtio_ops->notify_vq_eventfd)
55263b80e8SSasha Levin 		vtrans->virtio_ops->notify_vq_eventfd(kvm, vpci->dev, vq, ioevent.fd);
56263b80e8SSasha Levin 
571599d724SSasha Levin 	return 0;
581599d724SSasha Levin }
591599d724SSasha Levin 
6006f48103SSasha Levin static inline bool virtio_pci__msix_enabled(struct virtio_pci *vpci)
6106f48103SSasha Levin {
6206f48103SSasha Levin 	return vpci->pci_hdr.msix.ctrl & PCI_MSIX_FLAGS_ENABLE;
6306f48103SSasha Levin }
6406f48103SSasha Levin 
651c47ce69SSasha Levin static bool virtio_pci__specific_io_in(struct kvm *kvm, struct virtio_trans *vtrans, u16 port,
6636f5dc91SSasha Levin 					void *data, int size, int offset)
6736f5dc91SSasha Levin {
6836f5dc91SSasha Levin 	u32 config_offset;
691c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
7006f48103SSasha Levin 	int type = virtio__get_dev_specific_field(offset - 20,
7106f48103SSasha Levin 							virtio_pci__msix_enabled(vpci),
72*1382aba0SSasha Levin 							&config_offset);
7336f5dc91SSasha Levin 	if (type == VIRTIO_PCI_O_MSIX) {
7436f5dc91SSasha Levin 		switch (offset) {
7536f5dc91SSasha Levin 		case VIRTIO_MSI_CONFIG_VECTOR:
7636f5dc91SSasha Levin 			ioport__write16(data, vpci->config_vector);
7736f5dc91SSasha Levin 			break;
7836f5dc91SSasha Levin 		case VIRTIO_MSI_QUEUE_VECTOR:
7936f5dc91SSasha Levin 			ioport__write16(data, vpci->vq_vector[vpci->queue_selector]);
8036f5dc91SSasha Levin 			break;
8136f5dc91SSasha Levin 		};
8236f5dc91SSasha Levin 
8336f5dc91SSasha Levin 		return true;
8436f5dc91SSasha Levin 	} else if (type == VIRTIO_PCI_O_CONFIG) {
8536f5dc91SSasha Levin 		u8 cfg;
8636f5dc91SSasha Levin 
871c47ce69SSasha Levin 		cfg = vtrans->virtio_ops->get_config(kvm, vpci->dev, config_offset);
8836f5dc91SSasha Levin 		ioport__write8(data, cfg);
8936f5dc91SSasha Levin 		return true;
9036f5dc91SSasha Levin 	}
9136f5dc91SSasha Levin 
9236f5dc91SSasha Levin 	return false;
9336f5dc91SSasha Levin }
9436f5dc91SSasha Levin 
9536f5dc91SSasha Levin static bool virtio_pci__io_in(struct ioport *ioport, struct kvm *kvm, u16 port, void *data, int size)
9636f5dc91SSasha Levin {
9736f5dc91SSasha Levin 	unsigned long offset;
9836f5dc91SSasha Levin 	bool ret = true;
991c47ce69SSasha Levin 	struct virtio_trans *vtrans;
10036f5dc91SSasha Levin 	struct virtio_pci *vpci;
10136f5dc91SSasha Levin 	u32 val;
10236f5dc91SSasha Levin 
1031c47ce69SSasha Levin 	vtrans = ioport->priv;
1041c47ce69SSasha Levin 	vpci = vtrans->virtio;
10536f5dc91SSasha Levin 	offset = port - vpci->base_addr;
10636f5dc91SSasha Levin 
10736f5dc91SSasha Levin 	switch (offset) {
10836f5dc91SSasha Levin 	case VIRTIO_PCI_HOST_FEATURES:
1091c47ce69SSasha Levin 		val = vtrans->virtio_ops->get_host_features(kvm, vpci->dev);
11036f5dc91SSasha Levin 		ioport__write32(data, val);
11136f5dc91SSasha Levin 		break;
11236f5dc91SSasha Levin 	case VIRTIO_PCI_QUEUE_PFN:
1131c47ce69SSasha Levin 		val = vtrans->virtio_ops->get_pfn_vq(kvm, vpci->dev, vpci->queue_selector);
11436f5dc91SSasha Levin 		ioport__write32(data, val);
11536f5dc91SSasha Levin 		break;
11636f5dc91SSasha Levin 	case VIRTIO_PCI_QUEUE_NUM:
1171c47ce69SSasha Levin 		val = vtrans->virtio_ops->get_size_vq(kvm, vpci->dev, vpci->queue_selector);
11836f5dc91SSasha Levin 		ioport__write32(data, val);
11936f5dc91SSasha Levin 		break;
12036f5dc91SSasha Levin 		break;
12136f5dc91SSasha Levin 	case VIRTIO_PCI_STATUS:
12236f5dc91SSasha Levin 		ioport__write8(data, vpci->status);
12336f5dc91SSasha Levin 		break;
12436f5dc91SSasha Levin 	case VIRTIO_PCI_ISR:
12536f5dc91SSasha Levin 		ioport__write8(data, vpci->isr);
12636f5dc91SSasha Levin 		kvm__irq_line(kvm, vpci->pci_hdr.irq_line, VIRTIO_IRQ_LOW);
12736f5dc91SSasha Levin 		vpci->isr = VIRTIO_IRQ_LOW;
12836f5dc91SSasha Levin 		break;
12936f5dc91SSasha Levin 	default:
1301c47ce69SSasha Levin 		ret = virtio_pci__specific_io_in(kvm, vtrans, port, data, size, offset);
13136f5dc91SSasha Levin 		break;
13236f5dc91SSasha Levin 	};
13336f5dc91SSasha Levin 
13436f5dc91SSasha Levin 	return ret;
13536f5dc91SSasha Levin }
13636f5dc91SSasha Levin 
1371c47ce69SSasha Levin static bool virtio_pci__specific_io_out(struct kvm *kvm, struct virtio_trans *vtrans, u16 port,
13836f5dc91SSasha Levin 					void *data, int size, int offset)
13936f5dc91SSasha Levin {
1401c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
14136f5dc91SSasha Levin 	u32 config_offset, gsi, vec;
14206f48103SSasha Levin 	int type = virtio__get_dev_specific_field(offset - 20, virtio_pci__msix_enabled(vpci),
143*1382aba0SSasha Levin 							&config_offset);
14436f5dc91SSasha Levin 	if (type == VIRTIO_PCI_O_MSIX) {
14536f5dc91SSasha Levin 		switch (offset) {
14636f5dc91SSasha Levin 		case VIRTIO_MSI_CONFIG_VECTOR:
14736f5dc91SSasha Levin 			vec = vpci->config_vector = ioport__read16(data);
14836f5dc91SSasha Levin 
1491de74957SSasha Levin 			gsi = irq__add_msix_route(kvm, &vpci->msix_table[vec].msg);
15036f5dc91SSasha Levin 
15136f5dc91SSasha Levin 			vpci->config_gsi = gsi;
15236f5dc91SSasha Levin 			break;
15336f5dc91SSasha Levin 		case VIRTIO_MSI_QUEUE_VECTOR: {
15436f5dc91SSasha Levin 			vec = vpci->vq_vector[vpci->queue_selector] = ioport__read16(data);
15536f5dc91SSasha Levin 
1561de74957SSasha Levin 			gsi = irq__add_msix_route(kvm, &vpci->msix_table[vec].msg);
15736f5dc91SSasha Levin 			vpci->gsis[vpci->queue_selector] = gsi;
158263b80e8SSasha Levin 			if (vtrans->virtio_ops->notify_vq_gsi)
159263b80e8SSasha Levin 				vtrans->virtio_ops->notify_vq_gsi(kvm, vpci->dev,
160263b80e8SSasha Levin 							vpci->queue_selector, gsi);
16136f5dc91SSasha Levin 			break;
16236f5dc91SSasha Levin 		}
16336f5dc91SSasha Levin 		};
16436f5dc91SSasha Levin 
16536f5dc91SSasha Levin 		return true;
16636f5dc91SSasha Levin 	} else if (type == VIRTIO_PCI_O_CONFIG) {
1671c47ce69SSasha Levin 		vtrans->virtio_ops->set_config(kvm, vpci->dev, *(u8 *)data, config_offset);
16836f5dc91SSasha Levin 
16936f5dc91SSasha Levin 		return true;
17036f5dc91SSasha Levin 	}
17136f5dc91SSasha Levin 
17236f5dc91SSasha Levin 	return false;
17336f5dc91SSasha Levin }
17436f5dc91SSasha Levin 
17536f5dc91SSasha Levin static bool virtio_pci__io_out(struct ioport *ioport, struct kvm *kvm, u16 port, void *data, int size)
17636f5dc91SSasha Levin {
17736f5dc91SSasha Levin 	unsigned long offset;
17836f5dc91SSasha Levin 	bool ret = true;
1791c47ce69SSasha Levin 	struct virtio_trans *vtrans;
18036f5dc91SSasha Levin 	struct virtio_pci *vpci;
18136f5dc91SSasha Levin 	u32 val;
18236f5dc91SSasha Levin 
1831c47ce69SSasha Levin 	vtrans = ioport->priv;
1841c47ce69SSasha Levin 	vpci = vtrans->virtio;
18536f5dc91SSasha Levin 	offset = port - vpci->base_addr;
18636f5dc91SSasha Levin 
18736f5dc91SSasha Levin 	switch (offset) {
18836f5dc91SSasha Levin 	case VIRTIO_PCI_GUEST_FEATURES:
18936f5dc91SSasha Levin 		val = ioport__read32(data);
19013f9a438SSasha Levin 		vtrans->virtio_ops->set_guest_features(kvm, vpci->dev, val);
19136f5dc91SSasha Levin 		break;
19236f5dc91SSasha Levin 	case VIRTIO_PCI_QUEUE_PFN:
19336f5dc91SSasha Levin 		val = ioport__read32(data);
1941c47ce69SSasha Levin 		virtio_pci__init_ioeventfd(kvm, vtrans, vpci->queue_selector);
1951c47ce69SSasha Levin 		vtrans->virtio_ops->init_vq(kvm, vpci->dev, vpci->queue_selector, val);
19636f5dc91SSasha Levin 		break;
19736f5dc91SSasha Levin 	case VIRTIO_PCI_QUEUE_SEL:
19836f5dc91SSasha Levin 		vpci->queue_selector	= ioport__read16(data);
19936f5dc91SSasha Levin 		break;
20036f5dc91SSasha Levin 	case VIRTIO_PCI_QUEUE_NOTIFY:
20136f5dc91SSasha Levin 		val			= ioport__read16(data);
2021c47ce69SSasha Levin 		vtrans->virtio_ops->notify_vq(kvm, vpci->dev, val);
20336f5dc91SSasha Levin 		break;
20436f5dc91SSasha Levin 	case VIRTIO_PCI_STATUS:
20536f5dc91SSasha Levin 		vpci->status		= ioport__read8(data);
20636f5dc91SSasha Levin 		break;
20736f5dc91SSasha Levin 	default:
2081c47ce69SSasha Levin 		ret = virtio_pci__specific_io_out(kvm, vtrans, port, data, size, offset);
20936f5dc91SSasha Levin 		break;
21036f5dc91SSasha Levin 	};
21136f5dc91SSasha Levin 
21236f5dc91SSasha Levin 	return ret;
21336f5dc91SSasha Levin }
21436f5dc91SSasha Levin 
21536f5dc91SSasha Levin static struct ioport_operations virtio_pci__io_ops = {
21636f5dc91SSasha Levin 	.io_in	= virtio_pci__io_in,
21736f5dc91SSasha Levin 	.io_out	= virtio_pci__io_out,
21836f5dc91SSasha Levin };
21936f5dc91SSasha Levin 
22006f48103SSasha Levin static void callback_mmio_table(u64 addr, u8 *data, u32 len, u8 is_write, void *ptr)
22136f5dc91SSasha Levin {
22236f5dc91SSasha Levin 	struct virtio_pci *vpci = ptr;
22306f48103SSasha Levin 	void *table = &vpci->msix_table;
22436f5dc91SSasha Levin 
22536f5dc91SSasha Levin 	if (is_write)
22636f5dc91SSasha Levin 		memcpy(table + addr - vpci->msix_io_block, data, len);
22736f5dc91SSasha Levin 	else
22836f5dc91SSasha Levin 		memcpy(data, table + addr - vpci->msix_io_block, len);
22936f5dc91SSasha Levin }
23036f5dc91SSasha Levin 
23106f48103SSasha Levin static void callback_mmio_pba(u64 addr, u8 *data, u32 len, u8 is_write, void *ptr)
23206f48103SSasha Levin {
23306f48103SSasha Levin 	struct virtio_pci *vpci = ptr;
23406f48103SSasha Levin 	void *pba = &vpci->msix_pba;
23506f48103SSasha Levin 
23606f48103SSasha Levin 	if (is_write)
23706f48103SSasha Levin 		memcpy(pba + addr - vpci->msix_pba_block, data, len);
23806f48103SSasha Levin 	else
23906f48103SSasha Levin 		memcpy(data, pba + addr - vpci->msix_pba_block, len);
24006f48103SSasha Levin }
24106f48103SSasha Levin 
2421c47ce69SSasha Levin int virtio_pci__signal_vq(struct kvm *kvm, struct virtio_trans *vtrans, u32 vq)
24336f5dc91SSasha Levin {
2441c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
24506f48103SSasha Levin 	int tbl = vpci->vq_vector[vq];
24636f5dc91SSasha Levin 
24706f48103SSasha Levin 	if (virtio_pci__msix_enabled(vpci)) {
24806f48103SSasha Levin 		if (vpci->pci_hdr.msix.ctrl & PCI_MSIX_FLAGS_MASKALL ||
24906f48103SSasha Levin 			vpci->msix_table[tbl].ctrl & PCI_MSIX_ENTRY_CTRL_MASKBIT) {
25006f48103SSasha Levin 
25106f48103SSasha Levin 			vpci->msix_pba |= 1 << tbl;
25206f48103SSasha Levin 			return 0;
25306f48103SSasha Levin 		}
25406f48103SSasha Levin 
25506f48103SSasha Levin 		kvm__irq_trigger(kvm, vpci->gsis[vq]);
25606f48103SSasha Levin 	} else {
257a36eca7bSSasha Levin 		vpci->isr = VIRTIO_IRQ_HIGH;
25806f48103SSasha Levin 		kvm__irq_trigger(kvm, vpci->pci_hdr.irq_line);
25906f48103SSasha Levin 	}
26036f5dc91SSasha Levin 	return 0;
26136f5dc91SSasha Levin }
26236f5dc91SSasha Levin 
2631c47ce69SSasha Levin int virtio_pci__signal_config(struct kvm *kvm, struct virtio_trans *vtrans)
26436f5dc91SSasha Levin {
2651c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
26606f48103SSasha Levin 	int tbl = vpci->config_vector;
26706f48103SSasha Levin 
26806f48103SSasha Levin 	if (virtio_pci__msix_enabled(vpci)) {
26906f48103SSasha Levin 		if (vpci->pci_hdr.msix.ctrl & PCI_MSIX_FLAGS_MASKALL ||
27006f48103SSasha Levin 			vpci->msix_table[tbl].ctrl & PCI_MSIX_ENTRY_CTRL_MASKBIT) {
27106f48103SSasha Levin 
27206f48103SSasha Levin 			vpci->msix_pba |= 1 << tbl;
27306f48103SSasha Levin 			return 0;
27406f48103SSasha Levin 		}
27506f48103SSasha Levin 
27606f48103SSasha Levin 		kvm__irq_trigger(kvm, vpci->config_gsi);
27706f48103SSasha Levin 	} else {
27806f48103SSasha Levin 		vpci->isr = VIRTIO_PCI_ISR_CONFIG;
27906f48103SSasha Levin 		kvm__irq_trigger(kvm, vpci->pci_hdr.irq_line);
28006f48103SSasha Levin 	}
28136f5dc91SSasha Levin 
28236f5dc91SSasha Levin 	return 0;
28336f5dc91SSasha Levin }
28436f5dc91SSasha Levin 
2851c47ce69SSasha Levin int virtio_pci__init(struct kvm *kvm, struct virtio_trans *vtrans, void *dev,
286507e02d8SAsias He 			int device_id, int subsys_id, int class)
28736f5dc91SSasha Levin {
2881c47ce69SSasha Levin 	struct virtio_pci *vpci = vtrans->virtio;
28936f5dc91SSasha Levin 	u8 pin, line, ndev;
29036f5dc91SSasha Levin 
29136f5dc91SSasha Levin 	vpci->dev = dev;
29295d13a52SSasha Levin 	vpci->msix_io_block = pci_get_io_space_block(PCI_IO_SIZE);
29395d13a52SSasha Levin 	vpci->msix_pba_block = pci_get_io_space_block(PCI_IO_SIZE);
29436f5dc91SSasha Levin 
2951c47ce69SSasha Levin 	vpci->base_addr = ioport__register(IOPORT_EMPTY, &virtio_pci__io_ops, IOPORT_SIZE, vtrans);
29606f48103SSasha Levin 	kvm__register_mmio(kvm, vpci->msix_io_block, 0x100, callback_mmio_table, vpci);
29706f48103SSasha Levin 	kvm__register_mmio(kvm, vpci->msix_pba_block, 0x100, callback_mmio_pba, vpci);
29836f5dc91SSasha Levin 
29936f5dc91SSasha Levin 	vpci->pci_hdr = (struct pci_device_header) {
30036f5dc91SSasha Levin 		.vendor_id		= PCI_VENDOR_ID_REDHAT_QUMRANET,
30136f5dc91SSasha Levin 		.device_id		= device_id,
30236f5dc91SSasha Levin 		.header_type		= PCI_HEADER_TYPE_NORMAL,
30336f5dc91SSasha Levin 		.revision_id		= 0,
304507e02d8SAsias He 		.class			= class,
30536f5dc91SSasha Levin 		.subsys_vendor_id	= PCI_SUBSYSTEM_VENDOR_ID_REDHAT_QUMRANET,
30636f5dc91SSasha Levin 		.subsys_id		= subsys_id,
30736f5dc91SSasha Levin 		.bar[0]			= vpci->base_addr | PCI_BASE_ADDRESS_SPACE_IO,
30806f48103SSasha Levin 		.bar[1]			= vpci->msix_io_block | PCI_BASE_ADDRESS_SPACE_MEMORY
30906f48103SSasha Levin 					| PCI_BASE_ADDRESS_MEM_TYPE_64,
31006f48103SSasha Levin 		.bar[3]			= vpci->msix_pba_block | PCI_BASE_ADDRESS_SPACE_MEMORY
31106f48103SSasha Levin 					| PCI_BASE_ADDRESS_MEM_TYPE_64,
31236f5dc91SSasha Levin 		.status			= PCI_STATUS_CAP_LIST,
31336f5dc91SSasha Levin 		.capabilities		= (void *)&vpci->pci_hdr.msix - (void *)&vpci->pci_hdr,
31436f5dc91SSasha Levin 	};
31536f5dc91SSasha Levin 
31636f5dc91SSasha Levin 	vpci->pci_hdr.msix.cap = PCI_CAP_ID_MSIX;
31736f5dc91SSasha Levin 	vpci->pci_hdr.msix.next = 0;
31814bba8a0SAsias He 	/*
31914bba8a0SAsias He 	 * We at most have VIRTIO_PCI_MAX_VQ entries for virt queue,
32014bba8a0SAsias He 	 * VIRTIO_PCI_MAX_CONFIG entries for config.
32114bba8a0SAsias He 	 *
32214bba8a0SAsias He 	 * To quote the PCI spec:
32314bba8a0SAsias He 	 *
32414bba8a0SAsias He 	 * System software reads this field to determine the
32514bba8a0SAsias He 	 * MSI-X Table Size N, which is encoded as N-1.
32614bba8a0SAsias He 	 * For example, a returned value of "00000000011"
32714bba8a0SAsias He 	 * indicates a table size of 4.
32814bba8a0SAsias He 	 */
32914bba8a0SAsias He 	vpci->pci_hdr.msix.ctrl = (VIRTIO_PCI_MAX_VQ + VIRTIO_PCI_MAX_CONFIG - 1);
33006f48103SSasha Levin 
33106f48103SSasha Levin 	/*
33206f48103SSasha Levin 	 * Both table and PBA could be mapped on the same BAR, but for now
33306f48103SSasha Levin 	 * we're not in short of BARs
33406f48103SSasha Levin 	 */
33536f5dc91SSasha Levin 	vpci->pci_hdr.msix.table_offset = 1; /* Use BAR 1 */
33606f48103SSasha Levin 	vpci->pci_hdr.msix.pba_offset = 3; /* Use BAR 3 */
33736f5dc91SSasha Levin 	vpci->config_vector = 0;
33836f5dc91SSasha Levin 
339d6d239beSSasha Levin 	if (irq__register_device(subsys_id, &ndev, &pin, &line) < 0)
34036f5dc91SSasha Levin 		return -1;
34136f5dc91SSasha Levin 
34236f5dc91SSasha Levin 	vpci->pci_hdr.irq_pin	= pin;
34336f5dc91SSasha Levin 	vpci->pci_hdr.irq_line	= line;
34436f5dc91SSasha Levin 	pci__register(&vpci->pci_hdr, ndev);
34536f5dc91SSasha Levin 
34636f5dc91SSasha Levin 	return 0;
34736f5dc91SSasha Levin }
348