RFR (xs) 8145940: TempNewSymbol should have correct copy and assignment functions
Coleen Phillimore
coleen.phillimore at oracle.com
Thu Jan 7 20:02:41 UTC 2016
Summary: Add clear() to the assignment operator and add copy constructor.
Ran all jtreg, colocated and non-colocated tests. RunThese -jck with
PrintSymbolTableSizeHistogram statistics:
Percent removed 1.35
Reference counts 194583
clean:
Percent removed 1.53
Reference counts 194245
Without a reference counting copy constructor, we could remove a
TempNewSymbol's Symbol if a GC happens. Consider:
TempNewSymbol ts = SymbolTable::new_symbol("abc");
// Hit GC
The ref count for "abc" is 1 when created by new_symbol, and the
destructor could run after the copy into ts, decrementing the reference
count to 0 again. GC could unlink that symbol from the symbol table.
Fortunately, we haven't seen this bug.
open webrev at http://cr.openjdk.java.net/~coleenp/8145940/
bug link https://bugs.openjdk.java.net/browse/JDK-8145940
Thanks,
Coleen
More information about the hotspot-runtime-dev
mailing list