RFR (M): 8077144: Concurrent mark initialization takes too long (original) (raw)
Kim Barrett kim.barrett at oracle.com
Sat Mar 26 01:48:47 UTC 2016
- Previous message (by thread): RFR (M): 8077144: Concurrent mark initialization takes too long
- Next message (by thread): RFR (M): 8077144: Concurrent mark initialization takes too long
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
On Mar 25, 2016, at 9:38 PM, Kim Barrett <kim.barrett at oracle.com> wrote:
On Mar 15, 2016, at 6:12 PM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
Hi Mikael, updated webrev at http://cr.openjdk.java.net/~tschatzl/8077144/webrev.3/ (full) http://cr.openjdk.java.net/~tschatzl/8077144/webrev.2to3/ (diff) which implements the suggested changes. src/share/vm/utilities/bitMap.hpp 147 static idxt sizeinwords(sizet sizeinbits) { I find the name of this function confusing in conjunction with the no-arg ordinary member function; I'm not keen on overloads with with very different semantics and usage. Clearer (to me) here might be something like calcsizeinwords.
The existing BitMap::word_align_up could be used instead of introducing this new function.
- Previous message (by thread): RFR (M): 8077144: Concurrent mark initialization takes too long
- Next message (by thread): RFR (M): 8077144: Concurrent mark initialization takes too long
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]