On Tue, 30 Oct 2012 at 02:47 GMT, Zheng Li <[email protected]> wrote:
> +                     struct slave *tmp_slave = NULL;
> +                     int i = 0, found_mac = 0;
> +                     bond_for_each_slave(bond, tmp_slave, i) {
> +                             if (ether_addr_equal_64bits(arp->mac_src,
> +                                             tmp_slave->dev->dev_addr)) {
> +                                     found_mac = 1;
> +                                     break;
> +                             }
> +                     }
> +                     if (found_mac)
> +                             memcpy(arp->mac_src, tx_slave->dev->dev_addr,
> +                                     ETH_ALEN);

A nitpick: found_mac can be a bool.

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Reply via email to