Mercurial > hg > graal-jvmci-8
annotate src/share/vm/gc_implementation/g1/heapRegion.inline.hpp @ 1779:6eddcbe17c83
6981746: G1: SEGV with -XX:+TraceGen0Time
Summary: Pass correct value for length to NumberSeq constructor. Guard dereferences of "body_summary" pointer with a NULL check.
Reviewed-by: tonyp, ysr
author | johnc |
---|---|
date | Mon, 13 Sep 2010 10:00:24 -0700 |
parents | c18cbe5936b8 |
children | f95d63e2154a |
rev | line source |
---|---|
342 | 1 /* |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
342
diff
changeset
|
2 * Copyright (c) 2001, 2007, Oracle and/or its affiliates. All rights reserved. |
342 | 3 * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 * | |
5 * This code is free software; you can redistribute it and/or modify it | |
6 * under the terms of the GNU General Public License version 2 only, as | |
7 * published by the Free Software Foundation. | |
8 * | |
9 * This code is distributed in the hope that it will be useful, but WITHOUT | |
10 * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or | |
11 * FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License | |
12 * version 2 for more details (a copy is included in the LICENSE file that | |
13 * accompanied this code). | |
14 * | |
15 * You should have received a copy of the GNU General Public License version | |
16 * 2 along with this work; if not, write to the Free Software Foundation, | |
17 * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. | |
18 * | |
1552
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
342
diff
changeset
|
19 * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
342
diff
changeset
|
20 * or visit www.oracle.com if you need additional information or have any |
c18cbe5936b8
6941466: Oracle rebranding changes for Hotspot repositories
trims
parents:
342
diff
changeset
|
21 * questions. |
342 | 22 * |
23 */ | |
24 | |
25 inline HeapWord* G1OffsetTableContigSpace::allocate(size_t size) { | |
26 HeapWord* res = ContiguousSpace::allocate(size); | |
27 if (res != NULL) { | |
28 _offsets.alloc_block(res, size); | |
29 } | |
30 return res; | |
31 } | |
32 | |
33 // Because of the requirement of keeping "_offsets" up to date with the | |
34 // allocations, we sequentialize these with a lock. Therefore, best if | |
35 // this is used for larger LAB allocations only. | |
36 inline HeapWord* G1OffsetTableContigSpace::par_allocate(size_t size) { | |
37 MutexLocker x(&_par_alloc_lock); | |
38 // This ought to be just "allocate", because of the lock above, but that | |
39 // ContiguousSpace::allocate asserts that either the allocating thread | |
40 // holds the heap lock or it is the VM thread and we're at a safepoint. | |
41 // The best I (dld) could figure was to put a field in ContiguousSpace | |
42 // meaning "locking at safepoint taken care of", and set/reset that | |
43 // here. But this will do for now, especially in light of the comment | |
44 // above. Perhaps in the future some lock-free manner of keeping the | |
45 // coordination. | |
46 HeapWord* res = ContiguousSpace::par_allocate(size); | |
47 if (res != NULL) { | |
48 _offsets.alloc_block(res, size); | |
49 } | |
50 return res; | |
51 } | |
52 | |
53 inline HeapWord* G1OffsetTableContigSpace::block_start(const void* p) { | |
54 return _offsets.block_start(p); | |
55 } | |
56 | |
57 inline HeapWord* | |
58 G1OffsetTableContigSpace::block_start_const(const void* p) const { | |
59 return _offsets.block_start_const(p); | |
60 } |