bnx2x: Show port statistics in Multi-function
authorYuval Mintz <Yuval.Mintz@qlogic.com>
Thu, 19 Nov 2015 15:04:36 +0000 (17:04 +0200)
committerDavid S. Miller <davem@davemloft.net>
Fri, 20 Nov 2015 17:14:52 +0000 (12:14 -0500)
Today, port statistics are being presented when using `ethool -S' only
for single-function devices, but there are some port statistics which are
crucial for analyzing bottle-necks. E.g., HW Rx discards due to lack of
buffer space [when device isn't handling ingress traffic fast enough].

Judging the pros and cons, it was decided that in-order to better support
automatic dump-gathering tools, bnx2x should no longer hide those stats.
This leaves only VFs lacking the port statistics.

Signed-off-by: Yuval Mintz <Yuval.Mintz@qlogic.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ethernet/broadcom/bnx2x/bnx2x_ethtool.c

index 1dc77ac7cb21387306ddf1ac520f596722557aa9..a3ce9f2a23359755497921bb20d9d905846a9da3 100644 (file)
@@ -3069,9 +3069,7 @@ static void bnx2x_self_test(struct net_device *dev,
 #define IS_PORT_STAT(i) \
        ((bnx2x_stats_arr[i].flags & STATS_FLAGS_BOTH) == STATS_FLAGS_PORT)
 #define IS_FUNC_STAT(i)                (bnx2x_stats_arr[i].flags & STATS_FLAGS_FUNC)
-#define HIDE_PORT_STAT(bp) \
-               ((IS_MF(bp) && !(bp->msg_enable & BNX2X_MSG_STATS)) || \
-                IS_VF(bp))
+#define HIDE_PORT_STAT(bp)     IS_VF(bp)
 
 /* ethtool statistics are displayed for all regular ethernet queues and the
  * fcoe L2 queue if not disabled