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

copy packet in control path to avoid pool starvation #111

Closed
christophefontaine opened this issue Dec 17, 2024 · 2 comments
Closed

copy packet in control path to avoid pool starvation #111

christophefontaine opened this issue Dec 17, 2024 · 2 comments

Comments

@christophefontaine
Copy link
Collaborator 

          Just a note that holding on this packet in control path means that the data path packet pool may end up in a starvation situation. Imagine many packets being queued, with unresolved nh, and no packet left in the packet pool to receive a arp reply.

Grout would probably break from this situation later (as packets are held with a timeout), but still it would be more robust to hold on a packet copy and free the original mbuf.

Originally posted by @david-marchand in #109 (comment)
@david-marchand
Copy link
Member

david-marchand commented Dec 18, 2024 •
edited
Loading

Duplicate of issue #110 ?

@christophefontaine
Copy link
Collaborator Author

indeed, closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@christophefontaine @david-marchand