RFR (XXS): 8033106 Wrong predicate for checking whether the correct amount of symbol table entries have been processed in G1

Stefan Karlsson stefan.karlsson at oracle.com
Wed Jan 29 16:25:08 UTC 2014


On 2014-01-29 17:09, Thomas Schatzl wrote:
> Hi all,
>
>    can I have reviews for the following tiny change?
>
> In the review for the 7u60 backport for 8027476 (stringtable and symbol
> table parallelization) Mikael found this issue (thanks!) during review:
> the sanity checking code after string table and symbol table scan used
> one predicate meant for the string table checking in the symbol table
> checks.
>
> This went unnoticed because the symbol table is scanned always when
> calling the method that starts string/symbol table scan, so the second
> part of the expression always holds.
>
> Also fixes the copyright date.
>
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8033106/webrev/

Looks good.

StefanK

>
> Bug:
> https://bugs.openjdk.java.net/browse/JDK-8033106
>
> Testing:
> jprt
>
> Thanks,
>    Thomas
>




More information about the hotspot-gc-dev mailing list