[E1000-devel] igb hard-coded mac_table[0] in igb_probe

[James B. <Jam...@eu...>]

We've noticed that the igb_probe() function hard-codes mac_table index 0 when setting the PF MAC entry. Unfortunately this happens after igb_set_sriov_capability() has run and used mac_table[0] for the VF0 initial MAC and therefore the PF MAC silently obliterates the VF0 config. In practice this isn't a problem because the VF0 MAC will get re-added to the table later on igb_vf_reset_msg() etc.

The only reason this caused a problem for us was because we were using the number of free mac_table entries as a guide to how many additional MAC filters we could define - to work around this we simply have to allow for a spare entry for VF0 when it eventually gets used.

Regards,
James