hotspot/src/share/vm/utilities/hashtable.cpp
author iklam
Wed, 02 Aug 2017 18:06:38 -0700
changeset 46746 ea379ebb9447
parent 46742 24ec8a039c90
permissions -rw-r--r--
8072061: Automatically determine optimal sizes for the CDS regions Summary: See new C++ class MetaspaceClosure. Reviewed-by: coleenp, jiangli, mseledtsov
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     1
/*
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
     2
 * Copyright (c) 2003, 2017, Oracle and/or its affiliates. All rights reserved.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     4
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
489c9b5090e2 Initial load
duke
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
489c9b5090e2 Initial load
duke
parents:
diff changeset
     7
 * published by the Free Software Foundation.
489c9b5090e2 Initial load
duke
parents:
diff changeset
     8
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
489c9b5090e2 Initial load
duke
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
489c9b5090e2 Initial load
duke
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
489c9b5090e2 Initial load
duke
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
489c9b5090e2 Initial load
duke
parents:
diff changeset
    13
 * accompanied this code).
489c9b5090e2 Initial load
duke
parents:
diff changeset
    14
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
489c9b5090e2 Initial load
duke
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
489c9b5090e2 Initial load
duke
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
489c9b5090e2 Initial load
duke
parents:
diff changeset
    18
 *
5547
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1623
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1623
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
f4b087cbb361 6941466: Oracle rebranding changes for Hotspot repositories
trims
parents: 1623
diff changeset
    21
 * questions.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    22
 *
489c9b5090e2 Initial load
duke
parents:
diff changeset
    23
 */
489c9b5090e2 Initial load
duke
parents:
diff changeset
    24
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    25
#include "precompiled.hpp"
13199
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
    26
#include "classfile/altHashing.hpp"
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
    27
#include "classfile/dictionary.hpp"
35498
392b50de06c6 8146401: Clean up oop.hpp: add inline directives and fix header files
goetz
parents: 34257
diff changeset
    28
#include "classfile/javaClasses.inline.hpp"
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
    29
#include "classfile/moduleEntry.hpp"
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
    30
#include "classfile/packageEntry.hpp"
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
    31
#include "classfile/placeholders.hpp"
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
    32
#include "classfile/protectionDomainCache.hpp"
24426
0a69c8cdfca9 8038654: Separate SymbolTable and StringTable code
gziemski
parents: 23187
diff changeset
    33
#include "classfile/stringTable.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    34
#include "memory/allocation.inline.hpp"
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
    35
#include "memory/filemap.hpp"
7397
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    36
#include "memory/resourceArea.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    37
#include "oops/oop.inline.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    38
#include "runtime/safepoint.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    39
#include "utilities/dtrace.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    40
#include "utilities/hashtable.hpp"
5b173b4ca846 6989984: Use standard include model for Hospot
stefank
parents: 5547
diff changeset
    41
#include "utilities/hashtable.inline.hpp"
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
    42
