Skip to content

[FL-3289] Various Furi/FuriHal bug fixes and improvements #2637

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.

Sign up for GitHub

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

Jump to bottom
Merged
skotopes merged 2 commits into from
May 5, 2023
Merged

[FL-3289] Various Furi/FuriHal bug fixes and improvements #2637

skotopes merged 2 commits into from
May 5, 2023

Conversation

@skotopes
Copy link
Contributor

@skotopes skotopes commented May 5, 2023 •
edited
Loading

What's new

  • Furi: properly handle thread free before TCB scrapping
  • FuriHal: add DWT comparator api to cortex
  • Updater, RPC: refactor various thread shanenigans
  • Reset debug and logging on production update
  • Power: better low current gauge readings handling
  • Code cleanup

Verification

  • Ensure that no more crashes happens on reboot/update
  • Ensure that RPC is not crashing on disconnect anymore
  • Full release test is required

Checklist (For Reviewer)

  • PR has description of feature/bug or link to Confluence/Jira task
  • Description contains actions to verify feature/bugfix
  • I've built this code, uploaded it to the device and verified feature/bugfix

@skotopes
Furi: properly handle thread free before TCB scrapping, add furi_free…
2b8d624 
… - more invasive version of free to memmgr. FuriHal: add DWT comparator api to cortex. Updater, RPC: refactor various thread shanenigans. Code cleanup.
@skotopes skotopes marked this pull request as draft May 5, 2023 03:21
@github-actions
Copy link

github-actions bot commented May 5, 2023 •
edited
Loading

Compiled firmware for commit ac41471d:

@github-actions
Copy link

github-actions bot commented May 5, 2023

PVS-Studio report for commit 2b8d624f:

@skotopes skotopes changed the title Various Furi/FuriHal bug fixes and improvements [FL-3289] Various Furi/FuriHal bug fixes and improvements May 5, 2023
@skotopes skotopes marked this pull request as ready for review May 5, 2023 12:05
@skotopes skotopes merged commit 914129a into dev May 5, 2023
@skotopes skotopes deleted the aku/va_fixes branch May 5, 2023 12:40
@blackvault88 blackvault88 mentioned this pull request Jul 14, 2023
Open
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@hedger hedger hedger approved these changes

@DrZlo13 DrZlo13 Awaiting requested review from DrZlo13 DrZlo13 is a code owner

@Skorpionm Skorpionm Awaiting requested review from Skorpionm

@nminaylov nminaylov Awaiting requested review from nminaylov nminaylov is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@skotopes @hedger