* [PATCH] asm-generic/vmlinux.lds.h: Fix up RW_DATA_SECTION definition.
@ 2009-06-23 16:04 Paul Mundt
2009-06-23 21:10 ` Sam Ravnborg
0 siblings, 1 reply; 2+ messages in thread
From: Paul Mundt @ 2009-06-23 16:04 UTC (permalink / raw)
To: Sam Ravnborg; +Cc: linux-kernel
RW_DATA_SECTION is defined to take 4 different alignment parameters,
while NOSAVE_DATA currently uses a fixed PAGE_SIZE alignment as noted
in the comments.
There are presently no in-tree users of this at present, and I just
stumbled across this while implementing the simplified script on a new
architecture port, which subsequently resulted in a syntax error.
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
---
include/asm-generic/vmlinux.lds.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index 92b73b6..f92e730 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -704,7 +704,7 @@
* matches the requirment of PAGE_ALIGNED_DATA.
*
* use 0 as page_align if page_aligned data is not used */
-#define RW_DATA_SECTION(cacheline, nosave, pagealigned, inittask) \
+#define RW_DATA_SECTION(cacheline, pagealigned, inittask) \
. = ALIGN(PAGE_SIZE); \
.data : AT(ADDR(.data) - LOAD_OFFSET) { \
INIT_TASK(inittask) \
@@ -712,7 +712,7 @@
READ_MOSTLY_DATA(cacheline) \
DATA_DATA \
CONSTRUCTORS \
- NOSAVE_DATA(nosave) \
+ NOSAVE_DATA \
PAGE_ALIGNED_DATA(pagealigned) \
}
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] asm-generic/vmlinux.lds.h: Fix up RW_DATA_SECTION definition.
2009-06-23 16:04 [PATCH] asm-generic/vmlinux.lds.h: Fix up RW_DATA_SECTION definition Paul Mundt
@ 2009-06-23 21:10 ` Sam Ravnborg
0 siblings, 0 replies; 2+ messages in thread
From: Sam Ravnborg @ 2009-06-23 21:10 UTC (permalink / raw)
To: Paul Mundt, linux-kernel
On Wed, Jun 24, 2009 at 01:04:36AM +0900, Paul Mundt wrote:
> RW_DATA_SECTION is defined to take 4 different alignment parameters,
> while NOSAVE_DATA currently uses a fixed PAGE_SIZE alignment as noted
> in the comments.
>
> There are presently no in-tree users of this at present, and I just
> stumbled across this while implementing the simplified script on a new
> architecture port, which subsequently resulted in a syntax error.
>
> Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Thanks - applied.
And I'm glad this is used.
Sam
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2009-06-23 21:08 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2009-06-23 16:04 [PATCH] asm-generic/vmlinux.lds.h: Fix up RW_DATA_SECTION definition Paul Mundt
2009-06-23 21:10 ` Sam Ravnborg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).