Re: [PATCH] Spidernet: add ethtool -S (show statistics)

2006-09-22 Thread Jeff Garzik

ACK, but patch does not apply:

Applying 'Spidernet: add ethtool -S (show statistics)'

error: patch fragment without header at line 14: @@ -472,6 +472,7 @@ 
spider_net_prepare_rx_descr(struct spide
error: patch fragment without header at line 22: @@ -492,6 +493,7 @@ 
spider_net_prepare_rx_descr(struct spide
error: patch fragment without header at line 30: @@ -721,6 +723,7 @@ 
spider_net_prepare_tx_descr(struct spide
error: patch fragment without header at line 38: @@ -942,6 +945,7 @@ 
spider_net_xmit(struct sk_buff *skb, str
error: patch fragment without header at line 47: @@ -1047,6 +1051,7 @@ 
spider_net_pass_skb_up(struct spider_net
error: patch fragment without header at line 55: @@ -1144,9 +1149,11 @@ 
spider_net_decode_one_descr(struct spide
error: patch fragment without header at line 68: @@ -2112,6 +2119,7 @@ 
spider_net_tx_timeout(struct net_device
error: patch fragment without header at line 82: @@ -438,6 +438,15 @@ 
struct spider_net_options {
error: patch fragment without header at line 98: @@ -464,9 +473,9 @@ 
struct spider_net_card {

error: patch fragment without header at line 115: @@ -27,6 +27,27 @@
error: patch fragment without header at line 143: @@ -143,6 +164,38 @@ 
spider_net_ethtool_get_ringparam(struct
error: patch fragment without header at line 183: @@ -155,5 +208,8 @@ 
struct ethtool_ops spider_net_ethtool_op

fatal: No changes

-
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] Spidernet: add ethtool -S (show statistics)

2006-09-21 Thread Jim Lewis
This patch adds the ethtool -S (show statistics) feature to the
Spidernet ethernet driver. I have tested it extensively and believe it
is ready to be applied.


Signed-off-by: James K Lewis


---
 drivers/net/spider_net.c |   10 ++
 drivers/net/spider_net.h |   11 ++-
 drivers/net/spider_net_ethtool.c |   56
+++
 3 files changed, 75 insertions(+), 2 deletions(-)

Index: linux-2.6.18-rc2/drivers/net/spider_net.c
===
--- linux-2.6.18-rc2.orig/drivers/net/spider_net.c  2006-09-05
17:56:19.0 -0500
+++ linux-2.6.18-rc2/drivers/net/spider_net.c   2006-09-20
14:03:54.0 -0500
@@ -472,6 +472,7 @@ spider_net_prepare_rx_descr(struct spide
if (!descr->skb) {
if (netif_msg_rx_err(card) && net_ratelimit())
pr_err("Not enough memory to allocate rx buffer\n");
+   card->spider_stats.alloc_rx_skb_error++;
return -ENOMEM;
}
descr->buf_size = bufsize;
@@ -492,6 +493,7 @@ spider_net_prepare_rx_descr(struct spide
dev_kfree_skb_any(descr->skb);
if (netif_msg_rx_err(card) && net_ratelimit())
pr_err("Could not iommu-map rx buffer\n");
+   card->spider_stats.rx_iommu_map_error++;
descr->dmac_cmd_status = SPIDER_NET_DESCR_NOT_IN_USE;
} else {
descr->dmac_cmd_status = SPIDER_NET_DESCR_CARDOWNED |
@@ -721,6 +723,7 @@ spider_net_prepare_tx_descr(struct spide
if (netif_msg_tx_err(card) && net_ratelimit())
pr_err("could not iommu-map packet (%p, %i). "
  "Dropping packet\n", skb->data, skb->len);
+   card->spider_stats.tx_iommu_map_error++;
return -ENOMEM;
}
 
@@ -942,6 +945,7 @@ spider_net_xmit(struct sk_buff *skb, str
}
 
if (spider_net_get_descr_status(descr) != SPIDER_NET_DESCR_NOT_IN_USE)
{
+   card->netdev_stats.tx_dropped++;
result = NETDEV_TX_LOCKED;
netif_stop_queue(netdev);
goto out;
@@ -1047,6 +1051,7 @@ spider_net_pass_skb_up(struct spider_net
pr_err("error in received descriptor found, "
   "data_status=x%08x, data_error=x%08x\n",
   data_status, data_error);
+   card->spider_stats.rx_desc_error++;
return 0;
}
 
@@ -1144,9 +1149,11 @@ spider_net_decode_one_descr(struct spide
 
if ( (status != SPIDER_NET_DESCR_COMPLETE) &&
 (status != SPIDER_NET_DESCR_FRAME_END) ) {
-   if (netif_msg_rx_err(card))
+   if (netif_msg_rx_err(card)) {
pr_err("%s: RX descriptor with state %d\n",
   card->netdev->name, status);
+   card->spider_stats.rx_desc_unk_state++;
+   }
goto refill;
}
 
@@ -2112,6 +2119,7 @@ spider_net_tx_timeout(struct net_device 
schedule_work(&card->tx_timeout_task);
else
atomic_dec(&card->tx_timeout_task_counter);
+   card->spider_stats.tx_timeouts++;
 }
 
 /**
Index: linux-2.6.18-rc2/drivers/net/spider_net.h
===
--- linux-2.6.18-rc2.orig/drivers/net/spider_net.h  2006-09-07
17:45:15.0 -0500
+++ linux-2.6.18-rc2/drivers/net/spider_net.h   2006-09-20
14:02:10.0 -0500
@@ -438,6 +438,15 @@ struct spider_net_options {
  NETIF_MSG_HW | \
  NETIF_MSG_WOL )
 
+struct spider_net_extra_stats {
+   unsigned long rx_desc_error;
+   unsigned long tx_timeouts;
+   unsigned long alloc_rx_skb_error;
+   unsigned long rx_iommu_map_error;
+   unsigned long tx_iommu_map_error;
+   unsigned long rx_desc_unk_state;
+};
+
 struct spider_net_card {
struct net_device *netdev;
struct pci_dev *pdev;
@@ -464,9 +473,9 @@ struct spider_net_card {
 
/* for ethtool */
int msg_enable;
-
int rx_desc;
int tx_desc;
+   struct spider_net_extra_stats spider_stats;
 
struct spider_net_descr descr[0];
 };
Index: linux-2.6.18-rc2/drivers/net/spider_net_ethtool.c
===
--- linux-2.6.18-rc2.orig/drivers/net/spider_net_ethtool.c  2006-08-23
19:01:02.0 -0500
+++ linux-2.6.18-rc2/drivers/net/spider_net_ethtool.c   2006-09-20
14:06:28.0 -0500
@@ -27,6 +27,27 @@
 
 #include "spider_net.h"
 
+
+#define SPIDER_NET_NUM_STATS 13
+
+static struct {
+   const char str[ETH_GSTRING_LEN];
+} ethtool_stats_keys[] = {
+   { "tx_packets" },
+   { "tx_bytes" },
+   { "rx_packets" },
+   { "rx_bytes" },
+