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