RFR: 8190353: [Testbug] test/hotspot/jtreg/gc/logging/TestPrintReferences.java can still fail
sangheon.kim
sangheon.kim at oracle.com
Mon Oct 30 17:47:23 UTC 2017
Hi Stefan,
Thank you for finding and fixing this bug.
On 10/30/2017 08:04 AM, Stefan Johansson wrote:
> Hi,
>
> Please review this fix for:
> https://bugs.openjdk.java.net/browse/JDK-8190353
>
> Webrev:
> http://cr.openjdk.java.net/~sjohanss/8190353/00/
>
> Summary:
> The fix for JDK-8188245 was incomplete, it removed the check that if
> the reported time is longer than the sum of the sub-phases everything
> is ok. The new code requires the sum of the sub-phases to be within a
> give tolerance from the reported time. This is true in many cases but
> we can't know for sure that we won't get a context switch or some
> other stall between two sub-phases that causes the reported total time
> to be longer, but the sub-phases to not include this extra time.
>
> Testing:
> Verified that the new verification method returns correct values for
> different input values.
Looks good.
Just minor nit: you can ignore or no webrev is needed.
140 // Because of this we need method to verify that our measurements
and calculations are valid.
-> Because of this_*,*_ we need _*a*_ method ...
Thanks,
Sangheon
>
> Thanks,
> Stefan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20171030/56df7beb/attachment.htm>
More information about the hotspot-gc-dev
mailing list