Re: [Xen-devel] [PATCH 0/11] Add ixgbe support

[Steven Smith <steven.smith@xxxxxxxxxxxxx>]

> My first attempt to post this patch got silently dropped by Mailman,
> presumably because it was too big.  Here's the exact same code,
> chopped up 11 ways.
Thanks.  I've applied the whole series to the netchannel2 tree.

I had to apply an extra patch in order to get the thing to build:

diff -r 7f93f947b997 -r fcc044a90d40 drivers/net/ixgbe/ixgbe_ethtool.c
--- a/drivers/net/ixgbe/ixgbe_ethtool.c Thu Jan 29 10:36:26 2009 +0000
+++ b/drivers/net/ixgbe/ixgbe_ethtool.c Thu Jan 29 10:46:35 2009 +0000
@@ -129,8 +129,7 @@
                  sizeof(((struct ixgbe_adapter *)0)->stats.pxoffrxc) + \
                  sizeof(((struct ixgbe_adapter *)0)->stats.pxofftxc)) \
                 / sizeof(u64) : 0)
-#define IXGBE_STATS_LEN (IXGBE_GLOBAL_STATS_LEN + IXGBE_PB_STATS_LEN
- IXGBE_QUEUE_STATS_LEN)
+#define IXGBE_STATS_LEN (IXGBE_GLOBAL_STATS_LEN + IXGBE_PB_STATS_LEN + 
IXGBE_QUEUE_STATS_LEN)
 #define IXGBE_GLOBAL_STATS_LEN ARRAY_SIZE(ixgbe_gstrings_stats)
 #endif /* ETHTOOL_GSTATS */
 #ifdef ETHTOOL_TEST
@@ -1939,3 +1938,4 @@
 {
        SET_ETHTOOL_OPS(netdev, &ixgbe_ethtool_ops);
 }
+#endif

which presumably is due to some kind of patch screwiness.
Unfortunately, I don't actually have any ixgbes to test with, so I
don't have much idea whether that actually works.  Still, it looks
pretty safe. :)

Steven.

signature.asc
Description: Digital signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel