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

ISIS trap support for SAI #1238

Merged
merged 9 commits into from
Jul 6, 2021

Conversation

VenkatCisco
Copy link
Contributor

Signed-off-by: Venkat Garigipati venkatg@cisco.com

Extends SAI-v1.8 capabilities to support new ISIS header SAI_HOSTIF_TRAP_TYPE_ISIS.

@VenkatCisco VenkatCisco marked this pull request as ready for review May 20, 2021 15:19
@anshuv-mfst
Copy link

@rlhui - Could you please advise next steps on this PR. Maybe schedule in SAI community meeting.

inc/saihostif.h Outdated Show resolved Hide resolved
inc/saihostif.h Outdated Show resolved Hide resolved
inc/saihostif.h Outdated
@@ -322,6 +322,14 @@ typedef enum _sai_hostif_trap_type_t
/** Default packet action is forward */
SAI_HOSTIF_TRAP_TYPE_IPV6_NEIGHBOR_ADVERTISEMENT = 0x00002013,

/**
* @brief Intermediate System-to-Intermediate System (IS-IS) protocol
* traffic: L1: 01:80:c2:00:00:15, L2: 01:80:c2:00:00:15
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L1 mac should probably be :14 and not :15

@rlhui
Copy link
Collaborator

rlhui commented Jun 23, 2021

could the DCO be fixed?

@VenkatCisco
Copy link
Contributor Author

could the DCO be fixed?

Sure, Done.

@rlhui
Copy link
Collaborator

rlhui commented Jun 30, 2021

could the DCO be fixed?

Sure, Done.

Hi, it still shows DCO not yet passed? Thanks.

inc/saihostif.h Outdated Show resolved Hide resolved
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
SAI_HOSTIF_TRAP_TYPE_ISIS.

Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
@lguohan lguohan merged commit 68be575 into opencomputeproject:master Jul 6, 2021
tomzhu1 pushed a commit to Metaswitch/SAI that referenced this pull request Jul 20, 2021
* Extends SAI-v1.8 capabilities to support new ISIS header
SAI_HOSTIF_TRAP_TYPE_ISIS.

Signed-off-by: Venkat Garigipati <venkatg@cisco.com>
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.

7 participants