Yes, that's the one. Interestingly, in glibc-2.19, this change is reverted. It is present in glibc-2.17 & glibc-2.18 as released by GNU. Oddly, in glibc git, the buggy version appears.
On Tue, Apr 29, 2014 at 2:25 PM, Lennart Sorensen < lsore...@csclub.uwaterloo.ca> wrote: > On Tue, Apr 29, 2014 at 06:01:00PM +0200, Sébastien Bernard wrote: > > Le 29/04/2014 16:50, Kieron Gillespie a écrit : > > >I'll rebuild one of my SunBlade 2500 latter with sid and see if I > > >get the same result. If it doesn't show the symptom I will rebuild > > >my T2000 and see if it is something specific to the Niagara T1. > > > > > >-Kieron > > > > > > > > >On Tue, Apr 29, 2014 at 10:45 AM, Sébastien Bernard > > ><sbern...@nerim.net <mailto:sbern...@nerim.net>> wrote: > > > > > > Le 29/04/2014 16:34, Kieron Gillespie a écrit : > > > > > > I am currently investigating this unusual behavior with > > > strcmp, and I am unable to reproduce the problem using the > > > test_strcmp example provided. > > > > > > It returns the correct output of, > > > > > > result from strcmp('\0000','\0001' is -1) > > > > > > This was built on Debian Wheezy with a T2000 SPARC processor > > > using GCC 4.6.3-14 from Debian Wheezy Repo. > > > > > > Cheers, > > > Kieron > > > > > > Hum, I tried with a T5220 with debian 7.5 /wheezy. > > > Same result, the strcmp is ok. > > > Could you try with unstable ? > > > > > > Cheers. > > > > > > > > Tested on niagara T5220: > > with wheezy 2.13 glibc: > > root@skynet:~# ./test_strcmp > > result from strcmp('\0000','\0001' is -1) > > result from memcmp('\0000','\0001' is -1) > > > > Tested from debootstrap sid on the same niagara : > > root@skynet:/# ./test_strcmp > > result from strcmp('\0000','\0001' is 0) > > result from memcmp('\0000','\0001' is -1) > > > > > > So, glibc-2.18 is broken on sparc. > > So does that mean this commit broke it: > > commit ad69cc2652c0422ebac3296d914c25e470498ce1 > Author: David S. Miller <da...@davemloft.net> > Date: Wed Aug 24 01:32:24 2011 -0700 > > Rewritten v9/64-bit sparc strcmp. > > diff --git a/ChangeLog b/ChangeLog > index 4fde8c2..ab754e5 100644 > --- a/ChangeLog > +++ b/ChangeLog > @@ -1,3 +1,7 @@ > +2011-08-24 David S. Miller <da...@davemloft.net> > + > + * sysdeps/sparc/sparc64/strcmp.S: Rewrite. > + > 2011-08-24 Andreas Schwab <sch...@redhat.com> > > * elf/Makefile: Add rules to build and run unload8 test. > ...etc... > > -- > Len Sorensen >
