From: Andi Kleen <ak@suse.de>
To: Ray Bryant <raybry@sgi.com>
Cc: Kernel Mailing List <linux-kernel@vger.kernel.org>,
"linux-ia64@vger.kernel.org" <linux-ia64@vger.kernel.org>,
lse-tech <lse-tech@lists.sourceforge.net>,
holt@sgi.com, Dean Roe <roe@sgi.com>, Brian Sumner <bls@sgi.com>,
John Hawkes <hawkes@tomahawk.engr.sgi.com>
Subject: Re: [Lse-tech] scalability of signal delivery for Posix Threads
Date: Mon, 22 Nov 2004 16:22:14 +0000 [thread overview]
Message-ID: <20041122162214.GE21861@wotan.suse.de> (raw)
In-Reply-To: <41A20AF3.9030408@sgi.com>
On Mon, Nov 22, 2004 at 09:51:15AM -0600, Ray Bryant wrote:
> (Obviously, one solution is to recode the application to send fewer signals
> per thread as the number of threads increase. However, we are concerned by
> the fact that a user application, of any kind, can be constructed in a way
> that causes system to become responsive and would like to find a solutiuon
> that would let us correctly execute the program as described.)
I suspect there are hundreds or thousands of ways on such a big system to
exploit some lock to make the system unresponsive. If you wanted
to fix them all your would be in a large scale redesign effort.
It's not clear why this particular case is special.
> Since signals are sent much more often than sigaction() is called, it would
> seem to make more sense to make sigaction() take a heavier weight lock of
At least in traditional signal semantics you have to call sigaction
or signal in each signal handler to reset the signal. So that
assumption is not necessarily true.
> It seems to me that scalability would be improved if we moved the siglock
> from
> the sighand structure to the task_struct. (keep reading, please...) Code
> that manipulates the current task signal data only would just obtain that
> lock. Code that needs to change the sighand structure (e. g. sigaction())
> would obtain all of the siglock's of all tasks using the same sighand
> structure. A list of those task_struct's would be added to the sighand
> structure to enable finding these structurs without having to take the
> task_list_lock and search for them.
Taking all these locks without risking deadlock would be tricky.
You could just use a ring, but would need to point to a common
anchor and always start from there to make sure all lock grabbers
aquire the locks in the same order.
> Anyway, we would be interested in the community's ideas about dealing with
> this signal delivery scalability issue, and, comments on the solution above
> or suggestions for alternative solutions are welcome.
How about you figure out a fast path of some signals that can work
without locking: e.g. no load balancing needed, no queued signal, etc.
and then just do the delivery of SIGPROF lockless? Or just ignore it
since the original premise doesn't seem to useful.
-Andi
next prev parent reply other threads:[~2004-11-22 16:22 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-11-22 15:51 scalability of signal delivery for Posix Threads Ray Bryant
2004-11-22 16:07 ` Matthew Wilcox
2004-11-22 19:49 ` [Lse-tech] " Ray Bryant
2004-11-22 19:53 ` Andi Kleen
2004-11-22 16:22 ` Andi Kleen [this message]
2004-11-22 16:51 ` [Lse-tech] " Andreas Schwab
2004-11-22 16:54 ` Andi Kleen
2004-11-22 18:56 ` Ray Bryant
2004-11-22 19:22 ` Ray Bryant
2004-11-22 17:23 ` Philip J. Mucci
2004-11-22 21:26 ` Boehm, Hans
2004-11-22 21:34 ` Andi Kleen
2004-12-01 22:53 ` Brent Casavant
2004-11-22 21:27 ` Rick Lindsley
2004-11-22 23:39 ` Ray Bryant
2004-11-22 23:01 ` Boehm, Hans
2004-11-22 17:19 ` Robin Holt
2004-11-22 19:25 ` Ray Bryant
2004-11-23 20:42 ` Ray Bryant
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=20041122162214.GE21861@wotan.suse.de \
--to=ak@suse.de \
--cc=bls@sgi.com \
--cc=hawkes@tomahawk.engr.sgi.com \
--cc=holt@sgi.com \
--cc=linux-ia64@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lse-tech@lists.sourceforge.net \
--cc=raybry@sgi.com \
--cc=roe@sgi.com \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox