lkml.org 
[lkml]   [2008]   [May]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [bisected] Re: [PATCH 05 of 12] xen: add p2m mfn_list_list
On Wed, May 28, 2008 at 03:02:14PM +0100, Jeremy Fitzhardinge wrote:
>
> The use of __section(.data.page_aligned) (or worse
> __attribute__((section(".data.page_aligned"))) is fairly verbose and
> brittle. I've got a (totally untested) proposed patch below, to
> introduce __page_aligned_data|bss which sets the section and the
> alignment. This will work, but it requires that all page-aligned
> variables also have an alignment associated with them, so that mis-sized
> ones don't push the others around.
>
> There aren't very many users of .data|bss.page_aligned, so it should be
> easy enough to fix them all up.
>
> A link-time warning would be good too, of course.

I cooked up this:
diff --git a/arch/x86/kernel/vmlinux_32.lds.S b/arch/x86/kernel/vmlinux_32.lds.S
index ce5ed08..963b2ae 100644
--- a/arch/x86/kernel/vmlinux_32.lds.S
+++ b/arch/x86/kernel/vmlinux_32.lds.S
@@ -40,6 +40,7 @@ SECTIONS
.text : AT(ADDR(.text) - LOAD_OFFSET) {
. = ALIGN(PAGE_SIZE); /* not really needed, already page aligned */
*(.text.page_aligned)
+ end_text_page_aligned = .;
TEXT_TEXT
SCHED_TEXT
LOCK_TEXT
@@ -49,6 +50,9 @@ SECTIONS
_etext = .; /* End of text section */
} :text = 0x9090

+ ASSERT((end_text_page_aligned == ALIGN((end_text_page_aligned), PAGE_SIZE)),
+ "Text in .text.page_aligned are not modulo PAGE_SIZE")
+
. = ALIGN(16); /* Exception table */
__ex_table : AT(ADDR(__ex_table) - LOAD_OFFSET) {
__start___ex_table = .;
@@ -89,6 +93,8 @@ SECTIONS
*(.data.page_aligned)
*(.data.idt)
}
+ ASSERT((. == ALIGN(PAGE_SIZE)),
+ "Data in .data.page_aligned are not modulo PAGE_SIZE")

. = ALIGN(32);
.data.cacheline_aligned : AT(ADDR(.data.cacheline_aligned) - LOAD_OFFSET) {

But we should try to do it so all archs can benefit.
And it failed in the second ASSERT - I dunno why.

Soccer duties - so I have to run.

Sam


\
 
 \ /
  Last update: 2008-05-28 16:13    [W:0.114 / U:1.840 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site