hotspot/src/share/vm/utilities/hashtable.cpp
author jwilhelm
Thu, 06 Jul 2017 01:50:26 +0200
changeset 46630 75aa3e39d02c
parent 46545 b970b6e40209
child 46729 c62d2e8b2728
permissions -rw-r--r--
8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8 8182656: Make the required changes in GC code to build on OSX 10 + Xcode 8 8182657: Make the required changes in Runtime code to build on OSX 10 + Xcode 8 8182658: Make the required changes in Compiler code to build on OSX 10 + Xcode 8 Reviewed-by: jwilhelm, ehelin, phh Contributed-by: phh <hohensee@amazon.com>, jwilhelm <jesper.wilhelmsson@oracle.com>
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
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    87
// 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
    88
// 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
    89
// expected average bucket length, it's an unbalanced hashtable.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    90
// This is somewhat an arbitrary heuristic but if one bucket gets to
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    91
// rehash_count which is currently 100, there's probably something wrong.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    92
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    93
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
    94
  assert(this->table_size() != 0, "underflow");
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
    95
  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
    96
    // 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
    97
    // safepoint interval.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    98
    return true;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
    99
  }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   100
  return false;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   101
}
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   102
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   103
// 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
   104
// 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
   105
// and could in the future change the size of the table.
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   106
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   107
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
   108
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   109
  // Initialize the global seed for hashing.
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   110
  _seed = AltHashing::compute_seed();
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   111
  assert(seed() != 0, "shouldn't be zero");
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   112
025b0984feea 7181200: JVM new hashing code breaks SA in product mode
coleenp
parents: 13195
diff changeset
   113
  int saved_entry_count = this->number_of_entries();
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   114
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   115
  // 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
   116
  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
   117
    for (HashtableEntry<T, F>* p = this->bucket(i); p != NULL; ) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   118
      HashtableEntry<T, F>* next = p->next();
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   119
      T string = p->literal();
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   120
      // 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
   121
      unsigned int hashValue = string->new_hash(seed());
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   122
      // 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
   123
      int index = new_table->hash_to_index(hashValue);
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   124
      p->set_hash(hashValue);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   125
      // 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
   126
      // 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
   127
      // walking the hashtable past these entries requires
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   128
      // BasicHashtableEntry::make_ptr() call.
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   129
      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
   130
      this->unlink_entry(p);
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   131
      new_table->add_entry(index, p);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   132
      if (keep_shared) {
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   133
        p->set_shared();
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   134
      }
13087
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   135
      p = next;
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   136
    }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   137
  }
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   138
  // give the new table the free list as well
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   139
  new_table->copy_freelist(this);
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   140
  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
   141
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   142
  // Destroy memory used by the buckets in the hashtable.  The memory
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   143
  // 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
   144
  // destroyed.  The memory reuse will benefit resizing the SystemDictionary
673ea6efaf18 7158800: Improve storage of symbol tables
coleenp
parents: 10739
diff changeset
   145
  // to avoid a memory allocation spike at safepoint.
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   146
  BasicHashtable<F>::free_buckets();
13087
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
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   149
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
   150
  if (NULL != _buckets) {
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   151
    // 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
   152
    // allocated by os::malloc
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   153
    if (!UseSharedSpaces ||
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   154
        !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
   155
       FREE_C_HEAP_ARRAY(HashtableBucket, _buckets);
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   156
    }
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   157
    _buckets = NULL;
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   158
  }
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   159
}
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   160
45114
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   161
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
   162
  entry->set_next(_removed_head);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   163
  _removed_head = entry;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   164
  if (_removed_tail == NULL) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   165
    _removed_tail = entry;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   166
  }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   167
  _num_removed++;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   168
}
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   169
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   170
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
   171
  if (context->_num_removed == 0) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   172
    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
   173
           "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
   174
           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
   175
    return;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   176
  }
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
  // 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
   179
  BasicHashtableEntry<F>* current = _free_list;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   180
  while (true) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   181
    context->_removed_tail->set_next(current);
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   182
    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
   183
    if (old == current) {
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   184
      break;
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   185
    }
