OperatingSystemImpl should factor cpu shares when calculating system load

[David Holmes]

Hi Severin,

On 26/05/2021 6:30 pm, Severin Gehwolf wrote:
> On Mon, 2021-05-03 at 17:57 -0700, Laurence Cable wrote:
>>
>> On 5/3/21 4:50 PM, David Holmes wrote:
>>> On 4/05/2021 9:07 am, Argha C wrote:
>>>> Hello David,
>>>> It does look quite similar.
>>>> The crux is factoring in the sum of cpu time across all shares,
>>>> instead of a single cpu share.
>>>> My proposal for implementing the fix is slightly different from
>>>> the PR, but happy to trust your advice on taking this forward.
>>>
>>> I would suggest liaising with the contributors of
>>>
>>> https://github.com/openjdk/jdk/pull/3656
>>>
>>> so see if all of the issues can be addressed together.
>>
>> +1!
> 
> David, Larry:
> 
> https://github.com/openjdk/jdk/pull/3656
> 
> ... is ready for integration. Could we get a second review, please? It
> addresses wrong cpuLoad() calculation of cpu quota and cpu shares-based
> cpu limits in containers.
> 
> Thoughts?

Sorry, although I connected the two bugs reports I'm not familiar enough 
with this area to review anything. I don't work with containers and 
don't understand their resource management strategies or how the VM 
should report them.

David
-----

> Thanks,
> Severin
>