RFR (M): 8077144: Concurrent mark initialization takes too long

Kim Barrett kim.barrett at oracle.com
Sat Mar 26 01:48:47 UTC 2016


> 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.2_to_3/ (diff)
>> 
>> which implements the suggested changes.
> 
> src/share/vm/utilities/bitMap.hpp
> 147   static idx_t size_in_words(size_t size_in_bits) {
> 
> 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 calc_size_in_words.

The existing BitMap::word_align_up could be used instead of introducing this new function.



More information about the hotspot-gc-dev mailing list