[RFC] fs: make efivarfs a pseudo filesystem, built by default with EFI

[RFC] fs: make efivarfs a pseudo filesystem, built by default with EFI

[Leif Lindholm]

efivars is currently enabled under MISC_FILESYSTEMS, which is decribed
as "such as filesystems that came from other operating systems".
In reality, it is a pseudo filesystem, providing access to the kernel
UEFI variable interface.

Since this is the preferred interface for accessing UEFI variables, over
the legacy efivars interface, also build it by default as a module if
CONFIG_EFI.

Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
---
 fs/Kconfig          | 2 +-
 fs/efivarfs/Kconfig | 1 +
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/fs/Kconfig b/fs/Kconfig
index 664991a..a6bb530 100644
--- a/fs/Kconfig
+++ b/fs/Kconfig
@@ -165,6 +165,7 @@ config HUGETLB_PAGE
 	def_bool HUGETLBFS
 
 source "fs/configfs/Kconfig"
+source "fs/efivarfs/Kconfig"
 
 endmenu
 
@@ -209,7 +210,6 @@ source "fs/sysv/Kconfig"
 source "fs/ufs/Kconfig"
 source "fs/exofs/Kconfig"
 source "fs/f2fs/Kconfig"
-source "fs/efivarfs/Kconfig"
 
 endif # MISC_FILESYSTEMS
 
diff --git a/fs/efivarfs/Kconfig b/fs/efivarfs/Kconfig
index 367bbb1..c2499ef 100644
--- a/fs/efivarfs/Kconfig
+++ b/fs/efivarfs/Kconfig
@@ -1,6 +1,7 @@
 config EFIVAR_FS
 	tristate "EFI Variable filesystem"
 	depends on EFI
+	default m
 	help
 	  efivarfs is a replacement filesystem for the old EFI
 	  variable support via sysfs, as it doesn't suffer from the
-- 
2.1.3

Re: [RFC] fs: make efivarfs a pseudo filesystem, built by default with EFI

[Matt Fleming]

On Thu, 18 Dec, at 05:50:49PM, Leif Lindholm wrote:
> efivars is currently enabled under MISC_FILESYSTEMS, which is decribed
> as "such as filesystems that came from other operating systems".
> In reality, it is a pseudo filesystem, providing access to the kernel
> UEFI variable interface.
> 
> Since this is the preferred interface for accessing UEFI variables, over
> the legacy efivars interface, also build it by default as a module if
> CONFIG_EFI.
> 
> Signed-off-by: Leif Lindholm <leif.lindholm@linaro.org>
> ---
>  fs/Kconfig          | 2 +-
>  fs/efivarfs/Kconfig | 1 +
>  2 files changed, 2 insertions(+), 1 deletion(-)

Looks good to me Leif.

Unless I hear otherwise, I'll apply this to the 'next' branch of the EFI
tree this week.

-- 
Matt Fleming, Intel Open Source Technology Center