RFR (XS) 8181450: assert in BasicHashtable::verify_table

coleen.phillimore at oracle.com coleen.phillimore at oracle.com
Fri Jun 16 14:02:22 UTC 2017


Thank you for the code review.  I had already started the push job as a 
semi-trivial change.  Thank you for the consulting!
Coleen


On 6/16/17 9:49 AM, Daniel D. Daugherty wrote:
> On 6/15/17 6:21 PM, coleen.phillimore at oracle.com wrote:
>> Summary: remove assert as it has small probability of happening and 
>> add some logging
>>
>> Tested new logging with -Xlog:hashtables with failed tests. See bug 
>> report for more information.
>>
>> open webrev at http://cr.openjdk.java.net/~coleenp/8181450.01/webrev
>
> src/share/vm/classfile/placeholders.cpp
>     No comments.
>
> src/share/vm/utilities/hashtable.cpp
>     No comments.
>
> src/share/vm/utilities/hashtable.hpp
>     No comments.
>
> Thumbs up! This has been a very interesting hunt!
>
> Dan
>
>
>> bug link https://bugs.openjdk.java.net/browse/JDK-8181450
>>
>> Thanks,
>> Coleen
>>
>



More information about the hotspot-runtime-dev mailing list