8226863: [aix] jdk/java/lang/reflect/exeCallerAccessTest cannot launch on primordial thread

[Thomas Stüfe]

Hi Mandy,

On Wed, Jul 3, 2019, 00:20 Mandy Chung <mandy.chung at oracle.com> wrote:

> Hi Thomas,
>
> This is AIX-only.    It would be cleaner to move AIX-specific launcher
> to a new file like main_aix.c.  Have you considered that?
>

I rather not since this would unnecessarily increase complexity even
further.

However, if you really dislike the platform specific code, I would withdraw
the patch and just disable the test on AIX. I was on the fence about that
anyway.

Cheers, Thomas


>
> I don't know how to specify additional .c file in
> make/test/JtregNativeJdk.gmk  though.
>
> Mandy
>
>
>
> On 7/1/19 10:23 PM, Thomas Stüfe wrote:
> > Second, corrected Webrev:
> >
> >
> http://cr.openjdk.java.net/~stuefe/webrevs/execalleraccesstest-cannot-launch-on-primordial-thread/webrev.01/webrev/
> >
> > Run through SAP nightlies on all platforms.
> >
> > Cheers, Thomas
> >
> >
> > On Thu, Jun 27, 2019 at 9:02 AM Thomas Stüfe <thomas.stuefe at gmail.com>
> > wrote:
> >
> >> Hi all,
> >>
> >> Issue:
> >> https://bugs.openjdk.java.net/browse/JDK-8226863
> >> webrev:
> >>
> >>
> http://cr.openjdk.java.net/~stuefe/webrevs/8226863--jdk-java-lang-reflect-execalleraccesstest-cannot-launch-on-primordial-thread/webrev.00/webrev/
> >>
> >> we have this annoying issue on AIX that the libjvm cannot be invoked on
> a
> >> primordial thread. Therefore we need to change launchers which create
> the
> >> VM to spawn own threads (we only do this where it is worth the effort -
> >> this, admittedly, is a corner case, we could alternatively just disable
> the
> >> test on AIX).
> >>
> >> This is annoying and vaguely embarrassing :-/ Maybe IBM will step in
> some
> >> time and help us solve the underlying issues which have to do with the
> >> inability to create guard pages on the primordial thread stack (among
> other
> >> things IIRC).
> >>
> >> Thanks for reviewing,
> >>
> >> Thomas
> >>
> >>
> >>
>
>