[8u] RFR(XS): 8202600: [Zero] Undefined behaviour in src/os_cpu/linux_zero/vm/os_linux_zero.cpp

[Severin Gehwolf]

Hi,

On Thu, 2018-05-03 at 19:56 +0000, Thomas Stüfe wrote:
> Hi Severin,
> 
> Looks fine and trivial.

Thanks for the review!

Could I get another review from an JDK 8 Updates Project Reviewer,
please?

Thanks,
Severin

> Best regards, Thomas 
> 
> On Thu 3. May 2018 at 19:15, Severin Gehwolf <sgehwolf at redhat.com> wrote:
> > Hi,
> > 
> > Please review this trivial change which fixes a potential crasher bug
> > in JDK 8 Zero. JDK 10+ don't have this problem since they got fixed
> > with  JDK-8143245. I figured a smaller fix like this would be more
> > likely to get accepted as a backport. If you'd rather have JDK-8143245
> > backported to JDK 8, that would be fine with me too.
> > 
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8202600
> > webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8202600/webrev.jdk8.01/
> > 
> > Testing: We've been using this patch downstream for months now without
> > issues.
> > 
> > Thanks,
> > Severin