infernap12/jdk
1
0
Fork 0
mirror of https://github.com/openjdk/jdk.git synced 2026-02-07 08:59:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

8265064: Move clearing and setting of members into helpers in ReservedSpace

Browse Source 
Reviewed-by: tschatzl, iwalulya
This commit is contained in:
Stefan Johansson 2021-04-13 12:39:05 +00:00
parent 7006070fa3
commit a4f644eb2f
2 changed files with 51 additions and 40 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

80
src/hotspot/share/memory/virtualspace.cpp
View File

@ -73,12 +73,7 @@ ReservedSpace::ReservedSpace(char* base, size_t size, size_t alignment,
bool special, bool executable) : _fd_for_heap(-1) {
assert((size % os::vm_allocation_granularity()) == 0,
"size not allocation aligned");
_base = base;
_size = size;
_alignment = alignment;
_noaccess_prefix = 0;
_special = special;
_executable = executable;
initialize_members(base, size, alignment, special, executable);
}
// Helper method
@ -140,6 +135,21 @@ static bool failed_to_reserve_as_requested(char* base, char* requested_address,
return true;
}
void ReservedSpace::clear_members() {
initialize_members(NULL, 0, 0, false, false);
}
void ReservedSpace::initialize_members(char* base, size_t size, size_t alignment,
bool special, bool executable) {
_base = base;
_size = size;
_alignment = alignment;
_special = special;
_executable = executable;
_noaccess_prefix = 0;
}
void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
char* requested_address,
bool executable) {
@ -151,18 +161,14 @@ void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
assert(alignment == 0 || is_power_of_2((intptr_t)alignment),
"not a power of 2");
alignment = MAX2(alignment, (size_t)os::vm_page_size());
clear_members();
_base = NULL;
_size = 0;
_special = false;
_executable = executable;
_alignment = 0;
_noaccess_prefix = 0;
if (size == 0) {
return;
}
alignment = MAX2(alignment, (size_t)os::vm_page_size());
// If OS doesn't support demand paging for large page memory, we need
// to use reserve_memory_special() to reserve and pin the entire region.
// If there is a backing file directory for this space then whether
@ -193,9 +199,10 @@ void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
"Large pages returned a non-aligned address, base: "
PTR_FORMAT " alignment: " SIZE_FORMAT_HEX,
p2i(base), alignment);
_special = true;
} else {
// failed; try to reserve regular memory below
// failed; try to reserve regular memory below. Reservation
// should not be marked as special.
special = false;
if (UseLargePages && (!FLAG_IS_DEFAULT(UseLargePages) ||
!FLAG_IS_DEFAULT(LargePageSizeInBytes))) {
log_debug(gc, heap, coops)("Reserve regular memory without large pages");
@ -213,13 +220,13 @@ void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
// important. If available space is not detected, return NULL.
if (requested_address != 0) {
base = attempt_map_or_reserve_memory_at(requested_address, size, _fd_for_heap, _executable);
base = attempt_map_or_reserve_memory_at(requested_address, size, _fd_for_heap, executable);
if (failed_to_reserve_as_requested(base, requested_address, size, false, _fd_for_heap != -1)) {
// OS ignored requested address. Try different address.
base = NULL;
}
} else {
base = map_or_reserve_memory(size, _fd_for_heap, _executable);
base = map_or_reserve_memory(size, _fd_for_heap, executable);
}
if (base == NULL) return;
@ -231,7 +238,7 @@ void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
// Make sure that size is aligned
size = align_up(size, alignment);
base = map_or_reserve_memory_aligned(size, alignment, _fd_for_heap, _executable);
base = map_or_reserve_memory_aligned(size, alignment, _fd_for_heap, executable);
if (requested_address != 0 &&
failed_to_reserve_as_requested(base, requested_address, size, false, _fd_for_heap != -1)) {
@ -243,14 +250,13 @@ void ReservedSpace::initialize(size_t size, size_t alignment, bool large,
}
}
}
// Done
_base = base;
_size = size;
_alignment = alignment;
// If heap is reserved with a backing file, the entire space has been committed. So set the _special flag to true
// If heap is reserved with a backing file, the entire space has been committed. So set the special flag to true
if (_fd_for_heap != -1) {
_special = true;
special = true;
}
// Done
initialize_members(base, size, alignment, special, executable);
}
ReservedSpace ReservedSpace::first_part(size_t partition_size, size_t alignment) {
@ -315,12 +321,7 @@ void ReservedSpace::release() {
} else{
os::release_memory(real_base, real_size);
}
_base = NULL;
_size = 0;
_noaccess_prefix = 0;
_alignment = 0;
_special = false;
_executable = false;
clear_members();
}
}
@ -399,12 +400,13 @@ void ReservedHeapSpace::try_reserve_heap(size_t size,
"Large pages returned a non-aligned address, base: "
PTR_FORMAT " alignment: " SIZE_FORMAT_HEX,
p2i(base), alignment);
_special = true;
}
}
if (base == NULL) {
// Failed; try to reserve regular memory below
// Failed; try to reserve regular memory below. Reservation
// should not be marked as special.
special = false;
if (UseLargePages && (!FLAG_IS_DEFAULT(UseLargePages) ||
!FLAG_IS_DEFAULT(LargePageSizeInBytes))) {
log_debug(gc, heap, coops)("Reserve regular memory without large pages");
@ -422,18 +424,16 @@ void ReservedHeapSpace::try_reserve_heap(size_t size,
}
if (base == NULL) { return; }
// Done
_base = base;
_size = size;
_alignment = alignment;
// If heap is reserved with a backing file, the entire space has been committed. So set the _special flag to true
// If heap is reserved with a backing file, the entire space has been committed. So set the special flag to true
if (_fd_for_heap != -1) {
_special = true;
special = true;
}
// Done
initialize_members(base, size, alignment, special, false);
// Check alignment constraints
if ((((size_t)base) & (alignment - 1)) != 0) {
if (!is_aligned(base, alignment)) {
// Base not aligned, retry.
release();
}

11
src/hotspot/share/memory/virtualspace.hpp
View File

@ -48,6 +48,17 @@ class ReservedSpace {
ReservedSpace(char* base, size_t size, size_t alignment, bool special,
bool executable);
protected:
// Helpers to clear and set members during initialization. Two members
// require special treatment:
// * _fd_for_heap - The fd is set once and should not be cleared
// even if the reservation has to be retried.
// * _noaccess_prefix - Used for compressed heaps and updated after
// the reservation is initialized. Always set to
// 0 during initialization.
void clear_members();
void initialize_members(char* base, size_t size, size_t alignment,
bool special, bool executable);
void initialize(size_t size, size_t alignment, bool large,
char* requested_address,
bool executable);