JDK-8225638 : [lworld] Store inline classes in CDS archive
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: repo-valhalla
  • Priority: P4
  • Status: Resolved
  • Resolution: Fixed
  • Submitted: 2019-06-12
  • Updated: 2019-12-17
  • Resolved: 2019-12-17
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
Other
repo-valhallaFixed
Related Reports
Blocks :  
JDK-8231610 - Relocate the CDS archive if it cannot be mapped to the requested address
Duplicate :  
JDK-8210258 - [lworld] CDS broken
Description
http://hg.openjdk.java.net/valhalla/valhalla/file/fb97bb15ea8e/src/hotspot/share/oops/method.cpp#l1039

assert(!method_holder()->is_value(), "FIXME: valuetype not supported by CDS");

---

To support valuetypes, we need to implement two more trampoline entries for Method::{_from_compiled_value_ro_entry, _from_compiled_value_entry). We probably should also store EnableValhalla in the CDS archive header, and disable CDS if the run-time and dump-time value of EnableValhalla is different.
Comments
http://hg.openjdk.java.net/valhalla/valhalla/rev/d02b937ae15b
17-12-2019
JDK-8321610 changes the handling of the trampoline entries. We we should wait for JDK-8321610 before fixing this bug.
09-10-2019