diff options
author | Kyle Evans <kevans@FreeBSD.org> | 2021-01-07 21:51:18 +0000 |
---|---|---|
committer | Kyle Evans <kevans@FreeBSD.org> | 2021-01-08 16:41:50 +0000 |
commit | 9be9771c875259b9c99414feb3833c5af03baa8b (patch) | |
tree | 1f37662474330a8ea3144657af541ceb5acd7aa4 /sys/sys/efiio.h | |
parent | 40903394bf48c916f0790fac8a2cde5c487a2676 (diff) | |
download | src-9be9771c875259b9c99414feb3833c5af03baa8b.tar.gz src-9be9771c875259b9c99414feb3833c5af03baa8b.zip |
efidev: remove EFIIOC_GET_TABLE ioctl
This ioctl would instantly induce a panic, likely since near inception, up
until 0861c7d3e048. Lack of previous interest in fixing it combined with
the problematic interface (exports a pointer, really a physical address)
brings us to the natural conclusion: remove it until a useful consumer
forward.
If it eventually gets resurrected, the interface should definitely not
return in this exact form and likely needs to be reimagined.
The associated KPI, efi_get_table, is left intact for the time being.
Reviewed by: imp, jrtc27
Also discussed with: brooks, jhb
Differential Revision: https://reviews.freebsd.org/D28030
Diffstat (limited to 'sys/sys/efiio.h')
-rw-r--r-- | sys/sys/efiio.h | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/sys/sys/efiio.h b/sys/sys/efiio.h index 87fed9c3afd8..e5a0763536a3 100644 --- a/sys/sys/efiio.h +++ b/sys/sys/efiio.h @@ -32,12 +32,6 @@ #include <sys/uuid.h> #include <sys/efi.h> -struct efi_get_table_ioc -{ - struct uuid uuid; /* UUID to look up */ - void *ptr; /* Pointer to table in KVA space */ -}; - struct efi_var_ioc { efi_char *name; /* User pointer to name, in wide chars */ @@ -48,7 +42,6 @@ struct efi_var_ioc size_t datasize; /* Number of *bytes* in the data */ }; -#define EFIIOC_GET_TABLE _IOWR('E', 1, struct efi_get_table_ioc) #define EFIIOC_GET_TIME _IOR('E', 2, struct efi_tm) #define EFIIOC_SET_TIME _IOW('E', 3, struct efi_tm) #define EFIIOC_VAR_GET _IOWR('E', 4, struct efi_var_ioc) |