From: Jacob Keller <jacob.e.kel...@intel.com>

The original code that implemented this accidentally had an inverted
check, so it would never show the VF number properly.

Fix this code to correctly display the VF number when we have one.

Fixes: 36ee712b0fbc ("ethtool: support queue and VF fields for rxclass 
filters", 2017-03-23)
Signed-off-by: Jacob Keller <jacob.e.kel...@intel.com>
Tested-by: Andrew Bowers <andrewx.bow...@intel.com>
Signed-off-by: Jeff Kirsher <jeffrey.t.kirs...@intel.com>
---
 rxclass.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/rxclass.c b/rxclass.c
index ebc3e1a..144c8e5 100644
--- a/rxclass.c
+++ b/rxclass.c
@@ -252,11 +252,11 @@ static void rxclass_print_nfc_rule(struct 
ethtool_rx_flow_spec *fsp)
                u64 queue = ethtool_get_flow_spec_ring(fsp->ring_cookie);
 
                if (vf)
-                       fprintf(stdout, "\tAction: Direct to queue %llu\n",
-                               queue);
-               else
                        fprintf(stdout, "\tAction: Direct to VF %llu queue 
%llu\n",
                                vf, queue);
+               else
+                       fprintf(stdout, "\tAction: Direct to queue %llu\n",
+                               queue);
        } else {
                fprintf(stdout, "\tAction: Drop\n");
        }
-- 
2.14.3

Reply via email to