RFR(S) 8212127: Cleanup TLAB fast refill statistics, perf counters and etc.
zgu at redhat.com
zgu at redhat.com
Fri Feb 1 20:25:58 UTC 2019
Hi Serguei,
Thanks for reviewing.
> On Fri, 2019-02-01 at 12:11 -0800, serguei.spitsyn at oracle.com wrote:
> > Hi Zhengyu,
> >
> >
> >
> > I do not have enough expertise in this area but it looks good
> > to
> > me.
> >
> >
> >
> >
> >
> > http://cr.openjdk.java.net/~zgu/JDK-8212127/webrev.00/src/hotspot/s
> > hare/gc/shared/threadLocalAllocBuffer.hpp.udiff.html
> >
> >
> >
> > Minor comment:
> >
> > void update_fast_allocations(unsigned int refills,
> > size_t allocations,
> > size_t gc_waste,
> > - size_t fast_refill_waste,
> > size_t slow_refill_waste);
> > The slow_refill_waste
> > needs to be renamed to refill_waste.
Fixed in place.
-Zhengyu
> >
> >
> >
> >
> > Thanks,
> >
> > Serguei
> >
> >
> >
> >
> >
> > On 2/1/19 10:27,
> > zgu at redhat.com wrote:
> >
>
>
> > After FastTLABRefill was removed, let's cleanup related perf
> > counters.
> > I removed perf counters for fast refill, and renamed perf counters
> > for
> > slow refill.
> >
> > I am not all clear what implication of renaming/removing exported
> > symbols, and have no idea what aliasmap is for. So I ran all tests
> > I
> > think could be affected, showed no problem.
> >
> >
> > Bug: https://bugs.openjdk.java.net/browse/JDK-8212127
> > Webrev: http://cr.openjdk.java.net/~zgu/JDK-8212127/webrev.00/
> >
> > Test:
> > hotspot_runtime, hotspot_serviceability,
> > vmTestbase_nsk_monitoring,
> > vmTestbase_nsk_jdi, vmTestbase_nsk_jvmti, vmTestbase_vm_jdwp
> > on Linux x64
> >
> > Eyeball output of jsnap
> >
> > Thanks,
> >
> > -Zhengyu
> >
> >
>
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20190201/84c8577f/attachment.html>
More information about the serviceability-dev
mailing list