Your message dated Wed, 22 Feb 2017 15:18:48 +0000
with message-id <[email protected]>
and subject line Bug#854053: fixed in coreutils 8.26-3
has caused the Debian Bug report #854053,
regarding coreutils: improve 2x-3x sha256sum performance on ppc64le due to 
current gcc optimization bug
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
854053: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=854053
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: coreutils
Version: 8.26-2
Severity: important
Tags: patch

Dear Maintainer,

The sha256sum provided by coreutils (without openssl) is performing
poorly with versions >= 4.9 until 7.0 (currently under development). The
reason for that is the -fschedule-insns optimization that is used with
-O2. By simply deactivating it, there is a performance improvement of
2 to 3 times.

I'm attaching a patch that demonstrate that behavior but it lacks this
condition:
* If ppc64le
* If gcc being used is >= 4.9 and < 7.0

Notes:
1) gcc-7 is not affected by this bug (verified on 20170129 snapshot).
2) clang is not affected by this bug (verified on v3.8 and v3.9).
3) strangely the sha512 is not affected by this.

The #853846 bug is related, but it headed for a different kind of fix.


Below a demonstration of how it performs:

===================================================
$ (./configure && make -j9) > /dev/null && time src/sha256sum 
~/ubuntu-16.10-server-ppc64el.iso
configure: WARNING: libacl development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without ACL support.
configure: WARNING: libattr development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without xattr support.
configure: WARNING: libcap library was not found or not usable.
configure: WARNING: GNU coreutils will be built without capability support.
configure: WARNING: libgmp development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without GMP support.
src/who.c: In function 'print_user':
src/who.c:454:20: warning: initialization discards 'const' qualifier from 
pointer target type [-Wdiscarded-qualifiers]
         int   *a = utmp_ent->ut_addr_v6;
                    ^~~~~~~~
d14bdb413ea6cdc8d9354fcbc37a834b7de0c23f992deb0c6764d0fd5d65408e  
/home/gut/ubuntu-16.10-server-ppc64el.iso

real    0m18.670s
user    0m16.566s
sys     0m0.745s

$ # now with the following patch:
$ diff Makefile.in ../Makefile.in
8989c8989
< @am__fastdepCC_TRUE@  $(COMPILE) -MT $@ -MD -MP -MF $$depbase.Tpo -c -o $@ $< 
&&\
---
> @am__fastdepCC_TRUE@  $(COMPILE) $$([ "$@" == "lib/sha256.o" ] && echo 
> "-fno-schedule-insns") -MT $@ -MD -MP -MF $$depbase.Tpo -c -o $@ $< &&\
$ cp ../Makefile.in Makefile.in
$ (./configure && make -j9) > /dev/null && time src/sha256sum 
~/ubuntu-16.10-server-ppc64el.iso
configure: WARNING: libacl development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without ACL support.
configure: WARNING: libattr development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without xattr support.
configure: WARNING: libcap library was not found or not usable.
configure: WARNING: GNU coreutils will be built without capability support.
configure: WARNING: libgmp development library was not found or not usable.
configure: WARNING: GNU coreutils will be built without GMP support.
src/who.c: In function 'print_user':
src/who.c:454:20: warning: initialization discards 'const' qualifier from 
pointer target type [-Wdiscarded-qualifiers]
         int   *a = utmp_ent->ut_addr_v6;
                    ^~~~~~~~
d14bdb413ea6cdc8d9354fcbc37a834b7de0c23f992deb0c6764d0fd5d65408e  
/home/gut/ubuntu-16.10-server-ppc64el.iso

real    0m5.903s
user    0m5.560s
sys     0m0.255s

===================================================

*** Reporter, please consider answering these questions, where appropriate ***

   * What led up to the situation?
   * What exactly did you do (or not do) that was effective (or
     ineffective)?
   * What was the outcome of this action?
   * What outcome did you expect instead?

*** End of the template - remove these template lines ***


