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

Allow writing CBO bits in xenvcfg #635

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Timmmm
Copy link
Collaborator

@Timmmm Timmmm commented Dec 6, 2024

This was accidentally missed from the CBO implementation.

Copy link

github-actions bot commented Dec 6, 2024

Test Results

396 tests  ±0     0 ✅  - 396   0s ⏱️ ±0s
  4 suites ±0     0 💤 ±  0 
  1 files   ±0   396 ❌ +396 

For more details on these failures, see this check.

Results for commit 05fd1f9. ± Comparison against base commit 5f4a8e6.

♻️ This comment has been updated with latest results.

@jordancarlin
Copy link
Collaborator

jordancarlin commented Dec 6, 2024

Shouldn't these be gated on sys_enable_zicbom and sys_enable_zicboz? Or potentially even extensionEnabled(Ext_Zicbom) and extensionEnabled(Ext_Zicboz). The spec is slightly unclear on that.

@Timmmm
Copy link
Collaborator Author

Timmmm commented Dec 7, 2024

Ah yes you're right. I'll update it.

This was accidentally missed from the CBO implementation.
@Timmmm
Copy link
Collaborator Author

Timmmm commented Dec 16, 2024

I'll wait for #634 since this is exactly the sort of problem it helps with.

@Timmmm Timmmm self-assigned this Dec 16, 2024
@Timmmm Timmmm requested a review from jordancarlin December 17, 2024 15:38
@Timmmm Timmmm force-pushed the user/timh/cbo_menvcfg branch from 6f6b126 to 05fd1f9 Compare December 19, 2024 10:02
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