RFR(M): 8200126: [TESTBUG] Open source VM runtime signal tests

[Magnus Ihse Bursie]

On 2018-03-31 09:30, David Holmes wrote:
> Hi Misha.
>
> This all seems okay.
>
> On 30/03/2018 12:41 PM, Mikhailo Seledtsov wrote:
>> While testing I discovered build errors on Mac and Solaris. The 
>> following statement " BUILD_HOTSPOT_JTREG_EXECUTABLES_LIBS_exesigtest 
>> := -ljvm" was added to a Linux-only block. I have updated the make 
>> file to add this for any platform w/o conditions; the " exesigtest.c" 
>> is excluded from Windows anyway down below in the make file. I am not 
>> 100% sure this is the correct way to modify the make file; if not 
>> please advise the correct way.
>
> That works for me. I'm not sure how Magnus envisaged this being used 
> though. An alternative would be:
>
>   ifeq ($(OPENJDK_TARGET_OS), windows)
>       BUILD_HOTSPOT_JTREG_EXECUTABLES_CFLAGS_exeFPRegs := -MT
> +     BUILD_HOTSPOT_JTREG_EXCLUDE += exesigtest.c
> + else
> +     BUILD_HOTSPOT_JTREG_EXECUTABLES_LIBS_exesigtest := -ljvm
>   endif
>
> but I think your version is more future-proof.

Either works. If I "envisaged" anything, I think it was more in lines of 
David's suggestion here, but it really doesn't matter. The current 
version is fine as it is.

So, build changes looks good to me.

/Magnus


>
> Thanks,
> David
>
>> With this fix all 4 builds pass.
>>
>> Here is the updated webrev:
>> http://cr.openjdk.java.net/~mseledtsov/8200126.02.open/index.html
>>
>> Thank you,
>> Misha
>>
>>
>> On 3/29/18, 4:00 PM, mikhailo wrote:
>>> I have addressed feedback from Christian, David and Magnus. Here is 
>>> the updated webrev:
>>>
>>> http://cr.openjdk.java.net/~mseledtsov/8200126.01.open/index.html
>>>
>>>
>>> I have also confirmed that output from exesigtest.c printf() is 
>>> logged into .jtr files.
>>>
>>>       Grepped for "signal", I can see the output such as:
>>>           TestSigxfsz.jtr:SIGXFSZ: signal handler using function 
>>> 'sigset' has been set
>>>           TestSigxfsz.jtr:SIGXFSZ: signal handler for signal 25 has 
>>> been processed
>>>           TestSigxfsz.jtr:SIGXFSZ: signal has been sent successfully
>>>           TestSigxfsz.jtr:SIGXFSZ: signal has been received
>>>       Also can see other output from the printf, such as all initVM 
>>> logs.
>>>
>>>
>>> Thank you,
>>> Misha
>>>
>>> On 03/29/2018 02:49 PM, mikhailo wrote:
>>>> Magnus,
>>>>
>>>> Thank you for advice. I have updated the makefile accordingly. Will 
>>>> post updated webrev shortly.
>>>>
>>>>
>>>> Misha
>>>>
>>>>
>>>> On 03/28/2018 03:26 PM, Magnus Ihse Bursie wrote:
>>>>> Yes, you seem to have based this off an old version of 
>>>>> JtregNativeHotspot.gmk.
>>>>>
>>>>> If you update the file I think you see how you should do it, but 
>>>>> I'll give you some help:
>>>>>
>>>>> ifeq ($(OPENJDK_TARGET_OS), windows)
>>>>>   BUILD_HOTSPOT_JTREG_EXECUTABLES_CFLAGS_exeFPRegs := -MT
>>>>>   BUILD_HOTSPOT_JTREG_EXCLUDE += exesigtest.c
>>>>> endif
>>>>
>>>