-- System Information:
Debian Release: 9.0
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'unstable'), (500, 'stable')
Architecture: ppc64el (ppc64le)

Kernel: Linux 3.16.0-4-powerpc64le (SMP w/8 CPU cores)
Locale: LANG=en_US.UTF-8, LC_CTYPE=en_US.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages coreutils depends on:
ii  libacl1      2.2.52-3
ii  libattr1     1:2.4.47-2
ii  libc6        2.24-9
ii  libselinux1  2.6-3

coreutils recommends no packages.

coreutils suggests no packages.

-- no debconf information
--- Makefile.in	2016-11-30 16:34:55.000000000 -0200
+++ ../Makefile.in	2017-02-03 09:33:17.936000000 -0200
@@ -8986,7 +8986,7 @@
 
 .c.o:
 @am__fastdepCC_TRUE@	$(AM_V_CC)depbase=`echo $@ | sed 's|[^/]*$$|$(DEPDIR)/&|;s|\.o$$||'`;\
-@am__fastdepCC_TRUE@	$(COMPILE) -MT $@ -MD -MP -MF $$depbase.Tpo -c -o $@ $< &&\
+@am__fastdepCC_TRUE@  $(COMPILE) $$([ "$@" == "lib/sha256.o" ] && echo "-fno-schedule-insns") -MT $@ -MD -MP -MF $$depbase.Tpo -c -o $@ $< &&\
 @am__fastdepCC_TRUE@	$(am__mv) $$depbase.Tpo $$depbase.Po
 @AMDEP_TRUE@@am__fastdepCC_FALSE@	$(AM_V_CC)source='$<' object='$@' libtool=no @AMDEPBACKSLASH@
 @AMDEP_TRUE@@am__fastdepCC_FALSE@	DEPDIR=$(DEPDIR) $(CCDEPMODE) $(depcomp) @AMDEPBACKSLASH@

--- End Message ---
--- Begin Message ---
Source: coreutils
Source-Version: 8.26-3

