From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id E84C6199AC for ; Wed, 16 May 2018 16:42:46 +0200 (CEST) X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 16 May 2018 07:42:45 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.49,390,1520924400"; d="scan'208";a="229095346" Received: from silpixa00399777.ir.intel.com (HELO silpixa00399777.ger.corp.intel.com) ([10.237.222.236]) by fmsmga006.fm.intel.com with ESMTP; 16 May 2018 07:42:44 -0700 From: Ferruh Yigit To: dev@dpdk.org Cc: Ferruh Yigit , Christian Ehrhardt , Luca Boccassi , Maxime Coquelin , Neil Horman , Stephen Hemminger Date: Wed, 16 May 2018 15:42:20 +0100 Message-Id: <20180516144220.21235-1-ferruh.yigit@intel.com> X-Mailer: git-send-email 2.14.3 In-Reply-To: <20180516101851.2443-1-ferruh.yigit@intel.com> References: <20180516101851.2443-1-ferruh.yigit@intel.com> Subject: [dpdk-dev] [PATCH v3] igb_uio: fail and log if kernel lock down is enabled X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 16 May 2018 14:42:47 -0000 When EFI secure boot is enabled, it is possible to lock down kernel and prevent accessing device BARs and this makes igb_uio unusable. Lock down patches are not part of the vanilla kernel but they are applied and used by some distros already [1]. It is not possible to fix this issue, but intention of this patch is to detect and log if kernel lock down enabled and don't insert the module for that case. The challenge is since this feature enabled by distros, they have different config options and APIs for it. This patch is done based on Fedora and Ubuntu kernel source, may needs to add more distro specific support. [1] kernel.ubuntu.com/git/ubuntu/ubuntu-artful.git/commit/?id=99f9ef18d5b6 And a few more patches to Signed-off-by: Ferruh Yigit Acked-by: Luca Boccassi --- Cc: Christian Ehrhardt Cc: Luca Boccassi Cc: Maxime Coquelin Cc: Neil Horman Cc: Stephen Hemminger v2: * remove distro comments from checks Note: Since kernel_is_locked_down() is macro in one case, it can be used for comparison: #ifdef kernel_is_locked_down kernel_is_locked_down(NULL) #else kernel_is_locked_down() This will force all non macro defined cases to else and this may be broken in the feature if macro changed. To be more protective for changes, since this patch is not upstreamed to kernel yet, will keep config check although it is ugly. v3: * remove git artifact, fix build --- kernel/linux/igb_uio/compat.h | 23 +++++++++++++++++++---- kernel/linux/igb_uio/igb_uio.c | 5 +++++ 2 files changed, 24 insertions(+), 4 deletions(-) diff --git a/kernel/linux/igb_uio/compat.h b/kernel/linux/igb_uio/compat.h index d9f4d29fc..dfc66b1ef 100644 --- a/kernel/linux/igb_uio/compat.h +++ b/kernel/linux/igb_uio/compat.h @@ -125,10 +125,6 @@ static bool pci_check_and_mask_intx(struct pci_dev *pdev) #define HAVE_PCI_IS_BRIDGE_API 1 #endif -#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 8, 0) -#define HAVE_ALLOC_IRQ_VECTORS 1 -#endif - #if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 3, 0) #define HAVE_MSI_LIST_IN_GENERIC_DEVICE 1 #endif @@ -136,3 +132,22 @@ static bool pci_check_and_mask_intx(struct pci_dev *pdev) #if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 5, 0) #define HAVE_PCI_MSI_MASK_IRQ 1 #endif + +#if LINUX_VERSION_CODE >= KERNEL_VERSION(4, 8, 0) +#define HAVE_ALLOC_IRQ_VECTORS 1 +#endif + +static inline bool igbuio_kernel_is_locked_down(void) +{ +#ifdef CONFIG_LOCK_DOWN_KERNEL +#ifdef CONFIG_LOCK_DOWN_IN_EFI_SECURE_BOOT + return kernel_is_locked_down(NULL); +#elif CONFIG_EFI_SECURE_BOOT_LOCK_DOWN + return kernel_is_locked_down(); +#else + return false; +#endif +#else + return false; +#endif +} diff --git a/kernel/linux/igb_uio/igb_uio.c b/kernel/linux/igb_uio/igb_uio.c index cd9b7e721..b3233f18e 100644 --- a/kernel/linux/igb_uio/igb_uio.c +++ b/kernel/linux/igb_uio/igb_uio.c @@ -621,6 +621,11 @@ igbuio_pci_init_module(void) { int ret; + if (igbuio_kernel_is_locked_down()) { + pr_err("Not able to use module, kernel lock down is enabled\n"); + return -EINVAL; + } + ret = igbuio_config_intr_mode(intr_mode); if (ret < 0) return ret; -- 2.14.3