gatekeeper: move from the LPM libraries to the FIB libraries#618
Closed
AltraMayor wants to merge 5 commits intomasterfrom
Closed
gatekeeper: move from the LPM libraries to the FIB libraries#618AltraMayor wants to merge 5 commits intomasterfrom
AltraMayor wants to merge 5 commits intomasterfrom
Conversation
The macro G_LOG() in header "gatekeeper_main.h" uses the function rte_lcore_id(), which is defined in header <rte_lcore.h>.
DPDK will eventually drop the library rte_lpm in favor of the newest library rte_fib.
Not only will DPDK eventually drop the library rte_lpm6 in favor of the newest library rte_fib6, but the library rte_lpm6 has also been misbehaving under production edit loads.
The old libraries rte_lpm and rte_lpm6 do not support the zero-length prefix, but the new libraries rte_fib and rte_fib6 do. This commit removes the previous restrictions. This commit impacts the routing table of Gatekeeper servers, and the LPM tables used on Lua policies. This commit closes #91.
This patch logs out of memory for FIB and RIB to help operators to adjust the dimensions of their routing table as well as reports these errors to callers. Before this patch, the error code would not report out-of-memory errors because the return chain would change the error code.
Owner
Author
|
Library This pull request is going to be frozen until we implement our own FIB library. |
Owner
Author
|
Pull request #628 supersedes this pull request. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request replaces library
rte_lpmwithrte_fib, and libraryrte_lpm6withrte_fib6. Although DPDK will eventually drop the LPM libraries in favor of the newest FIB libraries, the motivation for this pull request is that the libraryrte_lpm6has been misbehaving under production edit loads.Thanks to the new libraries, this pull request enables the zero-length prefix on the routing table of Gatekeeper servers, and the LPM tables used on Lua policies.