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
AArch64: Disable j9sysinfo_test_get_levels_and_types() #8711
Conversation
Related PR: #8617 |
runtime/tests/port/si.c
Outdated
@@ -2273,7 +2273,7 @@ j9sysinfo_runTests(struct J9PortLibrary *portLibrary, char *argv0) | |||
rc |= j9sysinfo_test_get_groups(portLibrary); | |||
#endif /* !(defined(WIN32) || defined(WIN64)) */ | |||
rc |= j9sysinfo_test_get_l1dcache_line_size(portLibrary); | |||
#if !(defined(LINUXPPC) || defined(S390) || defined(J9ZOS390) || defined(J9ARM) || defined(OSX)) | |||
#if !(defined(LINUXPPC) || defined(S390) || defined(J9ZOS390) || defined(J9ARM) || defined(J9AARCH64) || defined(OSX)) | |||
rc |= j9sysinfo_test_get_levels_and_types(portLibrary); | |||
#endif /* !(defined(LINUXPPC) || defined(S390) || defined(J9ZOS390) || defined(J9ARM) || defined(OSX)) */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You should update this closing comment as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the comment part.
Is there an issue for tracking re-enabling this test? |
This commit disables running j9sysinfo_test_get_levels_and_types() on AArch64. Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@DanHeidinga I opened #8723 for tracking. |
jenkins test sanity alinux64 jdk11 |
I would like this PR to be merged if it is OK. |
@DanHeidinga : any objections if we merge this? |
My concern about the reminder issue has been addressed so I don't object (though I haven't reviewed the code) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Never mind, it's two lines - I have reviewed it and have no objections :)
This commit disables running j9sysinfo_test_get_levels_and_types()
on AArch64.
Signed-off-by: KONNO Kazuhiro konno@jp.ibm.com