src/hotspot/share/gc/g1/g1StringDedupTable.cpp
author tschatzl
Wed, 18 Apr 2018 11:36:48 +0200
changeset 49806 2d62570a615c
parent 49594 898ef81cbc0e
permissions -rw-r--r--
8200426: Make G1 code use _g1h members Summary: Consistently use _g1h member names for cached G1CollectedHeap* variables. Reviewed-by: sangheki, sjohanss
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     1
/*
49041
44122f767467 8198286: Direct memory accessors in typeArrayOop.hpp should use Access API
eosterlund
parents: 47679
diff changeset
     2
 * Copyright (c) 2014, 2018, Oracle and/or its affiliates. All rights reserved.
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     3
 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     4
 *
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     5
 * This code is free software; you can redistribute it and/or modify it
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     6
 * under the terms of the GNU General Public License version 2 only, as
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     7
 * published by the Free Software Foundation.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     8
 *
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
     9
 * This code is distributed in the hope that it will be useful, but WITHOUT
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    10
 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    11
 * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    12
 * version 2 for more details (a copy is included in the LICENSE file that
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    13
 * accompanied this code).
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    14
 *
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    15
 * You should have received a copy of the GNU General Public License version
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    16
 * 2 along with this work; if not, write to the Free Software Foundation,
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    17
 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    18
 *
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    19
 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    20
 * or visit www.oracle.com if you need additional information or have any
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    21
 * questions.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    22
 *
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    23
 */
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    24
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    25
#include "precompiled.hpp"
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    26
#include "classfile/altHashing.hpp"
29081
c61eb4914428 8072911: Remove includes of oop.inline.hpp from .hpp files
stefank
parents: 27880
diff changeset
    27
#include "classfile/javaClasses.inline.hpp"
49347
edb65305d3ac 8195148: Collapse G1SATBCardTableModRefBS and G1SATBCardTableLoggingModRefBS into a single G1BarrierSet
eosterlund
parents: 49041
diff changeset
    28
#include "gc/g1/g1BarrierSet.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30175
diff changeset
    29
#include "gc/g1/g1CollectedHeap.inline.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30175
diff changeset
    30
#include "gc/g1/g1StringDedup.hpp"
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30175
diff changeset
    31
#include "gc/g1/g1StringDedupTable.hpp"
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33628
diff changeset
    32
#include "logging/log.hpp"
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    33
#include "memory/padded.inline.hpp"
49041
44122f767467 8198286: Direct memory accessors in typeArrayOop.hpp should use Access API
eosterlund
parents: 47679
diff changeset
    34
#include "oops/arrayOop.inline.hpp"
30764
fec48bf5a827 8079792: GC directory structure cleanup
pliden
parents: 30175
diff changeset
    35
#include "oops/oop.inline.hpp"
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    36
#include "oops/typeArrayOop.hpp"
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    37
#include "runtime/mutexLocker.hpp"
49594
898ef81cbc0e 8200106: Move NoSafepointVerifier out from gcLocker.hpp
stefank
parents: 49347
diff changeset
    38
