RFR: 8296796: Provide clean, platform-agnostic interface to C-heap trimming
    David Holmes 
    dholmes at openjdk.org
       
    Mon Nov 14 01:39:31 UTC 2022
    
    
  
On Thu, 10 Nov 2022 13:23:34 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
> This is a breakout from [JDK-8293114](https://bugs.openjdk.org/browse/JDK-8293114), which is starved for reviews. So I attempt to break up that fix into smaller units which are hopefully easier to review separately.
> 
> We can trim the C-heap manually using jcmd since [JDK-8268893](https://bugs.openjdk.org/browse/JDK-8268893). This patch reshapes this code, cleaning it up in an OS-agnostic way. That will allow us to add implementions for other platforms (I have this on my list for AIX at least) and make review of 8293114 easier.
This looks good for doing what it says, but I have to wonder whether it is actually worthwhile doing this unless most OS/lib will support it? What will the implementation be in AIX?
Thanks
src/hotspot/share/utilities/globalDefinitions.hpp line 376:
> 374: 
> 375: #define PROPERFMT             SIZE_FORMAT "%s"
> 376: #define PROPERFMTARGS(S)      byte_size_in_proper_unit(S), proper_unit_for_byte_size(S)
style nit? lower-case 's'
-------------
PR: https://git.openjdk.org/jdk/pull/11089
    
    
More information about the serviceability-dev
mailing list