cxgb4: number of VFs supported is not always 16

Ganesh Goudar · davem330 · commit 1b974aa43ae2 · 2018-11-30T13:09:36.000-08:00
Total number of VFs supported by PF is used to determine the last
byte of VF's mac address. Number of VFs supported is not always
16, use the variable nvfs to get the number of VFs supported
rather than hard coding it to 16.

Signed-off-by: Casey Leedom &lt;leedom@chelsio.com&gt;
Signed-off-by: Ganesh Goudar &lt;ganeshgr@chelsio.com&gt;
Signed-off-by: David S. Miller &lt;davem@davemloft.net&gt;
diff --git a/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c b/drivers/net/ethernet/chelsio/cxgb4/cxgb4_main.c
@@ -2646,7 +2646,7 @@ static void cxgb4_mgmt_fill_vf_station_mac_addr(struct adapter *adap)
 
 	for (vf = 0, nvfs = pci_sriov_get_totalvfs(adap->pdev);
 		vf < nvfs; vf++) {
-		macaddr[5] = adap->pf * 16 + vf;
+		macaddr[5] = adap->pf * nvfs + vf;
 		ether_addr_copy(adap->vfinfo[vf].vf_mac_addr, macaddr);
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -2646,7 +2646,7 @@ static void cxgb4_mgmt_fill_vf_station_mac_addr(struct adapter *adap)`
`2646`	`2646`
`2647`	`2647`	`for (vf = 0, nvfs = pci_sriov_get_totalvfs(adap->pdev);`
`2648`	`2648`	`vf < nvfs; vf++) {`
`2649`		`- macaddr[5] = adap->pf * 16 + vf;`
	`2649`	`+ macaddr[5] = adap->pf * nvfs + vf;`
`2650`	`2650`	`ether_addr_copy(adap->vfinfo[vf].vf_mac_addr, macaddr);`
`2651`	`2651`	`}`
`2652`	`2652`	`}`