nuttxpr commented on PR #13973: URL: https://github.com/apache/nuttx/pull/13973#issuecomment-2401559789
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) The PR summary is missing key information such as what code is changed and how it works. The impact section claims "no impact" which is unlikely and needs justification. Testing only provides a code snippet without specifying build hosts, targets, or logs. **The PR does not meet the NuttX requirements.** Here's how to improve it: **Summary:** * **Expand on the "why":** What problem does thread_local support solve? What use cases does it enable? * **Be specific about the changes:** Which files are modified? Are you introducing new system calls? Modifying existing threading structures? * **Explain the implementation:** How is thread-local storage allocated and accessed? Are you using compiler extensions, platform-specific features, or a generic approach? **Impact:** * **Address all points:** Even if an item seems non-applicable, state "NO" explicitly. * **Justify "no impact" claims:** For example, if there's truly no user impact, explain why. If the build process doesn't change, mention that it's handled transparently. * **Consider potential implications:** Thread-local storage can have performance and memory usage implications. **Testing:** * **Provide build host details:** Operating systems, architectures, compilers used for testing. * **Specify target details:** Architectures, boards, and configurations tested on. * **Include actual logs:** Both before and after the changes, showing the difference in behavior. **Additional Considerations:** * **Documentation:** Since this is a new feature, documentation is crucial. Explain how to use thread-local storage in NuttX. * **Error Handling:** How are errors related to thread-local storage (e.g., allocation failures) handled and reported? * **Compatibility:** Are there any compatibility concerns with existing code or future extensions? -- 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]