#include "utilities/numberSeq.hpp"
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    43
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    44
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    45
// This hashtable is implemented as an open hash table with a fixed number of buckets.
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    46
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    47
template <MEMFLAGS F> BasicHashtableEntry<F>* BasicHashtable<F>::new_entry_free_list() {
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    48
  BasicHashtableEntry<F>* entry = NULL;
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    49
  if (_free_list != NULL) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    50
    entry = _free_list;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    51
    _free_list = _free_list->next();
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    52
  }
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    53
  return entry;
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    54
}
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    55
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    56
// HashtableEntrys are allocated in blocks to reduce the space overhead.
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    57
template <MEMFLAGS F> BasicHashtableEntry<F>* BasicHashtable<F>::new_entry(unsigned int hashValue) {
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    58
  BasicHashtableEntry<F>* entry = new_entry_free_list();
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    59
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    60
  if (entry == NULL) {
1551
b431de37a22c 6770949: minor tweaks before 6655638
jrose
parents: 1
diff changeset
    61
    if (_first_free_entry + _entry_size >= _end_block) {
b431de37a22c 6770949: minor tweaks before 6655638
jrose
parents: 1
diff changeset
    62
      int block_size = MIN2(512, MAX2((int)_table_size / 2, (int)_number_of_entries));
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    63
      int len = _entry_size * block_size;
1551
b431de37a22c 6770949: minor tweaks before 6655638
jrose
parents: 1
diff changeset
    64
      len = 1 << log2_intptr(len); // round down to power of 2
b431de37a22c 6770949: minor tweaks before 6655638
jrose
parents: 1
diff changeset
    65
      assert(len >= _entry_size, "");
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
    66
      _first_free_entry = NEW_C_HEAP_ARRAY2(char, len, F, CURRENT_PC);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    67
      _end_block = _first_free_entry + len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    68
    }
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
    69
    entry = (BasicHashtableEntry<F>*)_first_free_entry;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    70
    _first_free_entry += _entry_size;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    71
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
    72
1551
b431de37a22c 6770949: minor tweaks before 6655638
jrose
parents: 1
diff changeset
    73
  assert(_entry_size % HeapWordSize == 0, "");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    74
  entry->set_hash(hashValue);
489c9b5090e2 Initial load
duke
parents:
diff changeset
    75
  return entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    76
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    77
489c9b5090e2 Initial load
duke
parents:
diff changeset
    78
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
    79
template <class T, MEMFLAGS F> HashtableEntry<T, F>* Hashtable<T, F>::new_entry(unsigned int hashValue, T obj) {
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
    80
  HashtableEntry<T, F>* entry;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    81
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
    82
  entry = (HashtableEntry<T, F>*)BasicHashtable<F>::new_entry(hashValue);
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
    83
  entry->set_literal(obj);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
    84
  return entry;
489c9b5090e2 Initial load
duke
parents:
diff changeset
    85
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
    86
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    87
// Version of hashtable entry allocation that allocates in the C heap directly.
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    88
// The allocator in blocks is preferable but doesn't have free semantics.
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    89
template <class T, MEMFLAGS F> HashtableEntry<T, F>* Hashtable<T, F>::allocate_new_entry(unsigned int hashValue, T obj) {
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    90
  HashtableEntry<T, F>* entry = (HashtableEntry<T, F>*) NEW_C_HEAP_ARRAY(char, this->entry_size(), F);
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    91
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    92
  entry->set_hash(hashValue);
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    93
  entry->set_literal(obj);
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    94
  entry->set_next(NULL);
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    95
  return entry;
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    96
}
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
    97
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    98
// Check to see if the hashtable is unbalanced.  The caller set a flag to
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    99
// rehash at the next safepoint.  If this bucket is 60 times greater than the
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   100
// expected average bucket length, it's an unbalanced hashtable.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   101
// This is somewhat an arbitrary heuristic but if one bucket gets to
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   102
// rehash_count which is currently 100, there's probably something wrong.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   103
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   104
template <class T, MEMFLAGS F> bool RehashableHashtable<T, F>::check_rehash_table(int count) {
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   105
  assert(this->table_size() != 0, "underflow");
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   106
  if (count > (((double)this->number_of_entries()/(double)this->table_size())*rehash_multiple)) {
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   107
    // Set a flag for the next safepoint, which should be at some guaranteed
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   108
    // safepoint interval.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   109
    return true;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   110
  }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   111
  return false;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   112
}
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   113
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   114
// Create a new table and using alternate hash code, populate the new table
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   115
// with the existing elements.   This can be used to change the hash code
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   116
// and could in the future change the size of the table.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   117
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   118
template <class T, MEMFLAGS F> void RehashableHashtable<T, F>::move_to(RehashableHashtable<T, F>* new_table) {
13199
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   119
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   120
  // Initialize the global seed for hashing.
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   121
  _seed = AltHashing::compute_seed();
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   122
  assert(seed() != 0, "shouldn't be zero");
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   123
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   124
  int saved_entry_count = this->number_of_entries();
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   125
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   126
  // Iterate through the table and create a new entry for the new table
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   127
  for (int i = 0; i < new_table->table_size(); ++i) {
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   128
    for (HashtableEntry<T, F>* p = this->bucket(i); p != NULL; ) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   129
      HashtableEntry<T, F>* next = p->next();
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   130
      T string = p->literal();
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   131
      // Use alternate hashing algorithm on the symbol in the first table
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13342
diff changeset
   132
      unsigned int hashValue = string->new_hash(seed());
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   133
      // Get a new index relative to the new table (can also change size)
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   134
      int index = new_table->hash_to_index(hashValue);
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   135
      p->set_hash(hashValue);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   136
      // Keep the shared bit in the Hashtable entry to indicate that this entry
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   137
      // can't be deleted.   The shared bit is the LSB in the _next field so
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   138
      // walking the hashtable past these entries requires
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   139
      // BasicHashtableEntry::make_ptr() call.
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   140
      bool keep_shared = p->is_shared();
13342
76a5de64aa62 7186278: Build error after CR#6995781 / 7151532 with GCC 4.7.0
andrew
parents: 13199
diff changeset
   141
      this->unlink_entry(p);
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   142
      new_table->add_entry(index, p);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   143
      if (keep_shared) {
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   144
        p->set_shared();
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   145
      }
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   146
      p = next;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   147
    }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   148
  }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   149
  // give the new table the free list as well
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   150
  new_table->copy_freelist(this);
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   151
  assert(new_table->number_of_entries() == saved_entry_count, "lost entry on dictionary copy?");
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   152
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   153
  // Destroy memory used by the buckets in the hashtable.  The memory
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   154
  // for the elements has been used in a new table and is not
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   155
  // destroyed.  The memory reuse will benefit resizing the SystemDictionary
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   156
  // to avoid a memory allocation spike at safepoint.
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   157
  BasicHashtable<F>::free_buckets();
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   158
}
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   159
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   160
template <MEMFLAGS F> void BasicHashtable<F>::free_buckets() {
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   161
  if (NULL != _buckets) {
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   162
    // Don't delete the buckets in the shared space.  They aren't
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   163
    // allocated by os::malloc
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   164
    if (!UseSharedSpaces ||
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   165
        !FileMapInfo::current_info()->is_in_shared_space(_buckets)) {
27880
afb974a04396 8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents: 26422
diff changeset
   166
       FREE_C_HEAP_ARRAY(HashtableBucket, _buckets);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   167
    }
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   168
    _buckets = NULL;
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   169
  }
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   170
}
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   171
45114
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   172
template <MEMFLAGS F> void BasicHashtable<F>::BucketUnlinkContext::free_entry(BasicHashtableEntry<F>* entry) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   173
  entry->set_next(_removed_head);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   174
  _removed_head = entry;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   175
  if (_removed_tail == NULL) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   176
    _removed_tail = entry;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   177
  }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   178
  _num_removed++;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   179
}
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   180
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   181
template <MEMFLAGS F> void BasicHashtable<F>::bulk_free_entries(BucketUnlinkContext* context) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   182
  if (context->_num_removed == 0) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   183
    assert(context->_removed_head == NULL && context->_removed_tail == NULL,
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   184
           "Zero entries in the unlink context, but elements linked from " PTR_FORMAT " to " PTR_FORMAT,
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   185
           p2i(context->_removed_head), p2i(context->_removed_tail));
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   186
    return;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   187
  }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   188
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   189
  // MT-safe add of the list of BasicHashTableEntrys from the context to the free list.
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   190
  BasicHashtableEntry<F>* current = _free_list;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   191
  while (true) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   192
    context->_removed_tail->set_next(current);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   193
    BasicHashtableEntry<F>* old = (BasicHashtableEntry<F>*)Atomic::cmpxchg_ptr(context->_removed_head, &_free_list, current);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   194
    if (old == current) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   195
      break;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   196
    }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   197
    current = old;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   198
  }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   199
  Atomic::add(-context->_num_removed, &_number_of_entries);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   200
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
// Copy the table to the shared space.
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   202
template <MEMFLAGS F> size_t BasicHashtable<F>::count_bytes_for_table() {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   203
  size_t bytes = 0;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   204
  bytes += sizeof(intptr_t); // len
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   206
  for (int i = 0; i < _table_size; ++i) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   207
    for (BasicHashtableEntry<F>** p = _buckets[i].entry_addr();
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   208
         *p != NULL;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   209
         p = (*p)->next_addr()) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   210
      bytes += entry_size();
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   211
    }
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   212
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   214
  return bytes;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   215
}
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   216
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   217
// Dump the hash table entries (into CDS archive)
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   218
template <MEMFLAGS F> void BasicHashtable<F>::copy_table(char* top, char* end) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   219
  assert(is_aligned(top, sizeof(intptr_t)), "bad alignment");
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   220
  intptr_t *plen = (intptr_t*)(top);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   221
  top += sizeof(*plen);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
