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

[TCP] Close RAM usage optimization #6740

Merged
merged 1 commit into from
Jul 29, 2022
Merged

[TCP] Close RAM usage optimization #6740

merged 1 commit into from
Jul 29, 2022

Conversation

PetervdPerk-NXP
Copy link
Contributor

Summary

Addresses some of the impact described in #6721
Which optimizes the following PR #6398

Impact

Reduces static ram usage by ~ 200 bytes

peter@NXL04520:~/nuttx/nuttx$ arm-none-eabi-size nuttx
   text    data     bss     dec     hex filename
 143800     784   24080  168664   292d8 nuttx
peter@NXL04520:~/nuttx/nuttx$ arm-none-eabi-size nuttx
   text    data     bss     dec     hex filename
 143768     784   23856  168408   291d8 nuttx

Testing

I've briefly tested telnet and ipef, but I would like to have a proper review and test for this.

@xiaoxiang781216 xiaoxiang781216 linked an issue Jul 29, 2022 that may be closed by this pull request
NuttX with Networking Flash&RAM increase past 60 days. #6721
Closed
@xiaoxiang781216 xiaoxiang781216 merged commit 26dbdba into apache:master Jul 29, 2022
This was referenced Aug 25, 2022
Merged
Merged
@jerpelea jerpelea added this to To-Add in Release Notes - 11.0.0 Aug 30, 2022
@jerpelea jerpelea moved this from To-Add to Added in Release Notes - 11.0.0 Sep 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pkarashchenko pkarashchenko pkarashchenko approved these changes

@anchao anchao Awaiting requested review from anchao

@Ouss4 Ouss4 Awaiting requested review from Ouss4

@yamt yamt Awaiting requested review from yamt

Assignees
No one assigned
Labels
None yet
Projects
No open projects
Release Notes - 11.0.0
Added
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

NuttX with Networking Flash&RAM increase past 60 days.
3 participants
@PetervdPerk-NXP @pkarashchenko @xiaoxiang781216