Commit 8c0ab70a authored by Jesper Dangaard Brouer's avatar Jesper Dangaard Brouer Committed by David S. Miller

igb: Implement reading of reg RQDPC (Receive Queue Drop Packet Count)

Based on the previous patches from Jesper Dangaard Brouer <hawk@comx.dk>

Implement reading the per queue drop stats register
RQDPC (Receive Queue Drop Packet Count).  It counts the number of
packets dropped by a queue due to lack of descriptors available.

Notice RQDPC (Receive Queue Drop Packet Count) stats only gets
incremented, if the DROP_EN bit it set (in the SRRCTL register
for that queue).  If DROP_EN bit is NOT set, then the some what
equivalent count is stored in RNBC (not per queue basis).

The RQDPC register is only 12 bit, thus the precision might
suffer due to overrun in-netween the watchdog polling interval.
Signed-off-by: default avatarJeff Kirsher <jeffrey.t.kirsher@intel.com>
CC: Jesper Dangaard Brouer <hawk@comx.dk>
Signed-off-by: default avatarDavid S. Miller <davem@davemloft.net>
parent 3c514ce2
...@@ -137,11 +137,17 @@ struct igb_buffer { ...@@ -137,11 +137,17 @@ struct igb_buffer {
}; };
}; };
struct igb_queue_stats { struct igb_tx_queue_stats {
u64 packets; u64 packets;
u64 bytes; u64 bytes;
}; };
struct igb_rx_queue_stats {
u64 packets;
u64 bytes;
u64 drops;
};
struct igb_ring { struct igb_ring {
struct igb_adapter *adapter; /* backlink */ struct igb_adapter *adapter; /* backlink */
void *desc; /* descriptor ring memory */ void *desc; /* descriptor ring memory */
...@@ -167,12 +173,13 @@ struct igb_ring { ...@@ -167,12 +173,13 @@ struct igb_ring {
union { union {
/* TX */ /* TX */
struct { struct {
struct igb_queue_stats tx_stats; struct igb_tx_queue_stats tx_stats;
bool detect_tx_hung; bool detect_tx_hung;
}; };
/* RX */ /* RX */
struct { struct {
struct igb_queue_stats rx_stats; struct igb_rx_queue_stats rx_stats;
u64 rx_queue_drops;
struct napi_struct napi; struct napi_struct napi;
int set_itr; int set_itr;
struct igb_ring *buddy; struct igb_ring *buddy;
......
...@@ -96,9 +96,10 @@ static const struct igb_stats igb_gstrings_stats[] = { ...@@ -96,9 +96,10 @@ static const struct igb_stats igb_gstrings_stats[] = {
}; };
#define IGB_QUEUE_STATS_LEN \ #define IGB_QUEUE_STATS_LEN \
((((struct igb_adapter *)netdev_priv(netdev))->num_rx_queues + \ (((((struct igb_adapter *)netdev_priv(netdev))->num_rx_queues)* \
((struct igb_adapter *)netdev_priv(netdev))->num_tx_queues) * \ (sizeof(struct igb_rx_queue_stats) / sizeof(u64))) + \
(sizeof(struct igb_queue_stats) / sizeof(u64))) ((((struct igb_adapter *)netdev_priv(netdev))->num_tx_queues) * \
(sizeof(struct igb_tx_queue_stats) / sizeof(u64))))
#define IGB_GLOBAL_STATS_LEN \ #define IGB_GLOBAL_STATS_LEN \
sizeof(igb_gstrings_stats) / sizeof(struct igb_stats) sizeof(igb_gstrings_stats) / sizeof(struct igb_stats)
#define IGB_STATS_LEN (IGB_GLOBAL_STATS_LEN + IGB_QUEUE_STATS_LEN) #define IGB_STATS_LEN (IGB_GLOBAL_STATS_LEN + IGB_QUEUE_STATS_LEN)
...@@ -1960,7 +1961,8 @@ static void igb_get_ethtool_stats(struct net_device *netdev, ...@@ -1960,7 +1961,8 @@ static void igb_get_ethtool_stats(struct net_device *netdev,
{ {
struct igb_adapter *adapter = netdev_priv(netdev); struct igb_adapter *adapter = netdev_priv(netdev);
u64 *queue_stat; u64 *queue_stat;
int stat_count = sizeof(struct igb_queue_stats) / sizeof(u64); int stat_count_tx = sizeof(struct igb_tx_queue_stats) / sizeof(u64);
int stat_count_rx = sizeof(struct igb_rx_queue_stats) / sizeof(u64);
int j; int j;
int i; int i;
...@@ -1973,14 +1975,14 @@ static void igb_get_ethtool_stats(struct net_device *netdev, ...@@ -1973,14 +1975,14 @@ static void igb_get_ethtool_stats(struct net_device *netdev,
for (j = 0; j < adapter->num_tx_queues; j++) { for (j = 0; j < adapter->num_tx_queues; j++) {
int k; int k;
queue_stat = (u64 *)&adapter->tx_ring[j].tx_stats; queue_stat = (u64 *)&adapter->tx_ring[j].tx_stats;
for (k = 0; k < stat_count; k++) for (k = 0; k < stat_count_tx; k++)
data[i + k] = queue_stat[k]; data[i + k] = queue_stat[k];
i += k; i += k;
} }
for (j = 0; j < adapter->num_rx_queues; j++) { for (j = 0; j < adapter->num_rx_queues; j++) {
int k; int k;
queue_stat = (u64 *)&adapter->rx_ring[j].rx_stats; queue_stat = (u64 *)&adapter->rx_ring[j].rx_stats;
for (k = 0; k < stat_count; k++) for (k = 0; k < stat_count_rx; k++)
data[i + k] = queue_stat[k]; data[i + k] = queue_stat[k];
i += k; i += k;
} }
...@@ -2014,6 +2016,8 @@ static void igb_get_strings(struct net_device *netdev, u32 stringset, u8 *data) ...@@ -2014,6 +2016,8 @@ static void igb_get_strings(struct net_device *netdev, u32 stringset, u8 *data)
p += ETH_GSTRING_LEN; p += ETH_GSTRING_LEN;
sprintf(p, "rx_queue_%u_bytes", i); sprintf(p, "rx_queue_%u_bytes", i);
p += ETH_GSTRING_LEN; p += ETH_GSTRING_LEN;
sprintf(p, "rx_queue_%u_drops", i);
p += ETH_GSTRING_LEN;
} }
/* BUG_ON(p - data != IGB_STATS_LEN * ETH_GSTRING_LEN); */ /* BUG_ON(p - data != IGB_STATS_LEN * ETH_GSTRING_LEN); */
break; break;
......
...@@ -3588,6 +3588,23 @@ void igb_update_stats(struct igb_adapter *adapter) ...@@ -3588,6 +3588,23 @@ void igb_update_stats(struct igb_adapter *adapter)
/* Rx Errors */ /* Rx Errors */
if (hw->mac.type != e1000_82575) {
u32 rqdpc_tmp;
int i;
/* Read out drops stats per RX queue. Notice RQDPC (Receive
* Queue Drop Packet Count) stats only gets incremented, if
* the DROP_EN but it set (in the SRRCTL register for that
* queue). If DROP_EN bit is NOT set, then the some what
* equivalent count is stored in RNBC (not per queue basis).
* Also note the drop count is due to lack of available
* descriptors.
*/
for (i = 0; i < adapter->num_rx_queues; i++) {
rqdpc_tmp = rd32(E1000_RQDPC(i)) & 0xFFF;
adapter->rx_ring[i].rx_stats.drops += rqdpc_tmp;
}
}
/* RLEC on some newer hardware can be incorrect so build /* RLEC on some newer hardware can be incorrect so build
* our own version based on RUC and ROC */ * our own version based on RUC and ROC */
adapter->net_stats.rx_errors = adapter->stats.rxerrc + adapter->net_stats.rx_errors = adapter->stats.rxerrc +
......
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