RFR (XS) JDK-8072694: Need errno info when CDS archive creation fails
Lois Foltan
lois.foltan at oracle.com
Mon Feb 9 13:05:41 UTC 2015
Hi Harold,
Looks good. Really minor comment:
src/share/vm/memory/filemap.cpp:
- line #391 - you might want to format the error message
consistently with
how it is formatted in FileMapInfo::open_for_read(). In that
method the
errno info is put in parentheses. I don't have a preference
for either format
just that both formats are consistent.
I don't need to see a new webrev.
Thanks,
Lois
On 2/9/2015 7:42 AM, harold seigel wrote:
> Hi,
>
> Please review this small fix for bug 8072694.
>
> Open webrev: http://cr.openjdk.java.net/~hseigel/bug_8072694_errno/
>
> JBS bug: https://bugs.openjdk.java.net/browse/JDK-8072694
>
> The fix was regression tested with JPRT, JCK lang and VM tests,
> hotspot JTReg, and NSK quick tests. In addition, I ran tests that
> intentionally failed to create the archive and checked the generated
> errno messages.
>
> Thanks, Harold
More information about the hotspot-runtime-dev
mailing list