src/hotspot/share/jfr/recorder/checkpoint/types/jfrTypeSetUtils.cpp
author mgronlun
Thu, 24 Oct 2019 16:37:22 +0200
changeset 58786 7909763ad193
parent 58132 caa25ab47aca
child 58823 6a21dba79b81
permissions -rw-r--r--
8231081: TestMetadataRetention fails due to missing symbol id Reviewed-by: egahlin
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     1
/*
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
     2
 * Copyright (c) 2017, 2019, Oracle and/or its affiliates. All rights reserved.
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     4
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     7
 * published by the Free Software Foundation.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     8
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    13
 * accompanied this code).
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    14
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    18
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    21
 * questions.
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    22
 *
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    23
 */
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    24
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    25
#include "precompiled.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    26
#include "jfr/recorder/checkpoint/types/jfrTypeSetUtils.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    27
#include "oops/instanceKlass.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    28
#include "oops/oop.inline.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    29
#include "oops/symbol.hpp"
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    30
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    31
static JfrSymbolId::CStringEntry* bootstrap = NULL;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    32
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    33
JfrSymbolId::JfrSymbolId() :
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    34
  _sym_table(new SymbolTable(this)),
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    35
  _cstring_table(new CStringTable(this)),
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    36
  _sym_list(NULL),
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    37
  _cstring_list(NULL),
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    38
  _sym_query(NULL),
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    39
  _cstring_query(NULL),
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    40
  _symbol_id_counter(1),
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    41
  _class_unload(false) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    42
  assert(_sym_table != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    43
  assert(_cstring_table != NULL, "invariant");
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    44
  bootstrap = new CStringEntry(0, (const char*)&BOOTSTRAP_LOADER_NAME);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    45
  assert(bootstrap != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    46
  bootstrap->set_id(1);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    47
  _cstring_list = bootstrap;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    48
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    49
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    50
JfrSymbolId::~JfrSymbolId() {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    51
  clear();
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    52
  delete _sym_table;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    53
  delete _cstring_table;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    54
  delete bootstrap;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    55
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    56
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    57
void JfrSymbolId::clear() {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    58
  assert(_sym_table != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    59
  if (_sym_table->has_entries()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    60
    _sym_table->clear_entries();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    61
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    62
  assert(!_sym_table->has_entries(), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    63
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    64
  assert(_cstring_table != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    65
  if (_cstring_table->has_entries()) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    66
    _cstring_table->clear_entries();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    67
  }
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    68
  assert(!_cstring_table->has_entries(), "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    69
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    70
  _sym_list = NULL;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    71
  _symbol_id_counter = 1;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    72
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    73
  _sym_query = NULL;
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    74
  _cstring_query = NULL;
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    75
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    76
  assert(bootstrap != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    77
  bootstrap->reset();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    78
  _cstring_list = bootstrap;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    79
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    80
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    81
void JfrSymbolId::set_class_unload(bool class_unload) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    82
  _class_unload = class_unload;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    83
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    84
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    85
void JfrSymbolId::on_link(const SymbolEntry* entry) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    86
  assert(entry != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    87
  const_cast<Symbol*>(entry->literal())->increment_refcount();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    88
  assert(entry->id() == 0, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    89
  entry->set_id(++_symbol_id_counter);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    90
  entry->set_list_next(_sym_list);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    91
  _sym_list = entry;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    92
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    93
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
    94
bool JfrSymbolId::on_equals(uintptr_t hash, const SymbolEntry* entry) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    95
  assert(entry != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    96
  assert(entry->hash() == hash, "invariant");
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    97
  assert(_sym_query != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
    98
  return _sym_query == entry->literal();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
    99
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   100
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   101
void JfrSymbolId::on_unlink(const SymbolEntry* entry) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   102
  assert(entry != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   103
  const_cast<Symbol*>(entry->literal())->decrement_refcount();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   104
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   105
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   106
static const char* resource_to_cstring(const char* resource_str) {
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   107
  assert(resource_str != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   108
  const size_t length = strlen(resource_str);
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   109
  char* const c_string = JfrCHeapObj::new_array<char>(length + 1);
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   110
  assert(c_string != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   111
  strncpy(c_string, resource_str, length + 1);
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   112
  return c_string;
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   113
}
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   114
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   115
void JfrSymbolId::on_link(const CStringEntry* entry) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   116
  assert(entry != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   117
  assert(entry->id() == 0, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   118
  entry->set_id(++_symbol_id_counter);
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   119
  const_cast<CStringEntry*>(entry)->set_literal(resource_to_cstring(entry->literal()));
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   120
  entry->set_list_next(_cstring_list);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   121
  _cstring_list = entry;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   122
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   123
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   124
static bool string_compare(const char* query, const char* candidate) {
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   125
  assert(query != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   126
  assert(candidate != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   127
  const size_t length = strlen(query);
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   128
  return strncmp(query, candidate, length) == 0;
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   129
}
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   130
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   131
bool JfrSymbolId::on_equals(uintptr_t hash, const CStringEntry* entry) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   132
  assert(entry != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   133
  assert(entry->hash() == hash, "invariant");
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   134
  assert(_cstring_query != NULL, "invariant");
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   135
  return string_compare(_cstring_query, entry->literal());
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   136
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   137
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   138
void JfrSymbolId::on_unlink(const CStringEntry* entry) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   139
  assert(entry != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   140
  JfrCHeapObj::free(const_cast<char*>(entry->literal()), strlen(entry->literal() + 1));
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   141
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   142
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   143
traceid JfrSymbolId::bootstrap_name(bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   144
  assert(bootstrap != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   145
  if (leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   146
    bootstrap->set_leakp();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   147
  }
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   148
  return 1;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   149
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   150
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   151
traceid JfrSymbolId::mark(const Symbol* symbol, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   152
  assert(symbol != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   153
  return mark((uintptr_t)symbol->identity_hash(), symbol, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   154
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   155
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   156
traceid JfrSymbolId::mark(uintptr_t hash, const Symbol* data, bool leakp) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   157
  assert(data != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   158
  assert(_sym_table != NULL, "invariant");
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   159
  _sym_query = data;
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   160
  const SymbolEntry& entry = _sym_table->lookup_put(hash, data);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   161
  if (_class_unload) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   162
    entry.set_unloading();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   163
  }
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   164
  if (leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   165
    entry.set_leakp();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   166
  }
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   167
  return entry.id();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   168
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   169
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   170
traceid JfrSymbolId::mark(uintptr_t hash, const char* str, bool leakp) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   171
  assert(str != NULL, "invariant");
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   172
  assert(_cstring_table != NULL, "invariant");
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   173
  _cstring_query = str;
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   174
  const CStringEntry& entry = _cstring_table->lookup_put(hash, str);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   175
  if (_class_unload) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   176
    entry.set_unloading();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   177
  }
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   178
  if (leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   179
    entry.set_leakp();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   180
  }
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   181
  return entry.id();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   182
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   183
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   184
/*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   185
* jsr292 anonymous classes symbol is the external name +
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   186
* the identity_hashcode slash appended:
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   187
*   java.lang.invoke.LambdaForm$BMH/22626602
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   188
*
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   189
* caller needs ResourceMark
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   190
*/
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   191
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   192
uintptr_t JfrSymbolId::unsafe_anonymous_klass_name_hash(const InstanceKlass* ik) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   193
  assert(ik != NULL, "invariant");
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 51334
diff changeset
   194
  assert(ik->is_unsafe_anonymous(), "invariant");
53176
6af596144294 8215773: applications/kitchensink/Kitchensink.java crash with "assert(ZAddress::is_marked(addr)) failed: Should be marked"
eosterlund
parents: 51444
diff changeset
   195
  const oop mirror = ik->java_mirror_no_keepalive();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   196
  assert(mirror != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   197
  return (uintptr_t)mirror->identity_hash();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   198
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   199
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   200
static const char* create_unsafe_anonymous_klass_symbol(const InstanceKlass* ik, uintptr_t hash) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   201
  assert(ik != NULL, "invariant");
51444
3e5d28e6de32 8209301: JVM rename is_anonymous, host_klass to unsafe specific terminology ahead of Unsafe.defineAnonymousClass deprecation
lfoltan
parents: 51334
diff changeset
   202
  assert(ik->is_unsafe_anonymous(), "invariant");
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   203
  assert(hash != 0, "invariant");
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   204
  char* anonymous_symbol = NULL;
53176
6af596144294 8215773: applications/kitchensink/Kitchensink.java crash with "assert(ZAddress::is_marked(addr)) failed: Should be marked"
eosterlund
parents: 51444
diff changeset
   205
  const oop mirror = ik->java_mirror_no_keepalive();
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   206
  assert(mirror != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   207
  char hash_buf[40];
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   208
  sprintf(hash_buf, "/" UINTX_FORMAT, hash);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   209
  const size_t hash_len = strlen(hash_buf);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   210
  const size_t result_len = ik->name()->utf8_length();
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   211
  anonymous_symbol = NEW_RESOURCE_ARRAY(char, result_len + hash_len + 1);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   212
  ik->name()->as_klass_external_name(anonymous_symbol, (int)result_len + 1);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   213
  assert(strlen(anonymous_symbol) == result_len, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   214
  strcpy(anonymous_symbol + result_len, hash_buf);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   215
  assert(strlen(anonymous_symbol) == result_len + hash_len, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   216
  return anonymous_symbol;
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   217
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   218
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   219
bool JfrSymbolId::is_unsafe_anonymous_klass(const Klass* k) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   220
  assert(k != NULL, "invariant");
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   221
  return k->is_instance_klass() && ((const InstanceKlass*)k)->is_unsafe_anonymous();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   222
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   223
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   224
traceid JfrSymbolId::mark_unsafe_anonymous_klass_name(const InstanceKlass* ik, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   225
  assert(ik != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   226
  assert(ik->is_unsafe_anonymous(), "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   227
  const uintptr_t hash = unsafe_anonymous_klass_name_hash(ik);
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   228
  const char* const anonymous_klass_symbol = create_unsafe_anonymous_klass_symbol(ik, hash);
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   229
  return mark(hash, anonymous_klass_symbol, leakp);
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   230
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   231
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   232
traceid JfrSymbolId::mark(const Klass* k, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   233
  assert(k != NULL, "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   234
  traceid symbol_id = 0;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   235
  if (is_unsafe_anonymous_klass(k)) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   236
    assert(k->is_instance_klass(), "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   237
    symbol_id = mark_unsafe_anonymous_klass_name((const InstanceKlass*)k, leakp);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   238
  }
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   239
  if (0 == symbol_id) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   240
    Symbol* const sym = k->name();
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   241
    if (sym != NULL) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   242
      symbol_id = mark(sym, leakp);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   243
    }
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   244
  }
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   245
  assert(symbol_id > 0, "a symbol handler must mark the symbol for writing");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   246
  return symbol_id;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   247
}
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   248
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   249
JfrArtifactSet::JfrArtifactSet(bool class_unload) : _symbol_id(new JfrSymbolId()),
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   250
                                                    _klass_list(NULL),
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   251
                                                    _total_count(0) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   252
  initialize(class_unload);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   253
  assert(_klass_list != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   254
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   255
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   256
static const size_t initial_class_list_size = 200;
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   257
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   258
void JfrArtifactSet::initialize(bool class_unload, bool clear /* false */) {
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   259
  assert(_symbol_id != NULL, "invariant");
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   260
  if (clear) {
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   261
    _symbol_id->clear();
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   262
  }
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   263
  _symbol_id->set_class_unload(class_unload);
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   264
  _total_count = 0;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   265
  // resource allocation
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   266
  _klass_list = new GrowableArray<const Klass*>(initial_class_list_size, false, mtTracing);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   267
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   268
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   269
JfrArtifactSet::~JfrArtifactSet() {
58786
7909763ad193 8231081: TestMetadataRetention fails due to missing symbol id
mgronlun
parents: 58132
diff changeset
   270
  _symbol_id->clear();
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   271
  delete _symbol_id;
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   272
  // _klass_list will be cleared by a ResourceMark
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   273
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   274
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   275
traceid JfrArtifactSet::bootstrap_name(bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   276
  return _symbol_id->bootstrap_name(leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   277
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   278
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   279
traceid JfrArtifactSet::mark_unsafe_anonymous_klass_name(const Klass* klass, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   280
  assert(klass->is_instance_klass(), "invariant");
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   281
  return _symbol_id->mark_unsafe_anonymous_klass_name((const InstanceKlass*)klass, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   282
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   283
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   284
traceid JfrArtifactSet::mark(uintptr_t hash, const Symbol* sym, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   285
  return _symbol_id->mark(hash, sym, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   286
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   287
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   288
traceid JfrArtifactSet::mark(const Klass* klass, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   289
  return _symbol_id->mark(klass, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   290
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   291
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   292
traceid JfrArtifactSet::mark(const Symbol* symbol, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   293
  return _symbol_id->mark(symbol, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   294
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   295
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   296
traceid JfrArtifactSet::mark(uintptr_t hash, const char* const str, bool leakp) {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   297
  return _symbol_id->mark(hash, str, leakp);
50113
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   298
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   299
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   300
bool JfrArtifactSet::has_klass_entries() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   301
  return _klass_list->is_nonempty();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   302
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   303
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   304
int JfrArtifactSet::entries() const {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   305
  return _klass_list->length();
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   306
}
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   307
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   308
void JfrArtifactSet::register_klass(const Klass* k) {
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   309
  assert(k != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   310
  assert(_klass_list != NULL, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   311
  assert(_klass_list->find(k) == -1, "invariant");
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   312
  _klass_list->append(k);
caf115bb98ad 8199712: Flight Recorder
egahlin
parents:
diff changeset
   313
}
58132
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   314
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   315
size_t JfrArtifactSet::total_count() const {
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   316
  return _total_count;
caa25ab47aca 8225797: OldObjectSample event creates unexpected amount of checkpoint data
mgronlun
parents: 53176
diff changeset
   317
}