[13u] RFR: 8225797: OldObjectSample event creates unexpected amount of checkpoint data

[Andrew Brygin]

Hello,

 I would like to backport the fix for 8225797 to 13u. It is already in 11u.

 Bug: https://bugs.openjdk.java.net/browse/JDK-8225797
 Original change: https://hg.openjdk.java.net/jdk/jdk/rev/caa25ab47aca
 Webrev: http://cr.openjdk.java.net/~bae/13u/8225797/webrev.00/

 Prerequisite: JDK-8230398, applies cleanly to 13u.

 With the fix for JDK-8230398, this change apples to 14u cleanly with
two little exceptions:
 the directive '#include "oops/markWord.hpp"' is missed in files
src/hotspot/share/jfr/leakprofiler/chains/pathToGcRootsOperation.cpp and
src/hotspot/share/jfr/leakprofiler/checkpoint/rootResolver.cpp due to
absence of fix for JDK-8230398 in 13u, so two changes listed below are
not applicable to 13u codebase:

src/hotspot/share/jfr/leakprofiler/chains/pathToGcRootsOperation.cpp:
--- pathToGcRootsOperation.cpp
+++ pathToGcRootsOperation.cpp
@@ -43,7 +43,6 @@
 #include "jfr/leakprofiler/utilities/granularTimer.hpp"
 #include "logging/log.hpp"
 #include "memory/universe.hpp"
-#include "oops/markWord.hpp"
 #include "oops/oop.inline.hpp"
 #include "runtime/safepoint.hpp"
 #include "utilities/globalDefinitions.hpp"

src/hotspot/share/jfr/leakprofiler/checkpoint/rootResolver.cpp:
--- rootResolver.cpp
+++ rootResolver.cpp
@@ -32,7 +32,6 @@
 #include "memory/iterator.hpp"
 #include "memory/universe.hpp"
 #include "oops/klass.hpp"
-#include "oops/markWord.hpp"
 #include "oops/oop.hpp"
 #include "prims/jvmtiThreadState.hpp"
 #include "runtime/frame.inline.hpp"

The rest of the fix applies cleanly.

Testing: tier1 and jdk/jfr tests pass fine with the change.

Thanks,
Andrew