Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Migrate all protocol from si_send/si_sendto/si_recvfrom to si_sendmsg/si_recvmsg #2859

Closed
xiaoxiang781216 opened this issue Feb 16, 2021 · 0 comments · Fixed by #2959
Closed

Migrate all protocol from si_send/si_sendto/si_recvfrom to si_sendmsg/si_recvmsg #2859

xiaoxiang781216 opened this issue Feb 16, 2021 · 0 comments · Fixed by #2959
Labels
Area: Modularity Needed to support modular architecture Area: Standards NuttX application interfaces must compy to standards

Comments

@xiaoxiang781216
Copy link
Contributor

xiaoxiang781216 commented Feb 16, 2021 •
edited
Loading

si_sendmsg/si_recmsg is added to support CAN bus by: #581
Since si_sendmsg/si_recmsg is superset si_send/si_sendto/si_recvfrom, it's better to:

  1. Incorporate si_send/si_sendto/si_recvfrom into si_sendmsg/si_recmsg for each protocol
  2. Implement send/sendto/recv/recvfrom on top of sendmsg/recvmsg
  3. Remove si_send/si_sendto/si_recvfrom from sock_intf_s
  4. Improve tcp/udp protocol to support scatter/gather list

Then we can get more clean and fast network stack implementation.
@xiaoxiang781216 xiaoxiang781216 added Area: Modularity Needed to support modular architecture Area: Standards NuttX application interfaces must compy to standards labels Feb 16, 2021
@xiaoxiang781216 xiaoxiang781216 linked a pull request Mar 3, 2021 that will close this issue
net/socket: move si_send/recv into sendmsg/recvmsg #2959
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: Modularity Needed to support modular architecture Area: Standards NuttX application interfaces must compy to standards
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

net/socket: move si_send/recv into sendmsg/recvmsg PeterBee97/nuttx
1 participant
@xiaoxiang781216