489c9b5090e2 Initial load
duke
parents:
diff changeset
   223
  int i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   224
  for (i = 0; i < _table_size; ++i) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   225
    for (BasicHashtableEntry<F>** p = _buckets[i].entry_addr();
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   226
         *p != NULL;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   227
         p = (*p)->next_addr()) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   228
      *p = (BasicHashtableEntry<F>*)memcpy(top, (void*)*p, entry_size());
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   229
      top += entry_size();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   230
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   231
  }
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   232
  *plen = (char*)(top) - (char*)plen - sizeof(*plen);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   233
  assert(top == end, "count_bytes_for_table is wrong");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   234
  // Set the shared bit.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   235
489c9b5090e2 Initial load
duke
parents:
diff changeset
   236
  for (i = 0; i < _table_size; ++i) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   237
    for (BasicHashtableEntry<F>* p = bucket(i); p != NULL; p = p->next()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   238
      p->set_shared();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   239
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   240
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   241
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   242
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   243
// For oops and Strings the size of the literal is interesting. For other types, nobody cares.
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   244
static int literal_size(ConstantPool*) { return 0; }
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   245
static int literal_size(Klass*)        { return 0; }
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   246
#if INCLUDE_ALL_GCS
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   247
static int literal_size(nmethod*)      { return 0; }
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   248
#endif
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   249
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   250
static int literal_size(Symbol *symbol) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   251
  return symbol->size() * HeapWordSize;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   252
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   253
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   254
static int literal_size(oop obj) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   255
  // NOTE: this would over-count if (pre-JDK8) java_lang_Class::has_offset_field() is true,
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   256
  // and the String.value array is shared by several Strings. However, starting from JDK8,
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   257
  // the String.value array is not shared anymore.
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   258
  if (obj == NULL) {
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   259
    return 0;
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   260
  } else if (obj->klass() == SystemDictionary::String_klass()) {
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   261
    return (obj->size() + java_lang_String::value(obj)->size()) * HeapWordSize;
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   262
  } else {
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   263
    return obj->size();
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   264
  }
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   265
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   266
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   267
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   268
// Dump footprint and bucket length statistics
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   269
//
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   270
// Note: if you create a new subclass of Hashtable<MyNewType, F>, you will need to
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   271
// add a new function static int literal_size(MyNewType lit)
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   272
// because I can't get template <class T> int literal_size(T) to pick the specializations for Symbol and oop.
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   273
//
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   274
// The StringTable and SymbolTable dumping print how much footprint is used by the String and Symbol
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   275
// literals.
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   276
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   277
template <class T, MEMFLAGS F> void Hashtable<T, F>::print_table_statistics(outputStream* st,
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   278
                                                                            const char *table_name) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   279
  NumberSeq summary;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   280
  int literal_bytes = 0;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   281
  for (int i = 0; i < this->table_size(); ++i) {
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   282
    int count = 0;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   283
    for (HashtableEntry<T, F>* e = this->bucket(i);
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   284
         e != NULL; e = e->next()) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   285
      count++;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   286
      literal_bytes += literal_size(e->literal());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   287
    }
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   288
    summary.add((double)count);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   289
  }
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   290
  double num_buckets = summary.num();
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   291
  double num_entries = summary.sum();
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   292
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   293
  int bucket_bytes = (int)num_buckets * sizeof(HashtableBucket<F>);
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   294
  int entry_bytes  = (int)num_entries * sizeof(HashtableEntry<T, F>);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   295
  int total_bytes = literal_bytes +  bucket_bytes + entry_bytes;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   296
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   297
  int bucket_size  = (num_buckets <= 0) ? 0 : (bucket_bytes  / num_buckets);
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   298
  int entry_size   = (num_entries <= 0) ? 0 : (entry_bytes   / num_entries);
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   299
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   300
  st->print_cr("%s statistics:", table_name);
