RFR [jdk8] : 8193807 : AIX: avoid UnsatisfiedLinkError by providing empty basic implementations of getSystemCpuLoad and getProcessCpuLoad

[Erik Joelsson]

Looks ok to me.

/Erik


On 2017-12-20 10:32, Volker Simonis wrote:
> Hi Matthias,
>
> the change looks good!
> I can sponsor it once we get the approval.
>
> Also forwarded to buil-dev for the minimal build change.
>
> Thank you and best regards,
> Volker
>
>
> On Wed, Dec 20, 2017 at 9:50 AM, Baesken, Matthias
> <matthias.baesken at sap.com> wrote:
>> Hello  , Mark reported  this issue on AIX  with OpenJDK8 :
>>
>>> I'm getting an unsatisfied link error when using logstash on AIX but I suspect the issue is with AIX or the JRE, the exception is:
>>>
>>> java.lang.UnsatisfiedLinkError: sun.management.OperatingSystemImpl.getProcessCpuLoad()D
>>>                   getProcessCpuLoad at sun/management/OperatingSystemImpl.java:-2
>>>                              <init> at org/logstash/instrument/monitors/ProcessMonitor.java:40
>>>                              detect at org/logstash/instrument/monitors/ProcessMonitor.java:79
>>>                            generate at org/logstash/instrument/reports/ProcessReport.java:15
>>>
>>> this is the line in logstash:
>>>
>>> this.cpuProcessPercent = scaleLoadToPercent(unixOsBean.getProcessCpuLoad());
>>>
>>> https://github.com/elastic/logstash/blob/master/logstash-core/src/main/java/org/logstash/instrument/monitors/ProcessMonitor.java
>>>
>>>
>>> Could anybody help steer me in the right direction?
>>>
>> This fix addresses  the missing  getSystemCpuLoad and getProcessCpuLoad  on AIX  .
>> JDK8 is only affected , JDK9 and higher is not affected .
>>
>> Could I get a  review for this change   ?
>>
>>
>> Change :
>>
>> http://cr.openjdk.java.net/~mbaesken/webrevs/8193807/
>>
>> Bug :
>>
>> https://bugs.openjdk.java.net/browse/JDK-8193807
>>
>>
>> Thanks, Matthias