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 support for i40e #440

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from
Open

Conversation

Danielybl
Copy link
Contributor

@Danielybl Danielybl commented May 7, 2019

This patch is only for IPv4.

  1. x-port DPDK/dpdk@97bd4ef from DPDK 18.02
  2. uncomment DPVS_CFG_I40E_PMD

The commit 97bd4ef9a0fecfc660231d9547f94a8df616e8df has been modified as
0003-net-i40e-support-input-set-configuration.patch.

This patch is only for IPv4.

1. x-port DPDK/dpdk@97bd4ef from DPDK 18.02
2. uncomment DPVS_CFG_I40E_PMD

The commit 97bd4ef9a0fecfc660231d9547f94a8df616e8df has been modified as
0003-net-i40e-support-input-set-configuration.patch.
Copy link
Collaborator

@ytwang0320 ytwang0320 left a comment

Choose a reason for hiding this comment

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

It seems incomplete. Also need to add support for fdir mask for each queue.

@Danielybl
Copy link
Contributor Author

It seems incomplete. Also need to add support for fdir mask for each queue.

It's a bit hacking, but it can work. The fdir mask is set on port and no need to configure it on each queue.

@ytwang0320
Copy link
Collaborator

It seems incomplete. Also need to add support for fdir mask for each queue.

It's a bit hacking, but it can work. The fdir mask is set on port and no need to configure it on each queue.

Hi Daniel,
I have tried this patch using X722 NIC. The response really can hit the core where the request send out. But the traffic for one service always work on one same core and it cannot be well distributed on each core. I feel confused if the input set configuration break RSS logic.

@Danielybl
Copy link
Contributor Author

It seems incomplete. Also need to add support for fdir mask for each queue.

It's a bit hacking, but it can work. The fdir mask is set on port and no need to configure it on each queue.

Hi Daniel,
I have tried this patch using X722 NIC. The response really can hit the core where the request send out. But the traffic for one service always work on one same core and it cannot be well distributed on each core. I feel confused if the input set configuration break RSS logic.

Hi,
I checked the RSS on DPDK18.11.0 and I can distribute. I forgot to see if RSS works on DPDK17.11. I will check it again.

Thanks,
Daniel

@oupeng007
Copy link

i40e上你们有没有发现运行一段时间高概率软件收不到包,取包i40e_rx_scan_hw_ring函数中判断 I40E_RX_DESC_STATUS_DD_SHIFT位一直为0,但是硬件是收到包了

@ywc689 ywc689 added pr/to-confirm-needs consider whether the feature of pr is needed pr/to-review-codes review codes line by line and check if problem exists. pr/needs-confirmed the feature in the pr is what we need,and list what cases should be checked in later stages and removed pr/to-confirm-needs consider whether the feature of pr is needed labels Jul 16, 2020
@ywc689 ywc689 removed the pr/to-review-codes review codes line by line and check if problem exists. label Feb 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/needs-confirmed the feature in the pr is what we need,and list what cases should be checked in later stages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants