diff -r 278795ad438a -r 02cffb476ab0 src/hotspot/share/gc/shared/oopStorage.cpp --- a/src/hotspot/share/gc/shared/oopStorage.cpp Fri Jul 26 20:15:13 2019 -0400 +++ b/src/hotspot/share/gc/shared/oopStorage.cpp Fri Jul 26 23:37:51 2019 -0400 @@ -418,7 +418,7 @@ assert(!block->is_full(), "invariant"); if (block->is_empty()) { // Transitioning from empty to not empty. - log_debug(oopstorage, blocks)("%s: block not empty " PTR_FORMAT, name(), p2i(block)); + log_trace(oopstorage, blocks)("%s: block not empty " PTR_FORMAT, name(), p2i(block)); } oop* result = block->allocate(); assert(result != NULL, "allocation failed"); @@ -427,7 +427,7 @@ if (block->is_full()) { // Transitioning from not full to full. // Remove full blocks from consideration by future allocates. - log_debug(oopstorage, blocks)("%s: block full " PTR_FORMAT, name(), p2i(block)); + log_trace(oopstorage, blocks)("%s: block full " PTR_FORMAT, name(), p2i(block)); _allocation_list.unlink(*block); } log_trace(oopstorage, ref)("%s: allocated " PTR_FORMAT, name(), p2i(result)); @@ -481,7 +481,7 @@ } else if (!reduce_deferred_updates()) { // Once more before failure. // Attempt to add a block failed, no other thread added a block, // and no deferred updated added a block, then allocation failed. - log_debug(oopstorage, blocks)("%s: failed block allocation", name()); + log_info(oopstorage, blocks)("%s: failed block allocation", name()); return NULL; } } @@ -573,13 +573,15 @@ uintx old_allocated, const OopStorage* owner, const void* block) { - Log(oopstorage, blocks) log; - LogStream ls(log.debug()); - if (is_full_bitmask(old_allocated)) { - ls.print_cr("%s: block not full " PTR_FORMAT, owner->name(), p2i(block)); - } - if (releasing == old_allocated) { - ls.print_cr("%s: block empty " PTR_FORMAT, owner->name(), p2i(block)); + LogTarget(Trace, oopstorage, blocks) lt; + if (lt.is_enabled()) { + LogStream ls(lt); + if (is_full_bitmask(old_allocated)) { + ls.print_cr("%s: block not full " PTR_FORMAT, owner->name(), p2i(block)); + } + if (releasing == old_allocated) { + ls.print_cr("%s: block empty " PTR_FORMAT, owner->name(), p2i(block)); + } } } @@ -606,9 +608,7 @@ // updates and the associated locking here. if ((releasing == old_allocated) || is_full_bitmask(old_allocated)) { // Log transitions. Both transitions are possible in a single update. - if (log_is_enabled(Debug, oopstorage, blocks)) { - log_release_transitions(releasing, old_allocated, owner, this); - } + log_release_transitions(releasing, old_allocated, owner, this); // Attempt to claim responsibility for adding this block to the deferred // list, by setting the link to non-NULL by self-looping. If this fails, // then someone else has made such a claim and the deferred update has not @@ -631,7 +631,7 @@ if (releasing == old_allocated) { owner->record_needs_cleanup(); } - log_debug(oopstorage, blocks)("%s: deferred update " PTR_FORMAT, + log_trace(oopstorage, blocks)("%s: deferred update " PTR_FORMAT, owner->name(), p2i(this)); } } @@ -681,7 +681,7 @@ _allocation_list.push_back(*block); } - log_debug(oopstorage, blocks)("%s: processed deferred update " PTR_FORMAT, + log_trace(oopstorage, blocks)("%s: processed deferred update " PTR_FORMAT, name(), p2i(block)); return true; // Processed one pending update. }