Commit c78b5b6c authored by Huazhong Tan's avatar Huazhong Tan Committed by David S. Miller

net: hns3: modify parameter checks in the hns3_set_channels

The number of queues for each enabled TC should range from 1 to
the maximum available value, and return directly if the value
is same as the current one.
Signed-off-by: default avatarHuazhong Tan <tanhuazhong@huawei.com>
Signed-off-by: default avatarYunsheng Lin <linyunsheng@huawei.com>
Signed-off-by: default avatarPeng Li <lipeng321@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 73fc9c48
...@@ -4157,15 +4157,14 @@ int hns3_set_channels(struct net_device *netdev, ...@@ -4157,15 +4157,14 @@ int hns3_set_channels(struct net_device *netdev,
return -EINVAL; return -EINVAL;
if (new_tqp_num > hns3_get_max_available_channels(h) || if (new_tqp_num > hns3_get_max_available_channels(h) ||
new_tqp_num < kinfo->num_tc) { new_tqp_num < 1) {
dev_err(&netdev->dev, dev_err(&netdev->dev,
"Change tqps fail, the tqp range is from %d to %d", "Change tqps fail, the tqp range is from 1 to %d",
kinfo->num_tc,
hns3_get_max_available_channels(h)); hns3_get_max_available_channels(h));
return -EINVAL; return -EINVAL;
} }
if (kinfo->num_tqps == new_tqp_num) if (kinfo->rss_size == new_tqp_num)
return 0; return 0;
ret = hns3_reset_notify(h, HNAE3_DOWN_CLIENT); ret = hns3_reset_notify(h, HNAE3_DOWN_CLIENT);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment