From: Nick Piggin <nickpiggin@yahoo.com.au>
To: Arjan van de Ven <arjan@infradead.org>
Cc: Andi Kleen <ak@muc.de>, linux-kernel@vger.kernel.org
Subject: Re: 2.5isms
Date: Mon, 03 Jan 2005 11:49:31 +1100 [thread overview]
Message-ID: <41D8969B.2030701@yahoo.com.au> (raw)
In-Reply-To: <1104656340.4185.5.camel@laptopd505.fenrus.org>
Arjan van de Ven wrote:
>>I'm curious about a couple of points though. First, is that it is basically
>>just adding a cache colouring to the stack, right? In that case why do only
>>older HT CPUs have bad performance without it? And wouldn't it possibly make
>>even non HT CPUs possibly slightly more efficient WRT caching the stacks of
>>multiple processes?
>
>
> it's a win on more than older HT cpus. It's just that those suffer it
> the most... (since there you have 2 "cpus" share the cache, meaning you
> get double the aliasing)
>
>
>
>>Second, on what workloads does performance suffer, can you remember? I wonder
>>if natural variations in the stack pointer as the program runs would mitigate
>>the effect of this on all but micro benchmarks?
>
>
> one of the problem cases I remember is network daemons all waiting in
> accept() for connections. All from the same codepath basically.
> Randomizing the stackpointer is a gain for that on all cpus that have
> finite affinity on their caches.
>
I see. Yes, that would be a prime candidate.
>
>
>>But even if that were so so, it seems simple enough that I don't have any
>>real problem with keeping it of course.
>
>
> The reason my patch does it much more is that it makes it a step harder
> to write exploits for stack buffer overflows.
>
>
Oh yeah I realised that. I just meant specifically the code to do arch
specific stack colouring.
Thanks
Nick
next prev parent reply other threads:[~2005-01-03 0:50 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2004-12-31 23:06 2.5isms Justin Pryzby
2005-01-01 2:34 ` 2.5isms Nick Piggin
2005-01-01 8:40 ` 2.5isms Arjan van de Ven
2005-01-01 9:13 ` 2.5isms Andi Kleen
2005-01-02 0:43 ` 2.5isms Nick Piggin
2005-01-02 8:58 ` 2.5isms Arjan van de Ven
2005-01-03 0:49 ` Nick Piggin [this message]
2005-01-02 12:04 ` 2.5isms Andi Kleen
2005-01-03 0:44 ` 2.5isms Nick Piggin
-- strict thread matches above, loose matches on Subject: below --
2003-07-03 20:01 "Will be removed in 2.4" Justin Pryzby
2003-12-30 21:30 ` 2.5isms Justin Pryzby
2004-01-03 15:18 ` 2.5isms Pavel Machek
2004-01-07 7:28 ` 2.5isms Justin Pryzby
2004-03-29 15:40 ` 2.5isms Pavel Machek
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=41D8969B.2030701@yahoo.com.au \
--to=nickpiggin@yahoo.com.au \
--cc=ak@muc.de \
--cc=arjan@infradead.org \
--cc=linux-kernel@vger.kernel.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.