RFR 8144278 [TESTBUG] hotspot/runtime/StackGuardPages/testme.sh should use native library build support

[Gerald Thornbrugh]

Hi Dmitry,

Thanks!

Jerry
> Gerald,
>
> Looks good for me.
>
> -Dmitry
>
> On 2016-06-10 19:16, Gerald Thornbrugh wrote:
>> Hi Everyone,
>>
>> I have address Erik's, Dmitry's and David's concerns in a new webrev:
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8144278
>>
>> JDK Webrev: http://cr.openjdk.java.net/~gthornbr/8144278/jdk-webrev.01/
>> <http://cr.openjdk.java.net/%7Egthornbr/8144278/jdk-webrev.01/>
>>
>> Hotspot Webrev:
>> http://cr.openjdk.java.net/~gthornbr/8144278/hotspot-webrev.01/
>> <http://cr.openjdk.java.net/%7Egthornbr/8144278/hotspot-webrev.01/>
>>
>> Please let me know if you have any questions or concerns.
>>
>> Thanks,
>>
>> Gerald
>>> Hi Everyone,
>>>
>>> Please review my changes to address JDK-8144278.
>>>
>>> Bug: https://bugs.openjdk.java.net/browse/JDK-8144278
>>>
>>> JDK Webrev:
>>> http://cr.openjdk.java.net/~gthornbr/8144278/jdk-webrev.00/
>>> <http://cr.openjdk.java.net/%7Egthornbr/8144278/jdk-webrev.00/>
>>>
>>> Hotspot Webrev:
>>> http://cr.openjdk.java.net/~gthornbr/8144278/hotspot-webrev.00/
>>> <http://cr.openjdk.java.net/%7Egthornbr/8144278/hotspot-webrev.00/>
>>>
>>>
>>> The fix makes changes to two areas.
>>>
>>> The JDK changes where needed to add a path to the libjvm.so library
>>> when linking test executables.
>>> These changes where provide by Erik Joelsson.
>>>
>>> The Hotspot changes addressed the following issues:
>>>
>>> The StackGuardPages test directory needed to be added to the list of
>>> native executable tests.
>>>
>>> The libjvm.so and libpthread.so libraries needed to be added to the
>>> LDFLAGS definition for the
>>> "invoke" test executable.
>>>
>>> The StackGuardPages/testme.sh script needed to specify that
>>> DoOverflow.java must be complied,
>>> specify the new location of the "invoke" test executable and remove
>>> all portions of the script that
>>> compiled and linked DoOverflow.java and invoke.c.
>>>
>>> The native test source file "invoke.c" needed to be renamed
>>> "exeinvoke.c" so the build system will
>>> automatically build it.
>>>
>>> The native test source file exeinvoke.c needed to be changed to
>>> compile without warnings, pass the
>>> path of the DoOverflow.class file to the function which starts the JVM
>>> and keep the linux gcc compiler
>>> optimizer from in-lining the do_overflow() function which caused it
>>> not to get called.
>>>
>>> Please let me know if you have any questions or concerns.
>>>
>>> Thanks,
>>>
>>> Gerald
>>>
>