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

panic-semihosting docs are empty #287

Closed
jonas-schievink opened this issue Nov 10, 2020 · 0 comments
Closed

panic-semihosting docs are empty #287

jonas-schievink opened this issue Nov 10, 2020 · 0 comments
Labels

Comments

@jonas-schievink
Copy link
Contributor

https://docs.rs/panic-semihosting/0.5.4/panic_semihosting/ is empty, despite there being documentation on the crate.

Caused by #![cfg(all(target_arch = "arm", target_os = "none"))].

The fix is to tell docs.rs to use an embedded ARM target (see https://docs.rs/about/metadata).

bors bot added a commit that referenced this issue Nov 14, 2020
293: Build panic-semihosting for embedded target. Prepares v0.5.5. Closes #287. r=jonas-schievink a=adamgreig



Co-authored-by: Adam Greig <adam@adamgreig.com>
@bors bors bot closed this as completed in c4180df Nov 14, 2020
adamgreig pushed a commit that referenced this issue Jan 12, 2022
287: Fix common uses of INSERT AFTER with .bss and .text r=adamgreig a=mattico

Fixes #267 
Fixes #266

This fixes two related issues.

1. Named sections are often inserted after `.bss` or `.text` in order to have them handled as if they were part of that section. Defining the start/end symbols outside of the section allows this to work.
2. Uninitialized C statics will end up as common symbols which end up in the COMMON input section. If this section is orphaned, it will likely end up placed after `.bss`. C code often expects these statics to be zero initialized. The first change would cause these symbols to be placed before `__ebss` so they will get zeroed by the reset handler. Explicitly placing the common symbols into `.bss` ensures this happens. Users who want uninitialized symbols should use the `.uninit` section.

See rust-embedded/cortex-m-rt#287 (comment)

Co-authored-by: Matt Ickstadt <mattico8@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant