Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

8252921: NMT overwrite memory type for region assert when building dynamic archive #185

Closed
wants to merge 8 commits into from
+12 −14
Closed

8252921: NMT overwrite memory type for region assert when building dynamic archive #185

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
ad056b4
JDK-8252921
Sep 15, 2020
f4fbe38
JDK-8252921
Sep 15, 2020
467db25
Restore assertion
Sep 16, 2020
a7113de
Fix trailing whitespace
Sep 16, 2020
c49380e
Tighten recusive reserveration handling
Sep 17, 2020
6b927b5
Fix metaspace remapping on Windows
Sep 20, 2020
4764de4
Back out thread stack cleaning, to be addressed by JDK-8253429
Sep 21, 2020
7aaa383
Fix indents
Sep 21, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion src/hotspot/share/runtime/thread.hpp
View file
Original file line number Diff line number Diff line change
@@ -759,7 +759,11 @@ class Thread: public ThreadShadow {

public:
// Stack overflow support
address stack_base() const { return _stack_base; }
address stack_base() const {
assert(_stack_base != NULL || Thread::current() != this, "Sanity check");
return _stack_base;
}

void set_stack_base(address base) { _stack_base = base; }
size_t stack_size() const { return _stack_size; }
void set_stack_size(size_t size) { _stack_size = size; }