RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32 (original) (raw)
Nick Gasson Nick.Gasson at arm.com
Mon Nov 26 09:08:26 UTC 2018
- Previous message (by thread): RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
- Next message (by thread): RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
If you can then it would be great, if only to save others from looking at it and wondering if it should also be changed. Maybe for another issue but there are several other usages in the java launcher that should be looked at.
Hi Alan,
I've done this here:
http://cr.openjdk.java.net/~njian/8214077/webrev.3/
Thanks, Nick
-----Original Message----- From: Alan Bateman <Alan.Bateman at oracle.com> Sent: 23 November 2018 19:42 To: Nick Gasson <Nick.Gasson at arm.com>; Magnus Ihse Bursie <magnus.ihse.bursie at oracle.com>; build-dev <build-dev at openjdk.java.net>; core-libs-dev at openjdk.java.net Cc: nd <nd at arm.com> Subject: Re: RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
On 23/11/2018 11:13, Nick Gasson wrote: >> This looks right and reduces the stat usages in libjava down to one >> remaining case (ProcessHandlerImpllinux.c). > Hi Alan, > > Do you want me to do this one too for completeness? Although as > it's calling stat() on the /proc/ directory I don't think it's possible to > hit the EOVERFLOW case. If you can then it would be great, if only to save others from looking at it and wondering if it should also be changed. Maybe for another issue but there are several other usages in the java launcher that should be looked at. -Alan
- Previous message (by thread): RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
- Next message (by thread): RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]