nuttxpr commented on PR #14454: URL: https://github.com/apache/nuttx/pull/14454#issuecomment-2429318013
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The provided PR description does not meet all of the NuttX requirements. Here's a breakdown: **Strengths:** * **Clear Summary:** The summary explains the problem (incorrect assumption about CPU index stability), the solution (refreshing CPU ID), and the positive outcome (fixing a crash). * **Impact is addressed:** The impact section highlights the specific area affected (task/process exit logic) and clarifies that it only applies when CONFIG_SMP=y. **Weaknesses:** * **Missing Details in Impact:** * **User Impact:** While it mentions no other parts are affected, it doesn't explicitly state if users need to adapt. A clear "NO" is preferable. * **Build Impact:** Similarly, a clear "NO" is needed if the build process is unchanged. * **Other Impacts:** The description should include "NO" for all other impact categories (hardware, documentation, security, compatibility) to demonstrate these aspects were considered. * **Insufficient Testing Information:** * **Build Host Details:** Provide information about the build host OS, CPU, and compiler version used for testing. * **Target Details:** Be more specific about the unpublished MPFS SMP target. Mention the architecture and board if possible. * **Missing Logs:** The testing logs sections are empty. Include relevant logs demonstrating the issue before the change and the successful outcome after the change. **Recommendation:** Revise the PR description to address the missing information and provide more comprehensive testing details. -- 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]