45644c5f6b8e 8180048: Interned string and symbol table leak memory during parallel unlinking
tschatzl
parents: 42073
diff changeset
   186
    current = old;
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
  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
   189
}
13097
c146b608d91f 7178670: runtime/7158800/BadUtf8.java fails in SymbolTable::rehash_table
coleenp
parents: 13087
diff changeset
   190
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   191
// Copy the table to the shared space.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   192
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   193
template <MEMFLAGS F> void BasicHashtable<F>::copy_table(char** top, char* end) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   194
489c9b5090e2 Initial load
duke
parents:
diff changeset
   195
  // Dump the hash table entries.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   196
489c9b5090e2 Initial load
duke
parents:
diff changeset
   197
  intptr_t *plen = (intptr_t*)(*top);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   198
  *top += sizeof(*plen);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   199
489c9b5090e2 Initial load
duke
parents:
diff changeset
   200
  int i;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   201
  for (i = 0; i < _table_size; ++i) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   202
    for (BasicHashtableEntry<F>** p = _buckets[i].entry_addr();
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   203
                              *p != NULL;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   204
                               p = (*p)->next_addr()) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   205
      if (*top + entry_size() > end) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   206
        report_out_of_shared_space(SharedMiscData);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   207
      }
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46545
diff changeset
   208
      *p = (BasicHashtableEntry<F>*)memcpy(*top, (void*)*p, entry_size());
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   209
      *top += entry_size();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   210
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   211
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   212
  *plen = (char*)(*top) - (char*)plen - sizeof(*plen);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   213
489c9b5090e2 Initial load
duke
parents:
diff changeset
   214
  // Set the shared bit.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   215
