--- old/src/hotspot/share/jfr/periodic/sampling/jfrCallTrace.cpp 2019-04-24 10:49:00.918956969 +0900 +++ new/src/hotspot/share/jfr/periodic/sampling/jfrCallTrace.cpp 2019-04-24 10:49:00.712959057 +0900 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2012, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -27,6 +27,7 @@ #include "code/nmethod.hpp" #include "code/pcDesc.hpp" #include "jfr/periodic/sampling/jfrCallTrace.hpp" +#include "jfr/utilities/jfrTypes.hpp" #include "oops/method.hpp" #include "runtime/javaCalls.hpp" #include "runtime/frame.inline.hpp" @@ -37,7 +38,7 @@ assert(top_frame.cb() != NULL, "invariant"); RegisterMap map(_thread, false); frame candidate = top_frame; - for (int i = 0; i < MaxJavaStackTraceDepth * 2; ++i) { + for (u4 i = 0; i < MAX_STACK_DEPTH * 2; ++i) { if (candidate.is_entry_frame()) { JavaCallWrapper *jcw = candidate.entry_frame_call_wrapper_if_safe(_thread); if (jcw == NULL || jcw->is_first_frame()) { --- old/src/hotspot/share/jfr/recorder/service/jfrOptionSet.cpp 2019-04-24 10:49:01.439951691 +0900 +++ new/src/hotspot/share/jfr/recorder/service/jfrOptionSet.cpp 2019-04-24 10:49:01.148954639 +0900 @@ -1,5 +1,5 @@ /* - * Copyright (c) 2012, 2018, Oracle and/or its affiliates. All rights reserved. + * Copyright (c) 2012, 2019, Oracle and/or its affiliates. All rights reserved. * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. * * This code is free software; you can redistribute it and/or modify it @@ -27,6 +27,7 @@ #include "jfr/recorder/service/jfrMemorySizer.hpp" #include "jfr/recorder/service/jfrOptionSet.hpp" #include "jfr/utilities/jfrAllocation.hpp" +#include "jfr/utilities/jfrTypes.hpp" #include "logging/log.hpp" #include "memory/allocation.inline.hpp" #include "memory/resourceArea.hpp" @@ -106,10 +107,6 @@ return _stack_depth; } -static const u4 STACK_DEPTH_DEFAULT = 64; -static const u4 MIN_STACK_DEPTH = 1; -static const u4 MAX_STACK_DEPTH = 2048; - void JfrOptionSet::set_stackdepth(u4 depth) { if (depth < MIN_STACK_DEPTH) { _stack_depth = MIN_STACK_DEPTH; --- old/src/hotspot/share/jfr/recorder/stacktrace/jfrStackTraceRepository.cpp 2019-04-24 10:49:01.898947039 +0900 +++ new/src/hotspot/share/jfr/recorder/stacktrace/jfrStackTraceRepository.cpp 2019-04-24 10:49:01.706948984 +0900 @@ -69,8 +69,8 @@ } // handle general case - int loop_count = 0; - int loop_max = MaxJavaStackTraceDepth * 2; + u4 loop_count = 0; + u4 loop_max = MAX_STACK_DEPTH * 2; do { loop_count++; // By the time we get here we should never see unsafe but better safe then segv'd --- old/src/hotspot/share/jfr/utilities/jfrTypes.hpp 2019-04-24 10:49:02.393942025 +0900 +++ new/src/hotspot/share/jfr/utilities/jfrTypes.hpp 2019-04-24 10:49:02.124944749 +0900 @@ -33,6 +33,9 @@ typedef int fio_fd; const int invalid_fd = -1; const jlong invalid_offset = -1; +const u4 STACK_DEPTH_DEFAULT = 64; +const u4 MIN_STACK_DEPTH = 1; +const u4 MAX_STACK_DEPTH = 2048; enum EventStartTime { UNTIMED,