Skip to content

Conversation

dkalowsk
Copy link
Contributor

Introduce the THREAD_ANALYZER_LONG_FRAME_PER_INTERVAL Kconfig to reset the longest frame value to zero after printing enabling analysis where the longest frame can now represents the most recent interval. The longest overall frame is often at startup and this modification allows one to observe subsequent long frames.

@dkalowsk dkalowsk force-pushed the dkalowsky/update_profiler branch 2 times, most recently from 5d36b2e to 935f261 Compare August 28, 2025 16:00
@dkalowsk dkalowsk changed the title subsys: debug: add thread analyzer option to reset frame debug: thread_analyzer: add thread analyzer option to reset frame Aug 28, 2025
@dkalowsk dkalowsk force-pushed the dkalowsky/update_profiler branch from 935f261 to d713727 Compare September 2, 2025 21:31
ubieda
ubieda previously approved these changes Sep 2, 2025
Copy link

@jeppenodgaard jeppenodgaard left a comment

Choose a reason for hiding this comment

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

LGTM

@dkalowsk dkalowsk force-pushed the dkalowsky/update_profiler branch from d713727 to 3b83fa6 Compare September 15, 2025 22:23
@dkalowsk dkalowsk force-pushed the dkalowsky/update_profiler branch 3 times, most recently from abe5041 to 23948fb Compare October 15, 2025 21:13
Copy link
Contributor

@peter-mitsis peter-mitsis left a comment

Choose a reason for hiding this comment

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

I think that we are almost there--just a couple of slight tweaks.

Use THREAD_ANALYZER_LONG_FRAME_PER_INTERVAL config to reset
longest frame value to zero after printing so that longest frame
now represents longest frame for the most recent interval.  The
longest overall frame is often at startup and this modification
allows one to observe subsequent long frames.

Signed-off-by: Jay Lawlor <[email protected]>
Signed-off-by: Dan Kalowsky <[email protected]>
@dkalowsk dkalowsk force-pushed the dkalowsky/update_profiler branch from 23948fb to b9fda44 Compare October 16, 2025 20:43
@sonarqubecloud
Copy link

@dkalowsk dkalowsk added this to the v4.3.0 milestone Oct 16, 2025
@cfriedt cfriedt merged commit 27afe41 into zephyrproject-rtos:main Oct 17, 2025
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants