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

kernel/os: Fix os_msys_get() with several pools #3299

Merged
merged 2 commits into from
Sep 6, 2024

Conversation

kasjer
Copy link
Contributor

@kasjer kasjer commented Aug 23, 2024

When msys was configured with several mpools and one
of the pools run out of buffers, os_mbuf_get() returned
NULL even though other mpool did have buffers available.

Now search for suitable buffer checks if suitable pool
is empty and if not search continues.

Fixes #3234

@kasjer kasjer requested a review from sjanc August 23, 2024 10:02
@kasjer kasjer force-pushed the kasjer/msys-multi-mpool-fix branch 2 times, most recently from 3839468 to d054c6e Compare August 23, 2024 10:14
kasjer added 2 commits August 23, 2024 12:16
Tests check behavior when msys works on 1 or more mpools.
When msys was configured with several mpools and one
of the pools run out of buffers, os_mbuf_get() returned
NULL even though other mpool did have buffers available.

Now search for suitable buffer checks if suitable pool
is empty and if not search continues.
@kasjer kasjer merged commit ddae64e into apache:master Sep 6, 2024
19 checks passed
@kasjer kasjer deleted the kasjer/msys-multi-mpool-fix branch September 6, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants