net: hns3: call hns3_nic_set_real_num_queue with netdev down
hns3_client_setup_tc in enet is for updating TC configuration to stack, and hclge_setup_tc in hclge_dcb is mainly for setting the configuration to hardware. This patch removes the hns3_nic_set_real_num_queue from hns3_setup_tc in enet, and call hclge_client_setup_tc to update TC configuration to stack with netdev down, because the netdev down operation is done in hclge_dcb now. Signed-off-by: Yunsheng Lin <linyunsheng@huawei.com> Signed-off-by: Peng Li <lipeng321@huawei.com> Signed-off-by: Huazhong Tan <tanhuazhong@huawei.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
parent
a1ef124e6e
commit
1cce5eb650
|
@ -1482,7 +1482,6 @@ static int hns3_setup_tc(struct net_device *netdev, void *type_data)
|
||||||
u8 tc = mqprio_qopt->qopt.num_tc;
|
u8 tc = mqprio_qopt->qopt.num_tc;
|
||||||
u16 mode = mqprio_qopt->mode;
|
u16 mode = mqprio_qopt->mode;
|
||||||
u8 hw = mqprio_qopt->qopt.hw;
|
u8 hw = mqprio_qopt->qopt.hw;
|
||||||
int ret;
|
|
||||||
|
|
||||||
if (!((hw == TC_MQPRIO_HW_OFFLOAD_TCS &&
|
if (!((hw == TC_MQPRIO_HW_OFFLOAD_TCS &&
|
||||||
mode == TC_MQPRIO_MODE_CHANNEL) || (!hw && tc == 0)))
|
mode == TC_MQPRIO_MODE_CHANNEL) || (!hw && tc == 0)))
|
||||||
|
@ -1494,12 +1493,8 @@ static int hns3_setup_tc(struct net_device *netdev, void *type_data)
|
||||||
if (!netdev)
|
if (!netdev)
|
||||||
return -EINVAL;
|
return -EINVAL;
|
||||||
|
|
||||||
ret = (kinfo->dcb_ops && kinfo->dcb_ops->setup_tc) ?
|
return (kinfo->dcb_ops && kinfo->dcb_ops->setup_tc) ?
|
||||||
kinfo->dcb_ops->setup_tc(h, tc, prio_tc) : -EOPNOTSUPP;
|
kinfo->dcb_ops->setup_tc(h, tc, prio_tc) : -EOPNOTSUPP;
|
||||||
if (ret)
|
|
||||||
return ret;
|
|
||||||
|
|
||||||
return hns3_nic_set_real_num_queue(netdev);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
static int hns3_nic_setup_tc(struct net_device *dev, enum tc_setup_type type,
|
static int hns3_nic_setup_tc(struct net_device *dev, enum tc_setup_type type,
|
||||||
|
|
|
@ -384,13 +384,12 @@ static int hclge_setup_tc(struct hnae3_handle *h, u8 tc, u8 *prio_tc)
|
||||||
hclge_tm_prio_tc_info_update(hdev, prio_tc);
|
hclge_tm_prio_tc_info_update(hdev, prio_tc);
|
||||||
|
|
||||||
ret = hclge_tm_init_hw(hdev, false);
|
ret = hclge_tm_init_hw(hdev, false);
|
||||||
if (ret) {
|
if (ret)
|
||||||
if (hclge_notify_client(hdev, HNAE3_INIT_CLIENT))
|
goto err_out;
|
||||||
return ret;
|
|
||||||
|
|
||||||
hclge_notify_client(hdev, HNAE3_UP_CLIENT);
|
ret = hclge_client_setup_tc(hdev);
|
||||||
return ret;
|
if (ret)
|
||||||
}
|
goto err_out;
|
||||||
|
|
||||||
hdev->flag &= ~HCLGE_FLAG_DCB_ENABLE;
|
hdev->flag &= ~HCLGE_FLAG_DCB_ENABLE;
|
||||||
|
|
||||||
|
@ -404,6 +403,13 @@ static int hclge_setup_tc(struct hnae3_handle *h, u8 tc, u8 *prio_tc)
|
||||||
return ret;
|
return ret;
|
||||||
|
|
||||||
return hclge_notify_client(hdev, HNAE3_UP_CLIENT);
|
return hclge_notify_client(hdev, HNAE3_UP_CLIENT);
|
||||||
|
|
||||||
|
err_out:
|
||||||
|
if (hclge_notify_client(hdev, HNAE3_INIT_CLIENT))
|
||||||
|
return ret;
|
||||||
|
|
||||||
|
hclge_notify_client(hdev, HNAE3_UP_CLIENT);
|
||||||
|
return ret;
|
||||||
}
|
}
|
||||||
|
|
||||||
static const struct hnae3_dcb_ops hns3_dcb_ops = {
|
static const struct hnae3_dcb_ops hns3_dcb_ops = {
|
||||||
|
|
Loading…
Reference in New Issue