RFR: 8227631: Adjust AIX version check

Baesken, Matthias matthias.baesken at sap.com
Fri Jul 12 11:09:08 UTC 2019


Hello, please review this small AIX related change .

For some time, we do not support AIX 5.3 any more.
See (where AIX 7.1 or 7.2 is the supported build platform since OpenJDK11) :

https://wiki.openjdk.java.net/display/Build/Supported+Build+Platforms

The currently used xlc 16.1 (XL C/C++ Compilers) even  needs  minimum  AIX 7.1 to  run , see

http://www-01.ibm.com/support/docview.wss?uid=swg21326972

(and compiling for older releases on 7.1 / 7.2 would not work easily , at least not "out of the box"  to my knowledge .)

So we should adjust the minimum OS version check done in os_aix.cpp in os::Aix::initialize_os_info() .


Additionally the change removes a couple of warnings   [-Wwritable-strings   category]  .

/nightly/jdk/src/hotspot/os/aix/os_aix.cpp:4081:22: warning: ISO C++11 does not allow conversion from string literal to 'char *' [-Wwritable-strings]
    char *name_str = "unknown OS";
                     ^
/nightly/jdk/src/hotspot/os/aix/os_aix.cpp:4089:18: warning: ISO C++11 does not allow conversion from string literal to 'char *' [-Wwritable-strings]
      name_str = "OS/400 (pase)";
                 ^
/nightly/jdk/src/hotspot/os/aix/os_aix.cpp:4100:18: warning: ISO C++11 does not allow conversion from string literal to 'char *' [-Wwritable-strings]
      name_str = "AIX";



Bug/webrev :

https://bugs.openjdk.java.net/browse/JDK-8227631

http://cr.openjdk.java.net/~mbaesken/webrevs/8227631.0/

Thanks, Matthias


More information about the hotspot-dev mailing list