xiaoxiang781216 commented on PR #16431: URL: https://github.com/apache/nuttx/pull/16431#issuecomment-2906852037
> I completely agree with @pussuw here! There is nice cleanup available for the FDs in PR #16361, which fixes the operation to be acc. to the posix. > Yes, all agree #16361 is the right direction to go. > Please don't inject debug functionality which should NOT be inside kernel in there. Integrate first the #16361, and then implement the debug functionality in where it belongs. That is, in the kernel-userspace interface, not directly in the kernel. The patchset doesn't change or add any new debugging functionality, but fix the nasty bug found by fdsan/fdcheck. We can improve fdsan/fdcheck after merging this and #16361. -- 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-unsubscr...@nuttx.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