#include "runtime/safepointVerifiers.hpp"
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    39
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    40
//
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    41
// List of deduplication table entries. Links table
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    42
// entries together using their _next fields.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    43
//
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    44
class G1StringDedupEntryList : public CHeapObj<mtGC> {
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    45
private:
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    46
  G1StringDedupEntry* _list;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    47
  size_t              _length;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    48
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    49
public:
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    50
  G1StringDedupEntryList() :
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    51
    _list(NULL),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    52
    _length(0) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    53
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    54
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    55
  void add(G1StringDedupEntry* entry) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    56
    entry->set_next(_list);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    57
    _list = entry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    58
    _length++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    59
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    60
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    61
  G1StringDedupEntry* remove() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    62
    G1StringDedupEntry* entry = _list;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    63
    if (entry != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    64
      _list = entry->next();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    65
      _length--;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    66
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    67
    return entry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    68
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    69
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    70
  G1StringDedupEntry* remove_all() {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    71
    G1StringDedupEntry* list = _list;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    72
    _list = NULL;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    73
    return list;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    74
  }
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    75
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    76
  size_t length() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    77
    return _length;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    78
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    79
};
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    80
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    81
//
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    82
// Cache of deduplication table entries. This cache provides fast allocation and
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    83
// reuse of table entries to lower the pressure on the underlying allocator.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    84
// But more importantly, it provides fast/deferred freeing of table entries. This
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    85
// is important because freeing of table entries is done during stop-the-world
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    86
// phases and it is not uncommon for large number of entries to be freed at once.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    87
// Tables entries that are freed during these phases are placed onto a freelist in
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    88
// the cache. The deduplication thread, which executes in a concurrent phase, will
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    89
// later reuse or free the underlying memory for these entries.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    90
//
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    91
// The cache allows for single-threaded allocations and multi-threaded frees.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    92
// Allocations are synchronized by StringDedupTable_lock as part of a table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    93
// modification.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    94
//
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    95
class G1StringDedupEntryCache : public CHeapObj<mtGC> {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
    96
private:
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    97
  // One cache/overflow list per GC worker to allow lock less freeing of
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    98
  // entries while doing a parallel scan of the table. Using PaddedEnd to
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
    99
  // avoid false sharing.
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   100
  size_t                             _nlists;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   101
  size_t                             _max_list_length;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   102
  PaddedEnd<G1StringDedupEntryList>* _cached;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   103
  PaddedEnd<G1StringDedupEntryList>* _overflowed;
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   104
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   105
public:
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   106
  G1StringDedupEntryCache(size_t max_size);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   107
  ~G1StringDedupEntryCache();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   108
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   109
  // Set max number of table entries to cache.
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   110
  void set_max_size(size_t max_size);
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   111
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   112
  // Get a table entry from the cache, or allocate a new entry if the cache is empty.
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   113
  G1StringDedupEntry* alloc();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   114
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   115
  // Insert a table entry into the cache.
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   116
  void free(G1StringDedupEntry* entry, uint worker_id);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   117
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   118
  // Returns current number of entries in the cache.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   119
  size_t size();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   120
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   121
  // Deletes overflowed entries.
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   122
  void delete_overflowed();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   123
};
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   124
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   125
G1StringDedupEntryCache::G1StringDedupEntryCache(size_t max_size) :
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   126
  _nlists(ParallelGCThreads),
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   127
  _max_list_length(0),
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   128
  _cached(PaddedArray<G1StringDedupEntryList, mtGC>::create_unfreeable((uint)_nlists)),
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   129
  _overflowed(PaddedArray<G1StringDedupEntryList, mtGC>::create_unfreeable((uint)_nlists)) {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   130
  set_max_size(max_size);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   131
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   132
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   133
G1StringDedupEntryCache::~G1StringDedupEntryCache() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   134
  ShouldNotReachHere();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   135
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   136
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   137
void G1StringDedupEntryCache::set_max_size(size_t size) {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   138
  _max_list_length = size / _nlists;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   139
}
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   140
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   141
G1StringDedupEntry* G1StringDedupEntryCache::alloc() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   142
  for (size_t i = 0; i < _nlists; i++) {
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   143
    G1StringDedupEntry* entry = _cached[i].remove();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   144
    if (entry != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   145
      return entry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   146
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   147
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   148
  return new G1StringDedupEntry();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   149
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   150
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   151
void G1StringDedupEntryCache::free(G1StringDedupEntry* entry, uint worker_id) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   152
  assert(entry->obj() != NULL, "Double free");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   153
  assert(worker_id < _nlists, "Invalid worker id");
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   154
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   155
  entry->set_obj(NULL);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   156
  entry->set_hash(0);
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   157
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   158
  if (_cached[worker_id].length() < _max_list_length) {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   159
    // Cache is not full
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   160
    _cached[worker_id].add(entry);
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   161
  } else {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   162
    // Cache is full, add to overflow list for later deletion
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   163
    _overflowed[worker_id].add(entry);
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   164
  }
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   165
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   166
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   167
size_t G1StringDedupEntryCache::size() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   168
  size_t size = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   169
  for (size_t i = 0; i < _nlists; i++) {
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   170
    size += _cached[i].length();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   171
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   172
  return size;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   173
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   174
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   175
void G1StringDedupEntryCache::delete_overflowed() {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   176
  double start = os::elapsedTime();
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   177
  uintx count = 0;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   178
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   179
  for (size_t i = 0; i < _nlists; i++) {
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   180
    G1StringDedupEntry* entry;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   181
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   182
    {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   183
      // The overflow list can be modified during safepoints, therefore
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   184
      // we temporarily join the suspendible thread set while removing
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   185
      // all entries from the list.
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   186
      SuspendibleThreadSetJoiner sts_join;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   187
      entry = _overflowed[i].remove_all();
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   188
    }
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   189
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   190
    // Delete all entries
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   191
    while (entry != NULL) {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   192
      G1StringDedupEntry* next = entry->next();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   193
      delete entry;
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   194
      entry = next;
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   195
      count++;
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   196
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   197
  }
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   198
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   199
  double end = os::elapsedTime();
39690
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   200
  log_trace(gc, stringdedup)("Deleted " UINTX_FORMAT " entries, " G1_STRDEDUP_TIME_FORMAT_MS,
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   201
                             count, G1_STRDEDUP_TIME_PARAM_MS(end - start));
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   202
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   203
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   204
G1StringDedupTable*      G1StringDedupTable::_table = NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   205
G1StringDedupEntryCache* G1StringDedupTable::_entry_cache = NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   206
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   207
const size_t             G1StringDedupTable::_min_size = (1 << 10);   // 1024
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   208
const size_t             G1StringDedupTable::_max_size = (1 << 24);   // 16777216
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   209
const double             G1StringDedupTable::_grow_load_factor = 2.0; // Grow table at 200% load
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   210
const double             G1StringDedupTable::_shrink_load_factor = _grow_load_factor / 3.0; // Shrink table at 67% load
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   211
const double             G1StringDedupTable::_max_cache_factor = 0.1; // Cache a maximum of 10% of the table size
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   212
const uintx              G1StringDedupTable::_rehash_multiple = 60;   // Hash bucket has 60 times more collisions than expected
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   213
const uintx              G1StringDedupTable::_rehash_threshold = (uintx)(_rehash_multiple * _grow_load_factor);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   214
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   215
uintx                    G1StringDedupTable::_entries_added = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   216
uintx                    G1StringDedupTable::_entries_removed = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   217
uintx                    G1StringDedupTable::_resize_count = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   218
uintx                    G1StringDedupTable::_rehash_count = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   219
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   220
G1StringDedupTable::G1StringDedupTable(size_t size, jint hash_seed) :
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   221
  _size(size),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   222
  _entries(0),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   223
  _grow_threshold((uintx)(size * _grow_load_factor)),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   224
  _shrink_threshold((uintx)(size * _shrink_load_factor)),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   225
  _rehash_needed(false),
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   226
  _hash_seed(hash_seed) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   227
  assert(is_power_of_2(size), "Table size must be a power of 2");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   228
  _buckets = NEW_C_HEAP_ARRAY(G1StringDedupEntry*, _size, mtGC);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   229
  memset(_buckets, 0, _size * sizeof(G1StringDedupEntry*));
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   230
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   231
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   232
G1StringDedupTable::~G1StringDedupTable() {
27880
afb974a04396 8060074: os::free() takes MemoryTrackingLevel but doesn't need it
coleenp
parents: 23472
diff changeset
   233
  FREE_C_HEAP_ARRAY(G1StringDedupEntry*, _buckets);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   234
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   235
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   236
void G1StringDedupTable::create() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   237
  assert(_table == NULL, "One string deduplication table allowed");
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   238
  _entry_cache = new G1StringDedupEntryCache(_min_size * _max_cache_factor);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   239
  _table = new G1StringDedupTable(_min_size);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   240
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   241
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   242
void G1StringDedupTable::add(typeArrayOop value, bool latin1, unsigned int hash, G1StringDedupEntry** list) {
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   243
  G1StringDedupEntry* entry = _entry_cache->alloc();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   244
  entry->set_obj(value);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   245
  entry->set_hash(hash);
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   246
  entry->set_latin1(latin1);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   247
  entry->set_next(*list);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   248
  *list = entry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   249
  _entries++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   250
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   251
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   252
void G1StringDedupTable::remove(G1StringDedupEntry** pentry, uint worker_id) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   253
  G1StringDedupEntry* entry = *pentry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   254
  *pentry = entry->next();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   255
  _entry_cache->free(entry, worker_id);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   256
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   257
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   258
void G1StringDedupTable::transfer(G1StringDedupEntry** pentry, G1StringDedupTable* dest) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   259
  G1StringDedupEntry* entry = *pentry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   260
  *pentry = entry->next();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   261
  unsigned int hash = entry->hash();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   262
  size_t index = dest->hash_to_index(hash);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   263
  G1StringDedupEntry** list = dest->bucket(index);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   264
  entry->set_next(*list);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   265
  *list = entry;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   266
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   267
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   268
bool G1StringDedupTable::equals(typeArrayOop value1, typeArrayOop value2) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   269
  return (value1 == value2 ||
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   270
          (value1->length() == value2->length() &&
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   271
           (!memcmp(value1->base(T_BYTE),
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   272
                    value2->base(T_BYTE),
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   273
                    value1->length() * sizeof(jbyte)))));
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   274
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   275
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   276
typeArrayOop G1StringDedupTable::lookup(typeArrayOop value, bool latin1, unsigned int hash,
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   277
                                        G1StringDedupEntry** list, uintx &count) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   278
  for (G1StringDedupEntry* entry = *list; entry != NULL; entry = entry->next()) {
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   279
    if (entry->hash() == hash && entry->latin1() == latin1) {
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   280
      typeArrayOop existing_value = entry->obj();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   281
      if (equals(value, existing_value)) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   282
        // Match found
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   283
        return existing_value;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   284
      }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   285
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   286
    count++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   287
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   288
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   289
  // Not found
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   290
  return NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   291
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   292
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   293
typeArrayOop G1StringDedupTable::lookup_or_add_inner(typeArrayOop value, bool latin1, unsigned int hash) {
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   294
  size_t index = hash_to_index(hash);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   295
  G1StringDedupEntry** list = bucket(index);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   296
  uintx count = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   297
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   298
  // Lookup in list
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   299
  typeArrayOop existing_value = lookup(value, latin1, hash, list, count);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   300
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   301
  // Check if rehash is needed
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   302
  if (count > _rehash_threshold) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   303
    _rehash_needed = true;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   304
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   305
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   306
  if (existing_value == NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   307
    // Not found, add new entry
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   308
    add(value, latin1, hash, list);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   309
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   310
    // Update statistics
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   311
    _entries_added++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   312
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   313
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   314
  return existing_value;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   315
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   316
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   317
unsigned int G1StringDedupTable::hash_code(typeArrayOop value, bool latin1) {
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   318
  unsigned int hash;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   319
  int length = value->length();
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   320
  if (latin1) {
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   321
    const jbyte* data = (jbyte*)value->base(T_BYTE);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   322
    if (use_java_hash()) {
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   323
      hash = java_lang_String::hash_code(data, length);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   324
    } else {
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   325
      hash = AltHashing::murmur3_32(_table->_hash_seed, data, length);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   326
    }
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   327
  } else {
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   328
    length /= sizeof(jchar) / sizeof(jbyte); // Convert number of bytes to number of chars
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   329
    const jchar* data = (jchar*)value->base(T_CHAR);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   330
    if (use_java_hash()) {
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   331
      hash = java_lang_String::hash_code(data, length);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   332
    } else {
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   333
      hash = AltHashing::murmur3_32(_table->_hash_seed, data, length);
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   334
    }
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   335
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   336
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   337
  return hash;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   338
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   339
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   340
void G1StringDedupTable::deduplicate(oop java_string, G1StringDedupStat& stat) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   341
  assert(java_lang_String::is_instance(java_string), "Must be a string");
35492
c8c0273e6b91 8146690: Make all classes in GC follow the naming convention.
david
parents: 35061
diff changeset
   342
  NoSafepointVerifier nsv;
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   343
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   344
  stat.inc_inspected();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   345
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   346
  typeArrayOop value = java_lang_String::value(java_string);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   347
  if (value == NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   348
    // String has no value
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   349
    stat.inc_skipped();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   350
    return;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   351
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   352
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   353
  bool latin1 = java_lang_String::is_latin1(java_string);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   354
  unsigned int hash = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   355
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   356
  if (use_java_hash()) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   357
    // Get hash code from cache
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   358
    hash = java_lang_String::hash(java_string);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   359
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   360
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   361
  if (hash == 0) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   362
    // Compute hash
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   363
    hash = hash_code(value, latin1);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   364
    stat.inc_hashed();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   365
30619
f02421441c2e 8079840: G1StringDedupTable::deduplicate() reset String hash value unnecessarily.
jiangli
parents: 30175
diff changeset
   366
    if (use_java_hash() && hash != 0) {
f02421441c2e 8079840: G1StringDedupTable::deduplicate() reset String hash value unnecessarily.
jiangli
parents: 30175
diff changeset
   367
      // Store hash code in cache
f02421441c2e 8079840: G1StringDedupTable::deduplicate() reset String hash value unnecessarily.
jiangli
parents: 30175
diff changeset
   368
      java_lang_String::set_hash(java_string, hash);
f02421441c2e 8079840: G1StringDedupTable::deduplicate() reset String hash value unnecessarily.
jiangli
parents: 30175
diff changeset
   369
    }
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   370
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   371
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   372
  typeArrayOop existing_value = lookup_or_add(value, latin1, hash);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   373
  if (existing_value == value) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   374
    // Same value, already known
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   375
    stat.inc_known();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   376
    return;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   377
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   378
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   379
  // Get size of value array
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   380
  uintx size_in_bytes = value->size() * HeapWordSize;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   381
  stat.inc_new(size_in_bytes);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   382
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   383
  if (existing_value != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   384
    // Enqueue the reference to make sure it is kept alive. Concurrent mark might
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   385
    // otherwise declare it dead if there are no other strong references to this object.
49347
edb65305d3ac 8195148: Collapse G1SATBCardTableModRefBS and G1SATBCardTableLoggingModRefBS into a single G1BarrierSet
eosterlund
parents: 49041
diff changeset
   386
    G1BarrierSet::enqueue(existing_value);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   387
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   388
    // Existing value found, deduplicate string
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   389
    java_lang_String::set_value(java_string, existing_value);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   390
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   391
    if (G1CollectedHeap::heap()->is_in_young(value)) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   392
      stat.inc_deduped_young(size_in_bytes);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   393
    } else {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   394
      stat.inc_deduped_old(size_in_bytes);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   395
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   396
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   397
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   398
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   399
G1StringDedupTable* G1StringDedupTable::prepare_resize() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   400
  size_t size = _table->_size;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   401
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   402
  // Check if the hashtable needs to be resized
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   403
  if (_table->_entries > _table->_grow_threshold) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   404
    // Grow table, double the size
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   405
    size *= 2;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   406
    if (size > _max_size) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   407
      // Too big, don't resize
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   408
      return NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   409
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   410
  } else if (_table->_entries < _table->_shrink_threshold) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   411
    // Shrink table, half the size
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   412
    size /= 2;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   413
    if (size < _min_size) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   414
      // Too small, don't resize
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   415
      return NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   416
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   417
  } else if (StringDeduplicationResizeALot) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   418
    // Force grow
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   419
    size *= 2;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   420
    if (size > _max_size) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   421
      // Too big, force shrink instead
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   422
      size /= 4;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   423
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   424
  } else {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   425
    // Resize not needed
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   426
    return NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   427
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   428
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   429
  // Update statistics
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   430
  _resize_count++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   431
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   432
  // Update max cache size
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   433
  _entry_cache->set_max_size(size * _max_cache_factor);
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   434
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   435
  // Allocate the new table. The new table will be populated by workers
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   436
  // calling unlink_or_oops_do() and finally installed by finish_resize().
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   437
  return new G1StringDedupTable(size, _table->_hash_seed);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   438
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   439
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   440
void G1StringDedupTable::finish_resize(G1StringDedupTable* resized_table) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   441
  assert(resized_table != NULL, "Invalid table");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   442
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   443
  resized_table->_entries = _table->_entries;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   444
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   445
  // Free old table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   446
  delete _table;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   447
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   448
  // Install new table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   449
  _table = resized_table;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   450
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   451
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   452
void G1StringDedupTable::unlink_or_oops_do(G1StringDedupUnlinkOrOopsDoClosure* cl, uint worker_id) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   453
  // The table is divided into partitions to allow lock-less parallel processing by
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   454
  // multiple worker threads. A worker thread first claims a partition, which ensures
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   455
  // exclusive access to that part of the table, then continues to process it. To allow
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   456
  // shrinking of the table in parallel we also need to make sure that the same worker
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   457
  // thread processes all partitions where entries will hash to the same destination
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   458
  // partition. Since the table size is always a power of two and we always shrink by
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   459
  // dividing the table in half, we know that for a given partition there is only one
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   460
  // other partition whoes entries will hash to the same destination partition. That
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   461
  // other partition is always the sibling partition in the second half of the table.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   462
  // For example, if the table is divided into 8 partitions, the sibling of partition 0
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   463
  // is partition 4, the sibling of partition 1 is partition 5, etc.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   464
  size_t table_half = _table->_size / 2;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   465
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   466
  // Let each partition be one page worth of buckets
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   467
  size_t partition_size = MIN2(table_half, os::vm_page_size() / sizeof(G1StringDedupEntry*));
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   468
  assert(table_half % partition_size == 0, "Invalid partition size");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   469
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   470
  // Number of entries removed during the scan
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   471
  uintx removed = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   473
  for (;;) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   474
    // Grab next partition to scan
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   475
    size_t partition_begin = cl->claim_table_partition(partition_size);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   476
    size_t partition_end = partition_begin + partition_size;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   477
    if (partition_begin >= table_half) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   478
      // End of table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   479
      break;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   480
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   481
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   482
    // Scan the partition followed by the sibling partition in the second half of the table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   483
    removed += unlink_or_oops_do(cl, partition_begin, partition_end, worker_id);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   484
    removed += unlink_or_oops_do(cl, table_half + partition_begin, table_half + partition_end, worker_id);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   485
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   486
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   487
  // Delayed update to avoid contention on the table lock
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   488
  if (removed > 0) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   489
    MutexLockerEx ml(StringDedupTable_lock, Mutex::_no_safepoint_check_flag);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   490
    _table->_entries -= removed;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   491
    _entries_removed += removed;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   492
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   493
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   494
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   495
uintx G1StringDedupTable::unlink_or_oops_do(G1StringDedupUnlinkOrOopsDoClosure* cl,
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   496
                                            size_t partition_begin,
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   497
                                            size_t partition_end,
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   498
                                            uint worker_id) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   499
  uintx removed = 0;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   500
  for (size_t bucket = partition_begin; bucket < partition_end; bucket++) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   501
    G1StringDedupEntry** entry = _table->bucket(bucket);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   502
    while (*entry != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   503
      oop* p = (oop*)(*entry)->obj_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   504
      if (cl->is_alive(*p)) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   505
        cl->keep_alive(p);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   506
        if (cl->is_resizing()) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   507
          // We are resizing the table, transfer entry to the new table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   508
          _table->transfer(entry, cl->resized_table());
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   509
        } else {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   510
          if (cl->is_rehashing()) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   511
            // We are rehashing the table, rehash the entry but keep it
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   512
            // in the table. We can't transfer entries into the new table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   513
            // at this point since we don't have exclusive access to all
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   514
            // destination partitions. finish_rehash() will do a single
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   515
            // threaded transfer of all entries.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   516
            typeArrayOop value = (typeArrayOop)*p;
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   517
            bool latin1 = (*entry)->latin1();
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   518
            unsigned int hash = hash_code(value, latin1);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   519
            (*entry)->set_hash(hash);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   520
          }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   521
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   522
          // Move to next entry
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   523
          entry = (*entry)->next_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   524
        }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   525
      } else {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   526
        // Not alive, remove entry from table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   527
        _table->remove(entry, worker_id);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   528
        removed++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   529
      }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   530
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   531
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   532
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   533
  return removed;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   534
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   535
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   536
G1StringDedupTable* G1StringDedupTable::prepare_rehash() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   537
  if (!_table->_rehash_needed && !StringDeduplicationRehashALot) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   538
    // Rehash not needed
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   539
    return NULL;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   540
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   541
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   542
  // Update statistics
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   543
  _rehash_count++;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   544
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   545
  // Compute new hash seed
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   546
  _table->_hash_seed = AltHashing::compute_seed();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   547
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   548
  // Allocate the new table, same size and hash seed
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   549
  return new G1StringDedupTable(_table->_size, _table->_hash_seed);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   550
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   551
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   552
void G1StringDedupTable::finish_rehash(G1StringDedupTable* rehashed_table) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   553
  assert(rehashed_table != NULL, "Invalid table");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   554
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   555
  // Move all newly rehashed entries into the correct buckets in the new table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   556
  for (size_t bucket = 0; bucket < _table->_size; bucket++) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   557
    G1StringDedupEntry** entry = _table->bucket(bucket);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   558
    while (*entry != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   559
      _table->transfer(entry, rehashed_table);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   560
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   561
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   562
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   563
  rehashed_table->_entries = _table->_entries;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   564
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   565
  // Free old table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   566
  delete _table;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   567
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   568
  // Install new table
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   569
  _table = rehashed_table;
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   570
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   571
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   572
void G1StringDedupTable::verify() {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   573
  for (size_t bucket = 0; bucket < _table->_size; bucket++) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   574
    // Verify entries
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   575
    G1StringDedupEntry** entry = _table->bucket(bucket);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   576
    while (*entry != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   577
      typeArrayOop value = (*entry)->obj();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   578
      guarantee(value != NULL, "Object must not be NULL");
30173
13cf7580b000 8077413: Avoid use of Universe::heap() inside collectors
pliden
parents: 29081
diff changeset
   579
      guarantee(G1CollectedHeap::heap()->is_in_reserved(value), "Object must be on the heap");
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   580
      guarantee(!value->is_forwarded(), "Object must not be forwarded");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   581
      guarantee(value->is_typeArray(), "Object must be a typeArrayOop");
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   582
      bool latin1 = (*entry)->latin1();
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   583
      unsigned int hash = hash_code(value, latin1);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   584
      guarantee((*entry)->hash() == hash, "Table entry has inorrect hash");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   585
      guarantee(_table->hash_to_index(hash) == bucket, "Table entry has incorrect index");
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   586
      entry = (*entry)->next_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   587
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   588
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   589
    // Verify that we do not have entries with identical oops or identical arrays.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   590
    // We only need to compare entries in the same bucket. If the same oop or an
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   591
    // identical array has been inserted more than once into different/incorrect
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   592
    // buckets the verification step above will catch that.
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   593
    G1StringDedupEntry** entry1 = _table->bucket(bucket);
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   594
    while (*entry1 != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   595
      typeArrayOop value1 = (*entry1)->obj();
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   596
      bool latin1_1 = (*entry1)->latin1();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   597
      G1StringDedupEntry** entry2 = (*entry1)->next_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   598
      while (*entry2 != NULL) {
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   599
        typeArrayOop value2 = (*entry2)->obj();
33628
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   600
        bool latin1_2 = (*entry2)->latin1();
09241459a8b8 8141132: JEP 254: Compact Strings
thartmann
parents: 31592
diff changeset
   601
        guarantee(latin1_1 != latin1_2 || !equals(value1, value2), "Table entries must not have identical arrays");
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   602
        entry2 = (*entry2)->next_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   603
      }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   604
      entry1 = (*entry1)->next_addr();
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   605
    }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   606
  }
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   607
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   608
39452
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   609
void G1StringDedupTable::clean_entry_cache() {
eebe070746ad 8158871: Long response times with G1 and StringDeduplication
pliden
parents: 37242
diff changeset
   610
  _entry_cache->delete_overflowed();
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   611
}
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   612
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33628
diff changeset
   613
void G1StringDedupTable::print_statistics() {
37242
91e5f98fff6f 8152632: Rename LogHandle(...) to Log(...)
stefank
parents: 35492
diff changeset
   614
  Log(gc, stringdedup) log;
39690
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   615
  log.debug("  Table");
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   616
  log.debug("    Memory Usage: " G1_STRDEDUP_BYTES_FORMAT_NS,
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33628
diff changeset
   617
            G1_STRDEDUP_BYTES_PARAM(_table->_size * sizeof(G1StringDedupEntry*) + (_table->_entries + _entry_cache->size()) * sizeof(G1StringDedupEntry)));
39690
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   618
  log.debug("    Size: " SIZE_FORMAT ", Min: " SIZE_FORMAT ", Max: " SIZE_FORMAT, _table->_size, _min_size, _max_size);
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   619
  log.debug("    Entries: " UINTX_FORMAT ", Load: " G1_STRDEDUP_PERCENT_FORMAT_NS ", Cached: " UINTX_FORMAT ", Added: " UINTX_FORMAT ", Removed: " UINTX_FORMAT,
47679
4cfcb7be4984 8189666: Replace various inlined percentage calculations with global percent_of()
tschatzl
parents: 47216
diff changeset
   620
            _table->_entries, percent_of(_table->_entries, _table->_size), _entry_cache->size(), _entries_added, _entries_removed);
39690
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   621
  log.debug("    Resize Count: " UINTX_FORMAT ", Shrink Threshold: " UINTX_FORMAT "(" G1_STRDEDUP_PERCENT_FORMAT_NS "), Grow Threshold: " UINTX_FORMAT "(" G1_STRDEDUP_PERCENT_FORMAT_NS ")",
35061
be6025ebffea 8145092: Use Unified Logging for the GC logging
brutisso
parents: 33628
diff changeset
   622
            _resize_count, _table->_shrink_threshold, _shrink_load_factor * 100.0, _table->_grow_threshold, _grow_load_factor * 100.0);
39690
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   623
  log.debug("    Rehash Count: " UINTX_FORMAT ", Rehash Threshold: " UINTX_FORMAT ", Hash Seed: 0x%x", _rehash_count, _rehash_threshold, _table->_hash_seed);
09a3ee292336 8159974: G1 String deduplication logging not aligned with the rest of G1
pliden
parents: 39452
diff changeset
   624
  log.debug("    Age Threshold: " UINTX_FORMAT, StringDeduplicationAgeThreshold);
23472
35e93890ed88 8029075: String deduplication in G1
pliden
parents:
diff changeset
   625
}