DORMANTno replies

[PATCH] net/ixgbe: output details on failure to configure filter

From: Wei Dai <hidden>
Date: 2017-02-04 08:32:17
Subsystem: networking drivers, the rest · Maintainers: Andrew Lunn, "David S. Miller", Eric Dumazet, Jakub Kicinski, Paolo Abeni, Linus Torvalds

Add more details when the filter mode can't be successfully configured.
These details tell user what filter configuration is right.

Reported-by: Yulong Pei <redacted>
Signed-off-by: Wei Dai <redacted>
---
 drivers/net/ixgbe/ixgbe_fdir.c | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)
diff --git a/drivers/net/ixgbe/ixgbe_fdir.c b/drivers/net/ixgbe/ixgbe_fdir.c
index 3b9d60c..a23581f 100644
--- a/drivers/net/ixgbe/ixgbe_fdir.c
+++ b/drivers/net/ixgbe/ixgbe_fdir.c
@@ -628,16 +628,25 @@ ixgbe_fdir_configure(struct rte_eth_dev *dev)
 		hw->mac.type != ixgbe_mac_X540 &&
 		hw->mac.type != ixgbe_mac_X550 &&
 		hw->mac.type != ixgbe_mac_X550EM_x &&
-		hw->mac.type != ixgbe_mac_X550EM_a)
+		hw->mac.type != ixgbe_mac_X550EM_a) {
+		PMD_INIT_LOG(err, "Only when ixgbe MAC type is 82599EB or X540 or X550 "
+			"or X550EM_x or X550EM_a, a filter mode can be supported. "
+			"Current MAC type = %d", (int)(hw->mac.type));
 		return -ENOSYS;
+	}
 
 	/* x550 supports mac-vlan and tunnel mode but other NICs not */
 	if (hw->mac.type != ixgbe_mac_X550 &&
 	    hw->mac.type != ixgbe_mac_X550EM_x &&
 	    hw->mac.type != ixgbe_mac_X550EM_a &&
 	    mode != RTE_FDIR_MODE_SIGNATURE &&
-	    mode != RTE_FDIR_MODE_PERFECT)
+	    mode != RTE_FDIR_MODE_PERFECT) {
+		PMD_INIT_LOG(err, "If ixgbe MAC type is not X550 or X550EM_x or "
+			"X550EM_a, the filter mode must be signature or perfect, other mode "
+			"is not supported ! Current MAC type = %d filter mode = %d",
+			(int)(hw->mac.type), (int)mode);
 		return -ENOSYS;
+	}
 
 	err = configure_fdir_flags(&dev->data->dev_conf.fdir_conf, &fdirctrl);
 	if (err)
-- 
2.7.4
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help