[dpdk-dev] [PATCH v1 0/3] fix some problems of mtu, vlan, lro

2021-07-16 Thread Guoyang Zhou
the problem of MTU inconsistent Guoyang Zhou (3): net/hinic: increase the protection of the VLAN net/hinic/base: fix the problem of LRO net/hinic: fix the problem of MTU inconsistent drivers/net/hinic/base/hinic_pmd_niccfg.h | 9 - drivers/net/hinic/base/hinic_pmd_nicio.c | 4

[dpdk-dev] [PATCH v1 1/3] net/hinic: increase the protection of the VLAN

2021-07-16 Thread Guoyang Zhou
If the vlan id 0 is deleted for hinic, all packets without vlan will be discarded when the vlan filter is turned on. Fixes: 50ce3e7aec8f ("ethdev: fix VLAN offloads set if no relative capabilities") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/hinic_pmd_et

[dpdk-dev] [PATCH v1 3/3] net/hinic: fix the problem of MTU inconsistent

2021-07-16 Thread Guoyang Zhou
HINIC_MAX_JUMBO_FRAME_SIZE to calculate eth_overhead, so mtu will be inconsistent in the driver and firmware. Fixes: e542ab51ab27 ("net/hinic: fix jumbo frame flag condition for MTU set") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_niccfg.h | 9

[dpdk-dev] [PATCH v1 2/3] net/hinic/base: fix the problem of LRO

2021-07-16 Thread Guoyang Zhou
The rx queue must config as ceq disable, and must set msix state disable. Otherwise when lro is enable, there will be problems with packet aggregation because of firmware. Fixes: 9d02f40d6503 ("net/hinic: fix LRO") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/

[dpdk-dev] [PATCH v1 1/2] net/hinic/base: support two or more aeqs for chip

2020-11-04 Thread Guoyang Zhou
For device initialize, driver only supports four aeqs before, and now driver can supports two or more aeqs from chip config file. Fixes: 611faa5f46cc ("fix various typos found by Lintian") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_e

[dpdk-dev] [PATCH v1 0/2] support two or more aeqs for chip

