Withdrawn: 8334254: Cleanup CGroups initialization code

duke duke at openjdk.org
Fri Aug 23 18:53:10 UTC 2024


On Thu, 13 Jun 2024 18:20:33 GMT, Johan Sjölen <jsjolen at openjdk.org> wrote:

> Hi,
> 
> Please consider this patch which attempts to improve the code quality of some of the cgroups code, somewhat.
> 
> 1. Get rid of `#define`s and upgrade to a `CG_INFO` enum, with some helpers
> 2. Make some of the code prettier by removing copy-pasted and edited code
> 3. Make `CgroupInfo` RAII, ridding us of the `cleanup` function
> 4. `set_subsystem_path` is **only** used at construction time, only call it at that time and make those functions private. Also remove `virtual`ness, as this is unnecessary now.
> 
> Testing:
> 
> 1. cgroupTest group in gtest.
> 
> Thank you.

This pull request has been closed without being integrated.

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

PR: https://git.openjdk.org/jdk/pull/19709


More information about the hotspot-runtime-dev mailing list