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

LACP Mode Support for LAG #79

Merged
merged 4 commits into from
Jan 22, 2024
Merged

LACP Mode Support for LAG #79

merged 4 commits into from
Jan 22, 2024

Conversation

aashishkuma
Copy link
Collaborator

This PR includes the code changes done in krnlmon module to support LAG in LACP mode.

Copy link
Contributor

@ffoulkes ffoulkes left a comment

Choose a reason for hiding this comment

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

LGTM

I've (1) offered some suggested improvements and (2) pointed out a few possible errors, but I didn't see any definite showstoppers.

switchapi/es2k/switch_lag.c Outdated Show resolved Hide resolved
switchapi/es2k/switch_lag.c Outdated Show resolved Hide resolved
switchapi/es2k/switch_lag.c Outdated Show resolved Hide resolved
switchapi/es2k/switch_lag.c Outdated Show resolved Hide resolved
switchapi/es2k/switch_lag.c Outdated Show resolved Hide resolved
switchlink/sai/switchlink_handle_lag.c Outdated Show resolved Hide resolved
* SWITCHLINK_DB_STATUS_ITEM_NOT_FOUND otherwise
*/
switchlink_db_status_t switchlink_db_update_lag_member_oper_state(
switchlink_db_lag_member_info_t* lag_member_info) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can lag_member_info be a pointer to const?

switchlink_db_status_t switchlink_db_update_lag_member_oper_state(
switchlink_db_lag_member_info_t* lag_member_info) {
krnlmon_assert(lag_member_info != NULL);
tommy_node* node = tommy_list_head(&switchlink_db_lag_member_obj_list);
Copy link
Contributor

Choose a reason for hiding this comment

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

Can node be a pointer to const?

switchsai/sailag.c Show resolved Hide resolved
switchsai/sailag.c Show resolved Hide resolved
@ffoulkes ffoulkes added the major effort Significant effort required label Jan 7, 2024
Signed-off-by: Kumar, Aashish <[email protected]>
switchapi/es2k/switch_lag.c Show resolved Hide resolved
switchlink/sai/switchlink_handle_lag.c Show resolved Hide resolved
switchlink/sai/switchlink_handle_lag.c Show resolved Hide resolved
switchlink/sai/switchlink_handle_lag.c Outdated Show resolved Hide resolved
Signed-off-by: root <[email protected]>
Copy link
Contributor

@n-sandeep n-sandeep left a comment

Choose a reason for hiding this comment

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

LGTM

@aashishkuma aashishkuma merged commit 56cbef8 into main Jan 22, 2024
4 checks passed
@ffoulkes ffoulkes deleted the LACP_Mode_Support branch March 5, 2024 01:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
major effort Significant effort required
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants