[jdk8u-dev] RFR: 8305165: [8u] ServiceThread::nmethods_do is not called to keep nmethods from being zombied while in the queue [v2]

[Joshua Zhu]

> Various Crashes in JvmtiExport::post_compiled_method_load were found in
> our production environment after users started to receive the JVMTI
> COMPILED_METHOD_LOAD event.
> ServiceThread::nmethods_do should have been called to keep nmethods from
> being zombied while in the JvmtiDeferredEventQueue, but
> JavaThead::nmethods_do is called at present.
> Make a virtual function call to the correct method to fix this issue.
> With the fix, various crashes were gone.
> This issue only exists in 8u.

Joshua Zhu has updated the pull request incrementally with one additional commit since the last revision:

  Add 'virtual' in oops_do and nmethods_do declaration in JavaThread

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

Changes:
  - all: https://git.openjdk.org/jdk8u-dev/pull/293/files
  - new: https://git.openjdk.org/jdk8u-dev/pull/293/files/0c7e5a17..c3b83151

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=293&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk8u-dev&pr=293&range=00-01

  Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
  Patch: https://git.openjdk.org/jdk8u-dev/pull/293.diff
  Fetch: git fetch https://git.openjdk.org/jdk8u-dev.git pull/293/head:pull/293

PR: https://git.openjdk.org/jdk8u-dev/pull/293