[11u] RFR: 8255908: ExceptionInInitializerError due to UncheckedIOException while initializing cgroupv1 subsystem

[Severin Gehwolf]

On Wed, 2020-12-23 at 06:29 +0000, Langer, Christoph wrote:
> Hi Severin,
> 
> I've checked your backport and I think you've hit all spots for
> catching UncheckedIOException in the 11u cgroup implementation. Looks
> good.

Thanks for the review, Christoph!

Cheers,
Severin

> > -----Original Message-----
> > From: jdk-updates-dev <jdk-updates-dev-retn at openjdk.java.net> On
> > Behalf Of Severin Gehwolf
> > Sent: Freitag, 18. Dezember 2020 11:08
> > To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
> > Subject: Re: [11u] RFR: 8255908: ExceptionInInitializerError due to
> > UncheckedIOException while initializing cgroupv1 subsystem
> > 
> > On Thu, 2020-12-10 at 14:07 +0100, Severin Gehwolf wrote:
> > > Hi!
> > > 
> > > Could I please get a review of this bugfix which improves error
> > > checking in the Java Metrics container detection code? The patch
> > > is
> > > basically a rewrite, since there is no cgroups v2 in JDK 11, and
> > > places
> > > where UncheckedIOException need to be handled are different
> > > enough
> > for
> > > the JDK head patch to not apply properly.
> > > 
> > > The gist is to handle UncheckedIOException, possibly thrown
> > > because of
> > > interrupts, and exit gracefully rather than failing to initialize
> > > and
> > > crash.
> > > 
> > > Bug: https://bugs.openjdk.java.net/browse/JDK-8255908
> > > webrev: https://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-
> > 8255908/01/webrev/
> > > 
> > > Testing: Container tests on x86_64 Linux (cgroups v1) and tier 1.
> > > 
> > > Thoughts?
> > 
> > Anyone?
> > 
> > Thanks,
> > Severin
>