[PATCH] efi_runtime: remove efi_runtime dkms

Ivan Hu ivan.hu at canonical.com
Thu Jul 25 08:03:25 UTC 2019


efi_runtime driver provides the runtime UEFI interfaces for fwts to test
the UEFI runtime service implementiation and has upstream in Linux 4.9 named
efi_test. All supported Ubuntu kernels have efi_test included. As a result,
this dkms is no longer required.

Signed-off-by: Ivan Hu <ivan.hu at canonical.com>
---
 debian/control                    |   9 +-
 debian/fwts-efi-runtime-dkms.dkms |   6 -
 debian/rules                      |  18 +-
 efi_runtime/.gitignore            |   4 -
 efi_runtime/Makefile              |  28 --
 efi_runtime/efi_runtime.c         | 801 --------------------------------------
 efi_runtime/efi_runtime.h         | 135 -------
 7 files changed, 3 insertions(+), 998 deletions(-)
 delete mode 100644 debian/fwts-efi-runtime-dkms.dkms
 delete mode 100644 efi_runtime/.gitignore
 delete mode 100644 efi_runtime/Makefile
 delete mode 100644 efi_runtime/efi_runtime.c
 delete mode 100644 efi_runtime/efi_runtime.h

diff --git a/debian/control b/debian/control
index dcc7c19..2f88cd1 100644
--- a/debian/control
+++ b/debian/control
@@ -4,7 +4,7 @@ Priority: optional
 Maintainer: Firmware Testing Team <fwts-devel at lists.ubuntu.com>
 Uploaders: Colin King <colin.king at ubuntu.com>, Alex Hung <alex.hung at canonical.com>, Ivan Hu <ivan.hu at canonical.com>
 Standards-Version: 3.9.8
-Build-Depends: debhelper (>= 7.0.50~), autoconf, automake, libtool, libjson0-dev | libjson-c-dev (>= 0.9), flex, bison, dh-autoreconf, dkms, libglib2.0-dev, pkg-config, libfdt-dev, libbsd-dev
+Build-Depends: debhelper (>= 7.0.50~), autoconf, automake, libtool, libjson0-dev | libjson-c-dev (>= 0.9), flex, bison, dh-autoreconf, libglib2.0-dev, pkg-config, libfdt-dev, libbsd-dev
 
 Package: fwts
 Architecture: i386 amd64 armel armhf arm64 ppc64 ppc64el s390x
@@ -53,10 +53,3 @@ Description: Firmware Test Suite frontend interface
  to execute tests via Firmware Test Suite and is also used as the primary 
  interface within Firmware Test Suite live images.
 
-Package: fwts-efi-runtime-dkms
-Architecture: i386 amd64 armel armhf arm64 ppc64 ppc64el
-Priority: optional
-Depends: ${misc:Depends}, dkms
-Description: Firmware Test Suite UEFI Runtime Service kernel driver
- This package provides the efi_runtime kernel driver in DKMS format,
- which is required for accessing UEFI Runtime Services.
diff --git a/debian/fwts-efi-runtime-dkms.dkms b/debian/fwts-efi-runtime-dkms.dkms
deleted file mode 100644
index 94b3415..0000000
--- a/debian/fwts-efi-runtime-dkms.dkms
+++ /dev/null
@@ -1,6 +0,0 @@
-PACKAGE_NAME="fwts-efi-runtime-dkms"
-PACKAGE_VERSION="#MODULE_VERSION#"
-MAKE[0]="KVER=$kernelver make"
-BUILT_MODULE_NAME[0]="efi_runtime"
-DEST_MODULE_LOCATION[0]="/updates"
-AUTOINSTALL="yes"
diff --git a/debian/rules b/debian/rules
index c24df00..3ca21ad 100755
--- a/debian/rules
+++ b/debian/rules
@@ -2,27 +2,13 @@
 
 export DEB_BUILD_MAINT_OPTIONS=hardening=+all
 
+Package: fwts-efi-runtime-dkms
 DEB_BUILD_ARCH := $(shell dpkg-architecture -qDEB_BUILD_ARCH)
 
