Commit 34744a77 authored by Jiawen Wu's avatar Jiawen Wu Committed by Paolo Abeni

net: txgbe: add FDIR info to ethtool ops

Add flow director filter match and miss statistics to ethtool -S.
And change the number of queues when using flow director for ehtool -l.
Signed-off-by: default avatarJiawen Wu <jiawenwu@trustnetic.com>
Reviewed-by: default avatarSimon Horman <horms@kernel.org>
Signed-off-by: default avatarPaolo Abeni <pabeni@redhat.com>
parent 4bdb4411
...@@ -43,6 +43,11 @@ static const struct wx_stats wx_gstrings_stats[] = { ...@@ -43,6 +43,11 @@ static const struct wx_stats wx_gstrings_stats[] = {
WX_STAT("alloc_rx_buff_failed", alloc_rx_buff_failed), WX_STAT("alloc_rx_buff_failed", alloc_rx_buff_failed),
}; };
static const struct wx_stats wx_gstrings_fdir_stats[] = {
WX_STAT("fdir_match", stats.fdirmatch),
WX_STAT("fdir_miss", stats.fdirmiss),
};
/* drivers allocates num_tx_queues and num_rx_queues symmetrically so /* drivers allocates num_tx_queues and num_rx_queues symmetrically so
* we set the num_rx_queues to evaluate to num_tx_queues. This is * we set the num_rx_queues to evaluate to num_tx_queues. This is
* used because we do not have a good way to get the max number of * used because we do not have a good way to get the max number of
...@@ -55,13 +60,17 @@ static const struct wx_stats wx_gstrings_stats[] = { ...@@ -55,13 +60,17 @@ static const struct wx_stats wx_gstrings_stats[] = {
(WX_NUM_TX_QUEUES + WX_NUM_RX_QUEUES) * \ (WX_NUM_TX_QUEUES + WX_NUM_RX_QUEUES) * \
(sizeof(struct wx_queue_stats) / sizeof(u64))) (sizeof(struct wx_queue_stats) / sizeof(u64)))
#define WX_GLOBAL_STATS_LEN ARRAY_SIZE(wx_gstrings_stats) #define WX_GLOBAL_STATS_LEN ARRAY_SIZE(wx_gstrings_stats)
#define WX_FDIR_STATS_LEN ARRAY_SIZE(wx_gstrings_fdir_stats)
#define WX_STATS_LEN (WX_GLOBAL_STATS_LEN + WX_QUEUE_STATS_LEN) #define WX_STATS_LEN (WX_GLOBAL_STATS_LEN + WX_QUEUE_STATS_LEN)
int wx_get_sset_count(struct net_device *netdev, int sset) int wx_get_sset_count(struct net_device *netdev, int sset)
{ {
struct wx *wx = netdev_priv(netdev);
switch (sset) { switch (sset) {
case ETH_SS_STATS: case ETH_SS_STATS:
return WX_STATS_LEN; return (wx->mac.type == wx_mac_sp) ?
WX_STATS_LEN + WX_FDIR_STATS_LEN : WX_STATS_LEN;
default: default:
return -EOPNOTSUPP; return -EOPNOTSUPP;
} }
...@@ -70,6 +79,7 @@ EXPORT_SYMBOL(wx_get_sset_count); ...@@ -70,6 +79,7 @@ EXPORT_SYMBOL(wx_get_sset_count);
void wx_get_strings(struct net_device *netdev, u32 stringset, u8 *data) void wx_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
{ {
struct wx *wx = netdev_priv(netdev);
u8 *p = data; u8 *p = data;
int i; int i;
...@@ -77,6 +87,10 @@ void wx_get_strings(struct net_device *netdev, u32 stringset, u8 *data) ...@@ -77,6 +87,10 @@ void wx_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
case ETH_SS_STATS: case ETH_SS_STATS:
for (i = 0; i < WX_GLOBAL_STATS_LEN; i++) for (i = 0; i < WX_GLOBAL_STATS_LEN; i++)
ethtool_puts(&p, wx_gstrings_stats[i].stat_string); ethtool_puts(&p, wx_gstrings_stats[i].stat_string);
if (wx->mac.type == wx_mac_sp) {
for (i = 0; i < WX_FDIR_STATS_LEN; i++)
ethtool_puts(&p, wx_gstrings_fdir_stats[i].stat_string);
}
for (i = 0; i < netdev->num_tx_queues; i++) { for (i = 0; i < netdev->num_tx_queues; i++) {
ethtool_sprintf(&p, "tx_queue_%u_packets", i); ethtool_sprintf(&p, "tx_queue_%u_packets", i);
ethtool_sprintf(&p, "tx_queue_%u_bytes", i); ethtool_sprintf(&p, "tx_queue_%u_bytes", i);
...@@ -96,7 +110,7 @@ void wx_get_ethtool_stats(struct net_device *netdev, ...@@ -96,7 +110,7 @@ void wx_get_ethtool_stats(struct net_device *netdev,
struct wx *wx = netdev_priv(netdev); struct wx *wx = netdev_priv(netdev);
struct wx_ring *ring; struct wx_ring *ring;
unsigned int start; unsigned int start;
int i, j; int i, j, k;
char *p; char *p;
wx_update_stats(wx); wx_update_stats(wx);
...@@ -107,6 +121,13 @@ void wx_get_ethtool_stats(struct net_device *netdev, ...@@ -107,6 +121,13 @@ void wx_get_ethtool_stats(struct net_device *netdev,
sizeof(u64)) ? *(u64 *)p : *(u32 *)p; sizeof(u64)) ? *(u64 *)p : *(u32 *)p;
} }
if (wx->mac.type == wx_mac_sp) {
for (k = 0; k < WX_FDIR_STATS_LEN; k++) {
p = (char *)wx + wx_gstrings_fdir_stats[k].stat_offset;
data[i++] = *(u64 *)p;
}
}
for (j = 0; j < netdev->num_tx_queues; j++) { for (j = 0; j < netdev->num_tx_queues; j++) {
ring = wx->tx_ring[j]; ring = wx->tx_ring[j];
if (!ring) { if (!ring) {
...@@ -172,17 +193,21 @@ EXPORT_SYMBOL(wx_get_pause_stats); ...@@ -172,17 +193,21 @@ EXPORT_SYMBOL(wx_get_pause_stats);
void wx_get_drvinfo(struct net_device *netdev, struct ethtool_drvinfo *info) void wx_get_drvinfo(struct net_device *netdev, struct ethtool_drvinfo *info)
{ {
unsigned int stats_len = WX_STATS_LEN;
struct wx *wx = netdev_priv(netdev); struct wx *wx = netdev_priv(netdev);
if (wx->mac.type == wx_mac_sp)
stats_len += WX_FDIR_STATS_LEN;
strscpy(info->driver, wx->driver_name, sizeof(info->driver)); strscpy(info->driver, wx->driver_name, sizeof(info->driver));
strscpy(info->fw_version, wx->eeprom_id, sizeof(info->fw_version)); strscpy(info->fw_version, wx->eeprom_id, sizeof(info->fw_version));
strscpy(info->bus_info, pci_name(wx->pdev), sizeof(info->bus_info)); strscpy(info->bus_info, pci_name(wx->pdev), sizeof(info->bus_info));
if (wx->num_tx_queues <= WX_NUM_TX_QUEUES) { if (wx->num_tx_queues <= WX_NUM_TX_QUEUES) {
info->n_stats = WX_STATS_LEN - info->n_stats = stats_len -
(WX_NUM_TX_QUEUES - wx->num_tx_queues) * (WX_NUM_TX_QUEUES - wx->num_tx_queues) *
(sizeof(struct wx_queue_stats) / sizeof(u64)) * 2; (sizeof(struct wx_queue_stats) / sizeof(u64)) * 2;
} else { } else {
info->n_stats = WX_STATS_LEN; info->n_stats = stats_len;
} }
} }
EXPORT_SYMBOL(wx_get_drvinfo); EXPORT_SYMBOL(wx_get_drvinfo);
...@@ -383,6 +408,9 @@ void wx_get_channels(struct net_device *dev, ...@@ -383,6 +408,9 @@ void wx_get_channels(struct net_device *dev,
/* record RSS queues */ /* record RSS queues */
ch->combined_count = wx->ring_feature[RING_F_RSS].indices; ch->combined_count = wx->ring_feature[RING_F_RSS].indices;
if (test_bit(WX_FLAG_FDIR_CAPABLE, wx->flags))
ch->combined_count = wx->ring_feature[RING_F_FDIR].indices;
} }
EXPORT_SYMBOL(wx_get_channels); EXPORT_SYMBOL(wx_get_channels);
...@@ -400,6 +428,9 @@ int wx_set_channels(struct net_device *dev, ...@@ -400,6 +428,9 @@ int wx_set_channels(struct net_device *dev,
if (count > wx_max_channels(wx)) if (count > wx_max_channels(wx))
return -EINVAL; return -EINVAL;
if (test_bit(WX_FLAG_FDIR_CAPABLE, wx->flags))
wx->ring_feature[RING_F_FDIR].limit = count;
wx->ring_feature[RING_F_RSS].limit = count; wx->ring_feature[RING_F_RSS].limit = count;
return 0; return 0;
......
...@@ -2352,6 +2352,11 @@ void wx_update_stats(struct wx *wx) ...@@ -2352,6 +2352,11 @@ void wx_update_stats(struct wx *wx)
hwstats->b2ogprc += rd32(wx, WX_RDM_BMC2OS_CNT); hwstats->b2ogprc += rd32(wx, WX_RDM_BMC2OS_CNT);
hwstats->rdmdrop += rd32(wx, WX_RDM_DRP_PKT); hwstats->rdmdrop += rd32(wx, WX_RDM_DRP_PKT);
if (wx->mac.type == wx_mac_sp) {
hwstats->fdirmatch += rd32(wx, WX_RDB_FDIR_MATCH);
hwstats->fdirmiss += rd32(wx, WX_RDB_FDIR_MISS);
}
for (i = 0; i < wx->mac.max_rx_queues; i++) for (i = 0; i < wx->mac.max_rx_queues; i++)
hwstats->qmprc += rd32(wx, WX_PX_MPRC(i)); hwstats->qmprc += rd32(wx, WX_PX_MPRC(i));
} }
......
...@@ -157,6 +157,8 @@ ...@@ -157,6 +157,8 @@
#define WX_RDB_RA_CTL_RSS_IPV6_TCP BIT(21) #define WX_RDB_RA_CTL_RSS_IPV6_TCP BIT(21)
#define WX_RDB_RA_CTL_RSS_IPV4_UDP BIT(22) #define WX_RDB_RA_CTL_RSS_IPV4_UDP BIT(22)
#define WX_RDB_RA_CTL_RSS_IPV6_UDP BIT(23) #define WX_RDB_RA_CTL_RSS_IPV6_UDP BIT(23)
#define WX_RDB_FDIR_MATCH 0x19558
#define WX_RDB_FDIR_MISS 0x1955C
/******************************* PSR Registers *******************************/ /******************************* PSR Registers *******************************/
/* psr control */ /* psr control */
...@@ -1018,6 +1020,8 @@ struct wx_hw_stats { ...@@ -1018,6 +1020,8 @@ struct wx_hw_stats {
u64 crcerrs; u64 crcerrs;
u64 rlec; u64 rlec;
u64 qmprc; u64 qmprc;
u64 fdirmatch;
u64 fdirmiss;
}; };
enum wx_state { enum wx_state {
......
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