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

Fix flip-link, make SSP configurable #3203

Merged
merged 8 commits into from
Mar 5, 2025
Merged

Conversation

bugadani
Copy link
Contributor

@bugadani bugadani commented Mar 4, 2025

We have a reproducer showing that flip-link does not work. I've also tried a freshly generated project, and it also doesn't work. So it's time to add a test.

I had to modify config.toml a bit to get this to fail, but that should be okay because our template says the following:

// make sure linkall.x is the last linker script (otherwise might cause problems with flip-link)

I've ran the wifi_embassy_access_point_with_sta example on both S3 and C6 (with and without flip_link) and all three tests were successful.

Closes #3213

@bugadani bugadani force-pushed the flip-link branch 3 times, most recently from 9a8a1c5 to b23b6f2 Compare March 4, 2025 16:33
@bugadani bugadani changed the title Add a flip-link test Try to fix flip-link Mar 4, 2025
@bugadani bugadani marked this pull request as ready for review March 5, 2025 10:13
@bugadani bugadani changed the title Try to fix flip-link Fix flip-link, make SSP configurable Mar 5, 2025
Copy link
Contributor

@bjoernQ bjoernQ left a comment

Choose a reason for hiding this comment

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

Looking forward to rebase 3124 on this some day

Copy link
Member

@MabezDev MabezDev left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@MabezDev MabezDev added this pull request to the merge queue Mar 5, 2025
Merged via the queue into esp-rs:main with commit f535f1c Mar 5, 2025
41 of 42 checks passed
@bugadani bugadani deleted the flip-link branch March 5, 2025 14:37
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.

Add configuration option for stack protector location
3 participants