You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/12/21 04:23:35 UTC

[GitHub] [incubator-nuttx] anchao opened a new pull request #5045: net/usrsock: increase the sendto() length limit to UINT32_MAX

anchao opened a new pull request #5045:
URL: https://github.com/apache/incubator-nuttx/pull/5045


   ## Summary
   
   net/usrsock: increase the sendto() length limit to UINT32_MAX 
   
   change request type to uint32_t to the impove the throughput
   
   Signed-off-by: chao.an <an...@xiaomi.com>
   
   ## Impact
   
   prototype changes need to recompile usrsock client/server
   
   ## Testing
   
   sim usrsock test with vnc server


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #5045: net/usrsock: increase the sendto() length limit to UINT32_MAX

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #5045:
URL: https://github.com/apache/incubator-nuttx/pull/5045


   


-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] anchao commented on a change in pull request #5045: net/usrsock: increase the sendto() length limit to UINT32_MAX

Posted by GitBox <gi...@apache.org>.
anchao commented on a change in pull request #5045:
URL: https://github.com/apache/incubator-nuttx/pull/5045#discussion_r772838104



##########
File path: include/nuttx/net/usrsock.h
##########
@@ -155,7 +155,7 @@ begin_packed_struct struct usrsock_request_sendto_s
   int16_t usockid;
   int32_t flags;
   uint16_t addrlen;
-  uint16_t buflen;
+  uint32_t buflen;

Review comment:
       Done




-- 
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-unsubscribe@nuttx.apache.org

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



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on a change in pull request #5045: net/usrsock: increase the sendto() length limit to UINT32_MAX

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on a change in pull request #5045:
URL: https://github.com/apache/incubator-nuttx/pull/5045#discussion_r772821727



##########
File path: include/nuttx/net/usrsock.h
##########
@@ -155,7 +155,7 @@ begin_packed_struct struct usrsock_request_sendto_s
   int16_t usockid;
   int32_t flags;
   uint16_t addrlen;
-  uint16_t buflen;
+  uint32_t buflen;

Review comment:
       let's modify recv side too




-- 
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-unsubscribe@nuttx.apache.org

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