[GitHub] [incubator-nuttx-apps] masayuki2009 edited a comment on pull request #955: ostest: sighand.c: add sem_wait in signal handler

[GitBox <gi...@apache.org>]

masayuki2009 edited a comment on pull request #955:
URL: https://github.com/apache/incubator-nuttx-apps/pull/955#issuecomment-1005647806


   >@masayuki2009 cloud you try this patch with/without apache/incubator-nuttx#5120?
   
   I tried this PR with sabre-6quad:smp (QEMU) and confirmed that apache/incubator-nuttx#5120 works as expected.
   
   (1) without apache/incubator-nuttx#5120
   
   ```
   user_main: signal handler test
   sighand_test: Initializing semaphore to 0
   sighand_test: Starting waiter task
   waiter_main: Waiter started
   sighand_test: Started waiter_main pid=61
   waiter_main: Unmasking signal 17
   waiter_main: Registering signal handler
   waiter_main: oact.sigaction=0 oact.sa_flags=0 oact.sa_mask=0
   waiter_main: Waiting on semaphore
   sighand_test: Signaling pid=61 with signo=17 sigvalue=42
   wakeup_action: Received signal 17
   wakeup_action: sival_int=42
   wakeup_action: si_code=1
   wakeup_action: ucontext=0
   poster_main: Poster started
   poster_main: done
   wakeup_action: sem_wait() successfully!
   sighand_test: Started poster_main pid=62
   waiter_main: ERROR awakened with no error!
   waiter_main: done
   sighand_test: done
   ```
   
   (2) with apache/incubator-nuttx#5120
   
   ```
   user_main: signal handler test 
   sighand_test: Initializing semaphore to 0 
   sighand_test: Starting waiter task 
   sighand_test: Started waiter_main pid=61 
   waiter_main: Waiter started 
   waiter_main: Unmasking signal 17 
   waiter_main: Registering signal handler 
   waiter_main: oact.sigaction=0 oact.sa_flags=0 oact.sa_mask=0 
   waiter_main: Waiting on semaphore 
   sighand_test: Signaling pid=61 with signo=17 sigvalue=42 
   wakeup_action: Received signal 17 
   wakeup_action: sival_int=42 
   wakeup_action: si_code=1 
   wakeup_action: ucontext=0 
   poster_main: Poster started 
   poster_main: done 
   sighand_test: Started poster_main pid=62 
   wakeup_action: sem_wait() successfully! 
   waiter_main: sem_wait() successfully interrupted by signal 
   waiter_main: done 
   sighand_test: done 
   ```


-- 
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-unsubscribe@nuttx.apache.org

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