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.

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

Cli: top #3707

Merged
merged 9 commits into from
Jun 13, 2024
Merged

Cli: top #3707

merged 9 commits into from
Jun 13, 2024

Conversation

skotopes
Copy link
Member

@skotopes skotopes commented Jun 12, 2024 •
edited
Loading

What's new

  • Cli: top command
  • Furi: ThreadList container and thread enumeration routine.

Verification

  • Check cli top command

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

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2024 •
edited
Loading

PVS-Studio report for commit c1f5d71d:

@github-actions GitHub Actions
Copy link

github-actions bot commented Jun 12, 2024 •
edited
Loading

Compiled f7 firmware for commit 1f682f6b:

@CookiePLMonster
Copy link
Contributor

This PR should also update https://github.com/flipperdevices/flipperzero-firmware/blob/dev/documentation/AppManifests.md#parameters

  • stack_size: stack size in bytes to allocate for an application on its startup. Note that allocating a stack too small for an app to run will cause a system crash due to stack overflow, and allocating too much stack space will reduce usable heap memory size for apps to process data. Note: you can use ps and free CLI commands to profile your app's memory usage.

DrZlo13
DrZlo13 previously approved these changes Jun 13, 2024
View reviewed changes
Copy link
Member

@DrZlo13 DrZlo13 left a comment

Choose a reason for hiding this comment

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

such wow, much cool

gsurkov
gsurkov previously approved these changes Jun 13, 2024
View reviewed changes
@skotopes skotopes dismissed stale reviews from gsurkov and DrZlo13 via 1794838 June 13, 2024 16:56
@skotopes skotopes merged commit ca8517a into dev Jun 13, 2024
11 checks passed
@skotopes skotopes deleted the aku/top branch June 13, 2024 17:07
@skotopes skotopes mentioned this pull request Jun 13, 2024
Closed
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 DrZlo13 left review comments

@gsurkov gsurkov gsurkov left review comments

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

@drunkbatya drunkbatya Awaiting requested review from drunkbatya drunkbatya 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.
5 participants
@skotopes @CookiePLMonster @hedger @DrZlo13 @gsurkov