On Thu, 8 Oct 2020 13:36:20 GMT, Severin Gehwolf <sgehw...@openjdk.org> wrote:

>> src/java.base/linux/classes/jdk/internal/platform/CgroupSubsystemFactory.java
>>  line 185:
>> 
>>> 183:          } else {
>>> 184:            // fallback to old, pre JDK-8245543, behaviour
>>> 185:            return line.contains("cgroup");
>> 
>> Why are you doing this fallback rather than returning false??
>
> The idea was to have have some basics working even though the pattern doesn't 
> match for some reason. I guess we can
> remove that as it's rather unlikely to not match.

The change looks good.

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

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

Reply via email to