linux-kernel - Re: [PATCH] kaweth: use request

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]

Message-Id: <7D3DC891-CC9A-4CBD-8DCF-E684A2FB36B2@holtmann.org>
Date:	Sat, 31 May 2008 08:51:38 +0200
From:	Marcel Holtmann <marcel@...tmann.org>
To:	Alan Cox <alan@...rguk.ukuu.org.uk>
Cc:	David Woodhouse <dwmw2@...radead.org>,
	Oliver Neukum <oliver@...kum.org>,
	linux-kernel@...r.kernel.org, akpm@...ux-foundation.org,
	Michael Zappe <zapman@...erlan.net>,
	Stephane Alnet <stephane@...icardie.fr>,
	Brad Hards <bhards@...pond.net.au>
Subject: Re: [PATCH] kaweth: use request_firmware()

Hi Alan,

>> we tried to not put that policy into the drivers and I was looking
>> over this, but in the meantime driver slipped past me and made it in.
>> We should not do this at all and if people wanna have this, then they
>> should solve this within udev.
>
> Then you arrived at the party a year too late. The format name/ 
> firmware
> is commonly used all over the kernel. How that maps to a file or
> directory is indeed a matter for udev, but there is already a
> heirarchical namespace and pretending otherwise won't help.

I was not late to the party. I was the first one using  
request_firmware() and always keeping an eye on drivers using it and  
making sure using it correctly. However drivers sneaked in without  
that I realized it. Mainly because I don't have that hardware and  
reviewing every single driver obviously never happened.

I repeatedly said that no hierarchical namespace should be used (and  
this was a couple of years ago) and now since some developers misuse  
it, it makes it right. Don't think so.

Regards

Marcel

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