Skip to content

Display ThreadX byte pool info#89

Merged
haneefdm merged 2 commits intomcu-debug:mainfrom
raphaelmeyer:threadx
Feb 13, 2026
Merged

Display ThreadX byte pool info#89
haneefdm merged 2 commits intomcu-debug:mainfrom
raphaelmeyer:threadx

Conversation

@raphaelmeyer
Copy link
Copy Markdown
Contributor

Added tab to threadx to display byte pool usage.

@haneefdm
Copy link
Copy Markdown
Contributor

haneefdm commented Feb 5, 2026

Thanks for the submission.

Can you update the changelog and also post a screenshot? Did you add a tab or a column?

@raphaelmeyer
Copy link
Copy Markdown
Contributor Author

I added a new tab, because it as another kind of "resource".

image

@haneefdm
Copy link
Copy Markdown
Contributor

haneefdm commented Feb 6, 2026

Thats fine, from your description, I couldn't tell what you just did. We will wait for a review from @PhilippHaefele if he has some time.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new ThreadX UI tab to display byte pool usage information, extending the existing Threads/Semaphores/Mutexes RTOS views.

Changes:

  • Add byte pool table definition, data collection (_tx_byte_pool_created_count/_ptr traversal), and storage.
  • Render a new “Byte Pools” tab/panel and include its percentage-bar styling.
  • Document the feature in the changelog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/rtos/rtos-threadx.ts Adds byte pool tracking and a new HTML tab to display pool start address and usage percentage.
CHANGELOG.md Notes the addition of the ThreadX byte pool tab.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/rtos/rtos-threadx.ts
Comment thread src/rtos/rtos-threadx.ts Outdated
Comment thread src/rtos/rtos-threadx.ts
@haneefdm
Copy link
Copy Markdown
Contributor

haneefdm commented Feb 6, 2026

Sorry, you all, for experimenting on the PR. This is the first time I asked Copilot to review code. The issues pointed out, are real but very likely pre-existing.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@haneefdm
Copy link
Copy Markdown
Contributor

@copilot open a new pull request to apply changes based on the comments in this thread

@haneefdm haneefdm merged commit 61ecf95 into mcu-debug:main Feb 13, 2026
3 checks passed
Copy link
Copy Markdown
Collaborator

@PhilippHaefele PhilippHaefele left a comment

Choose a reason for hiding this comment

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

Even it got already merged. I dir have a look over the changes and they look good. Thanks for contributing

@raphaelmeyer
Copy link
Copy Markdown
Contributor Author

Sorry for my late response, my last days were a bit busy.

As soon as I can spare some time, I want to have a look at the copilot issue regarding the promises.

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.

4 participants