Sarthak-Pandey commented on PR #20702:
URL: https://github.com/apache/nuttx/pull/20702#issuecomment-6563320395

   Hi @casaroli,
   
   Thank you for the review! I have addressed both points:
   
   1. **Documentation**:
      - Added `iob_copyin_iov()`, `iob_trycopyin_iov()`, and 
`iob_copyout_iov()` to `Documentation/os/memory/iob.rst` alongside 
`iob_copyin()`, `iob_trycopyin()`, and `iob_copyout()`.
   
   2. **PR Description & Testing Alignment**:
      - Updated the PR body to reflect only the actual changes in this commit 
(removed the TCP send path modification claim and "Thread Safety" section).
      - Clarified Testing item 2 for `rv-virt:knsh` (noting that `AF_LOCAL`, 
`AF_INET UDP`, and `AF_INET TCP` passed multi-iov scatter-gather, while CAN 
validated `-ENODEV` and `-EMSGSIZE` guards due to the lack of netdev on knsh).
      - Embedded the full execution run logs for all 3 targets (`sim:can`, 
`rv-virt:knsh`, and `sim:tcploop` / `tcpblaster`) directly in the PR body.
   
   The updated commit has been pushed to the branch.
   


-- 
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]

Reply via email to