linux-kernel - Re: uninline check

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:	Mon, 16 Jul 2007 15:29:23 -0700
From:	Andrew Morton <akpm@...ux-foundation.org>
To:	Heiko Carstens <heiko.carstens@...ibm.com>
Cc:	Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
	Linus Torvalds <torvalds@...ux-foundation.org>,
	geert@...ux-m68k.org, zippel@...ux-m68k.org, jdike@...aya.com,
	alan@...rguk.ukuu.org.uk,
	Martin Schwidefsky <schwidefsky@...ibm.com>
Subject: Re: uninline check_signature()

On Tue, 17 Jul 2007 00:12:57 +0200
Heiko Carstens <heiko.carstens@...ibm.com> wrote:

> On Mon, Jul 16, 2007 at 06:02:55PM +0000, Linux Kernel Mailing List wrote:
> > Gitweb:     http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=cc2ea416b2aa04d0c34ff2281a23dae5b76b7b3b
> > Commit:     cc2ea416b2aa04d0c34ff2281a23dae5b76b7b3b
> > Parent:     4e7bd66318b3ae60fbba7d886d9a98b71ffbf74e
> > Author:     Andrew Morton <akpm@...ux-foundation.org>
> > AuthorDate: Sun Jul 15 23:41:38 2007 -0700
> > Committer:  Linus Torvalds <torvalds@...dy.linux-foundation.org>
> > CommitDate: Mon Jul 16 09:05:50 2007 -0700
> > 
> >     uninline check_signature()
> >     
> >     This is a rather bizarre thing to have inlined in io.h.  Stick it in lib/
> >     instead.
> >     
> >     While we're there, despaghetti it a bit, and fix its off-by-one behaviour when
> >     passed a zero length.
> >     
> >     Signed-off-by: Andrew Morton <akpm@...ux-foundation.org>
> >     Signed-off-by: Linus Torvalds <torvalds@...ux-foundation.org>
> > ---
> >  include/linux/io.h    |   29 ++---------------------------
> >  lib/Makefile          |    2 +-
> >  lib/check_signature.c |   26 ++++++++++++++++++++++++++
> >  3 files changed, 29 insertions(+), 28 deletions(-)
> 
> Hmm.. this breaks s390 and probably m68k and UML as well:
> 
>   CC      lib/check_signature.o
> lib/check_signature.c: In function 'check_signature':
> lib/check_signature.c:19: error: implicit declaration of function 'readb'

Yep.  They were already broken, only less so.

> Actually we tried to fix this, but it looks like there was no final
> conclusion: http://lkml.org/lkml/2007/6/28/98
> 
> Now what? Revert until things have been sorted out?

No, no revert, please.  If the architecture doesn't support readb() then we
need some reliable way of working that out within Kconfig.

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