2020-11-04 Thread Guoyang Zhou
This patch fixes can support two or more aeqs from chip config file and also support four aeqs before, and fix log info for pf command channel when something is error. -- v1: - support two or more aeqs for chip - fix log info for pf command channel when something is error Guoyang Zhou (2

[dpdk-dev] [PATCH v1 2/2] net/hinic/base: fix log info for PF command channel

2020-11-04 Thread Guoyang Zhou
When PF command channel is error, the variables in the log has been cleared, which is not printed yet. Fixes: 214164a6bf7f ("net/hinic/base: remove unused function parameters") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_mgmt.c | 4 +++- 1 fi

[dpdk-dev] [PATCH v1 0/1] add message id check for command channel

2020-11-10 Thread Guoyang Zhou
In the command channel, a message may has several fragments, and the several fragments should have same message id. To prevent problems, this check is added. -- v1: - add message id check for command channel Guoyang Zhou (1): net/hinic/base: add message check for command channel drivers

[dpdk-dev] [PATCH v1 1/1] net/hinic/base: add message check for command channel

2020-11-10 Thread Guoyang Zhou
In the command channel, a message may has several fragments, and the several fragments should have same message id. To prevent problems, this check is added. Fixes: 1e4593db1d58 ("net/hinic/base: fix log info for PF command channel") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --

[dpdk-dev] [PATCH v1 0/1] fix the problem of hugepage memory leaks

2020-11-19 Thread Guoyang Zhou
The nic_io memory sets to NULL before free it, this is wrong, and the freeing of some private port resources should be moved from the ".remove(device)" function to the ".dev_close(port)" function. -- v1: - fix the problem of hugepage memory leaks Guoyang Zhou (1): net/hi

[dpdk-dev] [PATCH v1 1/1] net/hinic: fix the problem of hugepage memory leaks

2020-11-19 Thread Guoyang Zhou
d queue xstats") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_nicio.c | 1 - drivers/net/hinic/hinic_pmd_ethdev.c | 17 - 2 files changed, 8 insertions(+), 10 deletions(-) diff --git a/drivers/net/hinic/base/hinic_pmd_nicio.c b/drivers/ne

[dpdk-dev] [PATCH v2 1/1] net/hinic: fix the problem of hugepage memory leaks

2020-11-20 Thread Guoyang Zhou
d queue xstats") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- v2: - correct the incorrect words in the comments. --- drivers/net/hinic/base/hinic_pmd_nicio.c | 1 - drivers/net/hinic/hinic_pmd_ethdev.c | 17 - 2 files changed, 8 insertions(+), 10 deletions(-) diff

[dpdk-dev] [PATCH v1 0/2] remove some limitations and operations

2020-11-23 Thread Guoyang Zhou
Guoyang Zhou (2): net/hinic/base: remove the limitation for max queue num net/hinic: remove the operation of optical modules drivers/net/hinic/base/hinic_pmd_niccfg.c | 38 --- drivers/net/hinic/base/hinic_pmd_niccfg.h | 2 -- drivers/net/hinic/base

[dpdk-dev] [PATCH v1 1/2] net/hinic/base: remove the limitation for max queue num

2020-11-23 Thread Guoyang Zhou
The maximum number of queues on the chip can be any value greater than 0, it does not have to be the power of two. Fixes: 511b7371d32b ("net/hinic: fix hugepage memory leaks") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_nicio.c | 5 - 1 fi

[dpdk-dev] [PATCH v1 2/2] net/hinic: remove the operation of optical modules

2020-11-23 Thread Guoyang Zhou
In the link up and link down functions, they do not need to operate the optical module. Fixes: 511b7371d32b ("net/hinic: fix hugepage memory leaks") Cc: sta...@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_niccfg.c | 38 --- d

[dpdk-dev] [PATCH v2 0/2] remove some limitations and operations

2020-11-23 Thread Guoyang Zhou
to kernel, it will be linked failed. -- v1->v2: - add some necessary comments for patches - correct fixed patch id v1: - remove the limitation for max queue num - remove the operation of optical modules in the link function Guoyang Zhou (2): net/hinic/base: remove the limitation

[dpdk-dev] [PATCH v2 1/2] net/hinic/base: remove the limitation for max queue num

2020-11-23 Thread Guoyang Zhou
org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_nicio.c | 5 - 1 file changed, 5 deletions(-) diff --git a/drivers/net/hinic/base/hinic_pmd_nicio.c b/drivers/net/hinic/base/hinic_pmd_nicio.c index 2736305..162308b 100644 --- a/drivers/net/hinic/base/hinic_pmd_nicio.c +++

[dpdk-dev] [PATCH v2 2/2] net/hinic: remove the operation of optical modules

2020-11-23 Thread Guoyang Zhou
.@dpdk.org Signed-off-by: Guoyang Zhou --- drivers/net/hinic/base/hinic_pmd_niccfg.c | 38 --- drivers/net/hinic/base/hinic_pmd_niccfg.h | 2 -- drivers/net/hinic/hinic_pmd_ethdev.c | 14 3 files changed, 54 deletions(-) diff --git a/drivers/net/

Re: [dpdk-dev] [PATCH v1 0/2] remove some limitations and operations

2020-11-24 Thread Guoyang Zhou
Hi ferruh, I hope that these two patches can be applied. Because in that way, the version can be stable. Thanks, Guoyang zhou 在 2020/11/24 17:31, Ferruh Yigit 写道: On 11/23/2020 4:09 PM, Ferruh Yigit wrote: On 11/23/2020 1:12 PM, Guoyang Zhou wrote: The maximum number of queues on the chip

[dpdk-dev] [PATCH v1 1/1] net/hinic: fix coredump when PMD used by fstack

2021-03-15 Thread Guoyang Zhou
The fstack will use secondary process to access the memory of eth_dev_ops , and it wants to get the info of dev, but hinic driver does not initialized it when in secondary process. Fixes: 66f64dd6dc86 ("net/hinic: fix secondary process") Cc: sta...@dpdk.org Signed-off-by: Gu

[dpdk-dev] [PATCH v2 0/1] fix coredump when in secondary process

2021-03-23 Thread Guoyang Zhou
Some apps, such as fstack, will use secondary process to access the memory of eth_dev_ops, and they want to get the info of dev, but hinic driver does not initialized it when in secondary process. -- v1->v2: - modify the patch title v1: - fix coredump when PMD used by fstack Guoyang Zhou

[dpdk-dev] [PATCH v2 1/1] net/hinic: fix coredump when in secondary process

2021-03-23 Thread Guoyang Zhou
Some apps, such as fstack, will use secondary process to access the memory of eth_dev_ops, and they want to get the info of dev, but hinic driver does not initialized it when in secondary process. Fixes: 66f64dd6dc86 ("net/hinic: fix secondary process") Cc: sta...@dpdk.org Signed-off-b