All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] x86 new setup: use appropriate sections for code and data
@ 2007-05-17 11:25 Alexander van Heukelum
  2007-05-17 17:05 ` H. Peter Anvin
  0 siblings, 1 reply; 3+ messages in thread
From: Alexander van Heukelum @ 2007-05-17 11:25 UTC (permalink / raw)
  To: H. Peter Anvin; +Cc: linux-kernel

An intermediate elf file is generated for the 16-bit setup code.
The generated code can be viewed using objdump -m i8086 -d. As it
stands, it also tries to disassemble the bugger_off_msg, which
results in garbage. This introduces two new sections to separate
the code and the data part of the bootsector stub. It also moves
some code from the .header section (a data section) to .inittext.

Signed-off-by: Alexander van Heukelum <heukelum@mailshack.com>
---

Hi,

I hope the new section names are ok? Alternatives would be
.bstext/.bsdata or .floppytext/.floppydata.

Greetings,
	Alexander


 arch/i386/boot/header.S |   10 +++++-----
 arch/i386/boot/setup.ld |    4 ++++
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/arch/i386/boot/header.S b/arch/i386/boot/header.S
index 02d0f7d..70b39cb 100644
--- a/arch/i386/boot/header.S
+++ b/arch/i386/boot/header.S
@@ -44,7 +44,7 @@ SWAP_DEV	= 0			/* SWAP_DEV is now written by "build" */
 #endif
 
 	.code16
-	.section ".header", "ax"
+	.section ".stubtext", "ax"
 
 	.global bootsect_start
 bootsect_start:
@@ -82,7 +82,7 @@ bs_die:
 	# invoke the BIOS reset code...
 	ljmp	$0xf000,$0xfff0
 
-
+	.section ".stubdata", "a"
 bugger_off_msg:
 	.ascii	"Direct booting from floppy is no longer supported.\r\n"
 	.ascii	"Please use a boot loader program instead.\r\n"
@@ -94,7 +94,7 @@ bugger_off_msg:
 	# Kernel attributes; used by setup.  This is part 1 of the
 	# header, from the old boot sector.
 
-	.org 497
+	.section ".header", "a"
 	.globl	hdr
 hdr:
 setup_sects:	.byte SETUPSECTS
@@ -208,10 +208,11 @@ pad3:			.word 0
 cmdline_size:   .long   COMMAND_LINE_SIZE-1     #length of the command line,
                                                 #added with boot protocol
                                                 #version 2.06
+start_of_setup:
 
 # End of setup header #####################################################
 
-start_of_setup:
+	.section ".inittext", "ax"
 #ifdef SAFE_RESET_DISK_CONTROLLER
 # Reset the disk controller.
 	movw	$0x0000, %ax		# Reset disk controller
@@ -225,7 +226,6 @@ start_of_setup:
 	pushw	$setup2
 	lretw
 
-	.section ".inittext", "ax"
 setup2:
 # Force %es = %ds
 	movw	%ds, %ax
diff --git a/arch/i386/boot/setup.ld b/arch/i386/boot/setup.ld
index a111572..8defc4f 100644
--- a/arch/i386/boot/setup.ld
+++ b/arch/i386/boot/setup.ld
@@ -10,6 +10,10 @@ ENTRY(_start)
 SECTIONS
 {
 	. = 0;
+	.stubtext	: { *(.stubtext) }
+	.stubdata	: { *(.stubdata) }
+
+	. = 497;
 	.header		: { *(.header) }
 	.inittext	: { *(.inittext) }
 	.initdata	: { *(.initdata) }
-- 
1.4.4.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread
* Re: [PATCH] x86 new setup: use appropriate sections for code and data
  2007-05-17 11:25 [PATCH] x86 new setup: use appropriate sections for code and data Alexander van Heukelum
@ 2007-05-17 17:05 ` H. Peter Anvin
  2007-05-17 18:54   ` Alexander van Heukelum
  0 siblings, 1 reply; 3+ messages in thread
From: H. Peter Anvin @ 2007-05-17 17:05 UTC (permalink / raw)
  To: Alexander van Heukelum; +Cc: linux-kernel

Alexander van Heukelum wrote:
> An intermediate elf file is generated for the 16-bit setup code.
> The generated code can be viewed using objdump -m i8086 -d. As it
> stands, it also tries to disassemble the bugger_off_msg, which
> results in garbage. This introduces two new sections to separate
> the code and the data part of the bootsector stub. It also moves
> some code from the .header section (a data section) to .inittext.
> 
> Signed-off-by: Alexander van Heukelum <heukelum@mailshack.com>
> ---
> 
> Hi,
> 
> I hope the new section names are ok? Alternatives would be
> .bstext/.bsdata or .floppytext/.floppydata.

I'd prefer .bstext or .bs_text and .bsdata/.bs_data; possibly .bsecttext
and .bsectdata.  The term "boot sector" for that particular piece of
code is firmly established.

	-hpa



^ permalink raw reply	[flat|nested] 3+ messages in thread
* Re: [PATCH] x86 new setup: use appropriate sections for code and data
  2007-05-17 17:05 ` H. Peter Anvin
