[PING] RFR: 8231111: Cgroups v2: Rework Metrics in java.base so as to recognize unified hierarchy
Mandy Chung
mandy.chung at oracle.com
Tue Feb 18 19:00:58 UTC 2020
On 2/18/20 4:50 AM, Severin Gehwolf wrote:
> Hi Mandy,
>
> Thanks again for the review!
>
> Updated webrev:
> incremental (only review changes): http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8231111/11/incremental/webrev/
> full: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8231111/11/webrev/
This looks good. I only skimmed on the tests and not reviewed in
details (I assume Bob has reviewed them).
All new cgroup-specific and metrics implementation classes are now
linux-specific classes which is good.
> More below.
>
>
>> test/lib/jdk/test/lib/containers/cgroup/MetricsTester.java
>> The Oracle copyright is taken out and the copyright is also changed from GPL to GPL+CP.
>> The Red Hat copyright can be added to the top of the file immediately before "DO NOT ALTER or REMOVE" line like [1].
>>
>> [1] http://hg.openjdk.java.net/jdk/jdk/file/tip/test/hotspot/jtreg/compiler/onSpinWait/TestOnSpinWaitEnableDisable.java
> Hmm, old MetricsTester got renamed with this patch to
> MetricsTesterCgroupV1. MetricsTesterCgroupV1 still has the old
> copyright. The version you've looked at is the common part and
> instantiates tester for cgroup v1 or cgroup v2 as required.
Thanks for clarifying. I now see that MetricsTester.java is a new file
in this patch but the webrev shows as an existing file.
> Aside: Not sure why old MetricsTester (or new MetricsTesterCgroupV1) is
> GPL (over GPL+CP).
> Either way, I've changed license to GPL over GPL+CP for the new test
> classes with Red Hat copyright.
I skimmed through the copyright header and license text. Looks fine to me.
Mandy
More information about the core-libs-dev
mailing list