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