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

Mode questions #529

Closed
knn-k opened this issue Apr 12, 2024 · 4 comments · Fixed by #531
Closed

Mode questions #529

knn-k opened this issue Apr 12, 2024 · 4 comments · Fixed by #531

Comments

@knn-k
Copy link
Contributor

knn-k commented Apr 12, 2024

I have some questions on ottawa.csv. What are these modes for?

Mode155:
It is enabled only on linux_390-64 and linux_aarch64 configurations.

<mode number="155">

variation:Mode155,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,yes,yes,yes,yes,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,yes,yes,no,no,no,no,no,yes,yes

Mode198 and Mode199:
These modes use the -Xaggressive option, and are enabled only on win_x86-64 and linux_aarch64 configurations.

<mode number="198">

<mode number="199">

TKG/resources/ottawa.csv

Lines 91 to 92 in 2326826

variation:Mode198,no,no,yes,yes,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,yes,yes,no,no,no,no,no,yes,yes
variation:Mode199,no,no,yes,yes,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,no,yes,yes,no,no,no,no,no,yes,yes

knn-k added a commit to knn-k/TKG that referenced this issue Apr 15, 2024
This commit disables Modes 155, 198, and 199 on AArch64 Linux.

Fixes: adoptium#529

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k
Copy link
Contributor Author

knn-k commented Apr 15, 2024

It seems that I enabled these modes on linux_aarch64_cmprssptrs by mistake when I added the platform by #40.
I opened PR #531 as a fix.

knn-k added a commit to knn-k/TKG that referenced this issue Apr 15, 2024
This commit disables Modes 155, 159, 198, and 199 on AArch64 Linux.

Fixes: adoptium#529

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k
Copy link
Contributor Author

knn-k commented Apr 15, 2024

PR #40 enabled Modes 158 and 159 by mistake.
PR #528 fixes Mode158, and #531 fixes Mode159.

@llxia
Copy link
Contributor

llxia commented Apr 16, 2024

What are these modes for?

The mode is a short name for a list of jvm options. We inherited the old logic from vmfarm.
As you listed above,

  • TKG/resources/modes.xml - defines the list of jvm options for each mode
  • TKG/resources/ottawa.csv - defines what mode can or cannot run on which platforms. The complexity escalates when it needs to support 64_nocmprssptrs, cmprssptrs, and mixed JVM using one file.

knn-k added a commit to knn-k/TKG that referenced this issue Apr 16, 2024
This commit disables Modes 155, 159, 198, and 199 on AArch64 Linux.

Fixes: adoptium#529

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
@knn-k
Copy link
Contributor Author

knn-k commented Apr 17, 2024

I think I should have been more clear.
I know what modes.xml and ottawa.csv are used for. I wanted to know the purposes of Modes 155, 198, and 199 specifically.
I am disabling those modes on AArch64 Linux by #531 any way.

llxia pushed a commit that referenced this issue Apr 17, 2024
This commit disables Modes 155, 159, 198, and 199 on AArch64 Linux.

Fixes: #529

Signed-off-by: KONNO Kazuhiro <konno@jp.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants