mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] Update the alignment for 4K stacks.
@ 2007-06-24 19:21 Robert P. J. Day
  2007-06-24 19:39 ` Andrew Morton
  0 siblings, 1 reply; 3+ messages in thread
From: Robert P. J. Day @ 2007-06-24 19:21 UTC (permalink / raw)
  To: Linux Kernel Mailing List; +Cc: Andrew Morton, Paul Mundt


Just following the advice in the comments, update the alignment when
4K stacks are selected, since gcc 3.0 shouldn't be an issue anymore.

Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>

---

 arch/i386/kernel/irq.c |    8 ++------
 arch/sh/kernel/irq.c   |    8 ++------
 2 files changed, 4 insertions(+), 12 deletions(-)

diff --git a/arch/i386/kernel/irq.c b/arch/i386/kernel/irq.c
index d2daf67..504f134 100644
--- a/arch/i386/kernel/irq.c
+++ b/arch/i386/kernel/irq.c
@@ -149,15 +149,11 @@ fastcall unsigned int do_IRQ(struct pt_regs *regs)

 #ifdef CONFIG_4KSTACKS

-/*
- * These should really be __section__(".bss.page_aligned") as well, but
- * gcc's 3.0 and earlier don't handle that correctly.
- */
 static char softirq_stack[NR_CPUS * THREAD_SIZE]
-		__attribute__((__aligned__(THREAD_SIZE)));
+		__attribute__((__section__(".bss.page_aligned")));

 static char hardirq_stack[NR_CPUS * THREAD_SIZE]
-		__attribute__((__aligned__(THREAD_SIZE)));
+		__attribute__((__section__(".bss.page_aligned")));

 /*
  * allocate per-cpu stacks for hardirq and for softirq processing
diff --git a/arch/sh/kernel/irq.c b/arch/sh/kernel/irq.c
index 27b923c..2789779 100644
--- a/arch/sh/kernel/irq.c
+++ b/arch/sh/kernel/irq.c
@@ -158,15 +158,11 @@ asmlinkage int do_IRQ(unsigned int irq, struct pt_regs *regs)
 }

 #ifdef CONFIG_4KSTACKS
-/*
- * These should really be __section__(".bss.page_aligned") as well, but
- * gcc's 3.0 and earlier don't handle that correctly.
- */
 static char softirq_stack[NR_CPUS * THREAD_SIZE]
-		__attribute__((__aligned__(THREAD_SIZE)));
+		__attribute__((__section__(".bss.page_aligned")));

 static char hardirq_stack[NR_CPUS * THREAD_SIZE]
-		__attribute__((__aligned__(THREAD_SIZE)));
+		__attribute__((__section__(".bss.page_aligned")));

 /*
  * allocate per-cpu stacks for hardirq and for softirq processing
diff --git a/include/asm-blackfin/macros.h b/include/asm-blackfin/macros.h
deleted file mode 100644
index e69de29..0000000
-- 
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Update the alignment for 4K stacks.
  2007-06-24 19:21 [PATCH] Update the alignment for 4K stacks Robert P. J. Day
@ 2007-06-24 19:39 ` Andrew Morton
  2007-06-24 22:12   ` Robert P. J. Day
  0 siblings, 1 reply; 3+ messages in thread
From: Andrew Morton @ 2007-06-24 19:39 UTC (permalink / raw)
  To: Robert P. J. Day; +Cc: Linux Kernel Mailing List, Paul Mundt

On Sun, 24 Jun 2007 15:21:37 -0400 (EDT) "Robert P. J. Day" <rpjday@mindspring.com> wrote:

>  arch/i386/kernel/irq.c |    8 ++------
>  arch/sh/kernel/irq.c   |    8 ++------

These two files are maintained by different developers who run
separate development trees.  Please do not try to alter them
both in the same patch.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] Update the alignment for 4K stacks.
  2007-06-24 19:39 ` Andrew Morton
@ 2007-06-24 22:12   ` Robert P. J. Day
  0 siblings, 0 replies; 3+ messages in thread
From: Robert P. J. Day @ 2007-06-24 22:12 UTC (permalink / raw)
  To: Andrew Morton; +Cc: Linux Kernel Mailing List, Paul Mundt

On Sun, 24 Jun 2007, Andrew Morton wrote:

> On Sun, 24 Jun 2007 15:21:37 -0400 (EDT) "Robert P. J. Day" <rpjday@mindspring.com> wrote:
>
> >  arch/i386/kernel/irq.c |    8 ++------
> >  arch/sh/kernel/irq.c   |    8 ++------
>
> These two files are maintained by different developers who run
> separate development trees.  Please do not try to alter them
> both in the same patch.

ok, i'll split them.

rday
-- 
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://fsdev.net/wiki/index.php?title=Main_Page
========================================================================

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2007-06-24 22:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-06-24 19:21 [PATCH] Update the alignment for 4K stacks Robert P. J. Day
2007-06-24 19:39 ` Andrew Morton
2007-06-24 22:12   ` Robert P. J. Day

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®