syslog/ramlog: add support of ramlog flush worker
Summary
syslog/ramlog: add support of ramlog flush worker
- add
RAMLOG_FLUSHoptions to support flush ramlog buffer toARCH_LOWPUTCdevice - add
RAMLOG_FLUSH_WORKERto support background ramlog flush from workqueue in timeoutRAMLOG_FLUSH_WORKER_TIMEOUT_MS
Signed-off-by: chao an [email protected]
Impact
N/A
Testing
ci-check
[Experimental Bot, please feedback here]
The PR does not fully meet the NuttX requirements, specifically in the following areas:
Summary:
- Insufficient detail: The summary provides a high-level overview but lacks specific details about the changes made. It should explain the rationale behind adding the
RAMLOG_FLUSHoption and theRAMLOG_FLUSH_WORKERfunctionality. - Missing references: It should include references to any related NuttX Issues or Pull Requests, if applicable.
Impact:
- Not Applicable (N/A) is not acceptable: The impact section needs to be filled out thoughtfully, addressing each point. While the change might not impact some areas, it's important to explicitly state that and provide justification. For instance, the change likely impacts the build process and potentially hardware (if it relies on specific architecture features).
Testing:
- Insufficient details:
- "ci-check" isn't descriptive enough. Specify the CI environment used (OS, compiler, architectures).
- Provide snippets of the actual testing logs showing the relevant changes in behavior. Simply stating "ci-check" doesn't offer proof of functionality.
Recommendations for Improvement:
- Expand the Summary: Clearly articulate the problem the PR solves and the technical approach used.
- Complete the Impact Section: Thoroughly analyze and describe the impact on each aspect (build, hardware, documentation, security, compatibility), providing details and justification.
- Provide Detailed Testing Information: List all build hosts and target environments used for testing. Include relevant snippets from the testing logs, highlighting the differences in behavior before and after the changes.
By addressing these points, the PR will better adhere to the NuttX requirements and provide reviewers with the necessary context for a comprehensive evaluation.
@anchao what to do with this PR? Did you address Xiang Xiao comments?
@anchao what to do with this PR? Did you address Xiang Xiao comments?
please hold this PR, I will enhance this capability in future.