<div dir="ltr"><div dir="ltr">Hi Aleksey,<div><br></div><div>The backport looks good to me. </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Feb 22, 2019 at 1:48 AM Aleksey Shipilev <<a href="mailto:shade@redhat.com">shade@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Please review the backport to 11u.<br>
<br>
Original bug:<br>
  <a href="https://bugs.openjdk.java.net/browse/JDK-8214118" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8214118</a><br>
<br>
Original fix:<br>
  <a href="http://hg.openjdk.java.net/jdk/jdk/rev/c9325aa887da" rel="noreferrer" target="_blank">http://hg.openjdk.java.net/jdk/jdk/rev/c9325aa887da</a><br>
<br>
The patch does not apply cleanly to 11u, because there are minute differences in the code. Notably,<br>
there is no closed_archive_heap_ranges, and as I see from the code, string_ranges is the old name<br>
for it. It was renamed in JDK-8212995.<br></blockquote><div><br></div><div>Right.  The renaming happened when graphs of non-string objects were supported in the closed region.</div><div><br></div><div>Thanks,</div><div>Jiangli</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
I also see there is Oracle-closed backport to 11u-oracle, can we compare the patches with Oracle<br>
folks, maybe?<br>
<br>
11u webrev:<br>
  <a href="http://cr.openjdk.java.net/~shade/8214118/webrev.11u.01/" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~shade/8214118/webrev.11u.01/</a><br>
<br>
Testing: Linux x86_64 tier1<br>
<br>
Thanks,<br>
-Aleksey<br>
<br>
</blockquote></div></div>