Commit e65a0231 authored by Hao Chen's avatar Hao Chen Committed by David S. Miller

net: hns3: add support to set/get rx buf len via ethtool for hns3 driver

Rx buf len is for rx BD buffer size, support setting it via ethtool -G
parameter and getting it via ethtool -g parameter.
Signed-off-by: default avatarHao Chen <chenhao288@hisilicon.com>
Signed-off-by: default avatarGuangbin Huang <huangguangbin2@huawei.com>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 74624944
...@@ -649,7 +649,7 @@ static void hns3_get_ringparam(struct net_device *netdev, ...@@ -649,7 +649,7 @@ static void hns3_get_ringparam(struct net_device *netdev,
{ {
struct hns3_nic_priv *priv = netdev_priv(netdev); struct hns3_nic_priv *priv = netdev_priv(netdev);
struct hnae3_handle *h = priv->ae_handle; struct hnae3_handle *h = priv->ae_handle;
int queue_num = h->kinfo.num_tqps; int rx_queue_index = h->kinfo.num_tqps;
if (hns3_nic_resetting(netdev)) { if (hns3_nic_resetting(netdev)) {
netdev_err(netdev, "dev resetting!"); netdev_err(netdev, "dev resetting!");
...@@ -660,7 +660,8 @@ static void hns3_get_ringparam(struct net_device *netdev, ...@@ -660,7 +660,8 @@ static void hns3_get_ringparam(struct net_device *netdev,
param->rx_max_pending = HNS3_RING_MAX_PENDING; param->rx_max_pending = HNS3_RING_MAX_PENDING;
param->tx_pending = priv->ring[0].desc_num; param->tx_pending = priv->ring[0].desc_num;
param->rx_pending = priv->ring[queue_num].desc_num; param->rx_pending = priv->ring[rx_queue_index].desc_num;
kernel_param->rx_buf_len = priv->ring[rx_queue_index].buf_size;
} }
static void hns3_get_pauseparam(struct net_device *netdev, static void hns3_get_pauseparam(struct net_device *netdev,
...@@ -1062,14 +1063,23 @@ static struct hns3_enet_ring *hns3_backup_ringparam(struct hns3_nic_priv *priv) ...@@ -1062,14 +1063,23 @@ static struct hns3_enet_ring *hns3_backup_ringparam(struct hns3_nic_priv *priv)
} }
static int hns3_check_ringparam(struct net_device *ndev, static int hns3_check_ringparam(struct net_device *ndev,
struct ethtool_ringparam *param) struct ethtool_ringparam *param,
struct kernel_ethtool_ringparam *kernel_param)
{ {
#define RX_BUF_LEN_2K 2048
#define RX_BUF_LEN_4K 4096
if (hns3_nic_resetting(ndev)) if (hns3_nic_resetting(ndev))
return -EBUSY; return -EBUSY;
if (param->rx_mini_pending || param->rx_jumbo_pending) if (param->rx_mini_pending || param->rx_jumbo_pending)
return -EINVAL; return -EINVAL;
if (kernel_param->rx_buf_len != RX_BUF_LEN_2K &&
kernel_param->rx_buf_len != RX_BUF_LEN_4K) {
netdev_err(ndev, "Rx buf len only support 2048 and 4096\n");
return -EINVAL;
}
if (param->tx_pending > HNS3_RING_MAX_PENDING || if (param->tx_pending > HNS3_RING_MAX_PENDING ||
param->tx_pending < HNS3_RING_MIN_PENDING || param->tx_pending < HNS3_RING_MIN_PENDING ||
param->rx_pending > HNS3_RING_MAX_PENDING || param->rx_pending > HNS3_RING_MAX_PENDING ||
...@@ -1082,6 +1092,22 @@ static int hns3_check_ringparam(struct net_device *ndev, ...@@ -1082,6 +1092,22 @@ static int hns3_check_ringparam(struct net_device *ndev,
return 0; return 0;
} }
static int hns3_change_rx_buf_len(struct net_device *ndev, u32 rx_buf_len)
{
struct hns3_nic_priv *priv = netdev_priv(ndev);
struct hnae3_handle *h = priv->ae_handle;
int i;
h->kinfo.rx_buf_len = rx_buf_len;
for (i = 0; i < h->kinfo.num_tqps; i++) {
h->kinfo.tqp[i]->buf_size = rx_buf_len;
priv->ring[i + h->kinfo.num_tqps].buf_size = rx_buf_len;
}
return 0;
}
static int hns3_set_ringparam(struct net_device *ndev, static int hns3_set_ringparam(struct net_device *ndev,
struct ethtool_ringparam *param, struct ethtool_ringparam *param,
struct kernel_ethtool_ringparam *kernel_param, struct kernel_ethtool_ringparam *kernel_param,
...@@ -1094,9 +1120,10 @@ static int hns3_set_ringparam(struct net_device *ndev, ...@@ -1094,9 +1120,10 @@ static int hns3_set_ringparam(struct net_device *ndev,
u32 old_tx_desc_num, new_tx_desc_num; u32 old_tx_desc_num, new_tx_desc_num;
u32 old_rx_desc_num, new_rx_desc_num; u32 old_rx_desc_num, new_rx_desc_num;
u16 queue_num = h->kinfo.num_tqps; u16 queue_num = h->kinfo.num_tqps;
u32 old_rx_buf_len;
int ret, i; int ret, i;
ret = hns3_check_ringparam(ndev, param); ret = hns3_check_ringparam(ndev, param, kernel_param);
if (ret) if (ret)
return ret; return ret;
...@@ -1105,8 +1132,10 @@ static int hns3_set_ringparam(struct net_device *ndev, ...@@ -1105,8 +1132,10 @@ static int hns3_set_ringparam(struct net_device *ndev,
new_rx_desc_num = ALIGN(param->rx_pending, HNS3_RING_BD_MULTIPLE); new_rx_desc_num = ALIGN(param->rx_pending, HNS3_RING_BD_MULTIPLE);
old_tx_desc_num = priv->ring[0].desc_num; old_tx_desc_num = priv->ring[0].desc_num;
old_rx_desc_num = priv->ring[queue_num].desc_num; old_rx_desc_num = priv->ring[queue_num].desc_num;
old_rx_buf_len = priv->ring[queue_num].buf_size;
if (old_tx_desc_num == new_tx_desc_num && if (old_tx_desc_num == new_tx_desc_num &&
old_rx_desc_num == new_rx_desc_num) old_rx_desc_num == new_rx_desc_num &&
kernel_param->rx_buf_len == old_rx_buf_len)
return 0; return 0;
tmp_rings = hns3_backup_ringparam(priv); tmp_rings = hns3_backup_ringparam(priv);
...@@ -1117,19 +1146,22 @@ static int hns3_set_ringparam(struct net_device *ndev, ...@@ -1117,19 +1146,22 @@ static int hns3_set_ringparam(struct net_device *ndev,
} }
netdev_info(ndev, netdev_info(ndev,
"Changing Tx/Rx ring depth from %u/%u to %u/%u\n", "Changing Tx/Rx ring depth from %u/%u to %u/%u, Changing rx buffer len from %d to %d\n",
old_tx_desc_num, old_rx_desc_num, old_tx_desc_num, old_rx_desc_num,
new_tx_desc_num, new_rx_desc_num); new_tx_desc_num, new_rx_desc_num,
old_rx_buf_len, kernel_param->rx_buf_len);
if (if_running) if (if_running)
ndev->netdev_ops->ndo_stop(ndev); ndev->netdev_ops->ndo_stop(ndev);
hns3_change_all_ring_bd_num(priv, new_tx_desc_num, new_rx_desc_num); hns3_change_all_ring_bd_num(priv, new_tx_desc_num, new_rx_desc_num);
hns3_change_rx_buf_len(ndev, kernel_param->rx_buf_len);
ret = hns3_init_all_ring(priv); ret = hns3_init_all_ring(priv);
if (ret) { if (ret) {
netdev_err(ndev, "Change bd num fail, revert to old value(%d)\n", netdev_err(ndev, "set ringparam fail, revert to old value(%d)\n",
ret); ret);
hns3_change_rx_buf_len(ndev, old_rx_buf_len);
hns3_change_all_ring_bd_num(priv, old_tx_desc_num, hns3_change_all_ring_bd_num(priv, old_tx_desc_num,
old_rx_desc_num); old_rx_desc_num);
for (i = 0; i < h->kinfo.num_tqps * 2; i++) for (i = 0; i < h->kinfo.num_tqps * 2; i++)
...@@ -1811,6 +1843,8 @@ static int hns3_set_tunable(struct net_device *netdev, ...@@ -1811,6 +1843,8 @@ static int hns3_set_tunable(struct net_device *netdev,
ETHTOOL_COALESCE_MAX_FRAMES | \ ETHTOOL_COALESCE_MAX_FRAMES | \
ETHTOOL_COALESCE_USE_CQE) ETHTOOL_COALESCE_USE_CQE)
#define HNS3_ETHTOOL_RING ETHTOOL_RING_USE_RX_BUF_LEN
static int hns3_get_ts_info(struct net_device *netdev, static int hns3_get_ts_info(struct net_device *netdev,
struct ethtool_ts_info *info) struct ethtool_ts_info *info)
{ {
...@@ -1889,6 +1923,7 @@ static int hns3_get_link_ext_state(struct net_device *netdev, ...@@ -1889,6 +1923,7 @@ static int hns3_get_link_ext_state(struct net_device *netdev,
static const struct ethtool_ops hns3vf_ethtool_ops = { static const struct ethtool_ops hns3vf_ethtool_ops = {
.supported_coalesce_params = HNS3_ETHTOOL_COALESCE, .supported_coalesce_params = HNS3_ETHTOOL_COALESCE,
.supported_ring_params = HNS3_ETHTOOL_RING,
.get_drvinfo = hns3_get_drvinfo, .get_drvinfo = hns3_get_drvinfo,
.get_ringparam = hns3_get_ringparam, .get_ringparam = hns3_get_ringparam,
.set_ringparam = hns3_set_ringparam, .set_ringparam = hns3_set_ringparam,
...@@ -1920,6 +1955,7 @@ static const struct ethtool_ops hns3vf_ethtool_ops = { ...@@ -1920,6 +1955,7 @@ static const struct ethtool_ops hns3vf_ethtool_ops = {
static const struct ethtool_ops hns3_ethtool_ops = { static const struct ethtool_ops hns3_ethtool_ops = {
.supported_coalesce_params = HNS3_ETHTOOL_COALESCE, .supported_coalesce_params = HNS3_ETHTOOL_COALESCE,
.supported_ring_params = HNS3_ETHTOOL_RING,
.self_test = hns3_self_test, .self_test = hns3_self_test,
.get_drvinfo = hns3_get_drvinfo, .get_drvinfo = hns3_get_drvinfo,
.get_link = hns3_get_link, .get_link = hns3_get_link,
......
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