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

Fix sjavac on all platforms in jprt

    Details

    • Subcomponent:
    • Resolved In Build:
      b29

      Backports

        Description

        To be able to make sjavac default, it needs to be a lot more stable. For a start, we should get it to build cleanly in normal jprt runs. The things that need to be fixed are:

        * Some issues in langtools where sjavac isn't checking platform system properties correctly to adjust behavior.
        * Configure needs to be smarter in constructing good jvm args to the sjavac jvm instance.

          Issue Links

            Activity

            Hide
            erikj Erik Joelsson added a comment -
            When JDK-8055767 is fixed, this seems to be just about correct sizing of the server process. I think I can handle that part.
            Show
            erikj Erik Joelsson added a comment - When JDK-8055767 is fixed, this seems to be just about correct sizing of the server process. I think I can handle that part.
            Hide
            erikj Erik Joelsson added a comment -
            I have changed the big if else construct to just do mem-size / 2 for mx. I also did some test runs locally and could not see any significant performance gain when using more than 2G heap so putting a cap there. Also lowering ms flag to avoid having a lot of jvm instances launch at the same time with a really big heap.
            Show
            erikj Erik Joelsson added a comment - I have changed the big if else construct to just do mem-size / 2 for mx. I also did some test runs locally and could not see any significant performance gain when using more than 2G heap so putting a cap there. Also lowering ms flag to avoid having a lot of jvm instances launch at the same time with a really big heap.
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk9/dev/rev/e0a080fd4047
            User: erikj
            Date: 2014-08-29 11:32:05 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/dev/rev/e0a080fd4047 User: erikj Date: 2014-08-29 11:32:05 +0000
            Hide
            hgupdate HG Updates added a comment -
            URL: http://hg.openjdk.java.net/jdk9/jdk9/rev/e0a080fd4047
            User: lana
            Date: 2014-09-03 20:44:28 +0000
            Show
            hgupdate HG Updates added a comment - URL: http://hg.openjdk.java.net/jdk9/jdk9/rev/e0a080fd4047 User: lana Date: 2014-09-03 20:44:28 +0000

              People

              • Assignee:
                erikj Erik Joelsson
                Reporter:
                erikj Erik Joelsson
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: