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

cleanup IA64 specific code in Hotspot

    Details

    • Subcomponent:
    • Resolved In Build:
      b18
    • CPU:
      itanium
    • OS:
      generic

      Backports

        Description

        There are about 40 #ifdef directives which mention IA64. (See attachment.)

        Recently a probable bug was noticed around line 150 of oop.inline.hpp,
        in IA64-specific code: It looks like the length value is garbage.

        On the other hand, another bug was fixed around line 1500 in graphKit.cpp,
        as part of 6470497, which had earlier (2002/11/05) been fixed only in an IA64 ifdef.
        Failure to backport this fix led to a customer VM crash.

        In both cases, it would probably have been better to eliminate or
        at least minimize the number and volume of IA64-specific #ifdefs.
        (And other #ifdefs for other platforms, too!)

        Somebody needs to make a pass through the sources and vigorously
        rethink, rewrite, and remove the platform-specific #ifdefs in our code.

          Issue Links

            Activity

            Hide
            kvn Vladimir Kozlov added a comment -
            Currently we have much less platform-specific #ifdefs.
            Show
            kvn Vladimir Kozlov added a comment - Currently we have much less platform-specific #ifdefs.
            Hide
            dholmes David Holmes added a comment -
            The removal of ia64 references should be undertaken across the board on the hotspot sources and build files. See also

            7188234 - which just modified an ia64-only command-line options
            8000412 - has been opened to clean up the SA
            Show
            dholmes David Holmes added a comment - The removal of ia64 references should be undertaken across the board on the hotspot sources and build files. See also 7188234 - which just modified an ia64-only command-line options 8000412 - has been opened to clean up the SA
            Show
            twisti Christian Thalinger added a comment - http://cr.openjdk.java.net/~twisti/6518907
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/9fae07c31641
            User: kvn
            Date: 2013-02-02 11:49:58 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/hsx/hotspot-comp/hotspot/rev/9fae07c31641 User: kvn Date: 2013-02-02 11:49:58 +0000
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/9fae07c31641
            User: amurillo
            Date: 2013-02-08 19:17:42 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/hsx/hsx25/hotspot/rev/9fae07c31641 User: amurillo Date: 2013-02-08 19:17:42 +0000

              People

              • Assignee:
                morris Morris Meyer (Inactive)
                Reporter:
                jrose John Rose
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Imported:
                  Indexed: