[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #6780: iob_s add io_private filed, user can use it to keep context

[GitBox <gi...@apache.org>]

pkarashchenko commented on PR #6780:
URL: https://github.com/apache/incubator-nuttx/pull/6780#issuecomment-1205203626

   The main issue is that all the network and not only the network use a single IOB mechanism. That has more drawbacks that advantages IMO, but still it is what we have now. For example LWIP pbuf allows you to chain the buffers and add any meta information into a chain if needed. Also each pbuf can be allocated with customised size. The NuttX IOB interface is simpler from this point of view, but quite a compact fixed sized interface.
   When I observe the current change I see that we are adding 8 bytes (for 32-bit systems) per IOB with no gain (4 for private data and 4 for pointer to data) in case if IP reassembly is not needed. That is quite a big punch to a systems with limited amount of memory.
   I agree that showing the draft version of IP reassembly and getting feedback on design points is the best way to move forward.


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