RFR(S): 8139673: NMT stack traces in output should show mtcomponent
Zhengyu Gu
zgu at redhat.com
Tue May 23 16:05:07 UTC 2017
Hi Aleksey,
Thanks for the review. I fixed double spaces in new webrev:
http://cr.openjdk.java.net/~zgu/8139673/webrev.02/
I do need sponsor the this, and please also add Max Ockner (mockner) as
an author.
Thanks,
-Zhengyu
On 05/23/2017 11:05 AM, Aleksey Shipilev wrote:
> On 05/23/2017 03:33 PM, Zhengyu Gu wrote:
>> Webrev: http://cr.openjdk.java.net/~zgu/8139673/webrev.01/index.html
>
> Looks good.
>
> Minor nit: there are some double spaces, fix them before pushing?
>
> 71 _malloc_site(stack, flags), _next(NULL) {
>
> ...
>
> 81 bool ret = MallocSiteTable::allocation_at(stack, size, bucket_idx,
> pos_idx, flags);
>
> Thanks,
> -Aleksey
>
>
>
More information about the hotspot-runtime-dev
mailing list