Skip to content

Commit

Permalink
enic: minimize pkt filter updates to firmware
Browse files Browse the repository at this point in the history
In set_multicast(), only push pkt filter changes down to firmware if
pkt filter actually changes.

Signed-off-by: Vasanthy Kolluri <vkolluri@cisco.com>
Signed-off-by: Scott Feldman <scofeldm@cisco.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
  • Loading branch information
Scott Feldman authored and David S. Miller committed Dec 24, 2009
1 parent 2d6ddce commit 9959a18
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
1 change: 1 addition & 0 deletions drivers/net/enic/enic.h
Original file line number Diff line number Diff line change
Expand Up @@ -89,6 +89,7 @@ struct enic {
spinlock_t devcmd_lock;
u8 mac_addr[ETH_ALEN];
u8 mc_addr[ENIC_MULTICAST_PERFECT_FILTERS][ETH_ALEN];
unsigned int flags;
unsigned int mc_count;
int csum_rx_enabled;
u32 port_mtu;
Expand Down
8 changes: 6 additions & 2 deletions drivers/net/enic/enic_main.c
Original file line number Diff line number Diff line change
Expand Up @@ -771,6 +771,7 @@ static void enic_set_multicast_list(struct net_device *netdev)
int promisc = (netdev->flags & IFF_PROMISC) ? 1 : 0;
int allmulti = (netdev->flags & IFF_ALLMULTI) ||
(netdev->mc_count > ENIC_MULTICAST_PERFECT_FILTERS);
unsigned int flags = netdev->flags | (allmulti ? IFF_ALLMULTI : 0);
u8 mc_addr[ENIC_MULTICAST_PERFECT_FILTERS][ETH_ALEN];
unsigned int mc_count = netdev->mc_count;
unsigned int i, j;
Expand All @@ -780,8 +781,11 @@ static void enic_set_multicast_list(struct net_device *netdev)

spin_lock(&enic->devcmd_lock);

vnic_dev_packet_filter(enic->vdev, directed,
multicast, broadcast, promisc, allmulti);
if (enic->flags != flags) {
enic->flags = flags;
vnic_dev_packet_filter(enic->vdev, directed,
multicast, broadcast, promisc, allmulti);
}

/* Is there an easier way? Trying to minimize to
* calls to add/del multicast addrs. We keep the
Expand Down

0 comments on commit 9959a18

Please sign in to comment.