Skip to content

[catalogue] Revamp the aarch64-ETS2 catalogue#1674

Open
relokin wants to merge 1 commit intoherd:masterfrom
relokin:catalogue-ets2
Open

[catalogue] Revamp the aarch64-ETS2 catalogue#1674
relokin wants to merge 1 commit intoherd:masterfrom
relokin:catalogue-ets2

Conversation

@relokin
Copy link
Copy Markdown
Member

@relokin relokin commented Jan 20, 2026

Previsouly, the aarch64-ETS2 was a soft link to the aarch64-VMSA catalogue. This meant that we would run the same tests as aarch64-VMSA with the addition of -variant ets2. In addition, there was a separate kinds.txt file because -variant ets2 meant that some tests would change to being architecturally forbidden.

This change switches to a custom set of tests which are designed to illustrate the most representative tests that become allowed with FEAT_ETS2.

@relokin
Copy link
Copy Markdown
Member Author

relokin commented Jan 20, 2026

@artkhyzha when you find some time, I would appreciate your opinion on this change.

Previsouly, the aarch64-ETS2 was a soft link to the aarch64-VMSA
catalogue. This meant that we would run the same tests as aarch64-VMSA
with the addition of -variant ets2. In addition, there was a separate
kinds.txt file because -variant ets2 meant that some tests would
change to being architecturally forbidden.

This change switches to a custom set of tests which are designed to
illustrate the most representative tests that become allowed with
FEAT_ETS2.

Signed-off-by: Nikos Nikoleris <nikos.nikoleris@arm.com>
Copy link
Copy Markdown
Contributor

@diaolo01 diaolo01 left a comment

Choose a reason for hiding this comment

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

Thanks, @relokin. I ran every test with and without ets2 and can confirm each goes from Allowed pre-ets2 to forbidden with ets2.

@@ -0,0 +1,15 @@
AArch64 SB+dsb.sy+dmb.syptev1p
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

According to the test name and references to this test, the file name needs updating.

@diaolo01
Copy link
Copy Markdown
Contributor

diaolo01 commented Apr 21, 2026

On the PR description:

This change switches to a custom set of tests which are designed to illustrate the most representative tests that become allowed with FEAT_ETS2.

is it meant to say become forbidden with FEAT_ETS2?

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