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

Remove un-supported CPU model on s390x #4235

Merged
merged 1 commit into from
Dec 20, 2024
Merged

Conversation

fbq815
Copy link
Contributor

@fbq815 fbq815 commented Dec 17, 2024

CPU_model: since z196 is deprecated start from 10, remove it

ID: 2646

Signed-off-by: Boqiao Fu [email protected]

@fbq815 fbq815 force-pushed the 2646 branch 2 times, most recently from 13b2dda to dc13dd8 Compare December 17, 2024 13:32
@fbq815
Copy link
Contributor Author

fbq815 commented Dec 19, 2024

Test result on s390x:
RHEL9:
PASS 8 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
RHEL10:
PASS 7 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1

@PaulYuuu this commit would only effect s390x, could you please help me review and merge it?

@@ -43,10 +44,14 @@
boot_cpu_models = 'z13,z13.2,z13s;z14,z14.2,z14ZR1;gen15a,gen15b;gen16a,gen16b'
RHEL.9:
boot_cpu_models = 'z14,z14.2,z14ZR1;gen15a,gen15b;gen16a,gen16b'
RHEL.10:
boot_cpu_models = 'z14,z14.2,z14ZR1;gen15a,gen15b;gen16a,gen16b'
- cpu_baseline:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can combine them.

@@ -10,6 +10,7 @@
type = s390x_cpu_model_expansion
variants:
- model_type_z196_z196_2_z114:
no RHEL.10
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Only old versions would be better than no

- cpu_baseline:
type = s390x_cpu_model_baseline
props1 = "msa1=True msa2=True msa3=False"
props2 = "msa1=True msa2=False msa3=True"
expected_props = "msa1"
not_expected_props = "msa2 msa3"
cpu_models = "z196 z114 zEC12 zBC12 z13 z13.2 z14 gen15b gen16b"
cpu_models = "z14 gen15b gen16b"
! RHEL.10:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

only 8 and 9?

CPU_model: since z196 is deprecated start from 10, remove it

Signed-off-by: Boqiao Fu <[email protected]>
@fbq815
Copy link
Contributor Author

fbq815 commented Dec 20, 2024

Test result on s390x:
Z15:
RHEL9
PASS 8 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
RHEL10
PASS 7 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
Z16:
RHEL9
PASS 8 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1
RHEL10
PASS 7 | ERROR 0 | FAIL 0 | SKIP 0 | WARN 0 | INTERRUPT 0 | CANCEL 1

@PaulYuuu updated, please help me review again

@PaulYuuu PaulYuuu merged commit a0de0cf into autotest:master Dec 20, 2024
7 checks passed
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 this pull request may close these issues.

2 participants