jerpelea opened a new pull request, #20262:
URL: https://github.com/apache/nuttx/pull/20262

   ## Summary
   
   Commit 
https://github.com/apache/nuttx/commit/b2a69ba781603fc7e1d477ea087b3e2f312a6858 
merged the critical monitor suspend and resume paths into 
nxsched_switch_critmon(), but introduced two state tracking problems:
   
       It reintroduced the obsolete premp_start field name. The field was 
previously renamed to preemp_start, causing configurations with preemption 
monitoring enabled to fail to build.
       It assigned the current timestamp to to->run_time instead of 
to->run_start. This overwrote the accumulated runtime and left a stale start 
timestamp for the next context-switch calculation when 
CONFIG_SCHED_CPULOAD_CRITMONITOR was disabled.
   
   This PR corrects the preemption field reference, restores the target 
thread's run_start timestamp, and updates the GDB TCB protocol annotations to 
match the current struct tcb_s fields.
   
   ## Impact
   
   RELEASE
   
   ## Testing
   
   CI


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to