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

Rewrite check_port_mtu() to use the new fields of struct rte_eth_dev_info #621

Closed
AltraMayor opened this issue Oct 27, 2022 · 2 comments
Closed

Rewrite check_port_mtu() to use the new fields of struct rte_eth_dev_info #621

AltraMayor opened this issue Oct 27, 2022 · 2 comments
Labels
enhancement
Milestone
Version 1.2

Comments

@AltraMayor
Copy link
Owner

When check_port_mtu() was first written, the fields min_mtu and max_mtu of struct rte_eth_dev_info were not available. Rewrite check_port_mtu() to take advantage of these fields.
@AltraMayor AltraMayor added this to the Version 1.2 milestone Oct 27, 2022
@AltraMayor
Copy link
Owner Author

The code of dependencies/dpdk/drivers/net/i40e/i40e_ethdev.c:i40e_dev_mtu_set() may be an useful reference for this issue.

AltraMayor added a commit that referenced this issue Mar 22, 2024
@AltraMayor
gatekeeper: rewrite MTU initialization
4ef3f91 
DPDK has changed the interface that configures the MTU of a port.
See the following DPDK commits for details:

ethdev: fix max Rx packet length
commit 1bb4a528c41f4af4847bd3d58cc2b2b9f1ec9a27

ethdev: remove jumbo offload flag
commit b563c1421282a1ec6038e5d26b4cd4fcbb01ada1

This commit closes #621.
AltraMayor added a commit that referenced this issue Mar 22, 2024
@AltraMayor
gatekeeper: rewrite MTU initialization
512cb30 
DPDK has changed the interface that configures the MTU of a port.
See the following DPDK commits for details:

ethdev: fix max Rx packet length
commit 1bb4a528c41f4af4847bd3d58cc2b2b9f1ec9a27

ethdev: remove jumbo offload flag
commit b563c1421282a1ec6038e5d26b4cd4fcbb01ada1

This commit closes #621.
@AltraMayor
Copy link
Owner Author

Pull request #680 addressed this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
Version 1.2
Development

No branches or pull requests
1 participant
@AltraMayor