489c9b5090e2 Initial load
duke
parents:
diff changeset
   216
  for (i = 0; i < _table_size; ++i) {
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   217
    for (BasicHashtableEntry<F>* p = bucket(i); p != NULL; p = p->next()) {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   218
      p->set_shared();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   219
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   220
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   221
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   222
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   223
template <class T, MEMFLAGS F> int RehashableHashtable<T, F>::literal_size(Symbol *symbol) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   224
  return symbol->size() * HeapWordSize;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   225
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   226
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   227
template <class T, MEMFLAGS F> int RehashableHashtable<T, F>::literal_size(oop oop) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   228
  // 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
   229
  // 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
   230
  // the String.value array is not shared anymore.
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   231
  assert(oop != NULL && oop->klass() == SystemDictionary::String_klass(), "only strings are supported");
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   232
  return (oop->size() + java_lang_String::value(oop)->size()) * HeapWordSize;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   233
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   234
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   235
// Dump footprint and bucket length statistics
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   236
//
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   237
// Note: if you create a new subclass of Hashtable<MyNewType, F>, you will need to
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   238
// add a new function Hashtable<T, F>::literal_size(MyNewType lit)
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   239
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   240
template <class T, MEMFLAGS F> void RehashableHashtable<T, F>::dump_table(outputStream* st, const char *table_name) {
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   241
  NumberSeq summary;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   242
  int literal_bytes = 0;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   243
  for (int i = 0; i < this->table_size(); ++i) {
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   244
    int count = 0;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   245
    for (HashtableEntry<T, F>* e = this->bucket(i);
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   246
       e != NULL; e = e->next()) {
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   247
      count++;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   248
      literal_bytes += literal_size(e->literal());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   249
    }
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   250
    summary.add((double)count);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   251
  }
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   252
  double num_buckets = summary.num();
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   253
  double num_entries = summary.sum();
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   254
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   255
  int bucket_bytes = (int)num_buckets * sizeof(HashtableBucket<F>);
17610
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   256
  int entry_bytes  = (int)num_entries * sizeof(HashtableEntry<T, F>);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   257
  int total_bytes = literal_bytes +  bucket_bytes + entry_bytes;
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   258
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   259
  double bucket_avg  = (num_buckets <= 0) ? 0 : (bucket_bytes  / num_buckets);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   260
  double entry_avg   = (num_entries <= 0) ? 0 : (entry_bytes   / num_entries);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   261
  double literal_avg = (num_entries <= 0) ? 0 : (literal_bytes / num_entries);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   262
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   263
  st->print_cr("%s statistics:", table_name);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   264
  st->print_cr("Number of buckets       : %9d = %9d bytes, avg %7.3f", (int)num_buckets, bucket_bytes,  bucket_avg);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   265
  st->print_cr("Number of entries       : %9d = %9d bytes, avg %7.3f", (int)num_entries, entry_bytes,   entry_avg);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   266
  st->print_cr("Number of literals      : %9d = %9d bytes, avg %7.3f", (int)num_entries, literal_bytes, literal_avg);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   267
  st->print_cr("Total footprint         : %9s = %9d bytes", "", total_bytes);
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   268
  st->print_cr("Average bucket size     : %9.3f", summary.avg());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   269
  st->print_cr("Variance of bucket size : %9.3f", summary.variance());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   270
  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
   271
  st->print_cr("Maximum bucket size     : %9d", (int)summary.maximum());
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   272
}
c6857feaac47 8014262: PrintStringTableStatistics should include more footprint info
iklam
parents: 13728
diff changeset
   273
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   274
489c9b5090e2 Initial load
duke
parents:
diff changeset
   275
// Dump the hash table buckets.
489c9b5090e2 Initial load
duke
parents:
diff changeset
   276
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   277
template <MEMFLAGS F> void BasicHashtable<F>::copy_buckets(char** top, char* end) {
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   278
  intptr_t len = _table_size * sizeof(HashtableBucket<F>);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   279
  *(intptr_t*)(*top) = len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   280
  *top += sizeof(intptr_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   281
489c9b5090e2 Initial load
duke
parents:
diff changeset
   282
  *(intptr_t*)(*top) = _number_of_entries;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   283
  *top += sizeof(intptr_t);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   284
489c9b5090e2 Initial load
duke
parents:
diff changeset
   285
  if (*top + len > end) {
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   286
    report_out_of_shared_space(SharedMiscData);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   287
  }
46630
75aa3e39d02c 8182299: Enable disabled clang warnings, build on OSX 10 + Xcode 8
jwilhelm
parents: 46545
diff changeset
   288
  _buckets = (HashtableBucket<F>*)memcpy(*top, (void*)_buckets, len);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   289
  *top += len;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   290
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   291
489c9b5090e2 Initial load
duke
parents:
diff changeset
   292
489c9b5090e2 Initial load
duke
parents:
diff changeset
   293
#ifndef PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   294
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   295
template <class T, MEMFLAGS F> void Hashtable<T, F>::print() {
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   296
  ResourceMark rm;
489c9b5090e2 Initial load
duke
parents:
diff changeset
   297
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   298
  for (int i = 0; i < BasicHashtable<F>::table_size(); i++) {
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   299
    HashtableEntry<T, F>* entry = bucket(i);
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   300
    while(entry != NULL) {
489c9b5090e2 Initial load
duke
parents:
diff changeset
   301
      tty->print("%d : ", i);
489c9b5090e2 Initial load
duke
parents:
diff changeset
   302
      entry->literal()->print();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   303
      tty->cr();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   304
      entry = entry->next();
489c9b5090e2 Initial load
duke
parents:
diff changeset
   305
    }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   306
  }
489c9b5090e2 Initial load
duke
parents:
diff changeset
   307
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   308
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   309
template <MEMFLAGS F>
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   310
template <class T> void BasicHashtable<F>::verify_table(const char* table_name) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   311
  int element_count = 0;
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   312
  int max_bucket_count = 0;
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   313
  int max_bucket_number = 0;
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   314
  for (int index = 0; index < table_size(); index++) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   315
    int bucket_count = 0;
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   316
    for (T* probe = (T*)bucket(index); probe != NULL; probe = probe->next()) {
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   317
      probe->verify();
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   318
      bucket_count++;
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   319
    }
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   320
    element_count += bucket_count;
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   321
    if (bucket_count > max_bucket_count) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   322
      max_bucket_count = bucket_count;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   323
      max_bucket_number = index;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   324
    }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   325
  }
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   326
  guarantee(number_of_entries() == element_count,
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   327
            "Verify of %s failed", table_name);
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   328
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   329
  // Log some statistics about the hashtable
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   330
  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
   331
                       max_bucket_count, max_bucket_number, _number_of_entries, _table_size);
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   332
  if (_number_of_entries > 0 && log_is_enabled(Debug, hashtables)) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   333
    for (int index = 0; index < table_size(); index++) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   334
      int bucket_count = 0;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   335
      for (T* probe = (T*)bucket(index); probe != NULL; probe = probe->next()) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   336
        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
   337
        bucket_count++;
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   338
      }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   339
      if (bucket_count > 0) {
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   340
        log_debug(hashtables)("bucket %d count %d", index, bucket_count);
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   341
      }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   342
    }
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   343
  }
1
489c9b5090e2 Initial load
duke
parents:
diff changeset
   344
}
489c9b5090e2 Initial load
duke
parents:
diff changeset
   345
