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

Add uSID endpoint enums for udx and udt #2098

Closed

Conversation

kperumalbfn
Copy link
Contributor

Enums for uDX4, uDX6, uDT4, uDT6 and uDT46

@kperumalbfn
Copy link
Contributor Author

@ashutosh-agrawal Please review

inc/saisrv6.h Outdated Show resolved Hide resolved
Copy link

@ahsalam ahsalam left a comment

Choose a reason for hiding this comment

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

The changes looks good. Thanks @kperumalbfn

@tjchadaga tjchadaga added the reviewed PR is discussed in SAI Meeting label Nov 14, 2024
kperumalbfn and others added 4 commits November 14, 2024 18:31
Signed-off-by: Kumaresh Perumal <[email protected]>
* Next hop group with members.

    Introduce a new type of Next hop group that allows the application
    to provide the list of next hops that are members of the Next
    hop group along with the list of weights.

Signed-off-by: Kishore Gummadidala <[email protected]>
Co-authored-by: Kishore Gummadidala <[email protected]>
Signed-off-by: Kumaresh Perumal <[email protected]>
* Prefix Compression feature addition to SAI

Signed-off-by: nshinoud <[email protected]>

Change BINCODE to META - PR change request

Signed-off-by: nshinoud <[email protected]>

Additional changes based on requests from git review
- Removed default meta attribute
- Added stage for table creation
- updated docs

Signed-off-by: nshinoud <[email protected]>

Fix typo: Changes SAI_ACL_TABLE_ATTR_PREFIX_COMPRESSION_STAGE to SAI_TABLE_ATTR_PREFIX_COMPRESSION_STAGE

Signed-off-by: nshinoud <[email protected]>

Fix metadata script error

Signed-off-by: nshinoud <[email protected]>

Another attempt to fix metadata issue. For some reason this passes on my local build

Signed-off-by: nshinoud <[email protected]>

Add support for Prefix Compression Table type: SRC, DST, BOTH

Signed-off-by: nshinoud <[email protected]>

Minor fix (change bincode to meta)

Signed-off-by: nshinoud <[email protected]>

Update comment

Signed-off-by: nshinoud <[email protected]>

* PR changes: Update an attribute and documentation

Signed-off-by: nshinoud <[email protected]>

* Update Docs: Packet pipeline

Signed-off-by: nshinoud <[email protected]>

* Update Prefix Compression Documentation: Pipeline

Signed-off-by: nshinoud <[email protected]>

---------

Signed-off-by: nshinoud <[email protected]>
Signed-off-by: Kumaresh Perumal <[email protected]>
@kperumalbfn
Copy link
Contributor Author

Closing this PR due to sanity check issue and force push failed. Opened #2102

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
reviewed PR is discussed in SAI Meeting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants