RFR(M): 7147848: [macosx] com.sun.management.UnixOperatingSystem uses hardcoded dummy values (original) (raw)

rhoover rhoover at apple.com
Tue Apr 10 10:51:33 PDT 2012

Scott, Steffan

(I am he one who put in the original lines of: return -1.0; // not available being replaced by this patch)

I was reluctant to implement these functions because the linux code was quite involved and it appeared to me that there might be some additional semantics to what was implemented than what was indicated by the function names alone.

I have not compared the code with the 'top' source, but it looks plausible. As a sanity test, the function values being returned could be printed by a java program and visually compared with the output of 'top' as a system is loaded up. It might also be wise to run the same java program on other platforms to make sure that the magnitude of the numbers is in the same ballpark.

On Apr 10, 2012, at 10:21 AM, Scott Kovatch wrote:

Regarding Apple, Roger Hoover would be a good person to look at this, as he's spent more time in the Darwin levels of the VM. I think he's still partially attached to the OpenJDK work. -- Scott On Apr 10, 2012, at 8:58 AM, Daniel D. Daugherty wrote:

Staffan,

I reviewed it and I think it looks OK. I tried looking at the code in MacosxOperatingSystem.c relative to the Linux version, but I think it is easily possible to miss something subtle here. You might try a direct ping to Mandy Chung since M&M was her area. You might also try a direct ping to Mike Swingler to get an Apple reviewer. Dan

On 4/10/12 3:30 AM, Staffan Larsen wrote: Any takers for this review? (added core-libs-dev as well) Thanks, /Staffan On 3 apr 2012, at 15:39, Staffan Larsen wrote: Please review the following fix: webrev: http://cr.openjdk.java.net/~sla/7147848/webrev.00/ <http://cr.openjdk.java.net/%7Esla/7147848/webrev.00/> bug: http://bugs.sun.com/bugdatabase/viewbug.do?bugid=7147848 This fix implements the missing functionality in UnixOperatingSystem for Mac OS X. Any feedback on the implementation is welcome as I am not very familiar with the APIs in Mac OS X. I have verified that the changes build on all platforms through JPRT. The correctness has been verified manually by looking in JConsole and running the tests in test/java/lang/management/OperatingSystemMXBean test/com/sun/management/OperatingSystemMXBean. Thanks, /Staffan

More information about the macosx-port-dev mailing list