#endif // PRODUCT
489c9b5090e2 Initial load
duke
parents:
diff changeset
   346
8076
96d498ec7ae1 6990754: Use native memory and reference counting to implement SymbolTable
coleenp
parents: 7397
diff changeset
   347
// Explicitly instantiate these types
26422
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   348
#if INCLUDE_ALL_GCS
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   349
template class Hashtable<nmethod*, mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   350
template class HashtableEntry<nmethod*, mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   351
template class BasicHashtable<mtGC>;
4ee5901e205e 8048268: G1 Code Root Migration performs poorly
mgerdin
parents: 26421
diff changeset
   352
#endif
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13342
diff changeset
   353
template class Hashtable<ConstantPool*, mtClass>;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   354
template class RehashableHashtable<Symbol*, mtSymbol>;
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   355
template class RehashableHashtable<oopDesc*, mtSymbol>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   356
template class Hashtable<Symbol*, mtSymbol>;
13728
882756847a04 6964458: Reimplement class meta-data storage to use native memory
coleenp
parents: 13342
diff changeset
   357
template class Hashtable<Klass*, mtClass>;
34257
4be3504cc03b 8140802: Clean up and refactor of class loading code for CDS
iklam
parents: 30593
diff changeset
   358
template class Hashtable<InstanceKlass*, mtClass>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   359
template class Hashtable<oop, mtClass>;
20282
7f9cbdf89af2 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 17610
diff changeset
   360
#if defined(SOLARIS) || defined(CHECK_UNHANDLED_OOPS)
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   361
template class Hashtable<oop, mtSymbol>;
26421
37d88e604ad0 8056084: Refactor Hashtable to allow implementations without rehashing support
mgerdin
parents: 24426
diff changeset
   362
template class RehashableHashtable<oop, mtSymbol>;
20282
7f9cbdf89af2 7195622: CheckUnhandledOops has limited usefulness now
hseigel
parents: 17610
diff changeset
   363
#endif // SOLARIS || CHECK_UNHANDLED_OOPS
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   364
template class Hashtable<oopDesc*, mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   365
template class Hashtable<Symbol*, mtClass>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   366
template class HashtableEntry<Symbol*, mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   367
template class HashtableEntry<Symbol*, mtClass>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   368
template class HashtableEntry<oop, mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   369
template class BasicHashtableEntry<mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   370
template class BasicHashtableEntry<mtCode>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   371
template class BasicHashtable<mtClass>;
34257
4be3504cc03b 8140802: Clean up and refactor of class loading code for CDS
iklam
parents: 30593
diff changeset
   372
template class BasicHashtable<mtClassShared>;
13195
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   373
template class BasicHashtable<mtSymbol>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   374
template class BasicHashtable<mtCode>;
be27e1b6a4b9 6995781: Native Memory Tracking (Phase 1)
zgu
parents: 13097
diff changeset
   375
template class BasicHashtable<mtInternal>;
38733
2b65f4db449e 8156156: Add module specific NMT MemoryType
hseigel
parents: 36384
diff changeset
   376
template class BasicHashtable<mtModule>;
36384
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   377
#if INCLUDE_TRACE
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   378
template class Hashtable<Symbol*, mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   379
template class HashtableEntry<Symbol*, mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   380
template class BasicHashtable<mtTracing>;
b0b41336a9a8 8143226: Minor updates to Event Based tracing
mgronlun
parents: 35498
diff changeset
   381
#endif
30593
69f942690128 8076284: Improve vectorization of parallel streams
kvn
parents: 27880
diff changeset
   382
template class BasicHashtable<mtCompiler>;
46475
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   383
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   384
template void BasicHashtable<mtClass>::verify_table<DictionaryEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   385
template void BasicHashtable<mtModule>::verify_table<ModuleEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   386
template void BasicHashtable<mtModule>::verify_table<PackageEntry>(char const*);
75902cea18af 8166848: Performance bug: SystemDictionary - optimization
coleenp
parents: 46435
diff changeset
   387
template void BasicHashtable<mtClass>::verify_table<ProtectionDomainCacheEntry>(char const*);
46545
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   388
template void BasicHashtable<mtClass>::verify_table<PlaceholderEntry>(char const*);
b970b6e40209 8181450: assert in BasicHashtable::verify_table
coleenp
parents: 46488
diff changeset
   389