Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8172670

AOT Platform Support for Windows and Mac OS X x64

    Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: P2
    • Resolution: Fixed
    • Affects Version/s: 9, 10
    • Fix Version/s: 10
    • Component/s: hotspot
    • Labels:
    • Subcomponent:
    • Introduced In Version:
      9
    • Resolved In Build:
      team
    • CPU:
      x86_64
    • OS:
      os_x

      Description

      The current ahead-of-time-compilation support in JDK 9 is limited to Linux x86_64.

      Windows and Mac OS X Operating system support should be added to the AOT feature in order to allow this technology to be used on these platforms.

      This RFE will also be used to remove the AOT technology's dependency on libelf since this library is only available for Linux and Solaris. The new BinaryContainer code required to satisfy this RFE will be written 100% in Java and generate object files for Linux, Solaris, Windows and Mac OSX.

        Issue Links

          Activity

          Hide
          zmajo Zoltan Majo (Inactive) added a comment -
          Bob [~bobv],
          Vladimir [~kvn],

          JDK-8168858 seems to target the removal of dependence on the ELF format (which is also targeted by this enhancement). Can we dedicate JDK-8168858 to removing the dependence on ELF? In that case, this enhancement would only target macosx_x64 support. Thanks!
          Show
          zmajo Zoltan Majo (Inactive) added a comment - Bob [~bobv], Vladimir [~kvn], JDK-8168858 seems to target the removal of dependence on the ELF format (which is also targeted by this enhancement). Can we dedicate JDK-8168858 to removing the dependence on ELF? In that case, this enhancement would only target macosx_x64 support. Thanks!
          Hide
          bobv Bob Vandette added a comment - - edited
          This enhancement is not removing a dependency on the ELF file format. It is removing the build and runtime dependency on the libELF library and header files. Rather than rely on libELF for the creation of the AOT shared
          library, I am generating ELF object files using 100% java code.

          I am hoping to publish a webrev containing the complete set of changes this week.

          Is there a reason that you would rather see the ELF changes split out from the Windows and Mac support?
          The addition of the Windows and Mac support is isolated to OS specific directories. Splitting out this work into three bug IDs will triple the amount of busy work required to integrate these changes with no reduction in the amount of files required to be reviewed. In fact there would be considerably more files needing review since there are a number of test that will need multiple updates for each platform that would be added (@requires updates).
          Show
          bobv Bob Vandette added a comment - - edited This enhancement is not removing a dependency on the ELF file format. It is removing the build and runtime dependency on the libELF library and header files. Rather than rely on libELF for the creation of the AOT shared library, I am generating ELF object files using 100% java code. I am hoping to publish a webrev containing the complete set of changes this week. Is there a reason that you would rather see the ELF changes split out from the Windows and Mac support? The addition of the Windows and Mac support is isolated to OS specific directories. Splitting out this work into three bug IDs will triple the amount of busy work required to integrate these changes with no reduction in the amount of files required to be reviewed. In fact there would be considerably more files needing review since there are a number of test that will need multiple updates for each platform that would be added (@requires updates).
          Hide
          kvn Vladimir Kozlov added a comment - - edited
          JDK-8168858 was created before Bob's implementation and was one of suggestion how we move forward. I would prefer to close JDK-8168858 as dup. But lets wait for Bob implementation review to see how it works (I still need to look on it in details). If it is enough for AOT_CDS work then we will close JDK-8168858.

          Show
          kvn Vladimir Kozlov added a comment - - edited JDK-8168858 was created before Bob's implementation and was one of suggestion how we move forward. I would prefer to close JDK-8168858 as dup. But lets wait for Bob implementation review to see how it works (I still need to look on it in details). If it is enough for AOT_CDS work then we will close JDK-8168858 .
          Hide
          zmajo Zoltan Majo (Inactive) added a comment -
          Thank you, for the clarification!
          Show
          zmajo Zoltan Majo (Inactive) added a comment - Thank you, for the clarification!
          Hide
          hgupdate HG Updates added a comment -
          URL: http://hg.openjdk.java.net/jdk10/hs/hotspot/rev/dda9ef4a90ae
          User: bobv
          Date: 2017-02-09 21:56:39 +0000
          Show
          hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk10/hs/hotspot/rev/dda9ef4a90ae User: bobv Date: 2017-02-09 21:56:39 +0000
          Hide
          hgupdate HG Updates added a comment -
          URL: http://hg.openjdk.java.net/jdk10/hs/rev/d71bedc6660e
          User: bobv
          Date: 2017-02-09 21:56:41 +0000
          Show
          hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk10/hs/rev/d71bedc6660e User: bobv Date: 2017-02-09 21:56:41 +0000
          Hide
          hgupdate HG Updates added a comment -
          URL: http://hg.openjdk.java.net/jdk10/hs/jdk/rev/aec70105919f
          User: bobv
          Date: 2017-02-09 21:56:43 +0000
          Show
          hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk10/hs/jdk/rev/aec70105919f User: bobv Date: 2017-02-09 21:56:43 +0000

            People

            • Assignee:
              bobv Bob Vandette
              Reporter:
              bobv Bob Vandette
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: