From: "Ryan S. Arnold" <ryan.arnold@gmail.com>
To: "Carlos O'Donell" <carlos@redhat.com>
Cc: "Will Newton" <will.newton@linaro.org>,
"libc-ports@sourceware.org" <libc-ports@sourceware.org>,
"Patch Tracking" <patches@linaro.org>,
"Ondřej Bílka" <neleai@seznam.cz>,
"Siddhesh Poyarekar" <siddhesh@redhat.com>
Subject: Re: [PATCH] sysdeps/arm/armv7/multiarch/memcpy_impl.S: Improve performance.
Date: Tue, 03 Sep 2013 20:56:00 -0000 [thread overview]
Message-ID: <CAAKybw_7VE3zYM1Vb4sfE-HRMMdCx2E9Obf45_11=bGjVZXeJQ@mail.gmail.com> (raw)
In-Reply-To: <52263E63.2080301@redhat.com>
On Tue, Sep 3, 2013 at 2:54 PM, Carlos O'Donell <carlos@redhat.com> wrote:
> The current set of performance preconditions are baked into the experience
> of the core developers reviewing patches. I want the experts out of the
> loop.
>
This is the clutch.
Developers working for the CPU manufacturers are privy to a lot of
unpublished timing, penalty/hazard information, as well as proprietary
pipeline analysis tools.
Will "J. Random Hacker" working for MegaCorp tell you that the reason
he's chosen a particular instruction sequence is because the system
he's working on has a tiny branch cache (the size of which might be
unpublished)?
>> PowerPC has had the luxury of not having their performance
>> pre-conditions contested. PowerPC string performance is optimized
>> based upon customer data-set analysis. So PowerPC's preconditions are
>> pretty concrete... Optimize for aligned data in excess of 128-bytes
>> (I believe).
>
> We should be documenting this somewhere, preferably in a Power-specific
> test that looks at just this kind of issue.
I might be mistaken, but I think you'll find these preconditions
explicitly defined in the string function implementation source files
for PowerPC.
> Documenting this statically is the first, in my opinion, stepping stone
> to having something like dynamic feedback.
Absolutely!
>> Unless technology evolves that you can statistically analyze data in
>> real time and adjust the implementation based on what you find (an
>> implementation with a different set of preconditions) to account for
>> this you're going to end up with a lot of in-fighting over
>> performance.
>
> Why do you assume we'll have a lot of in-fighting over performance?
I'm projecting here. If someone proposed to adjust the PowerPC
optimized string functions to their own preconditions and it
drastically changed the performance of existing customers, or future
customers you'd see me panic.
> At present we've split the performance intensive (or so we believe)
> routines on a per-machine basis. The arguments are then going to be
> had only on a per-machine basis, and even then for each hardware
> variant can have an IFUNC resolver select the right routine at
> runtime.
Right, selecting the right variant with IFUNC has certainly helped
platforms that didn't use optimized libraries. This is the low
hanging fruit. So now our concern is the proliferation of micro-tuned
variants and a lack of qualified eyes to objectively review the
patches.
> Then we come upon the tunables that should allow some dynamic adjustment
> of an algorithm based on realtime data.
Yes, you can do this with tunables if the developer knows something
about the data (more about that later).
>> I've run into situations where I recommended that a customer code
>> their own string function implementation because they continually
>> encountered unaligned-data when copying-by-value in C++ functions and
>> PowerPC's string function implementations penalized unaligned copies
>> in preference for aligned copies.
>
> Provide both in glibc and expose a tunable?
So do we (the glibc community) no longer consider the proliferation of
tunables to be a mortal sin? Or was that only with regard to
configuration options? Regardless, it still burdens the Linux
distributions and developers who have to provide QA.
If tunables are available, then trial-and-error would help where a
user doesn't know the particulars of his application's data usage.
Using tunables is potentially problematic as well. Often testing a
condition in highly optimized code is enough to obviate the
performance benefit you're attempting to provide. Checking for feature
availability might consume enough cycles to make it senseless to use
the facility itself. I believe this is what happened in the early
days trying to use VMX in string routines.
Additionally, while dynamically linked applications won't suffer from
using IFUNC resolved functions (because of mandatory PLT usage), glibc
internal usage of IFUNC resolved functions very likely will if/when
forced to go through the PLT, especially on systems like PowerPC where
indirect branching is more expensive than direct branching. When
Adhemerval's PowerPC IFUNC patches go in I'll probably argue for
keeping a 'generic' optimized version for internal libc usage. We'll
see how it all works together.
So using tunables alone isn't necessarily a win unless it's coupled
with IFUNC. But using IFUNC also isn't a guaranteed win in all cases.
For external usage, Using IFUNC in combination with a tunable should
be beneficial. For instance, on systems that don't have a concrete
cacheline size (e.g., the A2 processor), at process initialization we
query the system cacheline size, populate a static with the size, and
then the string routines will query that size at runtime. It'd be
nice to do that query at initialization and then pre-select an
implementation based on cacheline size so we don't have to test for
the cacheline size each time through the string function.
This of course increases the cost of maintaining the string routines
by having myriad of combinations.
These are all the trade-offs we weigh.
Ryan
next prev parent reply other threads:[~2013-09-03 20:56 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-08-12 7:55 Will Newton
2013-08-27 7:46 ` Will Newton
2013-08-30 17:14 ` Carlos O'Donell
2013-08-30 18:48 ` Will Newton
2013-08-30 19:26 ` Carlos O'Donell
2013-09-02 14:18 ` Will Newton
2013-09-03 16:14 ` Carlos O'Donell
[not found] ` <CANu=DmhA9QvSe6RS72Db2P=yyjC72fsE8d4QZKHEcNiwqxNMvw@mail.gmail.com>
2013-09-02 14:18 ` benchmark improvements (Was: Re: [PATCH] sysdeps/arm/armv7/multiarch/memcpy_impl.S: Improve performance.) Siddhesh Poyarekar
2013-09-03 13:46 ` Will Newton
2013-09-03 17:48 ` Ondřej Bílka
2013-09-02 19:57 ` [PATCH] sysdeps/arm/armv7/multiarch/memcpy_impl.S: Improve performance Ondřej Bílka
2013-09-03 16:18 ` Carlos O'Donell
2013-09-03 17:37 ` Ondřej Bílka
2013-09-03 17:52 ` Carlos O'Donell
2013-09-03 18:57 ` Ondřej Bílka
2013-09-03 19:15 ` Carlos O'Donell
2013-09-04 7:27 ` Siddhesh Poyarekar
2013-09-04 11:03 ` Ondřej Bílka
2013-09-04 11:43 ` Siddhesh Poyarekar
2013-09-04 17:37 ` Ryan S. Arnold
2013-09-05 8:04 ` Ondřej Bílka
2013-09-04 15:30 ` Carlos O'Donell
2013-09-04 17:35 ` Ryan S. Arnold
2013-09-05 11:07 ` Ondřej Bílka
2013-09-05 11:54 ` Joseph S. Myers
2013-09-03 19:34 ` Ryan S. Arnold
2013-09-07 11:55 ` Ondřej Bílka
2013-09-03 19:31 ` Ryan S. Arnold
2013-09-03 19:54 ` Carlos O'Donell
2013-09-03 20:56 ` Ryan S. Arnold [this message]
2013-09-03 23:29 ` Ondřej Bílka
2013-09-03 23:31 ` Carlos O'Donell
2013-09-03 22:27 ` Ondřej Bílka
2013-08-29 23:58 ` Joseph S. Myers
2013-08-30 14:56 ` Will Newton
2013-08-30 15:18 ` Joseph S. Myers
2013-08-30 18:46 ` Will Newton
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='CAAKybw_7VE3zYM1Vb4sfE-HRMMdCx2E9Obf45_11=bGjVZXeJQ@mail.gmail.com' \
--to=ryan.arnold@gmail.com \
--cc=carlos@redhat.com \
--cc=libc-ports@sourceware.org \
--cc=neleai@seznam.cz \
--cc=patches@linaro.org \
--cc=siddhesh@redhat.com \
--cc=will.newton@linaro.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 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 read-only IMAP folder(s) and NNTP newsgroup(s).