JDK 8253015 v2 #20

GlebSolovev · 2021-11-24T06:33:07Z

Ссылка на первую версию этого пулл реквеста, где мы обсудили, по-моему, все кроме graal-а: #18

Переношу этот рефакторинг кода: openjdk/jdk#154, ему как раз соответствует JDK 8253015.
Что и почему я при переносе изменил:

убрал все, что касалось SVE: его поддержка появилась только в jdk 16
- JBS: https://bugs.openjdk.java.net/browse/JDK-8231441
- jdk commit: openjdk/jdk@9b5a9b6
убрал MD5 Intrinsics, так как они тоже появились только в jdk 16
- появились в jdk 16 для x86, JBS: https://bugs.openjdk.java.net/browse/JDK-8250902
- для AArch64 еще unresolved, JBS: https://bugs.openjdk.java.net/browse/JDK-8251216
- собираются бэкпортировать в 11-pool (что это?), но еще не готово, JBS: https://bugs.openjdk.java.net/browse/JDK-8251319
убрал SHA512, так как его поддержали тоже только в jdk 16
- JBS: https://bugs.openjdk.java.net/browse/JDK-8165404
добавил комментарии с TODO к изменениям в graal-e, в которых проверялось условие JDK >= 16
- как я понял, основной рефактор кода, который я бэкпортирую, затронул и graal => после того, как этот рефактор ушел в jdk, изменения в graal-е попросили так же отдельно добавить в сам graal (чтобы не терялась синхронизация)
- что действительно произошло: Graal part of JDK-8253015: Aarch64: Move linux code out from generic CPU feature detection oracle/graal#2861
- однако как адаптировать проверку 16 версии, я пока не понимаю, поэтому и оставил TODO
- в текущей версии graal-а, например, эти изменения никак не модифицировались: раз, два
- @AntonKozlov здесь мне нужна ~~психо~~логическая помощь

Вместе с основным бэкпортом рефакторинга кода переношу еще 3 коммита, исправляющих в нем ошибки:

аккуратное удаление проверки guarantee(cpu_lines == os::processor_count(), "core count should be consistent"); и связанных с нею идейно штук:
- просто ее удаление: 8255716: AArch64: Regression: JVM crashes if manually offline a core openjdk/jdk#983
- удаление строк, связанных с не всегда работающим определением флага CPU_A53MAC через число cpu: 8255799: AArch64: CPU_A53MAC feature may be set incorrectly openjdk/jdk#1084
исправление static_assert-ов: 8255975: Fix AArch64 OpenJDK build failure with gcc-5 openjdk/jdk#1088

Backport ec9bee68660acd6abf0b4dd4023ae69514542256

… detection

Backport 36998b006d6a9ee8145004778534d1f35a55e068

Backport 2c8f4e202bd7e5b78edb32580b8a836660493d9c

Backport da48003abd2e88ad0d435c01a7c17073e1a61569

KaperD · 2021-11-24T17:27:58Z

src/hotspot/cpu/aarch64/vm_version_aarch64.cpp

+int VM_Version::_zva_length;
+int VM_Version::_dcache_line_size;
+int VM_Version::_icache_line_size;
+int VM_Version::_initial_sve_vector_length;


Это случайно попало, оно не нужно

Спасибо, действительно, случайно! Я даже помню, как его убивал, чорт)

AntonKozlov

Круто! Реально сложный бекпорт попался. Всё хорошо по большей части 👍

AntonKozlov · 2021-11-24T18:43:18Z

src/hotspot/cpu/aarch64/vm_version_aarch64.cpp

+#include "runtime/arguments.hpp"
+#include "runtime/globals_extension.hpp"


Выглядит как лишние include

AntonKozlov · 2021-11-24T18:44:36Z

src/hotspot/os_cpu/linux_aarch64/vm_version_linux_aarch64.cpp


+#include <asm/hwcap.h>
+#include <sys/auxv.h>
+#include <sys/prctl.h>


И этот один инклюд кажется лишний

#include <sys/prctl.h>

AntonKozlov · 2021-11-24T18:54:24Z

...ler.hotspot.aarch64/src/org/graalvm/compiler/hotspot/aarch64/AArch64HotSpotLIRGenerator.java

