linux-kernel - Re: [linux-dvb] DVB-S2 / Multiproto and future modulation support

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: <c8b4dbe10809051354r2747b42atf15b3b6f9346987c@mail.gmail.com>
Date:	Fri, 5 Sep 2008 21:54:51 +0100
From:	"Aidan Thornton" <makosoft@...glemail.com>
To:	"Greg KH" <greg@...ah.com>
Cc:	"Markus Rechberger" <mrechberger@...il.com>,
	"Kernel Mailing List" <linux-kernel@...r.kernel.org>,
	linux-dvb@...uxtv.org, mrechberger@...dtek.com
Subject: Re: [linux-dvb] DVB-S2 / Multiproto and future modulation support

On Sun, Aug 31, 2008 at 5:21 AM, Greg KH <greg@...ah.com> wrote:
> On Sun, Aug 31, 2008 at 05:57:46AM +0200, Markus Rechberger wrote:
>> Let's put another thing in here: Greg Kroah Hartman Linux Guy reverted
>
> If you're going to spell my full last name out, please get it right, you
> forgot a '-' :)
>
>> my patch in favour of supporting the binary Firmware upload tool of
>> Dell (I fully support Dell here too) although claiming to be
>> opensource but still running after someone (please comment this one -
>> it confused me at 'your' position). It was just like ok let's revert
>> it but not asking why?!
>
> What patch specifically are you referring to here?
>
> And what does this have to do with v4l and DVB issues?
>
> thanks,
>
> greg k-h

Hi,

Markus submitted a patch to the firmware loader code that fixed a
sysfs filename collision by appending a suffix to the sysfs filename
it used. This bug broke the use of the firmware loader from i2c device
drivers (specifically, the drivers for the xc3028 TV tuner chip) with
certain (not particularly unusual) kernel configurations - IIRC, it
affected kernels with I2C compiled as a module and a particular value
of some option related to sysfs depreciated support. The patch was
reverted by you because it broke binary-only firmware upload tools for
Dell hardware, screwing over normal desktop users in the process.

See, for example, http://lkml.org/lkml/2008/4/26/319 - this is fairly
typical. IIRC, the only drivers for the xc3028 that aren't affected
are Markus' recent ones, since they compile the firmware into the
driver (ugh). This may have been fixed since, but I'm not sure.

(Incidentally, looking at the conversation, I believe your remark that
"the i2c devices can fix things by changing their module names so this
collision doesn't happen :)" may be inaccurate. The firmware loader
copies the name it uses from the device passed to it, so I'm not sure
how much can be done, short of hacking around the issue by creating a
fake device to pass to the firmware loader or making potentially
compatibility-breaking changes to either the i2c core or the firmware
loader. Of course, I haven't looked at the issue that closely, so I
may be wrong.)

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