RFR(S) 8225801: Shenandoah: Adjust SA to reflect recent forwarding pointer changes

2019-06-14 Thread Zhengyu Gu
Please review this patch, that adjusts SA to reflect removal of extra header for forwarding pointer. Bug: https://bugs.openjdk.java.net/browse/JDK-8225801 Webrev: http://cr.openjdk.java.net/~zgu/JDK-8225801/webrev.00/ Test: hotspot_gc_shenandoah Manual test. Thanks, -Zhengyu

Re: RFR(S) 8225801: Shenandoah: Adjust SA to reflect recent forwarding pointer changes

2019-06-14 Thread Aleksey Shipilev
On 6/14/19 5:50 PM, Zhengyu Gu wrote: > Please review this patch, that adjusts SA to reflect removal of extra header > for forwarding pointer. > > Bug: https://bugs.openjdk.java.net/browse/JDK-8225801 > Webrev: http://cr.openjdk.java.net/~zgu/JDK-8225801/webrev.00/ Looks good. This comment is o

Re: RFR(S) 8225801: Shenandoah: Adjust SA to reflect recent forwarding pointer changes

2019-06-14 Thread Zhengyu Gu
Removed the comment and pushed. Thanks! -Zhengyu On 6/14/19 11:55 AM, Aleksey Shipilev wrote: On 6/14/19 5:50 PM, Zhengyu Gu wrote: Please review this patch, that adjusts SA to reflect removal of extra header for forwarding pointer. Bug: https://bugs.openjdk.java.net/browse/JDK-8225801 Webr

Re: RFR(S) 8225801: Shenandoah: Adjust SA to reflect recent forwarding pointer changes

2019-06-14 Thread Roman Kennke
Oops, totally forgot about that. Should this also go into jdk13 as bugfix? Roman Am 14. Juni 2019 17:50:57 MESZ schrieb Zhengyu Gu : >Please review this patch, that adjusts SA to reflect removal of extra >header for forwarding pointer. > >Bug: https://bugs.openjdk.java.net/browse/JDK-8225801 >W

Re: RFR(S) 8225801: Shenandoah: Adjust SA to reflect recent forwarding pointer changes

2019-06-14 Thread Zhengyu Gu
On 6/14/19 12:58 PM, Roman Kennke wrote: Oops, totally forgot about that. Should this also go into jdk13 as bugfix? Yes, I think so. -Zhengyu Roman Am 14. Juni 2019 17:50:57 MESZ schrieb Zhengyu Gu : Please review this patch, that adjusts SA to reflect removal of extra header f