jlaitine opened a new pull request, #3538:
URL: https://github.com/apache/nuttx-apps/pull/3538

   ## Summary
   
   Posix timers test doesn't really need signal actions, so we can change it to 
synchronously wait for the signal in sigwaitinfo, instead of registering a 
signal handler.
   
   This has the following advantages:
   1. Posix timers test works also in CONFIG_ENABLE_PARTIAL_SIGNALS
   2. It gets implicitly tested that the signals can wake up threads also in 
CONFIG_ENABLE_PARTIAL_SIGNALS
   
   The robust mutexes can't work withtout pthread_kill, which requires signals, 
so remove those if CONFIG_DISABLE_ALL_SIGNALS is defined
   
   ## Impact
   
   This corrects the functionality of ostest for other than 
CONFIG_ENABLE_ALL_SIGNALS.
   
   ## Testing
   
   Tested on qemu-armv8a:nsh , together with change in NuttX PR <todo>; 
   1. CONFIG_ENABLE_ALL_SIGNALS=y (unmodified defconfig): 
[qemu-armv8a_nsh_ostest_all.txt](https://github.com/user-attachments/files/28869127/qemu-armv8a_nsh_ostest_all.txt)
   2. +CONFIG_ENABLE_PARTIAL_SIGNALS=y: 
[qemu-armv8a_nsh_ostest_partial.txt](https://github.com/user-attachments/files/28869126/qemu-armv8a_nsh_ostest_partial.txt)
   3. +CONFIG_DISABLE_ALL_SIGNALS=y: 
[qemu-armv8a_nsh_ostest_disabled.txt](https://github.com/user-attachments/files/28869122/qemu-armv8a_nsh_ostest_disabled.txt)
   


-- 
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