build problems on sparc64-linux

[Andrew John Hughes]

2010/1/16 Matthias Klose <doko at ubuntu.com>:
> On 15.01.2010 03:10, Dennis Gilmore wrote:
>>
>> On Saturday 02 January 2010 09:02:47 am Matthias Klose wrote:
>>>
>>> I see a build failure of current 6b17 from the IcedTea6 branch on
>>> sparc64-linux using the original hotspot, but not on sparc-linux (32bit).
>>> The build fails building abstractCompiler.cpp in
>>> build/linux-sparcv9/hotspot/outputdir/linux_sparcv9_compiler2/product
>>>
>>> .../hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:1069: error: inline
>>> function 'void Assembler::emit_long(int)' used but never defined
>>> .../hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:1083: error: inline
>>> function 'void Assembler::add(RegisterImpl*, int, RegisterImpl*,
>>> relocInfo::relocType)' used but never defined
>>> .../hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:1514: error: inline
>>> function 'void Assembler::stb(RegisterImpl*, const Address&, int)' used
>>> but never defined .../hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:1291:
>>> error: inline function 'void Assembler::ldsb(const Address&,
>>> RegisterImpl*, int)' used but never defined
>>> .../hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:1507: error: inline
>>> function 'void Assembler::st(RegisterImpl*, RegisterImpl*,
>>> RegisterImpl*)'
>>> used but never defined
>>>
>>> these are all defined in assembler_sparc.inline.hpp. not using
>>> precompiled
>>> headers and -save-temps shows that assembler_sparc.inline.hpp is not
>>> included in the saved .ii file.
>>>
>>> I currently fail to see how assembler_x86.inline.hpp and
>>> assembler_zero.inline.hpp are included (or assembler_sparc.inline.hpp on
>>> sparc 32bit). Any hints?
>>>
>>> patches/icedtea-sparc-trapsfix.patch is patched to include asm/traps.h
>>> instead of asm-sparc/traps.h only for one sparc architecture. I don't see
>>> asm-sparc/traps.h anymore with recent kernel headers. maybe this should
>>> be
>>> build configure check depending on the kernel headers used?
>>
>> one of the sparc patches in fedora is dealing with the new header
>> locations.
>> kernel headers got moved when sparc and sparc64 kernel trees merged.
>
> ok, I'm checking in the attached patch as a replacement for
> patches/icedtea-sparc-trapsfix.patch, so the local fedora patches can be
> dropped.
>
>>> the fedora packaging repository does have two sparc patches not found in
>>> IcedTea. Are these still applied? Does the fedora build acctually build
>>> and
>>> work in it's current state?
>>
>> yes  and yes  at least with the last build we did
>> https://sparc.koji.fedoraproject.org/koji/buildinfo?buildID=42183
>>
>> If you have any sparc questions feel free to ask me.
>
> please could you try to build current IcedTea head or 1.7-branch configured
> with --with-hotspot=hs16 and see if it does build for you on sparc and
> sparc64?
>
>  Matthias
>

I looked at the Fedora patches yesterday and agree this is the right
change.  But I thought we were going to make this dependent on a
configure check for the /usr/include/asm-sparc directory?
-- 
Andrew :-)

Free Java Software Engineer
Red Hat, Inc. (http://www.redhat.com)

Support Free Java!
Contribute to GNU Classpath and the OpenJDK
http://www.gnu.org/software/classpath
http://openjdk.java.net

PGP Key: 94EFD9D8 (http://subkeys.pgp.net)
Fingerprint: F8EF F1EA 401E 2E60 15FA  7927 142C 2591 94EF D9D8