linux-kernel - Re: [GIT PULL] headers

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 Jun 2009 14:41:19 +0100
From:	Russell King <rmk+lkml@....linux.org.uk>
To:	Jaswinder Singh Rajput <jaswinder@...nel.org>
Cc:	Linus Torvalds <torvalds@...ux-foundation.org>,
	Sam Ravnborg <sam@...nborg.org>, Ingo Molnar <mingo@...e.hu>,
	Catalin Marinas <catalin.marinas@....com>,
	Tony Luck <tony.luck@...el.com>,
	Geert Uytterhoeven <geert@...ux-m68k.org>,
	Greg Ungerer <gerg@...pgear.com>,
	Ralf Baechle <ralf@...ux-mips.org>,
	Michael Abbott <michael@...neidae.co.uk>,
	Andrew Morton <akpm@...ux-foundation.org>,
	LKML <linux-kernel@...r.kernel.org>,
	David Howells <dhowells@...hat.com>
Subject: Re: [GIT PULL] headers_check fixes

On Sat, Jun 06, 2009 at 07:04:37PM +0530, Jaswinder Singh Rajput wrote:
> On Sat, 2009-06-06 at 14:02 +0100, Russell King wrote:
> > On Sat, Jun 06, 2009 at 06:24:17PM +0530, Jaswinder Singh Rajput wrote:
> > > Linus,
> > > 
> > > Please pull headers_check fixes:
> > > 
> > > The following changes since commit 064e38aaded5269e573ac1c765284fd65c8ebd13:
> > >   Linus Torvalds (1):
> > >         Merge git://git.kernel.org/.../mason/btrfs-unstable
> > > 
> > > are available in the git repository at:
> > > 
> > >   git://git.kernel.org/pub/scm/linux/kernel/git/jaswinder/headers-check-2.6.git master
> > > 
> > > Jaswinder Singh Rajput (6):
> > >       headers_check fix: arm, hwcap.h
> > 
> > So inspite of NAKing this patch you send it anyway.
> > 
> 
> Earlier it was already like this, because of Catalin Marinas
> fault (commit f884b1cf578e0) and your ignorance you converted :
> 
>  #ifdef __KERNEL__
>  #ifndef __ASSEMBLY__
> 
> to :
> 
>  #if defined(__KERNEL__) && !defined(__ASSEMBLY__)
> 
> I am just trying to rectify your mistake that __KERNEL__ should not be
> used with other defines to easily export headers to userspace. Whole
> kernel header files are following this protocol. Sam, Arnd and me are
> trying to convince you. Why you need an exception ?

You are not listening.

I believe that there is a much better way to fix this, and that is to
fix the tools.  Having looked at unifdef, I believe that the fix is
almost trivial in nature.

With fixed tools, we don't have to fuck around writing a special dialect
of C to work around their short comings - a dialect which isn't obvious.

What we actually have here is a small number of people who've suddenly
decided that a perfectly good bit of C code is no longer good enough for
their purposes, and are now requiring things to be written to match their
own rules.

Let's fix the tools and then we don't have to ever worry about this
problem again, or require people to write their C code in special ways.

-- 
Russell King
 Linux kernel    2.6 ARM Linux   - http://www.arm.linux.org.uk/
 maintainer of:
--
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/