linux-kernel - Re: [v4l-dvb-maintainer] [2.6 patch] remove DVB_AV7110

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] [day] [month] [year] [list]

Message-ID: <20061201123437.GA11084@stusta.de>
Date:	Fri, 1 Dec 2006 13:34:38 +0100
From:	Adrian Bunk <bunk@...sta.de>
To:	Oliver Endriss <o.endriss@....de>
Cc:	v4l-dvb-maintainer@...uxtv.org, Trent Piepho <xyzzy@...akeasy.org>,
	linux-kernel@...r.kernel.org
Subject: Re: [v4l-dvb-maintainer] [2.6 patch] remove DVB_AV7110_FIRMWARE

On Thu, Nov 30, 2006 at 01:58:32AM +0100, Oliver Endriss wrote:
> Adrian Bunk wrote:
> > On Tue, Nov 28, 2006 at 08:45:56PM -0800, Trent Piepho wrote:
> > > On Wed, 29 Nov 2006, Adrian Bunk wrote:
> > > > On Tue, Nov 28, 2006 at 01:06:02PM -0800, Trent Piepho wrote:
> > > > > On Sun, 26 Nov 2006, Adrian Bunk wrote:
> > > > > > DVB_AV7110_FIRMWARE was (except for some OSS drivers) the only option
> > > > > > that was still compiling a binary-only user-supplied firmware file at
> > > > > > build-time into the kernel.
> > > > > >
> > > > > > This patch changes the driver to always use the standard
> > > > > > request_firmware() way for firmware by removing DVB_AV7110_FIRMWARE.
> > > > >
> > > > > Doesn't this also prevent the AV7110 module from getting compiled
> > > > > into the kernel?  Shouldn't the Kconfig file be adjusted so
> > > > > that 'y' can't be selected anymore and it depends on MODULES?
> > > >
> > > > No.
> > > > No.
> > > >
> > > > request_firmware() works fine for built-in drivers.
> > > 
> > > Wouldn't that require loading the firmware file before the filesystems are
> > > mounted?
> > 
> > Sure.
> 
> And you have to create an initrd for the firmware!
> 
> As I wrote before: 
> I NAK any attempt to remove this option.
> 
> The option _is_ useful because it allows a user to build an av7110 driver
> without hotplug, initrd etc.
> 
> Nobody has to use this option, but it should be possible to do so.

Sorry, but the direction of kernel development is to replace statically 
into the kernel compiled firmware (that also brings some non-trivial 
legal questions - is distributing such a kernel a violation of the GPL?)
with request_firmware().

> CU
> Oliver

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
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/