* [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver
@ 2024-03-25 12:43 Jijie Shao
2024-03-25 12:43 ` [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats Jijie Shao
` (3 more replies)
0 siblings, 4 replies; 8+ messages in thread
From: Jijie Shao @ 2024-03-25 12:43 UTC (permalink / raw
To: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri
Cc: shenjian15, wangjie125, liuyonglong, shaojijie, netdev,
linux-kernel
There are some bugfix for the HNS3 ethernet driver
---
changeLog:
v3 -> v4:
- Fixed an unclear sentence in commit log, suggested by Michal Kubiak
- Used devl_lock to disallow reload to race with initialization, suggested by Jiri Pirko
v3: https://lore.kernel.org/all/20240318132948.3624333-1-shaojijie@huawei.com/
v2 -> v3:
- Fixed a syntax error in commit log, suggested by Ratheesh Kannoth
v2: https://lore.kernel.org/all/20240316111057.277591-1-shaojijie@huawei.com/
v1 -> v2:
- Fixed some syntax errors in commit log and comments, suggested by Michal Kubiak
- Optimized the code by using a flag, suggested by Michal Kubiak
v1: https://lore.kernel.org/all/20240315100748.2913882-1-shaojijie@huawei.com/
---
Jian Shen (1):
net: hns3: mark unexcuted loopback test result as UNEXECUTED
Jie Wang (1):
net: hns3: fix index limit to support all queue stats
Yonglong Liu (1):
net: hns3: fix kernel crash when devlink reload during pf
initialization
.../hns3/hns3_common/hclge_comm_tqp_stats.c | 2 +-
.../ethernet/hisilicon/hns3/hns3_ethtool.c | 19 +++++++++++++++++--
.../hisilicon/hns3/hns3pf/hclge_main.c | 4 ++++
3 files changed, 22 insertions(+), 3 deletions(-)
--
2.30.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats
2024-03-25 12:43 [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver Jijie Shao
@ 2024-03-25 12:43 ` Jijie Shao
2024-03-26 11:22 ` Simon Horman
2024-03-25 12:43 ` [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization Jijie Shao
` (2 subsequent siblings)
3 siblings, 1 reply; 8+ messages in thread
From: Jijie Shao @ 2024-03-25 12:43 UTC (permalink / raw
To: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri
Cc: shenjian15, wangjie125, liuyonglong, shaojijie, netdev,
linux-kernel
From: Jie Wang <wangjie125@huawei.com>
Currently, hns hardware supports more than 512 queues and the index limit
in hclge_comm_tqps_update_stats is wrong. So this patch removes it.
Fixes: 287db5c40d15 ("net: hns3: create new set of common tqp stats APIs for PF and VF reuse")
Signed-off-by: Jie Wang <wangjie125@huawei.com>
Signed-off-by: Jijie Shao <shaojijie@huawei.com>
Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
---
.../ethernet/hisilicon/hns3/hns3_common/hclge_comm_tqp_stats.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_tqp_stats.c b/drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_tqp_stats.c
index f3c9395d8351..618f66d9586b 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_tqp_stats.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3_common/hclge_comm_tqp_stats.c
@@ -85,7 +85,7 @@ int hclge_comm_tqps_update_stats(struct hnae3_handle *handle,
hclge_comm_cmd_setup_basic_desc(&desc, HCLGE_OPC_QUERY_TX_STATS,
true);
- desc.data[0] = cpu_to_le32(tqp->index & 0x1ff);
+ desc.data[0] = cpu_to_le32(tqp->index);
ret = hclge_comm_cmd_send(hw, &desc, 1);
if (ret) {
dev_err(&hw->cmq.csq.pdev->dev,
--
2.30.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization
2024-03-25 12:43 [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver Jijie Shao
2024-03-25 12:43 ` [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats Jijie Shao
@ 2024-03-25 12:43 ` Jijie Shao
2024-03-26 11:22 ` Simon Horman
2024-03-25 12:43 ` [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED Jijie Shao
2024-03-26 14:40 ` [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver patchwork-bot+netdevbpf
3 siblings, 1 reply; 8+ messages in thread
From: Jijie Shao @ 2024-03-25 12:43 UTC (permalink / raw
To: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri
Cc: shenjian15, wangjie125, liuyonglong, shaojijie, netdev,
linux-kernel
From: Yonglong Liu <liuyonglong@huawei.com>
The devlink reload process will access the hardware resources,
but the register operation is done before the hardware is initialized.
So, processing the devlink reload during initialization may lead to kernel
crash. This patch fixes this by taking devl_lock during initialization.
Fixes: b741269b2759 ("net: hns3: add support for registering devlink for PF")
Signed-off-by: Yonglong Liu <liuyonglong@huawei.com>
Signed-off-by: Jijie Shao <shaojijie@huawei.com>
---
drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
index b4afb66efe5c..ff6a2ed23ddb 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3pf/hclge_main.c
@@ -11626,6 +11626,8 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
if (ret)
goto err_pci_uninit;
+ devl_lock(hdev->devlink);
+
/* Firmware command queue initialize */
ret = hclge_comm_cmd_queue_init(hdev->pdev, &hdev->hw.hw);
if (ret)
@@ -11805,6 +11807,7 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
hclge_task_schedule(hdev, round_jiffies_relative(HZ));
+ devl_unlock(hdev->devlink);
return 0;
err_mdiobus_unreg:
@@ -11817,6 +11820,7 @@ static int hclge_init_ae_dev(struct hnae3_ae_dev *ae_dev)
err_cmd_uninit:
hclge_comm_cmd_uninit(hdev->ae_dev, &hdev->hw.hw);
err_devlink_uninit:
+ devl_unlock(hdev->devlink);
hclge_devlink_uninit(hdev);
err_pci_uninit:
pcim_iounmap(pdev, hdev->hw.hw.io_base);
--
2.30.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED
2024-03-25 12:43 [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver Jijie Shao
2024-03-25 12:43 ` [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats Jijie Shao
2024-03-25 12:43 ` [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization Jijie Shao
@ 2024-03-25 12:43 ` Jijie Shao
2024-03-26 11:30 ` Simon Horman
2024-03-26 14:40 ` [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver patchwork-bot+netdevbpf
3 siblings, 1 reply; 8+ messages in thread
From: Jijie Shao @ 2024-03-25 12:43 UTC (permalink / raw
To: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri
Cc: shenjian15, wangjie125, liuyonglong, shaojijie, netdev,
linux-kernel
From: Jian Shen <shenjian15@huawei.com>
Currently, loopback test may be skipped when resetting, but the test
result will still show as 'PASS', because the driver doesn't set
ETH_TEST_FL_FAILED flag. Fix it by setting the flag and
initializating the value to UNEXECUTED.
Fixes: 4c8dab1c709c ("net: hns3: reconstruct function hns3_self_test")
Signed-off-by: Jian Shen <shenjian15@huawei.com>
Signed-off-by: Jijie Shao <shaojijie@huawei.com>
Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
---
.../ethernet/hisilicon/hns3/hns3_ethtool.c | 19 +++++++++++++++++--
1 file changed, 17 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c b/drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c
index 999a0ee162a6..941cb529d671 100644
--- a/drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c
+++ b/drivers/net/ethernet/hisilicon/hns3/hns3_ethtool.c
@@ -78,6 +78,9 @@ static const struct hns3_stats hns3_rxq_stats[] = {
#define HNS3_NIC_LB_TEST_NO_MEM_ERR 1
#define HNS3_NIC_LB_TEST_TX_CNT_ERR 2
#define HNS3_NIC_LB_TEST_RX_CNT_ERR 3
+#define HNS3_NIC_LB_TEST_UNEXECUTED 4
+
+static int hns3_get_sset_count(struct net_device *netdev, int stringset);
static int hns3_lp_setup(struct net_device *ndev, enum hnae3_loop loop, bool en)
{
@@ -418,18 +421,26 @@ static void hns3_do_external_lb(struct net_device *ndev,
static void hns3_self_test(struct net_device *ndev,
struct ethtool_test *eth_test, u64 *data)
{
+ int cnt = hns3_get_sset_count(ndev, ETH_SS_TEST);
struct hns3_nic_priv *priv = netdev_priv(ndev);
struct hnae3_handle *h = priv->ae_handle;
int st_param[HNAE3_LOOP_NONE][2];
bool if_running = netif_running(ndev);
+ int i;
+
+ /* initialize the loopback test result, avoid marking an unexcuted
+ * loopback test as PASS.
+ */
+ for (i = 0; i < cnt; i++)
+ data[i] = HNS3_NIC_LB_TEST_UNEXECUTED;
if (hns3_nic_resetting(ndev)) {
netdev_err(ndev, "dev resetting!");
- return;
+ goto failure;
}
if (!(eth_test->flags & ETH_TEST_FL_OFFLINE))
- return;
+ goto failure;
if (netif_msg_ifdown(h))
netdev_info(ndev, "self test start\n");
@@ -451,6 +462,10 @@ static void hns3_self_test(struct net_device *ndev,
if (netif_msg_ifdown(h))
netdev_info(ndev, "self test end\n");
+ return;
+
+failure:
+ eth_test->flags |= ETH_TEST_FL_FAILED;
}
static void hns3_update_limit_promisc_mode(struct net_device *netdev,
--
2.30.0
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization
2024-03-25 12:43 ` [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization Jijie Shao
@ 2024-03-26 11:22 ` Simon Horman
0 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2024-03-26 11:22 UTC (permalink / raw
To: Jijie Shao
Cc: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri, shenjian15, wangjie125,
liuyonglong, netdev, linux-kernel
On Mon, Mar 25, 2024 at 08:43:10PM +0800, Jijie Shao wrote:
> From: Yonglong Liu <liuyonglong@huawei.com>
>
> The devlink reload process will access the hardware resources,
> but the register operation is done before the hardware is initialized.
> So, processing the devlink reload during initialization may lead to kernel
> crash. This patch fixes this by taking devl_lock during initialization.
>
> Fixes: b741269b2759 ("net: hns3: add support for registering devlink for PF")
> Signed-off-by: Yonglong Liu <liuyonglong@huawei.com>
> Signed-off-by: Jijie Shao <shaojijie@huawei.com>
Reviewed-by: Simon Horman <horms@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats
2024-03-25 12:43 ` [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats Jijie Shao
@ 2024-03-26 11:22 ` Simon Horman
0 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2024-03-26 11:22 UTC (permalink / raw
To: Jijie Shao
Cc: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri, shenjian15, wangjie125,
liuyonglong, netdev, linux-kernel
On Mon, Mar 25, 2024 at 08:43:09PM +0800, Jijie Shao wrote:
> From: Jie Wang <wangjie125@huawei.com>
>
> Currently, hns hardware supports more than 512 queues and the index limit
> in hclge_comm_tqps_update_stats is wrong. So this patch removes it.
>
> Fixes: 287db5c40d15 ("net: hns3: create new set of common tqp stats APIs for PF and VF reuse")
> Signed-off-by: Jie Wang <wangjie125@huawei.com>
> Signed-off-by: Jijie Shao <shaojijie@huawei.com>
> Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
> Reviewed-by: Kalesh AP <kalesh-anakkur.purayil@broadcom.com>
Reviewed-by: Simon Horman <horms@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED
2024-03-25 12:43 ` [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED Jijie Shao
@ 2024-03-26 11:30 ` Simon Horman
0 siblings, 0 replies; 8+ messages in thread
From: Simon Horman @ 2024-03-26 11:30 UTC (permalink / raw
To: Jijie Shao
Cc: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri, shenjian15, wangjie125,
liuyonglong, netdev, linux-kernel
On Mon, Mar 25, 2024 at 08:43:11PM +0800, Jijie Shao wrote:
> From: Jian Shen <shenjian15@huawei.com>
>
> Currently, loopback test may be skipped when resetting, but the test
> result will still show as 'PASS', because the driver doesn't set
> ETH_TEST_FL_FAILED flag. Fix it by setting the flag and
> initializating the value to UNEXECUTED.
>
> Fixes: 4c8dab1c709c ("net: hns3: reconstruct function hns3_self_test")
> Signed-off-by: Jian Shen <shenjian15@huawei.com>
> Signed-off-by: Jijie Shao <shaojijie@huawei.com>
> Reviewed-by: Michal Kubiak <michal.kubiak@intel.com>
Reviewed-by: Simon Horman <horms@kernel.org>
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver
2024-03-25 12:43 [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver Jijie Shao
` (2 preceding siblings ...)
2024-03-25 12:43 ` [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED Jijie Shao
@ 2024-03-26 14:40 ` patchwork-bot+netdevbpf
3 siblings, 0 replies; 8+ messages in thread
From: patchwork-bot+netdevbpf @ 2024-03-26 14:40 UTC (permalink / raw
To: Jijie Shao
Cc: yisen.zhuang, salil.mehta, davem, edumazet, kuba, pabeni,
michal.kubiak, rkannoth, jiri, shenjian15, wangjie125,
liuyonglong, netdev, linux-kernel
Hello:
This series was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@redhat.com>:
On Mon, 25 Mar 2024 20:43:08 +0800 you wrote:
> There are some bugfix for the HNS3 ethernet driver
>
> ---
> changeLog:
> v3 -> v4:
> - Fixed an unclear sentence in commit log, suggested by Michal Kubiak
> - Used devl_lock to disallow reload to race with initialization, suggested by Jiri Pirko
> v3: https://lore.kernel.org/all/20240318132948.3624333-1-shaojijie@huawei.com/
> v2 -> v3:
> - Fixed a syntax error in commit log, suggested by Ratheesh Kannoth
> v2: https://lore.kernel.org/all/20240316111057.277591-1-shaojijie@huawei.com/
> v1 -> v2:
> - Fixed some syntax errors in commit log and comments, suggested by Michal Kubiak
> - Optimized the code by using a flag, suggested by Michal Kubiak
> v1: https://lore.kernel.org/all/20240315100748.2913882-1-shaojijie@huawei.com/
>
> [...]
Here is the summary with links:
- [V4,net,1/3] net: hns3: fix index limit to support all queue stats
https://git.kernel.org/netdev/net/c/47e39d213e09
- [V4,net,2/3] net: hns3: fix kernel crash when devlink reload during pf initialization
https://git.kernel.org/netdev/net/c/93305b77ffcb
- [V4,net,3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED
https://git.kernel.org/netdev/net/c/5bd088d6c21a
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2024-03-26 14:40 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-03-25 12:43 [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver Jijie Shao
2024-03-25 12:43 ` [PATCH V4 net 1/3] net: hns3: fix index limit to support all queue stats Jijie Shao
2024-03-26 11:22 ` Simon Horman
2024-03-25 12:43 ` [PATCH V4 net 2/3] net: hns3: fix kernel crash when devlink reload during pf initialization Jijie Shao
2024-03-26 11:22 ` Simon Horman
2024-03-25 12:43 ` [PATCH V4 net 3/3] net: hns3: mark unexcuted loopback test result as UNEXECUTED Jijie Shao
2024-03-26 11:30 ` Simon Horman
2024-03-26 14:40 ` [PATCH V4 net 0/3] There are some bugfix for the HNS3 ethernet driver patchwork-bot+netdevbpf
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.