RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
Magnus Ihse Bursie
magnus.ihse.bursie at oracle.com
Mon Dec 10 11:05:54 UTC 2018
On 2018-12-10 11:56, Nick Gasson (Arm Technology China) wrote:
> Hi,
>
> Any update on this one? Or do we want to give up on it until JDK-8165620
> is implemented?
I think Alan reviewed it as OK with just a minor fix, and offered to
sponsor it for you.
Then the discussion started about some major changes, rewriting
configure to provide individual support for every possible system
function... I recommend that you go with the fix that solves your problem.
In fact, I disagree with the premise of JDK-8165620. I think it's better
to do what you're doing here, to update the source code, so that it is
clear that we know that we're handing 64-bit data.
/Magnus
>
> Thanks,
> Nick
>
>
> On 28/11/2018 11:33, Martin Buchholz wrote:
>>
>> On Tue, Nov 27, 2018 at 7:25 PM, Nick Gasson <Nick.Gasson at arm.com
>> <mailto:Nick.Gasson at arm.com>> wrote:
>>
>> > I missed one thing then looking at this. In TimeZone_md.c it should be
>> > #ifdef MACOSX rather than #ifndef. I can sponsor the change for you but
>> > I need to change this one line before pushing.
>>
>> Hi Alan,
>>
>> Yes feel free to modify it. I think looking at the at other files
>> with these #defines more closely, is it the case that the #ifndef
>> MACOSX check is only required for statvfs64? As in
>> e.g. UnixNativeDispatcher.c. So TimeZone_md.c should look like
>> this:
>>
>> #if defined(_ALLBSD_SOURCE)
>> #define stat64 stat
>> #define lstat64 lstat
>> #define fstat64 fstat
>> #endif
>>
>> I don't have access to any OS X machines to test unfortunately.
>>
>> But I wonder if a better way to handle this is to check for the
>> presence of the stat64 functions in the configure script, and
>> then we could just write something like this, which would be a
>> lot clearer:
>>
>> #if !defined(HAVE_STAT64)
>> #define stat64 stat
>> #endif
>>
>>
>>
>> The best way is to implement
>>
>> https://bugs.openjdk.java.net/browse/JDK-8165620
>> <https://bugs.openjdk.java.net/browse/JDK-8165620>
>> "Entire JDK should be built with -D_FILE_OFFSET_BITS=64"
>>
>> but yes, another good way is to do as you suggest, have configure define
>> HAVE_XXXX for all known functions with a 64-bit variant and then put
>> them into a header file with proper ifdefs for platforms that don't have
>> them.
>>
>> You could also "simply" add
>> #define _FILE_OFFSET_BITS 64
>> but you have to do it for cliques of files that share ambiguously sized
>> data simultaneously.
More information about the core-libs-dev
mailing list