+
+        boolean isDcZvaProhibited = true;
+        int zvaLength = 0;
+        if (GraalHotSpotVMConfig.JDK >= 16) { // TODO: fix GraalHotSpotVMConfig.JDK >= 16


Тут получается надо добавить проверку на JDK_UPDATE из src/jdk.internal.vm.compiler/share/classes/org.graalvm.compiler.hotspot/src/org/graalvm/compiler/hotspot/GraalHotSpotVMConfigAccess.java

т.е. JDK >= 16 || (JDK >= 15 && JDK_UPDATE >= xxx

А xxx можно посмотреть в make/autoconf/version-numbers:

DEFAULT_VERSION_UPDATE=6

Надо бы убедиться, что значение действительно сюда проходит и используется как надо.

Насколько я вижу, использование грааля включается -XX:+UnlockExperimentalVMOptions -XX:+UseJVMCICompiler (https://openjdk.java.net/jeps/317), плюс -Xcomp триггерит компиляцию каждого встреченного java метода. И тогда можно добавить отладочную печать прямо сюда.

AntonKozlov · 2021-11-24T18:54:57Z

...sses/org.graalvm.compiler.hotspot/src/org/graalvm/compiler/hotspot/GraalHotSpotVMConfig.java

    // * DZP, bit [4] of indicates whether use of DC ZVA instruction is prohibited.
-    public final int psrInfoDczidValue = getFieldValue("VM_Version::_psr_info.dczid_el0", Integer.class, "uint32_t", 0x10, (JVMCI ? jvmciGE(JVMCI_19_3_b04) : JDK >= 14) && osArch.equals("aarch64"));
+    public final int psrInfoDczidValue = getFieldValue("VM_Version::_psr_info.dczid_el0", Integer.class, "uint32_t", 0x10,
+                    (JVMCI ? jvmciGE(JVMCI_19_3_b04) : (JDK == 14 || JDK == 15)) && osArch.equals("aarch64"));


JDK == 15 должно теперь видимо превратиться в (JDK == 15 && JDK_UPDATE < xxx)

AntonKozlov · 2021-11-24T19:03:24Z

src/hotspot/os_cpu/linux_aarch64/vm_version_linux_aarch64.cpp

+  static_assert(CPU_FP      == HWCAP_FP,      "Flag CPU_FP must follow Linux HWCAP");
+  static_assert(CPU_ASIMD   == HWCAP_ASIMD,   "Flag CPU_ASIMD must follow Linux HWCAP");
+  static_assert(CPU_EVTSTRM == HWCAP_EVTSTRM, "Flag CPU_EVTSTRM must follow Linux HWCAP");
+  static_assert(CPU_AES     == HWCAP_AES,     "Flag CPU_AES must follow Linux HWCAP");
+  static_assert(CPU_PMULL   == HWCAP_PMULL,   "Flag CPU_PMULL must follow Linux HWCAP");
+  static_assert(CPU_SHA1    == HWCAP_SHA1,    "Flag CPU_SHA1 must follow Linux HWCAP");
+  static_assert(CPU_SHA2    == HWCAP_SHA2,    "Flag CPU_SHA2 must follow Linux HWCAP");
+  static_assert(CPU_CRC32   == HWCAP_CRC32,   "Flag CPU_CRC32 must follow Linux HWCAP");
+  static_assert(CPU_LSE     == HWCAP_ATOMICS, "Flag CPU_LSE must follow Linux HWCAP");
+  static_assert(CPU_DCPOP   == HWCAP_DCPOP,   "Flag CPU_DCPOP must follow Linux HWCAP");


А надо как-то запускать configure особо? у меня ругается на нашем кросс-компайлере

/home/anton/proj/jdk15u-dev/src/hotspot/os_cpu/linux_aarch64/vm_version_linux_aarch64.cpp:65:3: error: identifier 'static_assert' is a keyword in C++11 [-Werror=c++11-compat]

А в

jdk15u-dev/src/hotspot/share/utilities/debug.hpp

Line 184 in 29a64a5

#define STATIC_ASSERT(Cond) \

объявлен STATIC_ASSERT, который повсеместно используется в хостпоте. Кажется лучше использовать его.

AntonKozlov · 2021-11-25T08:24:21Z

Конфликты в edee095 отмечены в стиле по умолчанию. Реально удобнее работать с diff3 стилем. Глеб, на будущее добавь пожалуйста конфигурацию в git:

https://github.com/azul-research/macos-aarch64/blob/master/README.md#%D0%BD%D0%B0%D1%81%D1%82%D1%80%D0%BE%D0%B9%D0%BA%D0%B0

(параграф добавил только что :))

GlebSolovev added 6 commits November 24, 2021 07:09

8253015: Aarch64: Move linux code out from generic CPU feature detection

edee095

Backport ec9bee68660acd6abf0b4dd4023ae69514542256

fixup! 8253015: Aarch64: Move linux code out from generic CPU feature…

56c36c1

… detection

8255716: AArch64: Regression: JVM crashes if manually offline a core

1cdfe33

Backport 36998b006d6a9ee8145004778534d1f35a55e068

8255799: AArch64: CPU_A53MAC feature may be set incorrectly

3d971b4

Backport 2c8f4e202bd7e5b78edb32580b8a836660493d9c

8255975: Fix AArch64 OpenJDK build failure with gcc-5

a65be4c

Backport da48003abd2e88ad0d435c01a7c17073e1a61569

fixup! 8255975: Fix AArch64 OpenJDK build failure with gcc-5

d56ead6

GlebSolovev requested a review from AntonKozlov November 24, 2021 06:33

KaperD reviewed Nov 24, 2021

View reviewed changes

AntonKozlov reviewed Nov 24, 2021

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

JDK 8253015 v2 #20

JDK 8253015 v2 #20

Uh oh!

GlebSolovev commented Nov 24, 2021 •

edited

Loading

Uh oh!

KaperD Nov 24, 2021

Uh oh!

GlebSolovev Nov 24, 2021

Uh oh!

AntonKozlov left a comment

Uh oh!

AntonKozlov Nov 24, 2021

Uh oh!

AntonKozlov Nov 24, 2021

Uh oh!

AntonKozlov Nov 24, 2021

Uh oh!

AntonKozlov Nov 24, 2021

Uh oh!

AntonKozlov Nov 24, 2021

Uh oh!

AntonKozlov commented Nov 25, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

		#include "runtime/arguments.hpp"
		#include "runtime/globals_extension.hpp"

JDK 8253015 v2 #20

Are you sure you want to change the base?

JDK 8253015 v2 #20

Uh oh!

Conversation

GlebSolovev commented Nov 24, 2021 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

KaperD Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

GlebSolovev Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov left a comment

Choose a reason for hiding this comment

Uh oh!

AntonKozlov Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov Nov 24, 2021

Choose a reason for hiding this comment

Uh oh!

AntonKozlov commented Nov 25, 2021

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

GlebSolovev commented Nov 24, 2021 •

edited

Loading