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.

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

Fix recv_from allocating unnecessary memory #8187

Merged
merged 1 commit into from
Feb 10, 2020

Conversation

carllin
Copy link
Contributor

@carllin carllin commented Feb 10, 2020

Problem

recv_from() can allocate more memory in the Packets buffer than it needs

Summary of Changes

Make sure recv_from() doesn't allocate more than PACKETS_PER_BATCH number of packets

Fixes #

@codecov
Copy link

codecov bot commented Feb 10, 2020

Codecov Report

Merging #8187 into master will increase coverage by 0.0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #8187   +/-   ##
======================================
  Coverage    81.4%   81.4%           
======================================
  Files         252     252           
  Lines       54478   54497   +19     
======================================
+ Hits        44354   44371   +17     
- Misses      10124   10126    +2     

@mvines
Copy link
Member

mvines commented Feb 10, 2020

Fixes #

@carllin - does this attempt to fix #8175? I can still reproduce #8175 with this patch unfortunately.

@carllin
Copy link
Contributor Author

carllin commented Feb 10, 2020

@mvines, no it doesn't unfortunately :'(, although it probably makes the situation worse. This is a separate issue discovered while looking into that one.

@carllin carllin merged commit 0b263f8 into solana-labs:master Feb 10, 2020
mergify bot pushed a commit that referenced this pull request Feb 10, 2020
mergify bot pushed a commit that referenced this pull request Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants