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

[Ignore] test use of ldar rather than ldapr #24031

Closed
wants to merge 3 commits into from

Conversation

nhorman
Copy link
Contributor

@nhorman nhorman commented Apr 3, 2024

Please ignore - conducting a test of ldar vs ldapr in CI on a virtualized mac

@github-actions github-actions bot added the severity: fips change The pull request changes FIPS provider sources label Apr 3, 2024
@t8m
Copy link
Member

t8m commented Apr 4, 2024

@nhorman don't you have to revert #23967 to really test this?

@nhorman
Copy link
Contributor Author

nhorman commented Apr 4, 2024

crud, yes. Let me re-run this

@nhorman
Copy link
Contributor Author

nhorman commented Apr 4, 2024

after re-enabling the test, running 10 iterations of the macosx 14 ci job, the failure is still not observed. I believe prior findings hold here, making the single instruction change on the virtualized M1 core from ldapr to ldar makes the problem abate.

I should also note the cpu info indicates that we have 3 physical cpus on this system, with 3 separate l1 caches and a shared l2 core

@nhorman nhorman closed this Apr 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity: fips change The pull request changes FIPS provider sources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants