RFR: 8286212: Cgroup v1 initialization causes NPE on some systems [v2]

Severin Gehwolf sgehwolf at openjdk.java.net
Wed May 18 18:14:55 UTC 2022


On Tue, 17 May 2022 06:18:25 GMT, Ioi Lam <iklam at openjdk.org> wrote:

>> Severin Gehwolf has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use stringStream to simplify controller path assembly
>
> src/hotspot/os/linux/cgroupV1Subsystem_linux.cpp line 92:
> 
>> 90:           }
>> 91:           ss.print_raw(_root, last_matching_slash_pos);
>> 92:           _path = os::strdup(ss.base());
> 
> Do you mean `Find the longest common prefix`? Maybe give an example in the comments? Text parsing in C code is really difficult to understand.

@iklam yes I meant `Find the longest common prefix`. Fixed the comment.

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

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


More information about the core-libs-dev mailing list