All of lore.kernel.org
 help / color / mirror / Atom feed
From: tim.bird@am.sony.com (Tim Bird)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 3/3] ARM 4Kstacks: Decrease poll and select stack usage, when using 4K stacks
Date: Tue, 18 Oct 2011 16:34:02 -0700	[thread overview]
Message-ID: <4E9E0CEA.6030102@am.sony.com> (raw)
In-Reply-To: <4E9E0B71.9020708@am.sony.com>

Reduce the max allowed size for struct poll_wqueue to avoid
using too much stack space in do_select() and do_sys_poll(),
when the kernel is using 4K stacks.

Signed-off-by: Tim Bird <tim.bird@am.sony.com>
---
 include/linux/poll.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/include/linux/poll.h b/include/linux/poll.h
index cf40010..b7153a3 100644
--- a/include/linux/poll.h
+++ b/include/linux/poll.h
@@ -16,7 +16,11 @@
 extern struct ctl_table epoll_table[]; /* for sysctl */
 /* ~832 bytes of stack space used max in sys_select/sys_poll before allocating
    additional memory. */
+#ifndef CONFIG_4KSTACKS
 #define MAX_STACK_ALLOC 832
+#else
+#define MAX_STACK_ALLOC 448
+#endif
 #define FRONTEND_STACK_ALLOC	256
 #define SELECT_STACK_ALLOC	FRONTEND_STACK_ALLOC
 #define POLL_STACK_ALLOC	FRONTEND_STACK_ALLOC
-- 
1.6.6

WARNING: multiple messages have this Message-ID (diff)
From: Tim Bird <tim.bird@am.sony.com>
To: Russell King <rmk@arm.linux.org.uk>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Arnd Bergmann <arnd@arndb.de>, Andi Kleen <andi@firstfloor.org>,
	Thomas Gleixner <tglx@linutronix.de>,
	linux kernel <linux-kernel@vger.kernel.org>
Subject: [PATCH 3/3] ARM 4Kstacks: Decrease poll and select stack usage, when using 4K stacks
Date: Tue, 18 Oct 2011 16:34:02 -0700	[thread overview]
Message-ID: <4E9E0CEA.6030102@am.sony.com> (raw)
In-Reply-To: <4E9E0B71.9020708@am.sony.com>

Reduce the max allowed size for struct poll_wqueue to avoid
using too much stack space in do_select() and do_sys_poll(),
when the kernel is using 4K stacks.

Signed-off-by: Tim Bird <tim.bird@am.sony.com>
---
 include/linux/poll.h |    4 ++++
 1 files changed, 4 insertions(+), 0 deletions(-)

diff --git a/include/linux/poll.h b/include/linux/poll.h
index cf40010..b7153a3 100644
--- a/include/linux/poll.h
+++ b/include/linux/poll.h
@@ -16,7 +16,11 @@
 extern struct ctl_table epoll_table[]; /* for sysctl */
 /* ~832 bytes of stack space used max in sys_select/sys_poll before allocating
    additional memory. */
+#ifndef CONFIG_4KSTACKS
 #define MAX_STACK_ALLOC 832
+#else
+#define MAX_STACK_ALLOC 448
+#endif
 #define FRONTEND_STACK_ALLOC	256
 #define SELECT_STACK_ALLOC	FRONTEND_STACK_ALLOC
 #define POLL_STACK_ALLOC	FRONTEND_STACK_ALLOC
-- 
1.6.6


  parent reply	other threads:[~2011-10-18 23:34 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-10-18 23:27 [PATCH 0/3] ARM 4Kstacks: introduction Tim Bird
2011-10-18 23:27 ` Tim Bird
2011-10-18 23:29 ` [PATCH 1/3] ARM 4Kstacks: Add support for 4K kernel stacks to ARM Tim Bird
2011-10-18 23:29   ` Tim Bird
2011-10-18 23:31 ` [PATCH 2/3] ARM: Add static kernel function stack size analyzer, for ARM Tim Bird
2011-10-18 23:31   ` Tim Bird
2011-10-19 10:45   ` Arnd Bergmann
2011-10-19 10:45     ` Arnd Bergmann
2011-10-21 17:03     ` Andi Kleen
2011-10-21 17:03       ` Andi Kleen
2011-10-18 23:34 ` Tim Bird [this message]
2011-10-18 23:34   ` [PATCH 3/3] ARM 4Kstacks: Decrease poll and select stack usage, when using 4K stacks Tim Bird
2011-10-20 12:21   ` Arnd Bergmann
2011-10-20 12:21     ` Arnd Bergmann
2011-10-19  0:14 ` [PATCH 0/3] ARM 4Kstacks: introduction Joe Perches
2011-10-19  0:14   ` Joe Perches
2011-10-19  0:26   ` Tim Bird
2011-10-19  0:26     ` Tim Bird
2011-10-19  0:31     ` Joe Perches
2011-10-19  0:31       ` Joe Perches
2011-10-19 23:43       ` Tim Bird
2011-10-19 23:43         ` Tim Bird
2011-10-19  4:54     ` Dave Chinner
2011-10-19  4:54       ` Dave Chinner
2011-10-19  5:02       ` Andi Kleen
2011-10-19  5:02         ` Andi Kleen
2011-10-19 10:51     ` Arnd Bergmann
2011-10-19 10:51       ` Arnd Bergmann
2011-10-22  8:50       ` Ming Lei
2011-10-22  8:50         ` Ming Lei
2011-10-22 13:13         ` Måns Rullgård
2011-10-22 13:13           ` Måns Rullgård
2011-10-22 14:27           ` Andi Kleen
2011-10-22 14:27             ` Andi Kleen
2011-10-22 13:36         ` Russell King - ARM Linux
2011-10-22 13:36           ` Russell King - ARM Linux
2011-10-23 19:25           ` Tim Bird
2011-10-23 19:25             ` Tim Bird
2011-10-23 20:11             ` Russell King - ARM Linux
2011-10-23 20:11               ` Russell King - ARM Linux
2011-10-24 10:36           ` Ming Lei
2011-10-24 10:36             ` Ming Lei
2011-10-23 14:06       ` Bernd Petrovitsch
2011-10-23 14:06         ` Bernd Petrovitsch
2011-10-19  4:33 ` Dave Chinner
2011-10-19  4:33   ` Dave Chinner
2011-10-19  7:35 ` Russell King - ARM Linux
2011-10-19  7:35   ` Russell King - ARM Linux
2011-10-19 23:36   ` Tim Bird
2011-10-19 23:36     ` Tim Bird
2011-10-20  0:13     ` Måns Rullgård
2011-10-20  0:13       ` Måns Rullgård
2011-10-20  1:08       ` Tim Bird
2011-10-20  1:08         ` Tim Bird
2011-10-20  1:55         ` Måns Rullgård
2011-10-20  1:55           ` Måns Rullgård
2011-10-20 22:21     ` Dave Chinner
2011-10-20 22:21       ` Dave Chinner

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4E9E0CEA.6030102@am.sony.com \
    --to=tim.bird@am.sony.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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.