Re: [dpdk-dev] [PATCH] examples/vm_power_manager: fix set VF MAC address

[Hunt, David]

Replying to notify <sta...@dpdk.org>, which was missing from original mail.

-----Original Message-----
From: Coyle, David 
Sent: Tuesday, 23 January, 2018 2:14 PM
To: dev@dpdk.org
Cc: Hunt, David <david.h...@intel.com>; sta...@intel.com; Coyle, David 
<david.co...@intel.com>
Subject: [PATCH] examples/vm_power_manager: fix set VF MAC address

Current code only sets mac address of first VF. Fix code so that it continues 
through the loop and sets the mac address of each VF.

Fixes: c9a4779135c9 ("examples/vm_power_mgr: set MAC address of VF")

Signed-off-by: David Coyle <david.co...@intel.com>
---
 examples/vm_power_manager/main.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/examples/vm_power_manager/main.c b/examples/vm_power_manager/main.c
index a50984d..a9f5ad1 100644
--- a/examples/vm_power_manager/main.c
+++ b/examples/vm_power_manager/main.c
@@ -290,7 +290,7 @@ main(int argc, char **argv)
        for (portid = 0; portid < nb_ports; portid++) {
                struct ether_addr eth;
                int w, j;
-               int ret = -ENOTSUP;
+               int ret;
 
                if ((enabled_port_mask & (1 << portid)) == 0)
                        continue;
@@ -308,8 +308,7 @@ main(int argc, char **argv)
                for (w = 0; w < MAX_VFS; w++) {
                        eth.addr_bytes[5] = w + 0xf0;
 
-                       if (ret == -ENOTSUP)
-                               ret = rte_pmd_ixgbe_set_vf_mac_addr(portid,
+                       ret = rte_pmd_ixgbe_set_vf_mac_addr(portid,
                                                w, &eth);
                        if (ret == -ENOTSUP)
                                ret = rte_pmd_i40e_set_vf_mac_addr(portid,
--
2.7.5