[8u] RFR: 8217338: [Containers] Improve systemd slice memory limit support

Severin Gehwolf sgehwolf at redhat.com
Tue Dec 1 10:28:54 UTC 2020


Hi Paul,

On Thu, 2020-11-26 at 13:53 +0000, Hohensee, Paul wrote:
> I'd add the copyright update for 8u, and file an 11u-only issue to fix the copyright dates in the 11u backport.

This webrev brings all copyright lines to 2019. Only src/os/linux/vm/os
Container_linux.hpp needed the actual update. All others had 2019
already.

https://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8217338/jdk8/02/

Thoughts?

Thanks,
Severin


> On 11/26/20, 2:49 AM, "Severin Gehwolf" <sgehwolf at redhat.com> wrote:
> 
>     On Tue, 2020-11-24 at 23:00 +0000, Hohensee, Paul wrote:
>     > All files need a copyright update to 2019. Otherwise, lgtm.
> 
>     Thanks for the review!
> 
>     Note that neither the original change[1] nor the JDK 11u backport[2]
>     has the copyright year update. I'm not sure we should do that in the 8u
>     backport now. Thoughts?
> 
> 
>     Thanks,
>     Severin
> 
>     [1] http://hg.openjdk.java.net/jdk/jdk/rev/1c242c2d037f
>     [2] https://hg.openjdk.java.net/jdk-updates/jdk11u/rev/2173ddc0b886
> 
>     > On 11/4/20, 1:05 AM, "jdk8u-dev on behalf of Severin Gehwolf" <jdk8u-dev-retn at openjdk.java.net on behalf of sgehwolf at redhat.com> wrote:
>     >
>     >     Hi,
>     >
>     >     Please review this OpenJDK 8u port of JDK-8217338, which is a) an
>     >     Oracle JDK parity patch, b) a dependency of further container detection
>     >     backports like JDK-8232207 and JDK-8227006. The JDK 11 patch does not
>     >     apply cleanly. Changes I've done:
>     >
>     >      * Split of the patch for hotspot and jdk repositories. JDK 11u is a
>     >        unified repo.
>     >      * JDK 11u has UL, which is being translated to PrintContainerInfo
>     >        diagnostic flag in 8u.
>     >      * Some differences in context with respect to import statements in
>     >        Java code for Metrics
>     >
>     >     Otherwise the patch is the same. We've had this patch JDK 11 since
>     >     11.0.7 and hasn't caused any problems as far as I'm aware. As mentioned
>     >     previously, it makes getting further fixes (JDK-8232207, JDK-8227006)
>     >     easier and less risky. Therefore, I'd like to get it included for
>     >     8u282. The risk for this backport should be acceptable given the
>     >     circumstances. With that said, it's Linux-only.
>     >
>     >     Bug: https://bugs.openjdk.java.net/browse/JDK-8217338
>     >     webrev: https://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8217338/jdk8/01/
>     >
>     >     Testing: Container tests on Linux x86_64 (cgroup v1). Manual test[1].
>     >
>     >     Thoughts?
>     >
>     >     Thanks,
>     >     Severin
>     >
>     >     [1] https://bugs.openjdk.java.net/browse/JDK-8217338?focusedCommentId=14378230&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14378230
>     >
>     >
> 
> 



More information about the jdk8u-dev mailing list