Repository navigation
[ACL] Add support for REDIRECT_ORIGINAL_PACKET action - #2318
Merged
tjchadaga merged 1 commit intoOct 6, 2026
Merged
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
mobinmohan
force-pushed
the
redirect_original_packet
branch
2 times, most recently
from
July 20, 2026 22:26
556a3d4 to
0b78706
Compare
rck-innovium
approved these changes
Jul 23, 2026
Collaborator
|
@mobinmohan - could you please help address the comments? |
mobinmohan
force-pushed
the
redirect_original_packet
branch
3 times, most recently
from
October 2, 2026 21:45
fd7c494 to
5bf13c7
Compare
mobinmohan
commented
Oct 2, 2026
mobinmohan
left a comment
mobinmohan
left a comment
Contributor
Author
There was a problem hiding this comment.
Responded to comments.
Contributor
Author
@tjchadaga I have responded to the comments. |
Add SAI_ACL_ENTRY_ATTR_ACTION_REDIRECT_ORIGINAL_PACKET action to support redirecting packets to a physical Port or LAG destination same as received by the switch. This action sends packets to a Port or LAG without modifying them. It skips the egress stages like L2/L3 rewrite and VLAN editing, so the headers stay exactly the same as when they were first received by the switch. Signed-off-by: Mobin Mohan <mobinmohan@google.com>
mobinmohan
force-pushed
the
redirect_original_packet
branch
from
October 2, 2026 22:10
5bf13c7 to
7c3b5b4
Compare
AnandhiDhanabalan
approved these changes
Oct 3, 2026
jakumar
approved these changes
Oct 3, 2026
Collaborator
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
tjchadaga
approved these changes
Oct 6, 2026
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
6 participants
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.
Add SAI_ACL_ENTRY_ATTR_ACTION_REDIRECT_ORIGINAL_PACKET action to redirect packets to the destination without modification. This action is used to redirect the original copy of packet to a front panel port/lag.
How is this different from SAI_ACL_ACTION_TYPE_REDIRECT ?
Existing SAI_ACL_ACTION_TYPE_REDIRECT allows packets to go through egress editing(like L2/L3 rewrites) after ACL lookup. However, this new action sends packets to a Port or LAG without modifying them. It skips the egress stages like L2/L3 rewrite and VLAN editing, so the headers stay exactly the same as when they were first received by the switch.