On Thu, 19 Nov 2020 16:41:07 GMT, Poonam Bajaj <poo...@openjdk.org> 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

Reply via email to