<div dir="ltr">Hi Per,<div><br></div><div>I noticed when working on this code that I always disliked the resize method naming: really what it is doing is calculating the next size for a given TLAB to be done when the tlab is full. I don't think we'd automatically want to rename the resize method but it is only called once currently so changing its name could be done here easily and any reader would understand that tlab.resize() does not resize; it merely calculates the next size.</div><div><br></div><div>Were you to consider it, I think calculate_next_size would seem to be more appropriate (and we could remove the comment in the first line of the method since now it is redundant with the name of the method).</div><div><br></div><div>Anyway, whatever you do decide, it looks good to me (not a reviewer though),</div><div>Jc</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Sep 14, 2018 at 4:53 AM 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">Thanks Erik!<br>
<br>
/Per<br>
<br>
On 09/14/2018 01:47 PM, Per Liden wrote:<br>
> Make ThreadLocalAllocBuffer::resize() public to allow a GC to resize <br>
> TLABs in parallel. With resize() public I propose that we remove <br>
> ThreadLocalAllocBuffer::resize_all_tlabs() and let <br>
> CollectedHeap::resize_all_tlabs() iterate over the threads, just like <br>
> CollectedHeap::ensure_parsability() does.<br>
> <br>
> Bug: <a href="https://bugs.openjdk.java.net/browse/JDK-8210753" rel="noreferrer" target="_blank">https://bugs.openjdk.java.net/browse/JDK-8210753</a><br>
> Webrev: <a href="http://cr.openjdk.java.net/~pliden/8210753/webrev.0" rel="noreferrer" target="_blank">http://cr.openjdk.java.net/~pliden/8210753/webrev.0</a><br>
> <br>
> /Per<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><br></div>Thanks,<div>Jc</div></div></div>