You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2022/01/04 08:03:41 UTC
[GitHub] [incubator-nuttx-apps] zhaoxiu-zeng opened a new pull request #955: ostest: sighand.c: add sem_wait in signal handler
zhaoxiu-zeng opened a new pull request #955:
URL: https://github.com/apache/incubator-nuttx-apps/pull/955
task is blocked by semphore1, signal handler is blocked by semphore2,
after post semphore2, the task must get -EINTR.
Signed-off-by: Zeng Zhaoxiu <zh...@gmail.com>
## Summary
## Impact
## Testing
--
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
[GitHub] [incubator-nuttx-apps] masayuki2009 merged pull request #955: ostest: sighand.c: add sem_wait in signal handler
Posted by GitBox <gi...@apache.org>.
masayuki2009 merged pull request #955:
URL: https://github.com/apache/incubator-nuttx-apps/pull/955
--
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
[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #955: ostest: sighand.c: add sem_wait in signal handler
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #955:
URL: https://github.com/apache/incubator-nuttx-apps/pull/955#issuecomment-1005454104
@masayuki2009 could you try this PR?
--
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
[GitHub] [incubator-nuttx-apps] masayuki2009 edited a comment on pull request #955: ostest: sighand.c: add sem_wait in signal handler
Posted by 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
[GitHub] [incubator-nuttx-apps] masayuki2009 commented on pull request #955: ostest: sighand.c: add sem_wait in signal handler
Posted by GitBox <gi...@apache.org>.
masayuki2009 commented 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.
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
```
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
[GitHub] [incubator-nuttx-apps] xiaoxiang781216 commented on pull request #955: ostest: sighand.c: add sem_wait in signal handler
Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #955:
URL: https://github.com/apache/incubator-nuttx-apps/pull/955#issuecomment-1004690155
@masayuki2009 cloud you try this patch with/without https://github.com/apache/incubator-nuttx/pull/5120?
--
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