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 2021/05/15 07:54:14 UTC

[GitHub] [incubator-nuttx] Donny9 opened a new pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Donny9 opened a new pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719


   ## Summary
   Cancel dispatch signal when the number of members in the group is 
   zero of members in the group is zero.
   
   Repetition step:
   If using waitpid() to wait a task has exited, the waitpid will use
   nxsing_kill with signal number 0 to determine if that task is still
   alive, and will call group_signal to dispatch signal for each member
   of the group and perform signal handing checks, it will assert at
   sched/group/group_signal.c:261.
   
   Change-Id: I75ef015a261101277436d742897ce68fed44b5da
   Signed-off-by: Jiuzhu Dong <do...@xiaomi.com>
   ## Impact
   Don't assert when dispatch a signal to process doesn't exist. 
   ## Testing
   daily test.
   


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

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

[GitHub] [incubator-nuttx] Donny9 removed a comment on pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Posted by GitBox <gi...@apache.org>.
Donny9 removed a comment on pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719#issuecomment-841619538


   ```
   + mkdir -p /Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + export HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + rm -rf /usr/local/bin/2to3
   + brew update --quiet
   Error: Fetching /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core failed!
   Updated 4 taps (homebrew/cask, homebrew/bundle, homebrew/services and aws/tap).
   ```
   
   Trigger again?


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

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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719


   


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

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

[GitHub] [incubator-nuttx] Donny9 commented on pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Posted by GitBox <gi...@apache.org>.
Donny9 commented on pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719#issuecomment-841619538


   `+ mkdir -p /Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + export HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + rm -rf /usr/local/bin/2to3
   + brew update --quiet
   Error: Fetching /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core failed!
   Updated 4 taps (homebrew/cask, homebrew/bundle, homebrew/services and aws/tap).`
   
   Trigger again?


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

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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719


   


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

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

[GitHub] [incubator-nuttx] Donny9 edited a comment on pull request #3719: group/group_signal: cancel dispatch signal when child process doesn't exist

Posted by GitBox <gi...@apache.org>.
Donny9 edited a comment on pull request #3719:
URL: https://github.com/apache/incubator-nuttx/pull/3719#issuecomment-841619538


   ```
   + mkdir -p /Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + export HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + HOMEBREW_CACHE=/Users/runner/work/incubator-nuttx/incubator-nuttx/sources/prebuilt/homebrew
   + rm -rf /usr/local/bin/2to3
   + brew update --quiet
   Error: Fetching /usr/local/Homebrew/Library/Taps/homebrew/homebrew-core failed!
   Updated 4 taps (homebrew/cask, homebrew/bundle, homebrew/services and aws/tap).
   ```
   
   Trigger again?


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

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