<div dir="auto">Thanks!</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 10 Apr 2019, 11:54 Per Liden, <<a href="mailto:per.liden@oracle.com">per.liden@oracle.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Pushed<br>
<br>
/Per<br>
<br>
On 04/10/2019 11:22 AM, Erik Österlund wrote:<br>
> Hi Stuart,<br>
> <br>
> Good catch. Looks good.<br>
> <br>
> Thanks,<br>
> /Erik<br>
> <br>
> On 2019-04-09 11:52, Stuart Monteith wrote:<br>
>> Thanks Per.<br>
>><br>
>> I've opened JDK-8222180 and put the patch here:<br>
>>      <a href="http://cr.openjdk.java.net/~smonteith/8222180/webrev/" rel="noreferrer noreferrer" target="_blank">http://cr.openjdk.java.net/~smonteith/8222180/webrev/</a><br>
>><br>
>> It passes the gtests and runs fine with -XX:+ZVerifyForwarding enabled.<br>
>><br>
>> BR,<br>
>>     Stuart<br>
>><br>
>><br>
>> On Mon, 8 Apr 2019 at 21:18, Per Liden <<a href="mailto:per.liden@oracle.com" target="_blank" rel="noreferrer">per.liden@oracle.com</a>> wrote:<br>
>>> Hi Stuart,<br>
>>><br>
>>> On 04/08/2019 06:39 PM, Stuart Monteith wrote:<br>
>>> [...]<br>
>>>> This resolves the issue:<br>
>>>><br>
>>>> diff -r 6eb8c555644a src/hotspot/share/gc/z/zForwarding.cpp<br>
>>>> --- a/src/hotspot/share/gc/z/zForwarding.cpp    Mon Apr 08 09:44:49 <br>
>>>> 2019 +0100<br>
>>>> +++ b/src/hotspot/share/gc/z/zForwarding.cpp    Mon Apr 08 17:30:18 <br>
>>>> 2019 +0100<br>
>>>> @@ -69,6 +69,11 @@<br>
>>>>        // Check for duplicates<br>
>>>>        for (ZForwardingCursor j = i + 1; j < _entries.length(); j++) {<br>
>>>>          const ZForwardingEntry other = at(&j);<br>
>>>> +      if (!other.populated()) {<br>
>>>> +         // Skip empty entries<br>
>>>> +         continue;<br>
>>>> +       }<br>
>>>> +<br>
>>>>          guarantee(entry.from_index() != other.from_index(), <br>
>>>> "Duplicate from");<br>
>>>>          guarantee(entry.to_offset() != other.to_offset(), <br>
>>>> "Duplicate to");<br>
>>>>        }<br>
>>>><br>
>>>> Have you already encountered this, shall I create a bug+  patchset ?<br>
>>> Ah, yes. That looks like an oversight after JDK-8221540. Feel free to<br>
>>> create a bug and send a patch to hotspot-gc-dev. I'll sponsor it.<br>
>>><br>
>>> /Per<br>
> <br>
</blockquote></div>