@ 2007-05-17 18:54   ` Alexander van Heukelum
  0 siblings, 0 replies; 3+ messages in thread
From: Alexander van Heukelum @ 2007-05-17 18:54 UTC (permalink / raw)
  To: H. Peter Anvin; +Cc: linux-kernel

An intermediate elf file is generated for the 16-bit setup code.
The generated code can be viewed using objdump -m i8086 -d. As it
stands, it also tries to disassemble the bugger_off_msg, which
results in garbage. This introduces two new sections to separate
the code and the data part of the bootsector stub. It also moves
some code from the .header section (a data section) to .inittext.

Signed-off-by: Alexander van Heukelum <heukelum@mailshack.com>
---
> I'd prefer .bstext or .bs_text and .bsdata/.bs_data; possibly .bsecttext
> and .bsectdata.  The term "boot sector" for that particular piece of
> code is firmly established.
> 
> 	-hpa

Hi,

This version uses the section names .bstext/.bsdata.

Greetings,
	Alexander


 arch/i386/boot/header.S |   10 +++++-----
 arch/i386/boot/setup.ld |    4 ++++
 2 files changed, 9 insertions(+), 5 deletions(-)

diff --git a/arch/i386/boot/header.S b/arch/i386/boot/header.S
index 02d0f7d..c1c89e5 100644
--- a/arch/i386/boot/header.S
+++ b/arch/i386/boot/header.S
@@ -44,7 +44,7 @@ SWAP_DEV	= 0			/* SWAP_DEV is now written by "build" */
 #endif
 
 	.code16
-	.section ".header", "ax"
+	.section ".bstext", "ax"
 
 	.global bootsect_start
 bootsect_start:
@@ -82,7 +82,7 @@ bs_die:
 	# invoke the BIOS reset code...
 	ljmp	$0xf000,$0xfff0
 
-
+	.section ".bsdata", "a"
 bugger_off_msg:
 	.ascii	"Direct booting from floppy is no longer supported.\r\n"
 	.ascii	"Please use a boot loader program instead.\r\n"
@@ -94,7 +94,7 @@ bugger_off_msg:
 	# Kernel attributes; used by setup.  This is part 1 of the
 	# header, from the old boot sector.
 
-	.org 497
+	.section ".header", "a"
 	.globl	hdr
 hdr:
 setup_sects:	.byte SETUPSECTS
@@ -208,10 +208,11 @@ pad3:			.word 0
 cmdline_size:   .long   COMMAND_LINE_SIZE-1     #length of the command line,
                                                 #added with boot protocol
                                                 #version 2.06
+start_of_setup:
 
 # End of setup header #####################################################
 
-start_of_setup:
+	.section ".inittext", "ax"
 #ifdef SAFE_RESET_DISK_CONTROLLER
 # Reset the disk controller.
 	movw	$0x0000, %ax		# Reset disk controller
@@ -225,7 +226,6 @@ start_of_setup:
 	pushw	$setup2
 	lretw
 
-	.section ".inittext", "ax"
 setup2:
 # Force %es = %ds
 	movw	%ds, %ax
diff --git a/arch/i386/boot/setup.ld b/arch/i386/boot/setup.ld
index a111572..01816bb 100644
--- a/arch/i386/boot/setup.ld
+++ b/arch/i386/boot/setup.ld
@@ -10,6 +10,10 @@ ENTRY(_start)
 SECTIONS
 {
 	. = 0;
+	.bstext		: { *(.bstext) }
+	.bsdata		: { *(.bsdata) }
+
+	. = 497;
 	.header		: { *(.header) }
 	.inittext	: { *(.inittext) }
 	.initdata	: { *(.initdata) }
-- 
1.4.4.2


^ permalink raw reply related	[flat|nested] 3+ messages in thread
end of thread, other threads:[~2007-05-17 18:56 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-05-17 11:25 [PATCH] x86 new setup: use appropriate sections for code and data Alexander van Heukelum
2007-05-17 17:05 ` H. Peter Anvin
2007-05-17 18:54   ` Alexander van Heukelum

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.