nuttxpr commented on PR #13560: URL: https://github.com/apache/nuttx/pull/13560#issuecomment-2363587217
[**\[Experimental Bot, please feedback here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues) __Squash The Commits:__ This PR contains 2 Commits. Please Squash the Multiple Commits into a Single Commit. While the provided information gives a general idea about the PR, it lacks details to assess if it fully meets the NuttX requirements. Here's a breakdown: **Strengths:** * **Summary:** Clearly states the problem (incorrect assumption about translation levels) and the solution (dynamically setting levels based on VA bits). * **Impact:** Clearly mentions the added support for 4-level translations in ARM64. * **Testing:** Specifies the testing environment (QEMU + iMX93 with kernel mode). **Areas for Improvement:** * **Summary:** * **Functional part:** Needs more specifics. Which part of the codebase handles address translation? (e.g., MMU driver, virtual memory subsystem, etc.) * **NuttX Issue/PR references:** Are there any related issues or PRs this change addresses? If so, include their links. * **Impact:** * **User impact:** Will users need to make any configuration changes or be aware of this modification? * **Build impact:** Does this change affect the build process in any way? New dependencies, configuration options, etc.? * **Hardware impact:** Specifies ARM64, but are there specific architectures or boards affected? * **Documentation impact:** Does this change necessitate documentation updates? * **Security, compatibility:** While you might not anticipate impacts in these areas, it's still a good practice to explicitly state "NO" with a brief explanation (e.g., "NO - This is a bug fix within the internal address translation logic and has no direct impact on security or compatibility"). * **Testing:** * **Build Host(s):** Provide more specifics about your build host environment. OS version, compiler version are crucial for reproducibility. * **Testing logs:** The logs are placeholder comments. You need to include the actual logs from your testing both *before* and *after* the change. This is crucial for reviewers to understand the issue and verify the fix. **Recommendation:** Provide the missing details to make the PR review smoother and ensure it aligns with the NuttX requirements. -- 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]
