From 20ba16461ce232f5bb785b6b6189cf834ccfee1e Mon Sep 17 00:00:00 2001 From: Damjan Marion Date: Mon, 26 Mar 2018 15:35:33 +0200 Subject: vlib: add support for vfio no-iommu mode Change-Id: Ic83971d8d9d8d0bb90a35932e60761cd728457f3 Signed-off-by: Damjan Marion --- src/vlib/linux/pci.c | 13 +++++++++- src/vlib/linux/vfio.c | 67 +++++++++++++++++++++++++++------------------------ src/vlib/linux/vfio.h | 1 + src/vlib/pci/pci.h | 3 +++ 4 files changed, 51 insertions(+), 33 deletions(-) (limited to 'src/vlib') diff --git a/src/vlib/linux/pci.c b/src/vlib/linux/pci.c index 910f8c2f9cc..127d2ef167a 100644 --- a/src/vlib/linux/pci.c +++ b/src/vlib/linux/pci.c @@ -263,6 +263,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); @@ -695,7 +706,7 @@ add_device_uio (linux_pci_device_t * p, vlib_pci_device_info_t * di, err = r->init_function (lpm->vlib_main, p->handle); error: - free (s); + vec_free (s); if (err) { if (p->config_fd != -1) diff --git a/src/vlib/linux/vfio.c b/src/vlib/linux/vfio.c index 621dfb2ce01..956b4ab2e48 100644 --- a/src/vlib/linux/vfio.c +++ b/src/vlib/linux/vfio.c @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016 Cisco and/or its affiliates. + * Copyright (c) 2018 Cisco and/or its affiliates. * Licensed under the Apache License, Version 2.0 (the "License"); * you may not use this file except in compliance with the License. * You may obtain a copy of the License at: @@ -12,30 +12,6 @@ * See the License for the specific language governing permissions and * limitations under the License. */ -/* - * pci.c: Linux user space PCI bus management. - * - * Copyright (c) 2008 Eliot Dresselhaus - * - * Permission is hereby granted, free of charge, to any person obtaining - * a copy of this software and associated documentation files (the - * "Software"), to deal in the Software without restriction, including - * without limitation the rights to use, copy, modify, merge, publish, - * distribute, sublicense, and/or sell copies of the Software, and to - * permit persons to whom the Software is furnished to do so, subject to - * the following conditions: - * - * The above copyright notice and this permission notice shall be - * included in all copies or substantial portions of the Software. - * - * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, - * EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF - * MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND - * NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS BE - * LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION - * OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT OF OR IN CONNECTION - * WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE. - */ #include #include @@ -52,6 +28,10 @@ #include #include +#ifndef VFIO_NOIOMMU_IOMMU +#define VFIO_NOIOMMU_IOMMU 8 +#endif + linux_vfio_main_t vfio_main; static int @@ -103,7 +83,7 @@ get_vfio_iommu_group (int group) } static clib_error_t * -open_vfio_iommu_group (int group) +open_vfio_iommu_group (int group, int is_noiommu) { linux_vfio_main_t *lvm = &vfio_main; linux_pci_vfio_iommu_group_t *g; @@ -118,7 +98,7 @@ open_vfio_iommu_group (int group) g->refcnt++; return 0; } - s = format (s, "/dev/vfio/%u%c", group, 0); + s = format (s, "/dev/vfio/%s%u%c", is_noiommu ? "noiommu-" : "", group, 0); fd = open ((char *) s, O_RDWR); if (fd < 0) return clib_error_return_unix (0, "open '%s'", s); @@ -148,13 +128,17 @@ open_vfio_iommu_group (int group) if (lvm->iommu_mode == 0) { - if (ioctl (lvm->container_fd, VFIO_SET_IOMMU, VFIO_TYPE1_IOMMU) < 0) + if (is_noiommu) + lvm->iommu_mode = VFIO_NOIOMMU_IOMMU; + else + lvm->iommu_mode = VFIO_TYPE1_IOMMU; + + if (ioctl (lvm->container_fd, VFIO_SET_IOMMU, lvm->iommu_mode) < 0) { err = clib_error_return_unix (0, "ioctl(VFIO_SET_IOMMU) " "'/dev/vfio/vfio'"); goto error; } - lvm->iommu_mode = VFIO_TYPE1_IOMMU; } @@ -178,6 +162,7 @@ linux_vfio_group_get_device_fd (vlib_pci_addr_t * addr, int *fdp) int iommu_group; u8 *tmpstr; int fd; + int is_noiommu = 0; s = format (s, "/sys/bus/pci/devices/%U/iommu_group", format_vlib_pci_addr, addr); @@ -195,7 +180,20 @@ linux_vfio_group_get_device_fd (vlib_pci_addr_t * addr, int *fdp) } vec_reset_length (s); - if ((err = open_vfio_iommu_group (iommu_group))) + s = + format (s, "/sys/bus/pci/devices/%U/iommu_group/name", + format_vlib_pci_addr, addr); + err = clib_sysfs_read ((char *) s, "%s", &tmpstr); + if (err == 0) + { + if (strncmp ((char *) tmpstr, "vfio-noiommu", 12) == 0) + is_noiommu = 1; + vec_free (tmpstr); + } + else + clib_error_free (err); + vec_reset_length (s); + if ((err = open_vfio_iommu_group (iommu_group, is_noiommu))) return err; g = get_vfio_iommu_group (iommu_group); @@ -232,8 +230,13 @@ linux_vfio_init (vlib_main_t * vm) close (fd); fd = -1; } - else if (ioctl (fd, VFIO_CHECK_EXTENSION, VFIO_TYPE1_IOMMU) == 1) - lvm->flags |= LINUX_VFIO_F_HAVE_IOMMU; + else + { + if (ioctl (fd, VFIO_CHECK_EXTENSION, VFIO_TYPE1_IOMMU) == 1) + lvm->flags |= LINUX_VFIO_F_HAVE_IOMMU; + if (ioctl (fd, VFIO_CHECK_EXTENSION, VFIO_NOIOMMU_IOMMU) == 1) + lvm->flags |= LINUX_VFIO_F_HAVE_NOIOMMU; + } } lvm->iommu_pool_index_by_group = hash_create (0, sizeof (uword)); diff --git a/src/vlib/linux/vfio.h b/src/vlib/linux/vfio.h index e23cee19931..82490d805a8 100644 --- a/src/vlib/linux/vfio.h +++ b/src/vlib/linux/vfio.h @@ -27,6 +27,7 @@ typedef struct { u32 flags; #define LINUX_VFIO_F_HAVE_IOMMU (1 << 0) +#define LINUX_VFIO_F_HAVE_NOIOMMU (1 << 1) int container_fd; /* VFIO */ diff --git a/src/vlib/pci/pci.h b/src/vlib/pci/pci.h index 144eb3f598e..ec805b3ed53 100644 --- a/src/vlib/pci/pci.h +++ b/src/vlib/pci/pci.h @@ -59,6 +59,9 @@ typedef CLIB_PACKED (union typedef struct vlib_pci_device_info { + u32 flags; +#define VLIB_PCI_DEVICE_INFO_F_NOIOMMU (1 << 0); + /* addr */ vlib_pci_addr_t addr; -- cgit 1.2.3-korg