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