46742
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   301
  st->print_cr("Number of buckets       : %9d = %9d bytes, each %d", (int)num_buckets, bucket_bytes,  bucket_size);
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   302
  st->print_cr("Number of entries       : %9d = %9d bytes, each %d", (int)num_entries, entry_bytes,   entry_size);
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   303
  if (literal_bytes != 0) {
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   304
    double literal_avg = (num_entries <= 0) ? 0 : (literal_bytes / num_entries);
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   305
    st->print_cr("Number of literals      : %9d = %9d bytes, avg %7.3f", (int)num_entries, literal_bytes, literal_avg);
24ec8a039c90 8184994: Add Dictionary size logging and jcmd
coleenp
parents: 46729
diff changeset
   306
  }
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   307
  st->print_cr("Total footprint         : %9s = %9d bytes", "", total_bytes);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   308
  st->print_cr("Average bucket size     : %9.3f", summary.avg());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   309
  st->print_cr("Variance of bucket size : %9.3f", summary.variance());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   310
  st->print_cr("Std. dev. of bucket size: %9.3f", summary.sd());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   311
  st->print_cr("Maximum bucket size     : %9d", (int)summary.maximum());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   312
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   313
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   314
489c9b5090e2 Initial load
duke
parents:
diff changeset
   315
// Dump the hash table buckets.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   316
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   317
template <MEMFLAGS F> size_t BasicHashtable<F>::count_bytes_for_buckets() {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   318
  size_t bytes = 0;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   319
  bytes += sizeof(intptr_t); // len
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   320
  bytes += sizeof(intptr_t); // _number_of_entries
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   321
  bytes += _table_size * sizeof(HashtableBucket<F>); // the buckets
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   322
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   323
  return bytes;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   324
}
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   326
// Dump the buckets (into CDS archive)
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   327
template <MEMFLAGS F> void BasicHashtable<F>::copy_buckets(char* top, char* end) {
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   328
  assert(is_aligned(top, sizeof(intptr_t)), "bad alignment");
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   329
  intptr_t len = _table_size * sizeof(HashtableBucket<F>);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   330
  *(intptr_t*)(top) = len;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   331
  top += sizeof(intptr_t);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   332
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   333
  *(intptr_t*)(top) = _number_of_entries;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   334
  top += sizeof(intptr_t);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   335
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   336
  _buckets = (HashtableBucket<F>*)memcpy(top, (void*)_buckets, len);
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   337
  top += len;
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   338
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   339
  assert(top == end, "count_bytes_for_buckets is wrong");
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   340
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   341
489c9b5090e2 Initial load
duke
parents:
diff changeset
   342
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   343
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   344
template <class T, MEMFLAGS F> void Hashtable<T, F>::print() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   347
  for (int i = 0; i < BasicHashtable<F>::table_size(); i++) {
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   348
    HashtableEntry<T, F>* entry = bucket(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   349
    while(entry != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   350
      tty->print("%d : ", i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   351
      entry->literal()->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   352
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   353
      entry = entry->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   354
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   355
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   356
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   357
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   358
template <MEMFLAGS F>
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   359
template <class T> void BasicHashtable<F>::verify_table(const char* table_name) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   360
  int element_count = 0;
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   361
  int max_bucket_count = 0;
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   362
  int max_bucket_number = 0;
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   363
  for (int index = 0; index < table_size(); index++) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   364
    int bucket_count = 0;
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   365
    for (T* probe = (T*)bucket(index); probe != NULL; probe = probe->next()) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   366
      probe->verify();
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   367
      bucket_count++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   368
    }
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   369
    element_count += bucket_count;
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   370
    if (bucket_count > max_bucket_count) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   371
      max_bucket_count = bucket_count;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   372
      max_bucket_number = index;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   373
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   374
  }
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   375
  guarantee(number_of_entries() == element_count,
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   376
            "Verify of %s failed", table_name);
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   377
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   378
  // Log some statistics about the hashtable
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   379
  log_info(hashtables)("%s max bucket size %d bucket %d element count %d table size %d", table_name,
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   380
                       max_bucket_count, max_bucket_number, _number_of_entries, _table_size);
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   381
  if (_number_of_entries > 0 && log_is_enabled(Debug, hashtables)) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   382
    for (int index = 0; index < table_size(); index++) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   383
      int bucket_count = 0;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   384
      for (T* probe = (T*)bucket(index); probe != NULL; probe = probe->next()) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   385
        log_debug(hashtables)("bucket %d hash " INTPTR_FORMAT, index, (intptr_t)probe->hash());
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   386
        bucket_count++;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   387
      }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   388
      if (bucket_count > 0) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   389
        log_debug(hashtables)("bucket %d count %d", index, bucket_count);
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   390
      }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   391
    }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   392
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   393
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   394
#endif // PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   395
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   396
// Explicitly instantiate these types
26422
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   397
#if INCLUDE_ALL_GCS
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   398
template class Hashtable<nmethod*, mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   399
template class HashtableEntry<nmethod*, mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   400
template class BasicHashtable<mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   401
#endif
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13342
diff changeset
   402
