RFR(xxs): 8224023: Redo the fix for ErrorFile option does not handle pre-existing error files of the same name

[Thomas Stüfe]

Thanks for the quick review. Will run it through submit, then push.

Cheers, Thomas

On Thu, May 16, 2019, 11:12 David Holmes <david.holmes at oracle.com> wrote:

> Hi Thomas,
>
> This looks good to me. Thanks for addressing it so promptly.
>
> David
>
> On 16/05/2019 3:59 pm, Thomas Stüfe wrote:
> > Hi all,
> >
> > may I please have reviews for this small change.
> >
> > bug: https://bugs.openjdk.java.net/browse/JDK-8224023
> > cr:
> >
> http://cr.openjdk.java.net/~stuefe/webrevs/8224023-redo-errorfile-overrite-fix/webrev.00/webrev/
> >
> > 8221738 changed the overwrite logic for hs-err files (they are now
> > overwritten, which only affects cases where the file is specified with
> > -XX:ErrorFile=abc).
> >
> > Accidentally I also changed the logic for replay files, and for good
> > measure added a bug, which was a nice feat for a one line patch.
> >
> > Anyways, here is the new version: We open hs-err files with O_TRUNC, but
> > replay files with O_EXCL like we did before. Should we at some point
> decide
> > we want to apply the same overwrite logic to replay files, we can remove
> > this coding again.
> >
> > I manually tested the error handler tests and also all tests under
> > compiler/ciReplay, no issues.
> >
> > Thanks, Thomas
> >
>