We believe that the bug you reported is fixed in the latest version of
coreutils, which is due to be installed in the Debian FTP archive.

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Michael Stone <[email protected]> (supplier of updated coreutils package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA256

Format: 1.8
Date: Wed, 22 Feb 2017 07:23:45 -0500
Source: coreutils
Binary: coreutils mktemp realpath
Architecture: source amd64 all
Version: 8.26-3
Distribution: unstable
Urgency: medium
Maintainer: Michael Stone <[email protected]>
Changed-By: Michael Stone <[email protected]>
Description:
 coreutils  - GNU core utilities
 mktemp     - coreutils mktemp transitional package
 realpath   - coreutils realpath transitional package
Closes: 854053
Changes:
 coreutils (8.26-3) unstable; urgency=medium
 .
   * Update standards version to 3.9.8.0 (no changes)
   * Add -fno-schedule-insns to CFLAGS for sha256.o on PPC to fix
     performance regression (Closes: #854053)
Checksums-Sha1:
 c422be8b6112e638c4ffa185b8e2239c2075d82d 1955 coreutils_8.26-3.dsc
 281fbb158a4057975085d4c901c6a62ff1d41323 22392 coreutils_8.26-3.debian.tar.xz
 87d7ca0b33d9cd1dc884b1a25c0bbf7f80eb8791 6326638 
coreutils-dbgsym_8.26-3_amd64.deb
 a3c6cf94064a2fbf4430440a7c8a752f636c3a12 6390 coreutils_8.26-3_amd64.buildinfo
 abafd1b12a8d1388ef66009752719f4df947e272 2870090 coreutils_8.26-3_amd64.deb
 18b86b63746928cebc7b402ef3410a1b05e93f41 455422 mktemp_8.26-3_all.deb
 7ebe803288067a725cd7cefacc245fa60026f566 455420 realpath_8.26-3_all.deb
Checksums-Sha256:
 f62ab642e46e02c470cc045316643de530a0be50446151a5e449ca12da6485c4 1955 
coreutils_8.26-3.dsc
 cef6a15eb95d9e7bc992bca95010bc5ea9e25e98d8f4f668a698eee534d14b93 22392 
coreutils_8.26-3.debian.tar.xz
 b9873ccce15428aee9430529cb17876f45c03c2479936f01e6ee1161b4893cad 6326638 
coreutils-dbgsym_8.26-3_amd64.deb
 fc06addffd3f96dbc5105304b312414a9bc57ef02aa5c492e4ec4b615b13a7e2 6390 
coreutils_8.26-3_amd64.buildinfo
 ef6c0ab3d52a7d3e85ba4a9c04a1931264d34bab842da6e1428c8c4bda28a800 2870090 
coreutils_8.26-3_amd64.deb
 8dcf9cf8c20050fcd8c926610a4ce8bed86410550e88a7ddb91692523ffa9d23 455422 
mktemp_8.26-3_all.deb
 4e526b116f0bde0a8c1bf6bf3451f7951569a227de7b12edc10ec8e64382fc51 455420 
realpath_8.26-3_all.deb
Files:
 fb4ef58bfcaee0b663e6b3372edbd834 1955 utils required coreutils_8.26-3.dsc
 3a424bf130a0e2f201bfe1116f766489 22392 utils required 
coreutils_8.26-3.debian.tar.xz
 2e05609fce582e6d9fc14e13993eec89 6326638 debug extra 
coreutils-dbgsym_8.26-3_amd64.deb
 58c606c3924c2999be1ee2b58032c26a 6390 utils required 
coreutils_8.26-3_amd64.buildinfo
 ec067b553f90117d56022d8de993d47e 2870090 utils required 
coreutils_8.26-3_amd64.deb
 789fa7342475c9a94799df69441c5992 455422 oldlibs extra mktemp_8.26-3_all.deb
 43e8f31487b2885e8728bbf768bee094 455420 oldlibs extra realpath_8.26-3_all.deb

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEAtUxX/EfGGGGDh4C9hqs3PoR/94FAlitpMcACgkQ9hqs3PoR
/96ndw//e2Uci4WWaZDzmX6aNAKVV+NOvePSkyxDvUu4rbJwDwK8Do9Jj/680owR
6htYu10X7fSIMMRbcauZ4Ho5Ev4DigrlR2GxkGgYB5z488z6WHELLt6M7JwhFDc6
78p6MKR5KxLdYQbyYr+3WD2TLf9TJhXuLSGX4BM8VQVgEOv4JTEX0hOyOip6BA1B
d5AIfcc9sbCDgRF4hvgZsk2q2CvVfzeWawr1Eh9zdRuKT5sEk0SCyw2JA9Zw7L3C
t6Sy8Ow9IZIAy041tVR7H7BS9xd/eoXZVhX7ZF6cOpLvhKv6URnqfwWFtATMO9FY
z+EEJM+P9M2Xl32bUpTQ5zF5tCwAxYL3A0qQxAGGJ0mYmX7a438XLXyywD63zrr4
dwxgUOc+oupfoXsnUacMBXrexC/HOxw9pYGUfHeMT3IewXjhydXjICUMIwfEirUP
+SsraMUpf+VgJiOZwKwb6V5SFpY0QEKIpJlopvcFXl5H4jo1CfdbaWQZImVt7Xnt
8nFgrx2vUP8JQ07YUJeR8lLoXSUGicxz2gorcJYn3/COVoLasD8Ku9x/HucnbdX1
SYUntw+UCmcoT24mmZRmQRnJ5FD9S3CYnI1DwcOYpp3FeBHCkL1EjiMqAv3zTGDf
TWLuvx8UcWof3/ak7qDDBMrn26ZYNLnDOASRG4h75iweGxaVJ+4=
=iKaA
-----END PGP SIGNATURE-----

--- End Message ---

Reply via email to