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 switch attributes for hash offset configuration. #1195

Merged
merged 5 commits into from
Mar 18, 2021

Conversation

mikeberesford
Copy link
Contributor

Add ability to specify the hash offset (rotation) in addition to the
hash seed.

Signed-off-by: Mike Beresford mberes@google.com

Add ability to specify the hash offset (rotation) in addition to the
hash seed.

Signed-off-by: Mike Beresford <mberes@google.com>
Add ability to specify the hash offset (rotation) in addition to the
hash seed.

Signed-off-by: Mike Beresford <mberes@google.com>
Signed-off-by: Mike Beresford <mberes@google.com>
@mikeberesford
Copy link
Contributor Author

@JaiOCP @lguohan would you take a look and see if this covers what we discussed today (and help tag others that should review?)

@mikeberesford
Copy link
Contributor Author

@rlhui are there others I should be tagging for review here?

@rlhui
Copy link
Collaborator

rlhui commented Mar 3, 2021

@rlhui are there others I should be tagging for review here?

@marian-pritsak , @ashutosh-agrawal - please help review as well? Thanks.

inc/saiswitch.h Show resolved Hide resolved
inc/saiswitch.h Show resolved Hide resolved
inc/saiswitch.h Outdated Show resolved Hide resolved
Signed-off-by: Mike Beresford <mberes@google.com>
@mikeberesford
Copy link
Contributor Author

@lguohan any objections to the revised wording? If not, I think this can be merged.

@rlhui rlhui merged commit f053d89 into opencomputeproject:master Mar 18, 2021
@mikeberesford mikeberesford deleted the hashing_offset branch June 3, 2021 15:28
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