RFR(xxs, jdk10): 8171508: os::jvm_path -XXaltjvm processing error after 8066474

[Thomas Stüfe]

Hi Dmitry, thanks for reviewing!

..Thomas

On Wed, Apr 26, 2017 at 9:40 AM, Dmitry Samersoff <
dmitry.samersoff at oracle.com> wrote:

> Thomas,
>
> Looks good to me.
>
> -Dmitry
>
> On 2017-04-26 10:10, Thomas Stüfe wrote:
> > Hi all,
> >
> > may I please have a review for this tiny fix. 8066474 removed the <arch>
> > directory from the images and since then -XXaltjvm was slightly broken.
> > When handling XXaltjvm, os::jvm_path() examines the path of the libjvm.so
> > to check if it is part of what it considers a standard JDK by traversing
> a
> > number of slashes up the path and looking for "/jre/lib". That number of
> > slashes was off since 8066474.
> >
> > webrev:
> > http://cr.openjdk.java.net/~stuefe/webrevs/8171508-os_jvm_
> path_xxaltjvm_processing_error_after_8066474/jdk10-
> webrev.00/webrev/index.html
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8171508
> >
> > Note that this only affects cases where the alternate libjvm.so is part
> of
> > a full jdk, so it does not affect the gtestLauncher.
> >
> > Thanks & Regards, Thomas
> >
>
>
> --
> Dmitry Samersoff
> Oracle Java development team, Saint Petersburg, Russia
> * I would love to change the world, but they won't give me the sources.
>