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

network buffers aren't freed #3

Closed
convolvatron opened this issue Jan 4, 2018 · 2 comments
Closed

network buffers aren't freed #3

convolvatron opened this issue Jan 4, 2018 · 2 comments
Assignees
@wjhun

Comments

@convolvatron
Copy link
Contributor

look at using the lwip pool support or make a heap based pool
@convolvatron
Copy link
Contributor Author

i think there is some attempt to free buffers on some paths, regardless this issue requires a good look

@convolvatron convolvatron mentioned this issue Aug 24, 2018
Closed
@wjhun wjhun self-assigned this Sep 5, 2018
@wjhun wjhun mentioned this issue Sep 6, 2018
Closed
@wjhun
Copy link
Contributor

wjhun commented Sep 7, 2018

Resolved in #64 and #69

@wjhun wjhun closed this as completed Sep 7, 2018
This was referenced Sep 22, 2018
Closed
Closed
@tijoytom-zz tijoytom-zz mentioned this issue Nov 1, 2018
Closed
@mkhon mkhon mentioned this issue Feb 22, 2019
Closed
francescolavra added a commit that referenced this issue May 21, 2021
@francescolavra
lwIP performance optimization
dee60b1 
This commit changes a few lwIP configuration options, following the
hints at https://www.nongnu.org/lwip/2_1_x/optimization.html.
The lwip_htons() and lwip_htonl() macros have been defined so that
byte order inversion operations are executed inline instead of as
function calls (in #1299, the lwip_htons() function shows up as
taking a non-negligible share of CPU time).
Instead of the default checksum algorithm #2, algorithm #3 is now
being used, because it is faster on 64-bit platforms (checksum
calculation on 20-byte data is around 35% faster).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wjhun wjhun

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@convolvatron @wjhun @raocool