linux-kernel - Re: [PATCH] i386 kernel instant reboot with older binutils fix

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:	Wed, 3 Jan 2007 12:25:38 +0530
From:	Vivek Goyal <vgoyal@...ibm.com>
To:	"Eric W. Biederman" <ebiederm@...ssion.com>
Cc:	linux kernel mailing list <linux-kernel@...r.kernel.org>,
	Linus Torvalds <torvalds@...l.org>, Andi Kleen <ak@....de>,
	Morton Andrew Morton <akpm@...l.org>,
	Fastboot mailing list <fastboot@...ts.osdl.org>,
	Adrian Bunk <bunk@...sta.de>,
	Jean Delvare <khali@...ux-fr.org>,
	Segher Boessenkool <segher@...nel.crashing.org>
Subject: Re: [PATCH] i386 kernel instant reboot with older binutils fix

On Tue, Jan 02, 2007 at 11:44:34PM -0700, Eric W. Biederman wrote:
> Vivek Goyal <vgoyal@...ibm.com> writes:
> 
> > o i386 kernel reboots instantly if compiled with binutils older than
> >   2.6.15.
> >
> > o Older binutils required explicit flags to mark a section allocatable
> >   and executable(AX). Newer binutils automatically mark a section AX if
> >   the name starts with .text.
> >
> > o While defining a new section using assembler "section" directive,
> >   explicitly mention section flags.
> 
> As such this patch looks fine, and is certainly harmless.  But don't we
> also need to address the issue that .text.head is not listed in the
> linker script?
> 
> i.e.  Don't we also need?
> 
>   .text : AT(ADDR(.text) - LOAD_OFFSET) {
>   	_text = .;			/* Text and read-only data */
> +	*(.text.head)
> 	*(.text)
> 	SCHED_TEXT
> 	LOCK_TEXT
> 	KPROBES_TEXT
> 	*(.fixup)
> 	*(.gnu.warning)
>   	_etext = .;			/* End of text section */
>   } :text = 0x9090
> 
> 
> I'm not even certain how the i386 kernel links properly without the above.

Hi Eric,

This .text.head section is not part of vmlinux. This is part of uncompressed
portion in bzImage. arch/i386/boot/compressed/head.S.

Hence, arch/i386/boot/compressed/vmlinux.lds should take care of it which
already has entry for linking .text.head section.

        . =  0  ;
        .text.head : {
                _head = . ;
                *(.text.head)
                _ehead = . ;
        }

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