On Thu, 19 Nov 2020 20:49:07 GMT, Poonam Bajaj <poo...@openjdk.org> wrote:

>> Shouldn't you add a check in 
>> src/java.base/linux/classes/jdk/internal/platform/cgroupv2/CgroupV2Subsystem.java:sumTokensIOStat
>>  ?
>> 
>> What about getLongEntry in 
>> src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemController.java?
>>   There are streams operations occuring there as well.
>
> @bobvandette sumTokensIOStat() and getLongEntry() should also have the catch 
> blocks, and I have added those. Please review the latest commit.

@poonamparhar Wouldn't we be able to reproduce this by a test? E.g. 
`test/jdk/jdk/internal/platform/cgroup/TestCgroupSubsystemFactory.java` It 
should already be checking some of this with `mountInfoFileNotFound()` and 
`cgroupsFileNotFound()`. Thoughts?

-------------

PR: https://git.openjdk.java.net/jdk/pull/1303

Reply via email to