linux-kernel - Re: dell_rbu: Select CONFIG_FW_LOADER_USER

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sat, 6 Jul 2013 15:30:03 -0700
From:	Greg KH <gregkh@...uxfoundation.org>
To:	Dave Jones <davej@...hat.com>,
	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	tiwai@...e.de
Subject: Re: dell_rbu: Select CONFIG_FW_LOADER_USER_HELPER explicitly

On Sat, Jul 06, 2013 at 06:14:01PM -0400, Dave Jones wrote:
> On Tue, Jul 02, 2013 at 07:27:49PM +0000, Linux Kernel wrote:
>  > Gitweb:     http://git.kernel.org/linus/;a=commit;h=d05c39ea67f5786a549ac9d672d2951992b658c6
>  > Commit:     d05c39ea67f5786a549ac9d672d2951992b658c6
>  > Parent:     a3b2c8c7aa1ca860edcf0b0afa371d9eb2269c3c
>  > Author:     Takashi Iwai <tiwai@...e.de>
>  > AuthorDate: Wed May 22 18:28:37 2013 +0200
>  > Committer:  Greg Kroah-Hartman <gregkh@...uxfoundation.org>
>  > CommitDate: Mon Jun 3 13:57:29 2013 -0700
>  > 
>  >     dell_rbu: Select CONFIG_FW_LOADER_USER_HELPER explicitly
>  >     
>  >     The usermode helper is mandatory for this driver.
>  >     
>  >     Signed-off-by: Takashi Iwai <tiwai@...e.de>
>  >     Signed-off-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
>  > ---
>  >  drivers/firmware/Kconfig |    1 +
>  >  1 files changed, 1 insertions(+), 0 deletions(-)
>  > 
>  > diff --git a/drivers/firmware/Kconfig b/drivers/firmware/Kconfig
>  > index 9387630..0747872 100644
>  > --- a/drivers/firmware/Kconfig
>  > +++ b/drivers/firmware/Kconfig
>  > @@ -64,6 +64,7 @@ config DELL_RBU
>  >  	tristate "BIOS update support for DELL systems via sysfs"
>  >  	depends on X86
>  >  	select FW_LOADER
>  > +	select FW_LOADER_USER_HELPER
>  >  	help
>  >  	 Say m if you want to have the option of updating the BIOS for your
>  >  	 DELL system. Note you need a Dell OpenManage or Dell Update package (DUP)
> 
> 
> This is pretty crappy. Now every distro kernel has to have that enabled,
> which screws up for eg, the x86 microcode driver. (It takes 1 minute per cpu
> when this is enabled).

I'll let you and Takashi battle this one out, for some reason I thought
he added it _because_ a distro kernel needed it...

Takashi?

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/