On Thu, 19 Nov 2020 16:41:07 GMT, Poonam Bajaj <[email protected]> wrote:
>> src/java.base/linux/classes/jdk/internal/platform/CgroupUtil.java line 50:
>>
>>> 48: throw new InternalError(e.getCause());
>>> 49: } catch (UncheckedIOException e) {
>>> 50: throw e.getCause();
>>
>> There are other reader methods in the same file, should this block be added
>> to them as well?
>
> This change takes care of the case when reading of files /proc/self/cgroup or
> /proc/self/mountinfo might fail due to some reason that can cause the
> initialization of cgroupv1 subsystem (initSubSystem) to fail. Looking at the
> usages of other reader methods, I think it makes sense to catch
> UncheckedIOException there as well.
Updated the code changes, and added the catch blocks for the rest of the reader
methods and also for the cgroupv2 subsystem.
-------------
PR: https://git.openjdk.java.net/jdk/pull/1303
