Request for review: 8016302: Change type of the number of GC workers to unsigned int (2)
per.liden at oracle.com
Thu Sep 26 00:41:37 PDT 2013
On 2013-09-25 12:47, Vladimir Kempik wrote:
> Thanks for comments, updated webrev -
> On 24.09.2013 17:22, Thomas Schatzl wrote:
>> On Thu, 2013-09-19 at 16:22 +0200, Vladimir Kempik wrote:
>>> Thanks for comments, here is updated webrev -
>> - the comments in DirtyCardQueueSet::mut_process_buffer() should be
>> adapted that we use uints now. They talk of worker_i of -1.
>> 130 // If worker_i is not -1 then the thread has already claimed
>> 131 // a par_id. We make note of it using the already_claimed value
>> - in G1GCPhaseTimes::print_stats() we need to use UINT32_FORMAT instead
>> of %d as the printf() specifier for uints.
>> - same in ScanRSClosure::printCard().
More information about the hotspot-gc-dev