linux-kernel - Re: [discuss] [2.6 patch] x86_64: re-add a newline to RESTORE

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: <45A4B3E3.76E4.0078.0@novell.com>
Date:	Wed, 10 Jan 2007 08:37:39 +0000
From:	"Jan Beulich" <jbeulich@...ell.com>
To:	"Andrew Morton" <akpm@...l.org>, "Andi Kleen" <ak@...e.de>
Cc:	"Steven M. Christey" <coley@...re.org>,
	"Adrian Bunk" <bunk@...sta.de>, <linux-kernel@...r.kernel.org>,
	<discuss@...-64.org>
Subject: Re: [discuss] [2.6 patch] x86_64: re-add a newline to
	RESTORE_CONTEXT

>>> Andrew Morton <akpm@...l.org> 09.01.07 23:04 >>>
>On Tue, 9 Jan 2007 12:01:21 +0100
>Andi Kleen <ak@...e.de> wrote:
>
>> On Tuesday 09 January 2007 03:55, Adrian Bunk wrote:
>> > RESTORE_CONTEXT lost a newline in 
>> > commit 658fdbef66e5e9be79b457edc2cbbb3add840aa9:
>> > http://www.mail-archive.com/kgdb-bugreport@lists.sourceforge.net/msg00559.html 
>> 
>> I don't think we should add such changes for external patchkits.
>> 
>> In general kgdb shouldn't add any patches at all. If the existing 
>> hooks are not enough they should submit their changes needed so
>> that it can just work.
>> 
>
>But the patch is a bugfix.  Without it, you cannot do
>
>	RESTORE_CONTEXT	\
>	.globl ...	\

Their use was broken in the first place - they shouldn't have made
assumptions about the contents of the macro, by writing this like

	RESTORE_CONTEXT "\n\t"	\
	".globl ..."	\

if they really need to make use of the macro. This is similar to
requirements of other (assembly) macros that normally also
don't have a line terminator and hence require the users to
add appropriate line termination after the macro name (and
eventual arguments).

I would even go as far as asking for removing the \n\t on SAVE_CONTEXT
and the left \t on RESTORE_CONTEXT.

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