RFR: JDK-8202200: set INCLUDE_SA to false on s390x by default -was : RE: INCLUDE_SA/serviceability agent - support on s390x

[Jini George]

Looks good to me, Matthias.

Thank you,
Jini.

On 4/24/2018 9:39 PM, Baesken, Matthias wrote:
> Hi Jini,   the removal  of  the mentioned headers leads to build errors  so we better keep it .
> Error is when the headers are removed :
> 
> /nb/linuxs390x/nightly/jdk/src/hotspot/share/runtime/vmStructs.cpp:100:31: fatal error: vmStructs_s390.hpp: No such file or directory
>   #include CPU_HEADER(vmStructs)
> 
> 
> I  uploaded a webrev for review  :
> 
> 
> http://cr.openjdk.java.net/~mbaesken/webrevs/8202200/
> 
> bug :
> 
> https://bugs.openjdk.java.net/browse/JDK-8202200
> 
> 
> 
> Best regards, Matthias
> 
> 
>> -----Original Message-----
>> From: Jini George [mailto:jini.george at oracle.com]
>> Sent: Dienstag, 24. April 2018 08:58
>> To: Baesken, Matthias <matthias.baesken at sap.com>; 'build-
>> dev at openjdk.java.net' <build-dev at openjdk.java.net>
>> Cc: serviceability-dev at openjdk.java.net; Schmidt, Lutz
>> <lutz.schmidt at sap.com>
>> Subject: Re: INCLUDE_SA/serviceability agent - support on s390x
>>
>> Hi Matthias,
>>
>> Your change looks good to me. It might make sense to also remove the
>> following lines from:
>>
>> src/jdk.hotspot.agent/linux/native/libsaproc/libproc.h
>>
>>    78 #if defined(s390x)
>>    79 #include <asm/ptrace.h>
>>    80 #endif
>>
>> I am not sure if the following files are required either:
>> src/hotspot/cpu/s390/vmStructs_s390.hpp
>> src/hotspot/os_cpu/linux_s390/vmStructs_linux_s390.hpp
>>
>> Thanks,
>> Jini (Not a Reviewer).
>>
>>
>> On 4/23/2018 5:31 PM, Baesken, Matthias wrote:
>>> Hello,   as far as I know  the serviceability agent   is not  supported
>>> on  linux s390x .
>>>
>>> However  (unlike  on aix where it is not supported as well) ,
>>>    INCLUDE_SA=false    is not set  in the central configure  m4 files .
>>>
>>> Should we set it  ( suggested diff below)  ?
>>>
>>> Best regards, Matthias
>>>
>>> hg diff
>>>
>>> diff -r fcd5df7aa235 make/autoconf/jdk-options.m4
>>>
>>> --- a/make/autoconf/jdk-options.m4      Wed Apr 18 11:19:32 2018 +0200
>>>
>>> +++ b/make/autoconf/jdk-options.m4      Mon Apr 23 13:46:17 2018 +0200
>>>
>>> @@ -238,6 +238,9 @@
>>>
>>>      if test "x$OPENJDK_TARGET_OS" = xaix ; then
>>>
>>>        INCLUDE_SA=false
>>>
>>>      fi
>>>
>>> +  if test "x$OPENJDK_TARGET_CPU" = xs390x ; then
>>>
>>> +    INCLUDE_SA=false
>>>
>>> +  fi
>>>
>>>      AC_SUBST(INCLUDE_SA)
>>>
>>> # Compress jars
>>>
>>> Best regards, Matthias
>>>