-DEBVERS := $(shell dpkg-parsechangelog | grep ^Version: | cut -d' ' -f2 \
-		| cut -d- -f1)
-
-VERSION := $(shell echo '$(DEBVERS)' | sed -e 's/[+-].*//' -e 's/~//g')
-
-DKMS_SRC_DIR := $(CURDIR)/debian/fwts-efi-runtime-dkms/usr/src/fwts-efi-runtime-dkms-$(VERSION)
-
-override_dh_auto_install:
-	install -d $(DKMS_SRC_DIR)
-	cp -a efi_runtime/* $(DKMS_SRC_DIR)
-	dh_auto_install
-
-override_dh_dkms:
-	dh_dkms -V $(VERSION)
-
 override_dh_auto_test:
 ifneq ($(DEB_BUILD_ARCH),arm64)
 	dh_auto_test $@
 endif
 
 %:
-	dh $@ --with autoreconf,dkms
+	dh $@ --with autoreconf
diff --git a/efi_runtime/.gitignore b/efi_runtime/.gitignore
deleted file mode 100644
index a4b501b..0000000
--- a/efi_runtime/.gitignore
+++ /dev/null
@@ -1,4 +0,0 @@
-Module.symvers
-efi_runtime.ko
-efi_runtime.mod.c
-modules.order
diff --git a/efi_runtime/Makefile b/efi_runtime/Makefile
deleted file mode 100644
index ae86867..0000000
--- a/efi_runtime/Makefile
+++ /dev/null
@@ -1,28 +0,0 @@
-#
-# Copyright (C) 2012-2019 Canonical, Ltd.
-#
-# This program is free software; you can redistribute it and/or
-# modify it under the terms of the GNU General Public License
-# as published by the Free Software Foundation; either version 2
-# of the License, or (at your option) any later version.
-#
-# This program is distributed in the hope that it will be useful,
-# but WITHOUT ANY WARRANTY; without even the implied warranty of
-# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-# GNU General Public License for more details.
-#
-# You should have received a copy of the GNU General Public License
-# along with this program; if not, write to the Free Software
-# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
-#
-
-KVER ?= `uname -r`
-obj-m += efi_runtime.o
-all:
-	make -C /lib/modules/$(KVER)/build M=`pwd` modules
-
-install:
-	make -C /lib/modules/$(KVER)/build M=`pwd` modules_install
-
-clean:
-	make -C /lib/modules/$(KVER)/build M=`pwd` clean
diff --git a/efi_runtime/efi_runtime.c b/efi_runtime/efi_runtime.c
deleted file mode 100644
index 5dfa089..0000000
--- a/efi_runtime/efi_runtime.c
+++ /dev/null
@@ -1,801 +0,0 @@
-/*
- * EFI Runtime driver
- *
- * Copyright(C) 2012-2019 Canonical Ltd.
- *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, write to the Free Software
- *  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
- *  USA.
- */
-
-#include <linux/version.h>
-#include <linux/miscdevice.h>
-#include <linux/module.h>
-#include <linux/init.h>
-#include <linux/proc_fs.h>
-#include <linux/efi.h>
-#include <linux/slab.h>
-#include <linux/uaccess.h>
-
-#include "efi_runtime.h"
-
-MODULE_AUTHOR("Ivan Hu");
-MODULE_DESCRIPTION("EFI Runtime Driver");
-MODULE_LICENSE("GPL");
-
-/* commit 83e681897 turned efi_enabled into a function, so abstract it */
-#ifdef EFI_RUNTIME_SERVICES
-#define EFI_RUNTIME_ENABLED	efi_enabled(EFI_RUNTIME_SERVICES)
-#else
-#define EFI_RUNTIME_ENABLED	efi_enabled
-#endif
-
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(5, 0, 0)
-#define ACCESS_OK(type, addr, size)	access_ok(addr, size)
-#else
-#define ACCESS_OK(type, addr, size)	access_ok(type, addr, size)
-#endif
-
-/*
- * Count the bytes in 'str', including the terminating NULL.
- *
- * Note this function returns the number of *bytes*, not the number of
- * ucs2 characters.
- */
-static inline size_t user_ucs2_strsize(efi_char16_t  __user *str)
-{
-	efi_char16_t *s = str, c;
-	size_t len;
-
-	if (!str)
-		return 0;
-
-	/* Include terminating NULL */
-	len = sizeof(efi_char16_t);
-
-	if (get_user(c, s++)) {
-		/* Can't read userspace memory for size */
-		return 0;
-	}
-
-	while (c != 0) {
-		if (get_user(c, s++)) {
-			/* Can't read userspace memory for size */
-			return 0;
-		}
-		len += sizeof(efi_char16_t);
-	}
-	return len;
-}
-
-/*
- * Allocate a buffer and copy a ucs2 string from user space into it.
- */
-static inline int
-copy_ucs2_from_user_len(efi_char16_t **dst, efi_char16_t __user *src,
-			size_t len)
-{
-	efi_char16_t *buf;
-
-	if (!src) {
-		*dst = NULL;
-		return 0;
-	}
-
-	if (!ACCESS_OK(VERIFY_READ, src, 1))
-		return -EFAULT;
-
-	buf = memdup_user(src, len);
-	if (IS_ERR(buf)) {
-		*dst = NULL;
-		return PTR_ERR(buf);
-	}
-	*dst = buf;
-
-	return 0;
-}
-
-/*
- * Count the bytes in 'str', including the terminating NULL.
- *
- * Just a wrap for user_ucs2_strsize
- */
-static inline int
-get_ucs2_strsize_from_user(efi_char16_t __user *src, size_t *len)
-{
-	if (!ACCESS_OK(VERIFY_READ, src, 1))
-		return -EFAULT;
-
-	*len = user_ucs2_strsize(src);
-	if (*len == 0)
-		return -EFAULT;
-
-	return 0;
-}
-
-/*
- * Calculate the required buffer allocation size and copy a ucs2 string
- * from user space into it.
- *
- * This function differs from copy_ucs2_from_user_len() because it
- * calculates the size of the buffer to allocate by taking the length of
- * the string 'src'.
- *
- * If a non-zero value is returned, the caller MUST NOT access 'dst'.
- *
- * It is the caller's responsibility to free 'dst'.
- */
-static inline int
-copy_ucs2_from_user(efi_char16_t **dst, efi_char16_t __user *src)
-{
-	size_t len;
-
-	if (!ACCESS_OK(VERIFY_READ, src, 1))
-		return -EFAULT;
-
-	len = user_ucs2_strsize(src);
-	if (len == 0)
-		return -EFAULT;
-	return copy_ucs2_from_user_len(dst, src, len);
-}
-
-/*
- * Copy a ucs2 string to a user buffer.
- *
- * This function is a simple wrapper around copy_to_user() that does
- * nothing if 'src' is NULL, which is useful for reducing the amount of
- * NULL checking the caller has to do.
- *
- * 'len' specifies the number of bytes to copy.
- */
-static inline int
-copy_ucs2_to_user_len(efi_char16_t __user *dst, efi_char16_t *src, size_t len)
-{
-	if (!src)
-		return 0;
-
-	if (!ACCESS_OK(VERIFY_WRITE, dst, 1))
-		return -EFAULT;
-
-	return copy_to_user(dst, src, len);
-}
-
-static long efi_runtime_get_variable(unsigned long arg)
-{
-	struct efi_getvariable __user *getvariable_user;
-	struct efi_getvariable getvariable;
-	unsigned long datasize = 0, prev_datasize, *dz;
-	efi_guid_t vendor_guid, *vd = NULL;
-	efi_status_t status;
-	efi_char16_t *name = NULL;
-	u32 attr, *at;
-	void *data = NULL;
-	int rv = 0;
-
-	getvariable_user = (struct efi_getvariable __user *)arg;
-
-	if (copy_from_user(&getvariable, getvariable_user,
-			   sizeof(getvariable)))
-		return -EFAULT;
-	if (getvariable.data_size &&
-	    get_user(datasize, getvariable.data_size))
-		return -EFAULT;
-	if (getvariable.vendor_guid) {
-		if (copy_from_user(&vendor_guid, getvariable.vendor_guid,
-					sizeof(vendor_guid)))
-			return -EFAULT;
-		vd = &vendor_guid;
-	}
-
-	if (getvariable.variable_name) {
-		rv = copy_ucs2_from_user(&name, getvariable.variable_name);
-		if (rv)
-			return rv;
-	}
-
-	at = getvariable.attributes ? &attr : NULL;
-	dz = getvariable.data_size ? &datasize : NULL;
-
-	if (getvariable.data_size && getvariable.data) {
-		data = kmalloc(datasize, GFP_KERNEL);
-		if (!data) {
-			kfree(name);
-			return -ENOMEM;
-		}
-	}
-
-	prev_datasize = datasize;
-	status = efi.get_variable(name, vd, at, dz, data);
-	kfree(name);
-
-	if (put_user(status, getvariable.status)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	if (status != EFI_SUCCESS) {
-		if (status == EFI_BUFFER_TOO_SMALL) {
-			if (dz && put_user(datasize, getvariable.data_size)) {
-				rv = -EFAULT;
-				goto out;
-			}
-		}
-		rv = -EINVAL;
-		goto out;
-	}
-
-	if (prev_datasize < datasize) {
-		rv = -EINVAL;
-		goto out;
-	}
-
-	if (data) {
-		if (copy_to_user(getvariable.data, data, datasize)) {
-			rv = -EFAULT;
-			goto out;
-		}
-	}
-
-	if (at && put_user(attr, getvariable.attributes)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	if (dz && put_user(datasize, getvariable.data_size))
-		rv = -EFAULT;
-
-out:
-	kfree(data);
-	return rv;
-
-}
-
-static long efi_runtime_set_variable(unsigned long arg)
-{
-	struct efi_setvariable __user *setvariable_user;
-	struct efi_setvariable setvariable;
-	efi_guid_t vendor_guid;
-	efi_status_t status;
-	efi_char16_t *name = NULL;
-	void *data;
-	int rv = 0;
-
-	setvariable_user = (struct efi_setvariable __user *)arg;
-
-	if (copy_from_user(&setvariable, setvariable_user, sizeof(setvariable)))
-		return -EFAULT;
-	if (copy_from_user(&vendor_guid, setvariable.vendor_guid,
-				sizeof(vendor_guid)))
-		return -EFAULT;
-
-	if (setvariable.variable_name) {
-		rv = copy_ucs2_from_user(&name, setvariable.variable_name);
-		if (rv)
-			return rv;
-	}
-
-	data = memdup_user(setvariable.data, setvariable.data_size);
-	if (IS_ERR(data)) {
-		kfree(name);
-		return PTR_ERR(data);
-	}
-
-	status = efi.set_variable(name, &vendor_guid,
-				setvariable.attributes,
-				setvariable.data_size, data);
-
-	if (put_user(status, setvariable.status)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	rv = status == EFI_SUCCESS ? 0 : -EINVAL;
-
-out:
-	kfree(data);
-	kfree(name);
-
-	return rv;
-}
-
-static long efi_runtime_get_time(unsigned long arg)
-{
-	struct efi_gettime __user *gettime_user;
-	struct efi_gettime  gettime;
-	efi_status_t status;
-	efi_time_cap_t cap;
-	efi_time_t efi_time;
-
-	gettime_user = (struct efi_gettime __user *)arg;
-	if (copy_from_user(&gettime, gettime_user, sizeof(gettime)))
-		return -EFAULT;
-
-	status = efi.get_time(gettime.time ? &efi_time : NULL,
-			      gettime.capabilities ? &cap : NULL);
-
-	if (put_user(status, gettime.status))
-		return -EFAULT;
-
-	if (status != EFI_SUCCESS)
-		return -EINVAL;
-
-	if (gettime.capabilities) {
-		efi_time_cap_t __user *cap_local;
-
-		cap_local = (efi_time_cap_t *)gettime.capabilities;
-		if (put_user(cap.resolution, &(cap_local->resolution)) ||
-			put_user(cap.accuracy, &(cap_local->accuracy)) ||
-			put_user(cap.sets_to_zero, &(cap_local->sets_to_zero)))
-			return -EFAULT;
-	}
-	if (gettime.time) {
-		if (copy_to_user(gettime.time, &efi_time, sizeof(efi_time_t)))
-			return -EFAULT;
-	}
-
-	return 0;
-}
-
-static long efi_runtime_set_time(unsigned long arg)
-{
-	struct efi_settime __user *settime_user;
-	struct efi_settime settime;
-	efi_status_t status;
-	efi_time_t efi_time;
-
-	settime_user = (struct efi_settime __user *)arg;
-	if (copy_from_user(&settime, settime_user, sizeof(settime)))
-		return -EFAULT;
-	if (copy_from_user(&efi_time, settime.time,
-					sizeof(efi_time_t)))
-		return -EFAULT;
-	status = efi.set_time(&efi_time);
-
-	if (put_user(status, settime.status))
-		return -EFAULT;
-
-	return status == EFI_SUCCESS ? 0 : -EINVAL;
-}
-
-static long efi_runtime_get_waketime(unsigned long arg)
-{
-	struct efi_getwakeuptime __user *getwakeuptime_user;
-	struct efi_getwakeuptime getwakeuptime;
-	efi_bool_t enabled, pending;
-	efi_status_t status;
-	efi_time_t efi_time;
-
-	getwakeuptime_user = (struct efi_getwakeuptime __user *)arg;
-	if (copy_from_user(&getwakeuptime, getwakeuptime_user,
-				sizeof(getwakeuptime)))
-		return -EFAULT;
-
-	status = efi.get_wakeup_time(
-		getwakeuptime.enabled ? (efi_bool_t *)&enabled : NULL,
-		getwakeuptime.pending ? (efi_bool_t *)&pending : NULL,
-		getwakeuptime.time ? &efi_time : NULL);
-
-	if (put_user(status, getwakeuptime.status))
-		return -EFAULT;
-
-	if (status != EFI_SUCCESS)
-		return -EINVAL;
-
-	if (getwakeuptime.enabled && put_user(enabled,
-						getwakeuptime.enabled))
-		return -EFAULT;
-
-	if (getwakeuptime.time) {
-		if (copy_to_user(getwakeuptime.time, &efi_time,
-				sizeof(efi_time_t)))
-			return -EFAULT;
-	}
-
-	return 0;
-}
-
-static long efi_runtime_set_waketime(unsigned long arg)
-{
-	struct efi_setwakeuptime __user *setwakeuptime_user;
-	struct efi_setwakeuptime setwakeuptime;
-	efi_bool_t enabled;
-	efi_status_t status;
-	efi_time_t efi_time;
-
-	setwakeuptime_user = (struct efi_setwakeuptime __user *)arg;
-
-	if (copy_from_user(&setwakeuptime, setwakeuptime_user,
-				sizeof(setwakeuptime)))
-		return -EFAULT;
-
-	enabled = setwakeuptime.enabled;
-	if (setwakeuptime.time) {
-		if (copy_from_user(&efi_time, setwakeuptime.time,
-					sizeof(efi_time_t)))
-			return -EFAULT;
-
-		status = efi.set_wakeup_time(enabled, &efi_time);
-	} else
-		status = efi.set_wakeup_time(enabled, NULL);
-
-	if (put_user(status, setwakeuptime.status))
-		return -EFAULT;
-
-	return status == EFI_SUCCESS ? 0 : -EINVAL;
-}
-
-static long efi_runtime_get_nextvariablename(unsigned long arg)
-{
-	struct efi_getnextvariablename __user *getnextvariablename_user;
-	struct efi_getnextvariablename getnextvariablename;
-	unsigned long name_size, prev_name_size = 0, *ns = NULL;
-	efi_status_t status;
-	efi_guid_t *vd = NULL;
-	efi_guid_t vendor_guid;
-	efi_char16_t *name = NULL;
-	int rv = 0;
-
-	getnextvariablename_user = (struct efi_getnextvariablename __user *)arg;
-
-	if (copy_from_user(&getnextvariablename, getnextvariablename_user,
-			   sizeof(getnextvariablename)))
-		return -EFAULT;
-
-	if (getnextvariablename.variable_name_size) {
-		if (get_user(name_size, getnextvariablename.variable_name_size))
-			return -EFAULT;
-		ns = &name_size;
-		prev_name_size = name_size;
-	}
-
-	if (getnextvariablename.vendor_guid) {
-		if (copy_from_user(&vendor_guid,
-				getnextvariablename.vendor_guid,
-				sizeof(vendor_guid)))
-			return -EFAULT;
-		vd = &vendor_guid;
-	}
-
-	if (getnextvariablename.variable_name) {
-		size_t name_string_size = 0;
-
-		rv = get_ucs2_strsize_from_user(
-				getnextvariablename.variable_name,
-				&name_string_size);
-		if (rv)
-			return rv;
-		/*
-		 * The name_size may be smaller than the real buffer size where
-		 * variable name located in some use cases. The most typical
-		 * case is passing a 0 to get the required buffer size for the
-		 * 1st time call. So we need to copy the content from user
-		 * space for at least the string size of variable name, or else
-		 * the name passed to UEFI may not be terminated as we expected.
-		 */
-		rv = copy_ucs2_from_user_len(&name,
-				getnextvariablename.variable_name,
-				prev_name_size > name_string_size ?
-				prev_name_size : name_string_size);
-		if (rv)
-			return rv;
-	}
-
-	status = efi.get_next_variable(ns, name, vd);
-
-	if (put_user(status, getnextvariablename.status)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	if (status != EFI_SUCCESS) {
-		if (status == EFI_BUFFER_TOO_SMALL) {
-			if (ns && put_user(*ns,
-				getnextvariablename.variable_name_size)) {
-				rv = -EFAULT;
-				goto out;
-			}
-		}
-		rv = -EINVAL;
-		goto out;
-	}
-
-	if (name) {
-		if (copy_ucs2_to_user_len(getnextvariablename.variable_name,
-						name, prev_name_size)) {
-			rv = -EFAULT;
-			goto out;
-		}
-	}
-
-	if (ns) {
-		if (put_user(*ns, getnextvariablename.variable_name_size)) {
-			rv = -EFAULT;
-			goto out;
-		}
-	}
-
-	if (vd) {
-		if (copy_to_user(getnextvariablename.vendor_guid, vd,
-							sizeof(efi_guid_t)))
-			rv = -EFAULT;
-	}
-
-out:
-	kfree(name);
-	return rv;
-}
-
-static long efi_runtime_get_nexthighmonocount(unsigned long arg)
-{
-	struct efi_getnexthighmonotoniccount __user *getnexthighmonocount_user;
-	struct efi_getnexthighmonotoniccount getnexthighmonocount;
-	efi_status_t status;
-	u32 count;
-
-	getnexthighmonocount_user = (struct
-			efi_getnexthighmonotoniccount __user *)arg;
-
-	if (copy_from_user(&getnexthighmonocount,
-			   getnexthighmonocount_user,
-			   sizeof(getnexthighmonocount)))
-		return -EFAULT;
-
-	status = efi.get_next_high_mono_count(
-		getnexthighmonocount.high_count ? &count : NULL);
-
-	if (put_user(status, getnexthighmonocount.status))
-		return -EFAULT;
-
-	if (status != EFI_SUCCESS)
-		return -EINVAL;
-
-	if (getnexthighmonocount.high_count &&
-	    put_user(count, getnexthighmonocount.high_count))
-		return -EFAULT;
-
-	return 0;
-}
-
-static long efi_runtime_reset_system(unsigned long arg)
-{
-	struct efi_resetsystem __user *resetsystem_user;
-	struct efi_resetsystem resetsystem;
-	void *data = NULL;
-
-	resetsystem_user = (struct efi_resetsystem __user *)arg;
-	if (copy_from_user(&resetsystem, resetsystem_user,
-						sizeof(resetsystem)))
-		return -EFAULT;
-	if (resetsystem.data_size != 0) {
-		data = memdup_user((void *)resetsystem.data,
-						resetsystem.data_size);
-		if (IS_ERR(data))
-			return PTR_ERR(data);
-	}
-
-	efi.reset_system(resetsystem.reset_type, resetsystem.status,
-				resetsystem.data_size, (efi_char16_t *)data);
-
-	kfree(data);
-	return 0;
-}
-
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(3, 1, 0)
-static long efi_runtime_query_variableinfo(unsigned long arg)
-{
-	struct efi_queryvariableinfo __user *queryvariableinfo_user;
-	struct efi_queryvariableinfo queryvariableinfo;
-	efi_status_t status;
-	u64 max_storage, remaining, max_size;
-
-	queryvariableinfo_user = (struct efi_queryvariableinfo __user *)arg;
-
-	if (copy_from_user(&queryvariableinfo, queryvariableinfo_user,
-			   sizeof(queryvariableinfo)))
-		return -EFAULT;
-
-	status = efi.query_variable_info(queryvariableinfo.attributes,
-					 &max_storage, &remaining, &max_size);
-
-	if (put_user(status, queryvariableinfo.status))
-		return -EFAULT;
-
-	if (status != EFI_SUCCESS)
-		return -EINVAL;
-
-	if (put_user(max_storage,
-		     queryvariableinfo.maximum_variable_storage_size))
-		return -EFAULT;
-
-	if (put_user(remaining,
-		     queryvariableinfo.remaining_variable_storage_size))
-		return -EFAULT;
-
-	if (put_user(max_size, queryvariableinfo.maximum_variable_size))
-		return -EFAULT;
-
-	return 0;
-}
-
-static long efi_runtime_query_capsulecaps(unsigned long arg)
-{
-	struct efi_querycapsulecapabilities __user *qcaps_user;
-	struct efi_querycapsulecapabilities qcaps;
-	efi_capsule_header_t *capsules;
-	efi_status_t status;
-	u64 max_size;
-	int i, reset_type;
-	int rv = 0;
-
-	qcaps_user = (struct efi_querycapsulecapabilities __user *)arg;
-
-	if (copy_from_user(&qcaps, qcaps_user, sizeof(qcaps)))
-		return -EFAULT;
-
-	capsules = kcalloc(qcaps.capsule_count + 1,
-			   sizeof(efi_capsule_header_t), GFP_KERNEL);
-	if (!capsules)
-		return -ENOMEM;
-
-	for (i = 0; i < qcaps.capsule_count; i++) {
-		efi_capsule_header_t *c;
-		/*
-		 * We cannot dereference qcaps.capsule_header_array directly to
-		 * obtain the address of the capsule as it resides in the
-		 * user space
-		 */
-		if (get_user(c, qcaps.capsule_header_array + i)) {
-			rv = -EFAULT;
-			goto out;
-		}
-		if (copy_from_user(&capsules[i], c,
-				sizeof(efi_capsule_header_t))) {
-			rv = -EFAULT;
-			goto out;
-		}
-	}
-
-	qcaps.capsule_header_array = &capsules;
-
-	status = efi.query_capsule_caps((efi_capsule_header_t **)
-					qcaps.capsule_header_array,
-					qcaps.capsule_count,
-					&max_size, &reset_type);
-
-	if (put_user(status, qcaps.status)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	if (status != EFI_SUCCESS) {
-		rv = -EINVAL;
-		goto out;
-	}
-
-	if (put_user(max_size, qcaps.maximum_capsule_size)) {
-		rv = -EFAULT;
-		goto out;
-	}
-
-	if (put_user(reset_type, qcaps.reset_type))
-		rv = -EFAULT;
-
-out:
-	kfree(capsules);
-	return rv;
-}
-#endif
-
-static long efi_runtime_ioctl(struct file *file, unsigned int cmd,
-							unsigned long arg)
-{
-	switch (cmd) {
-	case EFI_RUNTIME_GET_VARIABLE:
-		return efi_runtime_get_variable(arg);
-
-	case EFI_RUNTIME_SET_VARIABLE:
-		return efi_runtime_set_variable(arg);
-
-	case EFI_RUNTIME_GET_TIME:
-		return efi_runtime_get_time(arg);
-
-	case EFI_RUNTIME_SET_TIME:
-		return efi_runtime_set_time(arg);
-
-	case EFI_RUNTIME_GET_WAKETIME:
-		return efi_runtime_get_waketime(arg);
-
-	case EFI_RUNTIME_SET_WAKETIME:
-		return efi_runtime_set_waketime(arg);
-
-	case EFI_RUNTIME_GET_NEXTVARIABLENAME:
-		return efi_runtime_get_nextvariablename(arg);
-
-	case EFI_RUNTIME_GET_NEXTHIGHMONOTONICCOUNT:
-		return efi_runtime_get_nexthighmonocount(arg);
-
-	case EFI_RUNTIME_RESET_SYSTEM:
-		return efi_runtime_reset_system(arg);
-
-#if LINUX_VERSION_CODE >= KERNEL_VERSION(3, 1, 0)
-	case EFI_RUNTIME_QUERY_VARIABLEINFO:
-		return efi_runtime_query_variableinfo(arg);
-
-	case EFI_RUNTIME_QUERY_CAPSULECAPABILITIES:
-		return efi_runtime_query_capsulecaps(arg);
-#endif
-	}
-
-	return -ENOTTY;
-}
-
-static int efi_runtime_open(struct inode *inode, struct file *file)
-{
-	/*
-	 * nothing special to do here
-	 * We do accept multiple open files at the same time as we
-	 * synchronize on the per call operation.
-	 */
-	return 0;
-}
-
-static int efi_runtime_close(struct inode *inode, struct file *file)
-{
-	return 0;
-}
-
-/*
- *	The various file operations we support.
- */
-static const struct file_operations efi_runtime_fops = {
-	.owner		= THIS_MODULE,
-	.unlocked_ioctl	= efi_runtime_ioctl,
-	.open		= efi_runtime_open,
-	.release	= efi_runtime_close,
-	.llseek		= no_llseek,
-};
-
-static struct miscdevice efi_runtime_dev = {
-	MISC_DYNAMIC_MINOR,
-	"efi_runtime",
-	&efi_runtime_fops
-};
-
-static int __init efi_runtime_init(void)
-{
-	int ret;
-
-	if (!EFI_RUNTIME_ENABLED) {
-		pr_err("EFI runtime services not enabled.\n");
-		return -ENODEV;
-	}
-
-	ret = misc_register(&efi_runtime_dev);
-	if (ret) {
-		pr_err("efi_runtime: can't misc_register on minor=%d\n",
-			MISC_DYNAMIC_MINOR);
-		return ret;
-	}
-
-	return 0;
-}
-
-static void __exit efi_runtime_exit(void)
-{
-	misc_deregister(&efi_runtime_dev);
-}
-
-module_init(efi_runtime_init);
-module_exit(efi_runtime_exit);
-
diff --git a/efi_runtime/efi_runtime.h b/efi_runtime/efi_runtime.h
deleted file mode 100644
index 1d9936b..0000000
--- a/efi_runtime/efi_runtime.h
+++ /dev/null
@@ -1,135 +0,0 @@
-/*
- * EFI Runtime driver
- *
- * Copyright(C) 2012-2019 Canonical Ltd.
- *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 2 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, write to the Free Software
- *  Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301,
- *  USA.
- */
-
-#ifndef _EFI_RUNTIME_H_
-#define _EFI_RUNTIME_H_
-
-#include <linux/efi.h>
-
-struct efi_getvariable {
-	efi_char16_t	*variable_name;
-	efi_guid_t	*vendor_guid;
-	u32		*attributes;
-	unsigned long	*data_size;
-	void		*data;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_setvariable {
-	efi_char16_t	*variable_name;
-	efi_guid_t	*vendor_guid;
-	u32		attributes;
-	unsigned long	data_size;
-	void		*data;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_getnextvariablename {
-	unsigned long	*variable_name_size;
-	efi_char16_t	*variable_name;
-	efi_guid_t	*vendor_guid;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_queryvariableinfo {
-	u32		attributes;
-	u64		*maximum_variable_storage_size;
-	u64		*remaining_variable_storage_size;
-	u64		*maximum_variable_size;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_gettime {
-	efi_time_t	*time;
-	efi_time_cap_t	*capabilities;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_settime {
-	efi_time_t	*time;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_getwakeuptime {
-	efi_bool_t	*enabled;
-	efi_bool_t	*pending;
-	efi_time_t	*time;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_setwakeuptime {
-	efi_bool_t	enabled;
-	efi_time_t	*time;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_getnexthighmonotoniccount {
-	u32		*high_count;
-	efi_status_t	*status;
-} __packed;
-
-struct efi_querycapsulecapabilities {
-	efi_capsule_header_t	**capsule_header_array;
-	unsigned long		capsule_count;
-	u64			*maximum_capsule_size;
-	int			*reset_type;
-	efi_status_t		*status;
-} __packed;
-
-struct efi_resetsystem {
-	int			reset_type;
-	efi_status_t		status;
-	unsigned long		data_size;
-	efi_char16_t		*data;
-} __packed;
-
-/* ioctl calls that are permitted to the /dev/efi_runtime interface. */
-#define EFI_RUNTIME_GET_VARIABLE \
-	_IOWR('p', 0x01, struct efi_getvariable)
-#define EFI_RUNTIME_SET_VARIABLE \
-	_IOW('p', 0x02, struct efi_setvariable)
-
-#define EFI_RUNTIME_GET_TIME \
-	_IOR('p', 0x03, struct efi_gettime)
-#define EFI_RUNTIME_SET_TIME \
-	_IOW('p', 0x04, struct efi_settime)
-
-#define EFI_RUNTIME_GET_WAKETIME \
-	_IOR('p', 0x05, struct efi_getwakeuptime)
-#define EFI_RUNTIME_SET_WAKETIME \
-	_IOW('p', 0x06, struct efi_setwakeuptime)
-
-#define EFI_RUNTIME_GET_NEXTVARIABLENAME \
-	_IOWR('p', 0x07, struct efi_getnextvariablename)
-
-#define EFI_RUNTIME_QUERY_VARIABLEINFO \
-	_IOR('p', 0x08, struct efi_queryvariableinfo)
-
-#define EFI_RUNTIME_GET_NEXTHIGHMONOTONICCOUNT \
-	_IOR('p', 0x09, struct efi_getnexthighmonotoniccount)
-
-#define EFI_RUNTIME_QUERY_CAPSULECAPABILITIES \
-	_IOR('p', 0x0A, struct efi_querycapsulecapabilities)
-
-#define EFI_RUNTIME_RESET_SYSTEM \
-	_IOW('p', 0x0B, struct efi_resetsystem)
-
-#endif /* _EFI_RUNTIME_H_ */
-- 
2.7.4




More information about the fwts-devel mailing list