RFR: 8214077: test java/io/File/SetLastModified.java fails on ARM32

[Nick Gasson]

> 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 (ProcessHandlerImpl_linux.c).
> > Hi Alan,
> >
> > Do you want me to do this one too for completeness? Although as
> > it's calling stat() on the /proc/<pid> 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