wengzhe opened a new pull request, #8947:
URL: https://github.com/apache/nuttx/pull/8947

   ## Summary
   Patches included:
   - usrsock_server: Use remain instead of iov[0] to figure out partial requests
     - Found a problem:
       When sendto handler gets an error, it will release all its rx buffer,
       then iov_base becomes NULL. But it cannot let client stop its request,
       then the next data from client cannot be handled by usrsock server
       correctly.
       It's better to note down the remaining bytes, then we can stop at
       correct time.
   - usrsock_server: Raise error earlier for large sendto request
     - It's better to raise error before client sends its (NIOVEC+1)th buffer
       (and release buffers held by server), otherwise the client may stuck at
       getting (NIOVEC+1)th tx buffer if NIOVEC is equal to rpmsg buffer num.
   
   ## Impact
   sendto logic (mainly error handling) on usrsock rpmsg server.
   
   ## Testing
   Manually & CI
   


-- 
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: commits-unsubscr...@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to