On 11/11/2015 05:35 PM, Stephen Hemminger wrote:
On Thu, 22 Oct 2015 16:26:36 -0700 Alexander Duyck <[email protected]> wrote:+static int ixgbe_uc_unsync(struct net_device *netdev, const unsigned char *addr) +{ + struct ixgbe_adapter *adapter = netdev_priv(netdev); + + ixgbe_del_mac_filter(adapter, addr, VMDQ_P(0)); + + return 0;Why add an internal function that always returns 0? Rather than making it void.
Because the function pointer is passed to the __dev_uc_sync call and it requires a return value on the unsync function. Basically if we returned an error it would delay flushing the address from the device until we could complete the call successfully, or __dev_uc_unsysnc was called without a function pointer.
- Alex -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
