[Feature] P9: Power9 CPU tuned libraries

Bug #1702917 reported by bugproxy
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
The Ubuntu-power-systems project
Won't Fix
Medium
Canonical Foundations Team
glibc (Ubuntu)
Won't Fix
Medium
Adam Conrad

Bug Description

glibc provides minimal support for POWER9 since version 2.23.
 glibc 2.25 is the first version to provide POWER9 optimizations via
 STT_GNU_IFUNC. The optimizations are enabled by default and will be disabled
 when using --disable-multi-arch (notice that multi-arch in glibc context is
 not the same thing as used by Debian).

 However, this feature request is proposing to provide a second glibc build,
 configured with --with-cpu=power9 and co-installed with the default glibc
 build, but under $(LIBDIR)/power9.
 Other distros usually install these under /lib64/power9. Ubuntu may need to
 use a different directory due to supporting Debian's multiarch.

 The support for this is known to work upstream since glibc 2.23.
 So, I think this is just pending changes in the distro build system.

bugproxy (bugproxy)
tags: added: architecture-ppc64le bugnameltc-145024 severity-low targetmilestone-inin1710
Changed in ubuntu:
assignee: nobody → Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage)
affects: ubuntu → glibc (Ubuntu)
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
assignee: nobody → Canonical Foundations Team (canonical-foundations)
Frank Heimes (fheimes)
Changed in ubuntu-power-systems:
importance: Undecided → Low
Revision history for this message
Frank Heimes (fheimes) wrote :

Due to the fact that this ticket is tagged with 'severity-low' does it mean the 'Power9 CPU tuned libraries' can be included with 18.04 (rather than with 17.10)?

bugproxy (bugproxy)
tags: added: severity-medium
removed: severity-low
Revision history for this message
bugproxy (bugproxy) wrote : Comment bridged from LTC Bugzilla

------- Comment From <email address hidden> 2017-07-18 15:56 EDT-------
Canonical - no we need this to be in 17.10. The priority has been re-adjusted. Let me know if there is an issue.

Revision history for this message
Matthias Klose (doko) wrote :

I'd like to see things done for the toolchain in 17.10 already, so that this kind of stuff can be tested for the LTS. It's another thing if we can do it, not having p9 hardware...

Revision history for this message
Adam Conrad (adconrad) wrote :

As a general rule, I'd prefer not to have extra glibc builds. The bug report points out that most of the critical optimizations are handles by IFUNC, is there a solid argument for why we should have two builds as well? Given that the followup request would almost certainly be to ship both by default, we'd end up with a fair amount of filesystem bloat, and it's also a bit of a packaging nightmare (there was great rejoicing from my end when I killed libc6-i686, the last "tuned" build of glibc we had).

For i386 versus i686, there were some very reasonable arguments. The mandatory FPU, the extra registers, cmov, etc, etc. Generic compiler tuninng of i386 versus i686 gave very obvious real world performance increases. I'm dubious that this would be true for p8 versus p9.

Manoj Iyer (manjo)
Changed in glibc (Ubuntu):
importance: Undecided → Low
assignee: Ubuntu on IBM Power Systems Bug Triage (ubuntu-power-triage) → Adam Conrad (adconrad)
tags: added: triage-g
Manoj Iyer (manjo)
Changed in glibc (Ubuntu):
importance: Low → Medium
Changed in ubuntu-power-systems:
importance: Low → Medium
tags: added: triage-a
removed: triage-g
Revision history for this message
bugproxy (bugproxy) wrote :

------- Comment From <email address hidden> 2017-09-05 09:53 EDT-------
> As a general rule, I'd prefer not to have extra glibc builds.
> ...
> (there was great rejoicing from my end when I
> killed libc6-i686, the last "tuned" build of glibc we had).

If this is a policy adopted by all architectures, there is no need to create an exception for POWER.

However, I kindly ask Canonical to contact us if/when another architecture starts having CPU-tuned libraries.

So, I'm changing the status of this feature request accordingly.

Thanks!

Changed in ubuntu-power-systems:
status: New → Won't Fix
Adam Conrad (adconrad)
Changed in glibc (Ubuntu):
status: New → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.