On Fri, 12 Feb 2021 12:49:19 GMT, Severin Gehwolf wrote:
>> Setting reviewers to 2 since I want @gnu-andrew to be OK with it too.
>
> Hi Harold,
>
>> Thanks for doing this! Sorry for taking so long to review this change. The
>> change looks good.
>
> Thanks for the review!
>
>> Before pushing
On Fri, 12 Feb 2021 10:19:03 GMT, Severin Gehwolf wrote:
>> Hi Severin,
>> Thanks for doing this! Sorry for taking so long to review this change. The
>> change looks good. Before pushing it, could you add a comment explaining
>> what the code in lines 185-194 of CgroupSubsystemFactory.java i
On Thu, 11 Feb 2021 19:51:43 GMT, Harold Seigel wrote:
>> Severin Gehwolf has updated the pull request with a new target base due to a
>> merge or a rebase. The incremental webrev excludes the unrelated changes
>> brought in by the merge/rebase. The pull request contains seven additional
>> co
On Tue, 9 Feb 2021 13:31:25 GMT, Severin Gehwolf wrote:
>> This is an enhancement which solves two issues:
>>
>> 1. Multiple reads of relevant cgroup interface files. Now interface files
>> are only read once per file (just like Hotspot).
>> 2. Proxies creation of the impl specific subsystem vi
> This is an enhancement which solves two issues:
>
> 1. Multiple reads of relevant cgroup interface files. Now interface files are
> only read once per file (just like Hotspot).
> 2. Proxies creation of the impl specific subsystem via `determineType()` as
> before, but now reads all relevant in