template class Hashtable<ConstantPool*, mtClass>;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   403
template class RehashableHashtable<Symbol*, mtSymbol>;
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   404
template class RehashableHashtable<oopDesc*, mtSymbol>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   405
template class Hashtable<Symbol*, mtSymbol>;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13342
diff changeset
   406
template class Hashtable<Klass*, mtClass>;
34257
4be3504cc03b 8140802: Clean up and refactor of class loading code for CDS
iklam
parents: 30593
diff changeset
   407
template class Hashtable<InstanceKlass*, mtClass>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   408
template class Hashtable<oop, mtClass>;
46729
c62d2e8b2728 7133093: Improve system dictionary performance
coleenp
parents: 46630
diff changeset
   409
template class Hashtable<Symbol*, mtModule>;
20282
7f9cbdf89af2 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 17610
diff changeset
   410
#if defined(SOLARIS) || defined(CHECK_UNHANDLED_OOPS)
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   411
template class Hashtable<oop, mtSymbol>;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   412
template class RehashableHashtable<oop, mtSymbol>;
20282
7f9cbdf89af2 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 17610
diff changeset
   413
#endif // SOLARIS || CHECK_UNHANDLED_OOPS
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   414
template class Hashtable<oopDesc*, mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   415
template class Hashtable<Symbol*, mtClass>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   416
template class HashtableEntry<Symbol*, mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   417
template class HashtableEntry<Symbol*, mtClass>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   418
template class HashtableEntry<oop, mtSymbol>;
46746
ea379ebb9447 8072061: Automatically determine optimal sizes for the CDS regions
iklam
parents: 46742
diff changeset
   419
