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

erts: Fix spectre mitigation configure test #8006

Conversation

garazdawi
Copy link
Contributor

On Ubuntu 20.04 and later gcc adds -fcf-protection=full by default, which cannot be used together with -mbranch-protect. So we set -fcf-protection=none so that we can use -mbranch-protect.

On Ubuntu 20.04 and later gcc adds -fcf-protection=full by
default, which cannot be used together with -mbranch-protect.
So we set -fcf-protection=none so that we can use -mbranch-protect.
@garazdawi garazdawi added team:VM Assigned to OTP team VM fix testing currently being tested, tag is used by OTP internal CI labels Jan 9, 2024
@garazdawi garazdawi requested a review from jhogberg January 9, 2024 08:40
@garazdawi garazdawi self-assigned this Jan 9, 2024
Copy link
Contributor

github-actions bot commented Jan 9, 2024

CT Test Results

Tests are running... https://github.com/erlang/otp/actions/runs/7458620478

Results for commit 13e1b29

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

  • No CT logs found
  • No HTML docs found
  • No Windows Installer found

// Erlang/OTP Github Action Bot

@garazdawi garazdawi merged commit 2592177 into erlang:master Jan 15, 2024
16 checks passed
@garazdawi garazdawi deleted the lukas/erts/fix-spectre-mitigation-configure-test branch January 15, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix team:VM Assigned to OTP team VM testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants