RFR: 8333290: NMT report should not print Metaspace info if Metaspace is not yet initialized

[Thomas Stuefe]

On Fri, 31 May 2024 15:42:26 GMT, Stefan Karlsson <stefank at openjdk.org> wrote:

>> Somewhat trivial fix to a small initialization problem ubsan uncovered. 
>> 
>> If hs-err file gets dumped after NMT initialization but before Metaspace initialization, we may see secondary crashes during error log writing, since NMT reports contain metaspace counters, and those are not ready yet. We should omit that info if metaspace has not yet been initialized.
>> 
>> (See prior discussions in https://github.com/openjdk/jdk/pull/19435)
>
> src/hotspot/share/nmt/memReporter.cpp line 275:
> 
>> 273: 
>> 274: void MemSummaryReporter::report_metadata(Metaspace::MetadataType type) const {
>> 275: 
> 
> You have an extra blank line here, but maybe that is intentional?

Thanks Stefan! Newline was intentional, better spacing.

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/19493#discussion_r1622679457