fjpanag commented on pull request #5266: URL: https://github.com/apache/incubator-nuttx/pull/5266#issuecomment-1017821283
Just a note: > As you ready said, the check is just for debugging, the user who enable CONFIG_MM_DEBUG should expect the slow response. I don't agree with this, because: * During debugging you want a system that behaves somewhat similarly to the production application. If it exhibits vastly different behaviour, then its debugging value is diminished. * Some systems, mostly safety-critical systems, continually check their state, validating that everything is working correctly. They need to catch any issue quickly and act to recover the system, instead of processing garbage. Such systems will want to keep these checks enabled in production firmwares too. This PR (and this feature in general) is not that it makes the system slower. It is that it changes how tasks are scheduled, and even can of violate priorities. -- 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]
