Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jvm: drop UseCompressedOops JVM option #7702

Open
wants to merge 1 commit into
base: 10.2
Choose a base branch
from
Open

jvm: drop UseCompressedOops JVM option #7702

wants to merge 1 commit into from

Commits on Nov 27, 2024

  1. jvm: drop UseCompressedOops JVM option

    Motivation:
    Starting java7, the UseCompressedOops is dynamically controed by heap
    size.
    
    ```
    $ java -Xmx32g -XX:+PrintFlagsFinal 2>/dev/null | grep UseCompressedOops
         bool UseCompressedOops                        = false                          {product lp64_product} {default}
    
    $ java -Xmx28g -XX:+PrintFlagsFinal 2>/dev/null | grep UseCompressedOops
         bool UseCompressedOops                        = true                           {product lp64_product} {ergonomic}
    ```
    
    The mismatch between UseCompressedOops endup with error:
    
    ```
    OpenJDK 64-Bit Server VM warning: Max heap size too large for Compressed Oops
    ***** WARNING! INCORRECT SYSTEM CONFIGURATION DETECTED! *****
    The system limit on number of memory mappings per process might be too low for the given
    [gc] max Java heap size (40960M). Please adjust /proc/sys/vm/max_map_count to allow for at
    [gc] least 73728 mappings (current limit is 65530). Continuing execution with the current
    ```
    
    Modification:
    drop UseCompressedOops JVM option from defaults.
    
    Result:
    correct behavior on JVMs with large heap
    
    Acked-by: Lea Morschel
    Target: master
    Require-book: no
    Require-notes: yes
    (cherry picked from commit 59ea69f)
    Signed-off-by: Tigran Mkrtchyan <[email protected]>
    kofemann committed Nov 27, 2024
    Configuration menu
    Copy the full SHA
    c350883 View commit details
    Browse the repository at this point in the history