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: <472051E8.2070005@garzik.org>
Date:	Thu, 25 Oct 2007 04:20:56 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	Andrew Morton <akpm@...ux-foundation.org>
CC:	LKML <linux-kernel@...r.kernel.org>, torvalds@...ux-foundation.org
Subject: Re: [PATCH] Permit silencing of __deprecated warnings.

Andrew Morton wrote:
> On Thu, 25 Oct 2007 04:06:13 -0400 (EDT) Jeff Garzik <jeff@...zik.org> wrote:
> 
>> The __deprecated marker is quite useful in highlighting the remnants of
>> old APIs that want removing.
>>
>> However, it is quite normal for one or more years to pass, before the
>> (usually ancient, bitrotten) code in question is either updated or
>> deleted.
>>
>> Thus, like __must_check, add a Kconfig option that permits the silencing
>> of this compiler warning.
>>
>> This change mimics the ifdef-ery and Kconfig defaults of MUST_CHECK as
>> closely as possible.
> 
> Sigh.  Can't we just fix the dud code?  Or mark it BROKEN and see what
> happens?

__deprecated has spread to just about every API that people don't 
consider fresh and up-to-date.

Like I noted in the patch description, rewriting grotty ISA/MCA/etc. 
probe code is a thankless, boring task that few are crazy enough to 
attempt :)

As you can see from the patch flood recently I /have/ been working 
through the dud code, but it will still take years.  The changes 
required for each are on average ~200 LOC changed, if not more.

But regardless...  I don't see any reason to force every kernel build to 
remind us of grotty drivers.  Where's the benefit?  Everybody knows they 
are grotty.

Like __must_check this option defaults to the current state of things -- 
warnings -- so you have to take an extra step to turn them off.

	Jeff



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

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