efi: Auto-load the efi-pstore module

efi: Auto-load the efi-pstore module

[Ben Hutchings]

[-- Attachment #1: Type: text/plain, Size: 640 bytes --]

efi-pstore should be auto-loaded on EFI systems, same as efivars.

Signed-off-by: Ben Hutchings <ben-/+tVBieCtBitmTQ+vhA3Yw@public.gmane.org>
---
This was included in Debian 8 "jessie" with no complaints so far
(and without CONFIG_EFI_VARS_PSTORE_DEFAULT_DISABLE set), so it seems
to be safe to do.

Ben.

--- a/drivers/firmware/efi/efi-pstore.c
+++ b/drivers/firmware/efi/efi-pstore.c
@@ -400,3 +400,4 @@ module_exit(efivars_pstore_exit);
 
 MODULE_DESCRIPTION("EFI variable backend for pstore");
 MODULE_LICENSE("GPL");
+MODULE_ALIAS("platform:efivars");
-- 
Ben Hutchings
All extremists should be taken out and shot.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 811 bytes --]

Re: efi: Auto-load the efi-pstore module

[Matt Fleming]

On Mon, 28 Sep, at 01:44:16AM, Ben Hutchings wrote:
> efi-pstore should be auto-loaded on EFI systems, same as efivars.
> 
> Signed-off-by: Ben Hutchings <ben-/+tVBieCtBitmTQ+vhA3Yw@public.gmane.org>
> ---
> This was included in Debian 8 "jessie" with no complaints so far
> (and without CONFIG_EFI_VARS_PSTORE_DEFAULT_DISABLE set), so it seems
> to be safe to do.
> 
> Ben.
> 
> --- a/drivers/firmware/efi/efi-pstore.c
> +++ b/drivers/firmware/efi/efi-pstore.c
> @@ -400,3 +400,4 @@ module_exit(efivars_pstore_exit);
>  
>  MODULE_DESCRIPTION("EFI variable backend for pstore");
>  MODULE_LICENSE("GPL");
> +MODULE_ALIAS("platform:efivars");

I'm OK with this change because it still relies on distributions to
actually enable the Kconfig option too, before it works.

Folks on Cc, please let me know if you have concerns.

-- 
Matt Fleming, Intel Open Source Technology Center