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

Much nearly duplicated code for PerfMemory support

    XMLWordPrintable

Details

    • b06
    • generic
    • generic

    Description

      For Linux, Solaris, BSD, and AIX the implementations of PerfMemory support (perfMemory_<os>.cpp) are very similar, with many of the differences being merely code formatting and spelling, capitalization, and formatting in comments. Many of the remaining differences appear to be unimportant and could be reconciled.

      I think these perfMemory_<os>.cpp files could be mostly merged into a single implementation in os/posix.

      Attachments

        Issue Links

          duplicates

          Enhancement - null JDK-8260534 Factor out common POSIX code: perfMemory

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          relates to

          Enhancement - null JDK-7015652 src/os/solaris, src/os/linux and src/os/bsd should not contain duplicate code

          • P3 - Major loss of function.
          • Resolved

          Enhancement - null JDK-8202353 os::readdir should use readdir instead of readdir_r

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8223699 cleanup perfMemory_aix.cpp O_NOFOLLOW coding on aix

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8260262 Use common code in function unmap_shared() in perfMemory_posix.cpp

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Enhancement - null JDK-8260533 Umbrella: Factor out common POSIX code

          • P4 - Minor loss of function, or other problem where easy workaround is present.
          • Closed
          links to

          Review Review openjdk/jdk/2037

          Activity

            People

              hseigel Harold Seigel (Inactive)
              kbarrett Kim Barrett
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: