< prev index next >
src/share/vm/gc/g1/g1AllocRegion.cpp
Print this page
@@ -89,29 +89,28 @@
"post-condition");
return result;
}
size_t G1AllocRegion::retire(bool fill_up) {
- assert(_alloc_region != NULL, G1_ALLOC_REGION_MSG("not initialized properly"));
+ G1_ALLOC_REGION_ASSERT(_alloc_region != NULL, "not initialized properly");
size_t result = 0;
trace("retiring");
HeapRegion* alloc_region = _alloc_region;
if (alloc_region != _dummy_region) {
// We never have to check whether the active region is empty or not,
// and potentially free it if it is, given that it's guaranteed that
// it will never be empty.
- assert(!alloc_region->is_empty(),
- G1_ALLOC_REGION_MSG("the alloc region should never be empty"));
+ G1_ALLOC_REGION_ASSERT(!alloc_region->is_empty(),
+ "the alloc region should never be empty");
if (fill_up) {
result = fill_up_remaining_space(alloc_region, _bot_updates);
}
- assert(alloc_region->used() >= _used_bytes_before,
- G1_ALLOC_REGION_MSG("invariant"));
+ G1_ALLOC_REGION_ASSERT(alloc_region->used() >= _used_bytes_before, "invariant");
size_t allocated_bytes = alloc_region->used() - _used_bytes_before;
retire_region(alloc_region, allocated_bytes);
_used_bytes_before = 0;
_alloc_region = _dummy_region;
}
@@ -120,21 +119,21 @@
return result;
}
HeapWord* G1AllocRegion::new_alloc_region_and_allocate(size_t word_size,
bool force) {
- assert(_alloc_region == _dummy_region, G1_ALLOC_REGION_MSG("pre-condition"));
- assert(_used_bytes_before == 0, G1_ALLOC_REGION_MSG("pre-condition"));
+ G1_ALLOC_REGION_ASSERT(_alloc_region == _dummy_region, "pre-condition");
+ G1_ALLOC_REGION_ASSERT(_used_bytes_before == 0, "pre-condition");
trace("attempting region allocation");
HeapRegion* new_alloc_region = allocate_new_region(word_size, force);
if (new_alloc_region != NULL) {
new_alloc_region->reset_pre_dummy_top();
// Need to do this before the allocation
_used_bytes_before = new_alloc_region->used();
HeapWord* result = allocate(new_alloc_region, word_size, _bot_updates);
- assert(result != NULL, G1_ALLOC_REGION_MSG("the allocation should succeeded"));
+ G1_ALLOC_REGION_ASSERT(result != NULL, "the allocation should succeeded");
OrderAccess::storestore();
// Note that we first perform the allocation and then we store the
// region in _alloc_region. This is the reason why an active region
// can never be empty.
@@ -148,27 +147,25 @@
ShouldNotReachHere();
}
void G1AllocRegion::init() {
trace("initializing");
- assert(_alloc_region == NULL && _used_bytes_before == 0,
- G1_ALLOC_REGION_MSG("pre-condition"));
- assert(_dummy_region != NULL, G1_ALLOC_REGION_MSG("should have been set"));
+ G1_ALLOC_REGION_ASSERT(_alloc_region == NULL && _used_bytes_before == 0, "pre-condition");
+ G1_ALLOC_REGION_ASSERT(_dummy_region != NULL, "should have been set");
_alloc_region = _dummy_region;
_count = 0;
trace("initialized");
}
void G1AllocRegion::set(HeapRegion* alloc_region) {
trace("setting");
// We explicitly check that the region is not empty to make sure we
// maintain the "the alloc region cannot be empty" invariant.
- assert(alloc_region != NULL && !alloc_region->is_empty(),
- G1_ALLOC_REGION_MSG("pre-condition"));
- assert(_alloc_region == _dummy_region &&
+ G1_ALLOC_REGION_ASSERT(alloc_region != NULL && !alloc_region->is_empty(), "pre-condition");
+ G1_ALLOC_REGION_ASSERT(_alloc_region == _dummy_region &&
_used_bytes_before == 0 && _count == 0,
- G1_ALLOC_REGION_MSG("pre-condition"));
+ "pre-condition");
_used_bytes_before = alloc_region->used();
_alloc_region = alloc_region;
_count += 1;
trace("set");
@@ -176,12 +173,11 @@
void G1AllocRegion::update_alloc_region(HeapRegion* alloc_region) {
trace("update");
// We explicitly check that the region is not empty to make sure we
// maintain the "the alloc region cannot be empty" invariant.
- assert(alloc_region != NULL && !alloc_region->is_empty(),
- G1_ALLOC_REGION_MSG("pre-condition"));
+ G1_ALLOC_REGION_ASSERT(alloc_region != NULL && !alloc_region->is_empty(), "pre-condition");
_alloc_region = alloc_region;
_alloc_region->set_allocation_context(allocation_context());
_count += 1;
trace("updated");
@@ -189,12 +185,11 @@
HeapRegion* G1AllocRegion::release() {
trace("releasing");
HeapRegion* alloc_region = _alloc_region;
retire(false /* fill_up */);
- assert(_alloc_region == _dummy_region,
- G1_ALLOC_REGION_MSG("post-condition of retire()"));
+ G1_ALLOC_REGION_ASSERT(_alloc_region == _dummy_region, "post-condition of retire()");
_alloc_region = NULL;
trace("released");
return (alloc_region == _dummy_region) ? NULL : alloc_region;
}
< prev index next >