--- old/src/share/vm/gc/g1/dirtyCardQueue.cpp 2016-02-11 13:39:06.441431343 -0500 +++ new/src/share/vm/gc/g1/dirtyCardQueue.cpp 2016-02-11 13:39:05.721390515 -0500 @@ -166,8 +166,8 @@ void DirtyCardQueueSet::initialize(CardTableEntryClosure* cl, Monitor* cbl_mon, Mutex* fl_lock, - int process_completed_threshold, - int max_completed_queue, + size_t process_completed_threshold, + size_t max_completed_queue, Mutex* lock, DirtyCardQueueSet* fl_owner, bool init_free_ids) { @@ -207,11 +207,11 @@ } -BufferNode* DirtyCardQueueSet::get_completed_buffer(int stop_at) { +BufferNode* DirtyCardQueueSet::get_completed_buffer(size_t stop_at) { BufferNode* nd = NULL; MutexLockerEx x(_cbl_mon, Mutex::_no_safepoint_check_flag); - if ((int)_n_completed_buffers <= stop_at) { + if ((size_t)_n_completed_buffers <= stop_at) { _process_completed = false; return NULL; } @@ -221,8 +221,8 @@ _completed_buffers_head = nd->next(); if (_completed_buffers_head == NULL) _completed_buffers_tail = NULL; + assert(_n_completed_buffers > 0, "Invariant"); _n_completed_buffers--; - assert(_n_completed_buffers >= 0, "Invariant"); } DEBUG_ONLY(assert_completed_buffer_list_len_correct_locked()); return nd; @@ -252,7 +252,7 @@ bool DirtyCardQueueSet::apply_closure_to_completed_buffer(CardTableEntryClosure* cl, uint worker_i, - int stop_at, + size_t stop_at, bool during_pause) { assert(!during_pause || stop_at == 0, "Should not leave any completed buffers during a pause"); BufferNode* nd = get_completed_buffer(stop_at); @@ -330,7 +330,7 @@ // Iterate over all the threads, if we find a partial log add it to // the global list of logs. Temporarily turn off the limit on the number // of outstanding buffers. - int save_max_completed_queue = _max_completed_queue; + size_t save_max_completed_queue = _max_completed_queue; _max_completed_queue = max_jint; assert(SafepointSynchronize::is_at_safepoint(), "Must be at safepoint."); for (JavaThread* t = Threads::first(); t; t = t->next()) {