template class HashtableBucket<mtClass>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   420
template class BasicHashtableEntry<mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   421
template class BasicHashtableEntry<mtCode>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   422
template class BasicHashtable<mtClass>;
34257
4be3504cc03b 8140802: Clean up and refactor of class loading code for CDS
iklam
parents: 30593
diff changeset
   423
template class BasicHashtable<mtClassShared>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   424
template class BasicHashtable<mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   425
template class BasicHashtable<mtCode>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   426
template class BasicHashtable<mtInternal>;
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 36384
diff changeset
   427
template class BasicHashtable<mtModule>;
36384
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   428
#if INCLUDE_TRACE
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   429
template class Hashtable<Symbol*, mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   430
template class HashtableEntry<Symbol*, mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   431
template class BasicHashtable<mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   432
#endif
30593
69f942690128 8076284: Improve vectorization of parallel streams
kvn
parents: 27880
diff changeset
   433
template class BasicHashtable<mtCompiler>;
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   434
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   435
template void BasicHashtable<mtClass>::verify_table<DictionaryEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   436
template void BasicHashtable<mtModule>::verify_table<ModuleEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   437
template void BasicHashtable<mtModule>::verify_table<PackageEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   438
template void BasicHashtable<mtClass>::verify_table<ProtectionDomainCacheEntry>(char const*);
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   439
template void BasicHashtable<mtClass>::verify_table<PlaceholderEntry>(char const*);