RFR: 8328507: Move StackWatermark code from safepoint cleanup [v2]

[Erik Österlund]

> There is some stack watermark code in safepoint cleanup that flushes out concurrent stack processing, as a defensive measure to guard safepoint operations that assume the stacks have been fixed up. However, we wish to get rid of safepoint cleanup so this operation should move out from the safepoint. This proposal moves it into CollectedHeap::safepoint_synchronize_begin instead which runs before we start shutting down Java threads. This moves the code into more obvious GC places and reduces time spent inside of non-GC safepoints.

Erik Österlund has updated the pull request incrementally with one additional commit since the last revision:

  Remove incorrect assert

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/18378/files
  - new: https://git.openjdk.org/jdk/pull/18378/files/f18381c8..e830146a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=18378&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=18378&range=00-01

  Stats: 1 line in 1 file changed: 0 ins; 1 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/18378.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/18378/head:pull/18378

PR: https://git.openjdk.org/jdk/pull/18378