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.

Sign up for GitHub

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

Jump to bottom

feature: mtu config #716

Merged
merged 10 commits into from
Mar 31, 2021
Merged

feature: mtu config #716

merged 10 commits into from
Mar 31, 2021

Conversation

zos43
Copy link
Contributor

@zos43 zos43 commented Feb 24, 2021

Add mtu configuration to the conf file 。

Set Bond mtu to all slave nics

DPDK 18.11 Bond driver add a function “bond_ethdev_mtu_set”

@ywc689 ywc689 added the pr/to-review-codes review codes line by line and check if problem exists. label Feb 26, 2021
ywc689
ywc689 reviewed Feb 26, 2021
View reviewed changes
Copy link
Collaborator

@ywc689 ywc689 left a comment

Choose a reason for hiding this comment

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

Please fix the alignments in the changed files. Do not use tab in alignments.

@zos43
Copy link
Contributor Author

zos43 commented Mar 1, 2021

done

@ywc689 ywc689 added pr/to-test-codes Compile and test the patch of pr to verify if it works. pr/codes-reviewed-ok code review passed and no problem found and removed pr/to-review-codes review codes line by line and check if problem exists. labels Mar 10, 2021
@zos43 zos43 closed this Mar 11, 2021
@zos43 zos43 reopened this Mar 11, 2021
@ywc689 ywc689 added pr/codes-tested-ok compile ok and specified tests passed pr/accepted the pr passed all review stages and await to be merged and removed pr/to-test-codes Compile and test the patch of pr to verify if it works. labels Mar 31, 2021
@ywc689
Copy link
Collaborator

ywc689 commented Mar 31, 2021

DPVS should support jumbo frame to make better use of this feature.

@ywc689 ywc689 merged commit 64c4ec2 into iqiyi:devel Mar 31, 2021
@zos43 zos43 deleted the devel branch March 31, 2021 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ywc689 ywc689 ywc689 approved these changes

Assignees
No one assigned
Labels
pr/accepted the pr passed all review stages and await to be merged pr/codes-reviewed-ok code review passed and no problem found pr/codes-tested-ok compile ok and specified tests passed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@zos43 @ywc689