Re: [PR] arch: set current regs firstly in undefinedinsn [nuttx]

[via GitHub]

XuNeo commented on code in PR #14135:
URL: https://github.com/apache/nuttx/pull/14135#discussion_r1797540997


##########
arch/arm/src/arm/arm_undefinedinsn.c:
##########
@@ -42,6 +42,12 @@
 
 void arm_undefinedinsn(uint32_t *regs)
 {
+  /* Save the saved processor context in current_regs where it can be
+   * accessed for register dumps and possibly context switching.
+   */
+
+  up_set_current_regs(regs);

Review Comment:
   `syslog` could trigger another crash and we don't have the registers saved. 
For example the below one we met.
   
   
![image](https://github.com/user-attachments/assets/ebaecc3d-0e53-43cf-972d-92a29e249694)
   



-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org