X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=src%2Fvlib%2Flinux%2Fpci.c;h=b99f54f2a62b7bf5a5ceb0638f0926cd9fef0411;hb=d2bfb78f4f8fbfae204424467a4106530b89e608;hp=8954bccd93f14a65972dd180079c79463157d223;hpb=f9a968e2f928111e53de43fdf33dbf5145210074;p=vpp.git diff --git a/src/vlib/linux/pci.c b/src/vlib/linux/pci.c index 8954bccd93f..b99f54f2a62 100644 --- a/src/vlib/linux/pci.c +++ b/src/vlib/linux/pci.c @@ -42,6 +42,7 @@ #include #include #include +#include #include #include @@ -59,13 +60,47 @@ static const char *sysfs_pci_drv_path = "/sys/bus/pci/drivers"; static char *sysfs_mod_vfio_noiommu = "/sys/module/vfio/parameters/enable_unsafe_noiommu_mode"; +#define pci_log_debug(vm, dev, f, ...) \ + vlib_log(VLIB_LOG_LEVEL_DEBUG, pci_main.log_default, "%U: " f, \ + format_vlib_pci_addr, vlib_pci_get_addr(vm, dev->handle), ## __VA_ARGS__) +#define pci_log_err(vm, dev, f, ...) \ + vlib_log(VLIB_LOG_LEVEL_ERR, pci_main.log_default, "%U: " f, \ + format_vlib_pci_addr, vlib_pci_get_addr(vm, dev->handle), ## __VA_ARGS__) + +typedef struct +{ + int fd; + void *addr; + size_t size; +} linux_pci_region_t; + typedef struct { + int fd; + u32 clib_file_index; + union + { + pci_intx_handler_function_t *intx_handler; + pci_msix_handler_function_t *msix_handler; + }; +} linux_pci_irq_t; + +typedef enum +{ + LINUX_PCI_DEVICE_TYPE_UNKNOWN, + LINUX_PCI_DEVICE_TYPE_UIO, + LINUX_PCI_DEVICE_TYPE_VFIO, +} linux_pci_device_type_t; + +typedef struct +{ + linux_pci_device_type_t type; vlib_pci_dev_handle_t handle; vlib_pci_addr_t addr; + u32 numa_node; /* Resource file descriptors. */ - int *resource_fds; + linux_pci_region_t *regions; /* File descriptor for config space read/write. */ int config_fd; @@ -74,26 +109,23 @@ typedef struct /* Device File descriptor */ int fd; + /* read/write file descriptor for io bar */ + int io_fd; + u64 io_offset; + /* Minor device for uio device. */ u32 uio_minor; - /* Index given by clib_file_add. */ - u32 clib_file_index; - - /* Interrupt handler */ - void (*interrupt_handler) (vlib_pci_dev_handle_t h); + /* Interrupt handlers */ + linux_pci_irq_t intx_irq; + linux_pci_irq_t *msix_irqs; /* private data */ uword private_data; -} linux_pci_device_t; + u8 supports_va_dma; -typedef struct -{ - int group; - int fd; - int refcnt; -} linux_pci_vfio_iommu_group_t; +} linux_pci_device_t; /* Pool of PCI devices. */ typedef struct @@ -101,21 +133,11 @@ typedef struct vlib_main_t *vlib_main; linux_pci_device_t *linux_pci_devices; - /* VFIO */ - int vfio_container_fd; - int vfio_iommu_mode; - - /* pool of IOMMU groups */ - linux_pci_vfio_iommu_group_t *iommu_groups; - - /* iommu group pool index by group id hash */ - uword *iommu_pool_index_by_group; - } linux_pci_main_t; extern linux_pci_main_t linux_pci_main; -linux_pci_device_t * +static linux_pci_device_t * linux_pci_get_device (vlib_pci_dev_handle_t h) { linux_pci_main_t *lpm = &linux_pci_main; @@ -123,24 +145,32 @@ linux_pci_get_device (vlib_pci_dev_handle_t h) } uword -vlib_pci_get_private_data (vlib_pci_dev_handle_t h) +vlib_pci_get_private_data (vlib_main_t * vm, vlib_pci_dev_handle_t h) { linux_pci_device_t *d = linux_pci_get_device (h); return d->private_data; } void -vlib_pci_set_private_data (vlib_pci_dev_handle_t h, uword private_data) +vlib_pci_set_private_data (vlib_main_t * vm, vlib_pci_dev_handle_t h, + uword private_data) { linux_pci_device_t *d = linux_pci_get_device (h); d->private_data = private_data; } vlib_pci_addr_t * -vlib_pci_get_addr (vlib_pci_dev_handle_t h) +vlib_pci_get_addr (vlib_main_t * vm, vlib_pci_dev_handle_t h) +{ + linux_pci_device_t *d = linux_pci_get_device (h); + return &d->addr; +} + +u32 +vlib_pci_get_numa_node (vlib_main_t * vm, vlib_pci_dev_handle_t h) { - linux_pci_device_t *lpm = linux_pci_get_device (h); - return &lpm->addr; + linux_pci_device_t *d = linux_pci_get_device (h); + return d->numa_node; } /* Call to allocate/initialize the pci subsystem. @@ -151,9 +181,10 @@ clib_error_t *pci_bus_init (vlib_main_t * vm); linux_pci_main_t linux_pci_main; vlib_pci_device_info_t * -vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) +vlib_pci_get_device_info (vlib_main_t * vm, vlib_pci_addr_t * addr, + clib_error_t ** error) { - linux_pci_main_t *lpm = &linux_pci_main; + linux_vfio_main_t *lvm = &vfio_main; clib_error_t *err; vlib_pci_device_info_t *di; u8 *f = 0; @@ -161,7 +192,7 @@ vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) int fd; di = clib_mem_alloc (sizeof (vlib_pci_device_info_t)); - memset (di, 0, sizeof (vlib_pci_device_info_t)); + clib_memset (di, 0, sizeof (vlib_pci_device_info_t)); di->addr.as_u32 = addr->as_u32; u8 *dev_dir_name = format (0, "%s/%U", sysfs_pci_dev_path, @@ -182,10 +213,8 @@ vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) /* You can only read more that 64 bytes of config space as root; so we try to read the full space but fall back to just the first 64 bytes. */ - if (read (fd, &di->config_data, sizeof (di->config_data)) != - sizeof (di->config_data) - && read (fd, &di->config0, - sizeof (di->config0)) != sizeof (di->config0)) + if (read (fd, &di->config_data, sizeof (di->config_data)) < + sizeof (di->config0)) { err = clib_error_return_unix (0, "read `%s'", f); close (fd); @@ -195,7 +224,7 @@ vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) { static pci_config_header_t all_ones; if (all_ones.vendor_id == 0) - memset (&all_ones, ~0, sizeof (all_ones)); + clib_memset (&all_ones, ~0, sizeof (all_ones)); if (!memcmp (&di->config0.header, &all_ones, sizeof (all_ones))) { @@ -246,7 +275,7 @@ vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) di->driver_name = clib_sysfs_link_to_name ((char *) f); di->iommu_group = -1; - if (lpm->vfio_container_fd != -1) + if (lvm->container_fd != -1) { u8 *tmpstr; vec_reset_length (f); @@ -257,6 +286,17 @@ vlib_pci_get_device_info (vlib_pci_addr_t * addr, clib_error_t ** error) di->iommu_group = atoi ((char *) tmpstr); vec_free (tmpstr); } + vec_reset_length (f); + f = format (f, "%v/iommu_group/name%c", dev_dir_name, 0); + err = clib_sysfs_read ((char *) f, "%s", &tmpstr); + if (err == 0) + { + if (strncmp ((char *) tmpstr, "vfio-noiommu", 12) == 0) + di->flags |= VLIB_PCI_DEVICE_INFO_F_NOIOMMU; + vec_free (tmpstr); + } + else + clib_error_free (err); } close (fd); @@ -325,7 +365,8 @@ directory_exists (char *path) } clib_error_t * -vlib_pci_bind_to_uio (vlib_pci_addr_t * addr, char *uio_drv_name) +vlib_pci_bind_to_uio (vlib_main_t * vm, vlib_pci_addr_t * addr, + char *uio_drv_name) { clib_error_t *error = 0; u8 *s = 0, *driver_name = 0; @@ -336,7 +377,7 @@ vlib_pci_bind_to_uio (vlib_pci_addr_t * addr, char *uio_drv_name) u8 *dev_dir_name = format (0, "%s/%U", sysfs_pci_dev_path, format_vlib_pci_addr, addr); - di = vlib_pci_get_device_info (addr, &error); + di = vlib_pci_get_device_info (vm, addr, &error); if (error) return error; @@ -422,10 +463,11 @@ vlib_pci_bind_to_uio (vlib_pci_addr_t * addr, char *uio_drv_name) if (e->d_name[0] == '.') /* skip . and .. */ continue; - memset (&ifr, 0, sizeof ifr); - memset (&drvinfo, 0, sizeof drvinfo); + clib_memset (&ifr, 0, sizeof ifr); + clib_memset (&drvinfo, 0, sizeof drvinfo); ifr.ifr_data = (char *) &drvinfo; - strncpy (ifr.ifr_name, e->d_name, IFNAMSIZ - 1); + strncpy (ifr.ifr_name, e->d_name, sizeof (ifr.ifr_name)); + ifr.ifr_name[ARRAY_LEN (ifr.ifr_name) - 1] = '\0'; drvinfo.cmd = ETHTOOL_GDRVINFO; if (ioctl (fd, SIOCETHTOOL, &ifr) < 0) { @@ -439,8 +481,9 @@ vlib_pci_bind_to_uio (vlib_pci_addr_t * addr, char *uio_drv_name) if (strcmp ((char *) s, drvinfo.bus_info)) continue; - memset (&ifr, 0, sizeof (ifr)); - strncpy (ifr.ifr_name, e->d_name, IFNAMSIZ - 1); + clib_memset (&ifr, 0, sizeof (ifr)); + strncpy (ifr.ifr_name, e->d_name, sizeof (ifr.ifr_name)); + ifr.ifr_name[ARRAY_LEN (ifr.ifr_name) - 1] = '\0'; if (ioctl (fd, SIOCGIFFLAGS, &ifr) < 0) { error = clib_error_return_unix (0, "ioctl fetch intf %s flags", @@ -516,42 +559,87 @@ scan_uio_dir (void *arg, u8 * path_name, u8 * file_name) return 0; } +static clib_error_t * +vfio_set_irqs (vlib_main_t * vm, linux_pci_device_t * p, u32 index, u32 start, + u32 count, u32 flags, int *efds) +{ + int data_len = efds ? count * sizeof (int) : 0; + u8 buf[sizeof (struct vfio_irq_set) + data_len]; + struct vfio_irq_info ii = { 0 }; + struct vfio_irq_set *irq_set = (struct vfio_irq_set *) buf; + + + ii.argsz = sizeof (struct vfio_irq_info); + ii.index = index; + + if (ioctl (p->fd, VFIO_DEVICE_GET_IRQ_INFO, &ii) < 0) + return clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_IRQ_INFO) " + "'%U'", format_vlib_pci_addr, &p->addr); + + pci_log_debug (vm, p, "%s index:%u count:%u flags: %s%s%s%s(0x%x)", + __func__, ii.index, ii.count, + ii.flags & VFIO_IRQ_INFO_EVENTFD ? "eventfd " : "", + ii.flags & VFIO_IRQ_INFO_MASKABLE ? "maskable " : "", + ii.flags & VFIO_IRQ_INFO_AUTOMASKED ? "automasked " : "", + ii.flags & VFIO_IRQ_INFO_NORESIZE ? "noresize " : "", + ii.flags); + + if (ii.count < start + count) + return clib_error_return_unix (0, "vfio_set_irq: unexistng interrupt on " + "'%U'", format_vlib_pci_addr, &p->addr); + + + if (efds) + { + flags |= VFIO_IRQ_SET_DATA_EVENTFD; + clib_memcpy_fast (&irq_set->data, efds, data_len); + } + else + flags |= VFIO_IRQ_SET_DATA_NONE; + + ASSERT ((flags & (VFIO_IRQ_SET_DATA_NONE | VFIO_IRQ_SET_DATA_EVENTFD)) != + (VFIO_IRQ_SET_DATA_NONE | VFIO_IRQ_SET_DATA_EVENTFD)); + + irq_set->argsz = sizeof (struct vfio_irq_set) + data_len; + irq_set->index = index; + irq_set->start = start; + irq_set->count = count; + irq_set->flags = flags; + + if (ioctl (p->fd, VFIO_DEVICE_SET_IRQS, irq_set) < 0) + return clib_error_return_unix (0, "%U:ioctl(VFIO_DEVICE_SET_IRQS) " + "[index = %u, start = %u, count = %u, " + "flags = 0x%x]", + format_vlib_pci_addr, &p->addr, + index, start, count, flags); + return 0; +} + static clib_error_t * linux_pci_uio_read_ready (clib_file_t * uf) { + vlib_main_t *vm = vlib_get_main (); int __attribute__ ((unused)) rv; vlib_pci_dev_handle_t h = uf->private_data; linux_pci_device_t *p = linux_pci_get_device (h); + linux_pci_irq_t *irq = &p->intx_irq; u32 icount; rv = read (uf->file_descriptor, &icount, 4); - if (p->interrupt_handler) - p->interrupt_handler (h); + if (irq->intx_handler) + irq->intx_handler (vm, h); - vlib_pci_intr_enable (h); + vlib_pci_intr_enable (vm, h); return /* no error */ 0; } static clib_error_t * -linux_pci_vfio_unmask_intx (linux_pci_device_t * d) +linux_pci_vfio_unmask_intx (vlib_main_t * vm, linux_pci_device_t * d) { - clib_error_t *err = 0; - struct vfio_irq_set i = { - .argsz = sizeof (struct vfio_irq_set), - .start = 0, - .count = 1, - .index = VFIO_PCI_INTX_IRQ_INDEX, - .flags = VFIO_IRQ_SET_DATA_NONE | VFIO_IRQ_SET_ACTION_UNMASK, - }; - - if (ioctl (d->fd, VFIO_DEVICE_SET_IRQS, &i) < 0) - { - err = clib_error_return_unix (0, "ioctl(VFIO_DEVICE_SET_IRQS) '%U'", - format_vlib_pci_addr, &d->addr); - } - return err; + return vfio_set_irqs (vm, d, VFIO_PCI_INTX_IRQ_INDEX, 0, 1, + VFIO_IRQ_SET_ACTION_UNMASK, 0); } static clib_error_t * @@ -563,19 +651,40 @@ linux_pci_uio_error_ready (clib_file_t * uf) } static clib_error_t * -linux_pci_vfio_read_ready (clib_file_t * uf) +linux_pci_vfio_msix_read_ready (clib_file_t * uf) +{ + vlib_main_t *vm = vlib_get_main (); + int __attribute__ ((unused)) rv; + vlib_pci_dev_handle_t h = uf->private_data >> 16; + u16 line = uf->private_data & 0xffff; + linux_pci_device_t *p = linux_pci_get_device (h); + linux_pci_irq_t *irq = vec_elt_at_index (p->msix_irqs, line); + + u64 icount; + rv = read (uf->file_descriptor, &icount, sizeof (icount)); + + if (irq->msix_handler) + irq->msix_handler (vm, h, line); + + return /* no error */ 0; +} + +static clib_error_t * +linux_pci_vfio_intx_read_ready (clib_file_t * uf) { + vlib_main_t *vm = vlib_get_main (); int __attribute__ ((unused)) rv; vlib_pci_dev_handle_t h = uf->private_data; linux_pci_device_t *p = linux_pci_get_device (h); + linux_pci_irq_t *irq = &p->intx_irq; u64 icount; rv = read (uf->file_descriptor, &icount, sizeof (icount)); - if (p->interrupt_handler) - p->interrupt_handler (h); + if (irq->intx_handler) + irq->intx_handler (vm, h); - linux_pci_vfio_unmask_intx (p); + linux_pci_vfio_unmask_intx (vm, p); return /* no error */ 0; } @@ -592,12 +701,12 @@ static clib_error_t * add_device_uio (vlib_main_t * vm, linux_pci_device_t * p, vlib_pci_device_info_t * di, pci_device_registration_t * r) { + linux_pci_main_t *lpm = &linux_pci_main; clib_error_t *err = 0; - clib_file_t t = { 0 }; u8 *s = 0; - p->addr.as_u32 = di->addr.as_u32; p->fd = -1; + p->type = LINUX_PCI_DEVICE_TYPE_UIO; s = format (s, "%s/%U/config%c", sysfs_pci_dev_path, format_vlib_pci_addr, &di->addr, 0); @@ -626,17 +735,14 @@ add_device_uio (vlib_main_t * vm, linux_pci_device_t * p, goto error; } - t.read_function = linux_pci_uio_read_ready; - t.file_descriptor = p->fd; - t.error_function = linux_pci_uio_error_ready; - t.private_data = p->handle; + if (r && r->interrupt_handler) + vlib_pci_register_intx_handler (vm, p->handle, r->interrupt_handler); - p->clib_file_index = clib_file_add (&file_main, &t); - p->interrupt_handler = r->interrupt_handler; - err = r->init_function (vm, p->handle); + if (r && r->init_function) + err = r->init_function (lpm->vlib_main, p->handle); error: - free (s); + vec_free (s); if (err) { if (p->config_fd != -1) @@ -647,117 +753,172 @@ error: return err; } -static linux_pci_vfio_iommu_group_t * -get_vfio_iommu_group (int group) +clib_error_t * +vlib_pci_register_intx_handler (vlib_main_t * vm, vlib_pci_dev_handle_t h, + pci_intx_handler_function_t * intx_handler) { - linux_pci_main_t *lpm = &linux_pci_main; - uword *p; + linux_pci_device_t *p = linux_pci_get_device (h); + clib_file_t t = { 0 }; + linux_pci_irq_t *irq = &p->intx_irq; + ASSERT (irq->fd == -1); - p = hash_get (lpm->iommu_pool_index_by_group, group); + if (p->type == LINUX_PCI_DEVICE_TYPE_VFIO) + { + struct vfio_irq_info ii = { 0 }; + ii.argsz = sizeof (struct vfio_irq_info); + ii.index = VFIO_PCI_INTX_IRQ_INDEX; + if (ioctl (p->fd, VFIO_DEVICE_GET_IRQ_INFO, &ii) < 0) + return clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_IRQ_INFO) '" + "%U'", format_vlib_pci_addr, &p->addr); + pci_log_debug (vm, p, "%s index:%u count:%u flags: %s%s%s%s(0x%x)", + __func__, ii.index, ii.count, + ii.flags & VFIO_IRQ_INFO_EVENTFD ? "eventfd " : "", + ii.flags & VFIO_IRQ_INFO_MASKABLE ? "maskable " : "", + ii.flags & VFIO_IRQ_INFO_AUTOMASKED ? "automasked " : "", + ii.flags & VFIO_IRQ_INFO_NORESIZE ? "noresize " : "", + ii.flags); + if (ii.count != 1) + return clib_error_return (0, "INTx interrupt does not exist on device" + "'%U'", format_vlib_pci_addr, &p->addr); + + irq->fd = eventfd (0, EFD_NONBLOCK); + if (irq->fd == -1) + return clib_error_return_unix (0, "eventfd"); + + t.file_descriptor = irq->fd; + t.read_function = linux_pci_vfio_intx_read_ready; + } + else if (p->type == LINUX_PCI_DEVICE_TYPE_UIO) + { + t.file_descriptor = p->fd; + t.read_function = linux_pci_uio_read_ready; + } + else + return 0; - return p ? pool_elt_at_index (lpm->iommu_groups, p[0]) : 0; + t.error_function = linux_pci_uio_error_ready; + t.private_data = p->handle; + t.description = format (0, "PCI %U INTx", format_vlib_pci_addr, &p->addr); + irq->clib_file_index = clib_file_add (&file_main, &t); + irq->intx_handler = intx_handler; + return 0; } -static clib_error_t * -open_vfio_iommu_group (int group) +clib_error_t * +vlib_pci_register_msix_handler (vlib_main_t * vm, vlib_pci_dev_handle_t h, + u32 start, u32 count, + pci_msix_handler_function_t * msix_handler) { - linux_pci_main_t *lpm = &linux_pci_main; - linux_pci_vfio_iommu_group_t *g; clib_error_t *err = 0; - struct vfio_group_status group_status; - u8 *s = 0; - int fd; - - g = get_vfio_iommu_group (group); - if (g) - { - g->refcnt++; - return 0; - } - s = format (s, "/dev/vfio/%u%c", group, 0); - fd = open ((char *) s, O_RDWR); - if (fd < 0) - return clib_error_return_unix (0, "open '%s'", s); + linux_pci_device_t *p = linux_pci_get_device (h); + u32 i; - group_status.argsz = sizeof (group_status); - if (ioctl (fd, VFIO_GROUP_GET_STATUS, &group_status) < 0) - { - err = clib_error_return_unix (0, "ioctl(VFIO_GROUP_GET_STATUS) '%s'", - s); - goto error; - } + if (p->type != LINUX_PCI_DEVICE_TYPE_VFIO) + return clib_error_return (0, "vfio driver is needed for MSI-X interrupt " + "support"); - if (!(group_status.flags & VFIO_GROUP_FLAGS_VIABLE)) - { - err = clib_error_return (0, "iommu group %d is not viable (not all " - "devices in this group bound to vfio-pci)", - group); - goto error; - } + /* *INDENT-OFF* */ + vec_validate_init_empty (p->msix_irqs, start + count - 1, (linux_pci_irq_t) + { .fd = -1}); + /* *INDENT-ON* */ - if (ioctl (fd, VFIO_GROUP_SET_CONTAINER, &lpm->vfio_container_fd) < 0) + for (i = start; i < start + count; i++) { - err = clib_error_return_unix (0, "ioctl(VFIO_GROUP_SET_CONTAINER) '%s'", - s); - goto error; - } + clib_file_t t = { 0 }; + linux_pci_irq_t *irq = vec_elt_at_index (p->msix_irqs, i); + ASSERT (irq->fd == -1); - if (lpm->vfio_iommu_mode == 0) - { - if (ioctl (lpm->vfio_container_fd, VFIO_SET_IOMMU, VFIO_TYPE1_IOMMU) < - 0) + irq->fd = eventfd (0, EFD_NONBLOCK); + if (irq->fd == -1) { - err = clib_error_return_unix (0, "ioctl(VFIO_SET_IOMMU) " - "'/dev/vfio/vfio'"); + err = clib_error_return_unix (0, "eventfd"); goto error; } - lpm->vfio_iommu_mode = VFIO_TYPE1_IOMMU; - } + t.read_function = linux_pci_vfio_msix_read_ready; + t.file_descriptor = irq->fd; + t.error_function = linux_pci_vfio_error_ready; + t.private_data = p->handle << 16 | i; + t.description = format (0, "PCI %U MSI-X #%u", format_vlib_pci_addr, + &p->addr, i); + irq->clib_file_index = clib_file_add (&file_main, &t); + irq->msix_handler = msix_handler; + } - pool_get (lpm->iommu_groups, g); - g->fd = fd; - g->refcnt = 1; - hash_set (lpm->iommu_pool_index_by_group, group, g - lpm->iommu_groups); - vec_free (s); return 0; + error: - close (fd); + while (i-- > start) + { + linux_pci_irq_t *irq = vec_elt_at_index (p->msix_irqs, i); + if (irq->fd != -1) + { + clib_file_del_by_index (&file_main, irq->clib_file_index); + close (irq->fd); + irq->fd = -1; + } + } return err; } +clib_error_t * +vlib_pci_enable_msix_irq (vlib_main_t * vm, vlib_pci_dev_handle_t h, + u16 start, u16 count) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + int fds[count]; + int i; + + if (p->type != LINUX_PCI_DEVICE_TYPE_VFIO) + return clib_error_return (0, "vfio driver is needed for MSI-X interrupt " + "support"); + + for (i = start; i < start + count; i++) + { + linux_pci_irq_t *irq = vec_elt_at_index (p->msix_irqs, i); + fds[i] = irq->fd; + } + + return vfio_set_irqs (vm, p, VFIO_PCI_MSIX_IRQ_INDEX, start, count, + VFIO_IRQ_SET_ACTION_TRIGGER, fds); +} + +clib_error_t * +vlib_pci_disable_msix_irq (vlib_main_t * vm, vlib_pci_dev_handle_t h, + u16 start, u16 count) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + int i, fds[count]; + + if (p->type != LINUX_PCI_DEVICE_TYPE_VFIO) + return clib_error_return (0, "vfio driver is needed for MSI-X interrupt " + "support"); + + for (i = start; i < start + count; i++) + fds[i] = -1; + + return vfio_set_irqs (vm, p, VFIO_PCI_MSIX_IRQ_INDEX, start, count, + VFIO_IRQ_SET_ACTION_TRIGGER, fds); +} + static clib_error_t * add_device_vfio (vlib_main_t * vm, linux_pci_device_t * p, vlib_pci_device_info_t * di, pci_device_registration_t * r) { - linux_pci_vfio_iommu_group_t *g; + linux_pci_main_t *lpm = &linux_pci_main; struct vfio_device_info device_info = { 0 }; struct vfio_region_info reg = { 0 }; - struct vfio_irq_info irq_info = { 0 }; clib_error_t *err = 0; u8 *s = 0; + int is_noiommu; - p->addr.as_u32 = di->addr.as_u32; - - if (di->driver_name == 0 || - (strcmp ("vfio-pci", (char *) di->driver_name) != 0)) - return clib_error_return (0, "Device '%U' (iommu group %d) not bound to " - "vfio-pci", format_vlib_pci_addr, &di->addr, - di->iommu_group); + p->type = LINUX_PCI_DEVICE_TYPE_VFIO; - if ((err = open_vfio_iommu_group (di->iommu_group))) + if ((err = linux_vfio_group_get_device_fd (&p->addr, &p->fd, &is_noiommu))) return err; - g = get_vfio_iommu_group (di->iommu_group); - - s = format (s, "%U%c", format_vlib_pci_addr, &di->addr, 0); - if ((p->fd = ioctl (g->fd, VFIO_GROUP_GET_DEVICE_FD, (char *) s)) < 0) - { - err = clib_error_return_unix (0, "ioctl(VFIO_GROUP_GET_DEVICE_FD) '%U'", - format_vlib_pci_addr, &di->addr); - goto error; - } - vec_reset_length (s); + if (is_noiommu == 0) + p->supports_va_dma = 1; device_info.argsz = sizeof (device_info); if (ioctl (p->fd, VFIO_DEVICE_GET_INFO, &device_info) < 0) @@ -767,15 +928,26 @@ add_device_vfio (vlib_main_t * vm, linux_pci_device_t * p, goto error; } - irq_info.argsz = sizeof (struct vfio_irq_info); - irq_info.index = VFIO_PCI_INTX_IRQ_INDEX; - if (ioctl (p->fd, VFIO_DEVICE_GET_IRQ_INFO, &irq_info) < 0) + reg.argsz = sizeof (struct vfio_region_info); + reg.index = VFIO_PCI_CONFIG_REGION_INDEX; + if (ioctl (p->fd, VFIO_DEVICE_GET_REGION_INFO, ®) < 0) { - err = clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_IRQ_INFO) " - "'%U'", format_vlib_pci_addr, &di->addr); + err = clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_INFO) '%U'", + format_vlib_pci_addr, &di->addr); goto error; } + pci_log_debug (vm, p, "%s region_info index:%u size:0x%lx offset:0x%lx " + "flags: %s%s%s(0x%x)", __func__, + reg.index, reg.size, reg.offset, + reg.flags & VFIO_REGION_INFO_FLAG_READ ? "rd " : "", + reg.flags & VFIO_REGION_INFO_FLAG_WRITE ? "wr " : "", + reg.flags & VFIO_REGION_INFO_FLAG_MMAP ? "mmap " : "", + reg.flags); + + p->config_offset = reg.offset; + p->config_fd = p->fd; + /* reset if device supports it */ if (device_info.flags & VFIO_DEVICE_FLAGS_RESET) if (ioctl (p->fd, VFIO_DEVICE_RESET) < 0) @@ -785,51 +957,29 @@ add_device_vfio (vlib_main_t * vm, linux_pci_device_t * p, goto error; } - if ((irq_info.flags & VFIO_IRQ_INFO_EVENTFD) && irq_info.count == 1) + if (r && r->interrupt_handler) { - u8 buf[sizeof (struct vfio_irq_set) + sizeof (int)] = { 0 }; - struct vfio_irq_set *irq_set = (struct vfio_irq_set *) buf; - clib_file_t t = { 0 }; - int efd = eventfd (0, EFD_NONBLOCK); - - irq_set->argsz = sizeof (buf); - irq_set->count = 1; - irq_set->index = VFIO_PCI_INTX_IRQ_INDEX; - irq_set->flags = - VFIO_IRQ_SET_DATA_EVENTFD | VFIO_IRQ_SET_ACTION_TRIGGER; - clib_memcpy (&irq_set->data, &efd, sizeof (int)); - - if (ioctl (p->fd, VFIO_DEVICE_SET_IRQS, irq_set) < 0) - { - err = clib_error_return_unix (0, "ioctl(VFIO_DEVICE_SET_IRQS) '%U'", - format_vlib_pci_addr, &di->addr); - goto error; - } - - t.read_function = linux_pci_vfio_read_ready; - t.file_descriptor = efd; - t.error_function = linux_pci_vfio_error_ready; - t.private_data = p->handle; - p->clib_file_index = clib_file_add (&file_main, &t); - - /* unmask the interrupt */ - linux_pci_vfio_unmask_intx (p); + vlib_pci_register_intx_handler (vm, p->handle, r->interrupt_handler); + linux_pci_vfio_unmask_intx (vm, p); } - p->interrupt_handler = r->interrupt_handler; - - reg.argsz = sizeof (struct vfio_region_info); - reg.index = VFIO_PCI_CONFIG_REGION_INDEX; - if (ioctl (p->fd, VFIO_DEVICE_GET_REGION_INFO, ®) < 0) + if (p->supports_va_dma) { - err = clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_REGION_INFO) " - "'%U'", format_vlib_pci_addr, &di->addr); - goto error; + vlib_buffer_pool_t *bp; + /* *INDENT-OFF* */ + vec_foreach (bp, buffer_main.buffer_pools) + { + u32 i; + vlib_physmem_map_t *pm; + pm = vlib_physmem_get_map (vm, bp->physmem_map_index); + for (i = 0; i < pm->n_pages; i++) + vfio_map_physmem_page (vm, pm->base + (i << pm->log2_page_size)); + } + /* *INDENT-ON* */ } - p->config_offset = reg.offset; - p->config_fd = p->fd; - err = r->init_function (vm, p->handle); + if (r && r->init_function) + err = r->init_function (lpm->vlib_main, p->handle); error: vec_free (s); @@ -837,15 +987,16 @@ error: { if (p->fd != -1) close (p->fd); - if (p->config_fd != -1) + if (p->config_fd != -1 && p->config_fd != p->fd) close (p->config_fd); + p->config_fd = p->fd = -1; } return err; } /* Configuration space read/write. */ clib_error_t * -vlib_pci_read_write_config (vlib_pci_dev_handle_t h, +vlib_pci_read_write_config (vlib_main_t * vm, vlib_pci_dev_handle_t h, vlib_read_or_write_t read_or_write, uword address, void *data, u32 n_bytes) { @@ -866,71 +1017,305 @@ vlib_pci_read_write_config (vlib_pci_dev_handle_t h, } static clib_error_t * -vlib_pci_map_resource_int (vlib_pci_dev_handle_t h, - u32 resource, u8 * addr, void **result) +vlib_pci_region (vlib_main_t * vm, vlib_pci_dev_handle_t h, u32 bar, int *fd, + u64 * size, u64 * offset) { linux_pci_device_t *p = linux_pci_get_device (h); - struct stat stat_buf; - u8 *file_name; - int fd; - clib_error_t *error; - int flags = MAP_SHARED; + clib_error_t *error = 0; + int _fd = -1; + u64 _size = 0, _offset = 0; - error = 0; + ASSERT (bar <= 5); - file_name = format (0, "%s/%U/resource%d%c", sysfs_pci_dev_path, - format_vlib_pci_addr, &p->addr, resource, 0); + error = 0; - fd = open ((char *) file_name, O_RDWR); - if (fd < 0) + if (p->type == LINUX_PCI_DEVICE_TYPE_UIO) { - error = clib_error_return_unix (0, "open `%s'", file_name); - goto done; - } + u8 *file_name; + struct stat stat_buf; + file_name = format (0, "%s/%U/resource%d%c", sysfs_pci_dev_path, + format_vlib_pci_addr, &p->addr, bar, 0); + + _fd = open ((char *) file_name, O_RDWR); + if (_fd < 0) + { + error = clib_error_return_unix (0, "open `%s'", file_name); + vec_free (file_name); + return error; + } - if (fstat (fd, &stat_buf) < 0) + if (fstat (_fd, &stat_buf) < 0) + { + error = clib_error_return_unix (0, "fstat `%s'", file_name); + vec_free (file_name); + close (_fd); + return error; + } + + vec_free (file_name); + _size = stat_buf.st_size; + _offset = 0; + } + else if (p->type == LINUX_PCI_DEVICE_TYPE_VFIO) { - error = clib_error_return_unix (0, "fstat `%s'", file_name); - goto done; + struct vfio_region_info reg = { 0 }; + reg.argsz = sizeof (struct vfio_region_info); + reg.index = bar; + if (ioctl (p->fd, VFIO_DEVICE_GET_REGION_INFO, ®) < 0) + return clib_error_return_unix (0, "ioctl(VFIO_DEVICE_GET_INFO) " + "'%U'", format_vlib_pci_addr, + &p->addr); + _fd = p->fd; + _size = reg.size; + _offset = reg.offset; + pci_log_debug (vm, p, "%s region_info index:%u size:0x%lx offset:0x%lx " + "flags: %s%s%s(0x%x)", __func__, + reg.index, reg.size, reg.offset, + reg.flags & VFIO_REGION_INFO_FLAG_READ ? "rd " : "", + reg.flags & VFIO_REGION_INFO_FLAG_WRITE ? "wr " : "", + reg.flags & VFIO_REGION_INFO_FLAG_MMAP ? "mmap " : "", + reg.flags); } + else + ASSERT (0); + + *fd = _fd; + *size = _size; + *offset = _offset; + + return error; +} - vec_validate (p->resource_fds, resource); - p->resource_fds[resource] = fd; - if (addr != 0) +static clib_error_t * +vlib_pci_map_region_int (vlib_main_t * vm, vlib_pci_dev_handle_t h, + u32 bar, u8 * addr, void **result) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + int fd = -1; + clib_error_t *error; + int flags = MAP_SHARED; + u64 size = 0, offset = 0; + + pci_log_debug (vm, p, "map region %u to va %p", bar, addr); + + if ((error = vlib_pci_region (vm, h, bar, &fd, &size, &offset))) + return error; + + if (p->type == LINUX_PCI_DEVICE_TYPE_UIO && addr != 0) flags |= MAP_FIXED; - *result = mmap (addr, - /* size */ stat_buf.st_size, - PROT_READ | PROT_WRITE, flags, - /* file */ fd, - /* offset */ 0); + *result = mmap (addr, size, PROT_READ | PROT_WRITE, flags, fd, offset); if (*result == (void *) -1) { - error = clib_error_return_unix (0, "mmap `%s'", file_name); - goto done; - } - -done: - if (error) - { - if (fd >= 0) + error = clib_error_return_unix (0, "mmap `BAR%u'", bar); + if (p->type == LINUX_PCI_DEVICE_TYPE_UIO) close (fd); + return error; } - vec_free (file_name); + + /* *INDENT-OFF* */ + vec_validate_init_empty (p->regions, bar, + (linux_pci_region_t) { .fd = -1}); + /* *INDENT-ON* */ + if (p->type == LINUX_PCI_DEVICE_TYPE_UIO) + p->regions[bar].fd = fd; + p->regions[bar].addr = *result; + p->regions[bar].size = size; + return 0; +} + +clib_error_t * +vlib_pci_map_region (vlib_main_t * vm, vlib_pci_dev_handle_t h, u32 resource, + void **result) +{ + return (vlib_pci_map_region_int (vm, h, resource, 0 /* addr */ , result)); +} + +clib_error_t * +vlib_pci_map_region_fixed (vlib_main_t * vm, vlib_pci_dev_handle_t h, + u32 resource, u8 * addr, void **result) +{ + return (vlib_pci_map_region_int (vm, h, resource, addr, result)); +} + +clib_error_t * +vlib_pci_io_region (vlib_main_t * vm, vlib_pci_dev_handle_t h, u32 resource) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + clib_error_t *error = 0; + int fd = -1; + u64 size = 0, offset = 0; + + if ((error = vlib_pci_region (vm, h, resource, &fd, &size, &offset))) + return error; + + p->io_fd = fd; + p->io_offset = offset; return error; } clib_error_t * -vlib_pci_map_resource (vlib_pci_dev_handle_t h, u32 resource, void **result) +vlib_pci_read_write_io (vlib_main_t * vm, vlib_pci_dev_handle_t h, + vlib_read_or_write_t read_or_write, + uword offset, void *data, u32 length) { - return (vlib_pci_map_resource_int (h, resource, 0 /* addr */ , result)); + linux_pci_device_t *p = linux_pci_get_device (h); + int n = 0; + + if (read_or_write == VLIB_READ) + n = pread (p->io_fd, data, length, p->io_offset + offset); + else + n = pwrite (p->io_fd, data, length, p->io_offset + offset); + + if (n != length) + return clib_error_return_unix (0, "%s", + read_or_write == VLIB_READ + ? "read" : "write"); + return 0; +} + +clib_error_t * +vlib_pci_map_dma (vlib_main_t * vm, vlib_pci_dev_handle_t h, void *ptr) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + + if (!p->supports_va_dma) + return 0; + + return vfio_map_physmem_page (vm, ptr); +} + +int +vlib_pci_supports_virtual_addr_dma (vlib_main_t * vm, vlib_pci_dev_handle_t h) +{ + linux_pci_device_t *p = linux_pci_get_device (h); + + return p->supports_va_dma != 0; } clib_error_t * -vlib_pci_map_resource_fixed (vlib_pci_dev_handle_t h, - u32 resource, u8 * addr, void **result) +vlib_pci_device_open (vlib_main_t * vm, vlib_pci_addr_t * addr, + pci_device_id_t ids[], vlib_pci_dev_handle_t * handle) +{ + linux_pci_main_t *lpm = &linux_pci_main; + vlib_pci_device_info_t *di; + linux_pci_device_t *p; + clib_error_t *err = 0; + pci_device_id_t *i; + + di = vlib_pci_get_device_info (vm, addr, &err); + + if (err) + return err; + for (i = ids; i->vendor_id != 0; i++) + if (i->vendor_id == di->vendor_id && i->device_id == di->device_id) + break; + + if (i->vendor_id == 0) + return clib_error_return (0, "Wrong vendor or device id"); + + pool_get (lpm->linux_pci_devices, p); + p->handle = p - lpm->linux_pci_devices; + p->addr.as_u32 = di->addr.as_u32; + p->intx_irq.fd = -1; + p->numa_node = di->numa_node; + /* + * pci io bar read/write fd + */ + p->io_fd = -1; + + pci_log_debug (vm, p, "open vid:0x%04x did:0x%04x driver:%s iommu_group:%d", + di->vendor_id, di->device_id, di->driver_name, + di->iommu_group); + + if (strncmp ("vfio-pci", (char *) di->driver_name, 8) == 0) + err = add_device_vfio (vm, p, di, 0); + else if (strncmp ("uio_pci_generic", (char *) di->driver_name, 8) == 0) + err = add_device_uio (vm, p, di, 0); + else + err = clib_error_create ("device not bound to 'vfio-pci' or " + "'uio_pci_generic' kernel module"); + if (err) + goto error; + + *handle = p->handle; + +error: + vlib_pci_free_device_info (di); + if (err) + { + pci_log_err (vm, p, "%U", format_clib_error, err); + clib_memset (p, 0, sizeof (linux_pci_device_t)); + pool_put (lpm->linux_pci_devices, p); + } + + return err; +} + +void +vlib_pci_device_close (vlib_main_t * vm, vlib_pci_dev_handle_t h) { - return (vlib_pci_map_resource_int (h, resource, addr, result)); + linux_pci_main_t *lpm = &linux_pci_main; + linux_pci_device_t *p = linux_pci_get_device (h); + linux_pci_irq_t *irq; + linux_pci_region_t *res; + clib_error_t *err = 0; + + if (p->type == LINUX_PCI_DEVICE_TYPE_UIO) + { + irq = &p->intx_irq; + clib_file_del_by_index (&file_main, irq->clib_file_index); + close (p->config_fd); + if (p->io_fd != -1) + close (p->io_fd); + } + else if (p->type == LINUX_PCI_DEVICE_TYPE_VFIO) + { + irq = &p->intx_irq; + /* close INTx irqs */ + if (irq->fd != -1) + { + err = vfio_set_irqs (vm, p, VFIO_PCI_INTX_IRQ_INDEX, 0, 0, + VFIO_IRQ_SET_ACTION_TRIGGER, 0); + clib_error_free (err); + clib_file_del_by_index (&file_main, irq->clib_file_index); + close (irq->fd); + } + + /* close MSI-X irqs */ + if (vec_len (p->msix_irqs)) + { + err = vfio_set_irqs (vm, p, VFIO_PCI_MSIX_IRQ_INDEX, 0, 0, + VFIO_IRQ_SET_ACTION_TRIGGER, 0); + clib_error_free (err); + /* *INDENT-OFF* */ + vec_foreach (irq, p->msix_irqs) + { + if (irq->fd == -1) + continue; + clib_file_del_by_index (&file_main, irq->clib_file_index); + close (irq->fd); + } + /* *INDENT-ON* */ + vec_free (p->msix_irqs); + } + } + + /* *INDENT-OFF* */ + vec_foreach (res, p->regions) + { + if (res->size == 0) + continue; + munmap (res->addr, res->size); + if (res->fd != -1) + close (res->fd); + } + /* *INDENT-ON* */ + vec_free (p->regions); + + close (p->fd); + clib_memset (p, 0, sizeof (linux_pci_device_t)); + pool_put (lpm->linux_pci_devices, p); } void @@ -945,6 +1330,7 @@ init_device_from_registered (vlib_main_t * vm, vlib_pci_device_info_t * di) pool_get (lpm->linux_pci_devices, p); p->handle = p - lpm->linux_pci_devices; + p->intx_irq.fd = -1; r = pm->pci_device_registrations; @@ -967,7 +1353,7 @@ init_device_from_registered (vlib_main_t * vm, vlib_pci_device_info_t * di) } /* No driver, close the PCI config-space FD */ - memset (p, 0, sizeof (linux_pci_device_t)); + clib_memset (p, 0, sizeof (linux_pci_device_t)); pool_put (lpm->linux_pci_devices, p); } @@ -1041,10 +1427,8 @@ clib_error_t * linux_pci_init (vlib_main_t * vm) { vlib_pci_main_t *pm = &pci_main; - linux_pci_main_t *lpm = &linux_pci_main; vlib_pci_addr_t *addr = 0, *addrs; clib_error_t *error; - int fd; pm->vlib_main = vm; @@ -1053,29 +1437,12 @@ linux_pci_init (vlib_main_t * vm) ASSERT (sizeof (vlib_pci_addr_t) == sizeof (u32)); - fd = open ("/dev/vfio/vfio", O_RDWR); - - if ((fd != -1) && (ioctl (fd, VFIO_GET_API_VERSION) != VFIO_API_VERSION)) - { - close (fd); - fd = -1; - } - - if ((fd != -1) && (ioctl (fd, VFIO_CHECK_EXTENSION, VFIO_TYPE1_IOMMU) == 0)) - { - close (fd); - fd = -1; - } - - lpm->vfio_container_fd = fd; - lpm->iommu_pool_index_by_group = hash_create (0, sizeof (uword)); - addrs = vlib_pci_get_all_dev_addrs (); /* *INDENT-OFF* */ vec_foreach (addr, addrs) { vlib_pci_device_info_t *d; - if ((d = vlib_pci_get_device_info (addr, 0))) + if ((d = vlib_pci_get_device_info (vm, addr, 0))) { init_device_from_registered (vm, d); vlib_pci_free_device_info (d);