wangchdo commented on PR #17357: URL: https://github.com/apache/nuttx/pull/17357#issuecomment-3561841018
> > > The detailed implementation is different, both approach could benefit in the different usage. It's better to introduce the disable level concept, so both pr could be merged. > > > > > > Why do we need two PRs for this? I already have a PR that disables signals. If the level-control approach works, it would be straightforward to integrate it into #17352. Opening a new PR would only consume additional reviewer time. I think we should focus on improving the existing PR instead. > > It's fine to use one pr to do the work, but someone need help to merge them. For example, for example, this pr contain more clean than #17352: https://github.com/apache/nuttx/pull/17357/files#diff-9e6b14f89a1f89e46914da0086d8e74e4d511ce0907333ca592dfda73368207fR132-R138 > > Bascially, we can arrange the pr into three part: > > 1. Skip the signal dispatch in each arch. Both pr do the same thing, but we need merge the change to ensure the skip completeness. > 2. One patch skip the signal dispatch in the common code > 3. Final patch skip all signal related function > > Who volunteers to complete this work? @wangchdo or @extinguish. I can take care of this, and I'm happy to do so. By the way, since this is still under VOTING on the mailing list, I think this work is not very urgent. -- 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]
