# HG changeset patch # User dcubed # Date 1511823081 18000 # Node ID 87b6295b3f4f95e7f0545b603dfd4a2e9e27fdde # Parent d851eb254409d8a6eaa52e6ddf0c836b171e5d56# Parent 3c0ef95d12e3ffcddc50780fcf08b11683bb87fe Merge diff -r d851eb254409 -r 87b6295b3f4f src/hotspot/share/gc/parallel/mutableNUMASpace.cpp --- a/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp Mon Nov 27 15:26:55 2017 -0500 +++ b/src/hotspot/share/gc/parallel/mutableNUMASpace.cpp Mon Nov 27 17:51:21 2017 -0500 @@ -91,14 +91,14 @@ MutableSpace *s = ls->space(); if (s->top() < top()) { // For all spaces preceding the one containing top() if (s->free_in_words() > 0) { - intptr_t cur_top = (intptr_t)s->top(); + HeapWord* cur_top = s->top(); size_t words_left_to_fill = pointer_delta(s->end(), s->top());; while (words_left_to_fill > 0) { size_t words_to_fill = MIN2(words_left_to_fill, CollectedHeap::filler_array_max_size()); assert(words_to_fill >= CollectedHeap::min_fill_size(), "Remaining size (" SIZE_FORMAT ") is too small to fill (based on " SIZE_FORMAT " and " SIZE_FORMAT ")", words_to_fill, words_left_to_fill, CollectedHeap::filler_array_max_size()); - CollectedHeap::fill_with_object((HeapWord*)cur_top, words_to_fill); + CollectedHeap::fill_with_object(cur_top, words_to_fill); if (!os::numa_has_static_binding()) { size_t touched_words = words_to_fill; #ifndef ASSERT @@ -108,19 +108,19 @@ } #endif MemRegion invalid; - HeapWord *crossing_start = align_up((HeapWord*)cur_top, os::vm_page_size()); - HeapWord *crossing_end = align_down((HeapWord*)(cur_top + touched_words), os::vm_page_size()); + HeapWord *crossing_start = align_up(cur_top, os::vm_page_size()); + HeapWord *crossing_end = align_down(cur_top + touched_words, os::vm_page_size()); if (crossing_start != crossing_end) { // If object header crossed a small page boundary we mark the area // as invalid rounding it to a page_size(). - HeapWord *start = MAX2(align_down((HeapWord*)cur_top, page_size()), s->bottom()); - HeapWord *end = MIN2(align_up((HeapWord*)(cur_top + touched_words), page_size()), s->end()); + HeapWord *start = MAX2(align_down(cur_top, page_size()), s->bottom()); + HeapWord *end = MIN2(align_up(cur_top + touched_words, page_size()), s->end()); invalid = MemRegion(start, end); } ls->add_invalid_region(invalid); } - cur_top = cur_top + (words_to_fill * HeapWordSize); + cur_top += words_to_fill; words_left_to_fill -= words_to_fill; } }