]> Pileus Git - ~andy/linux/commitdiff
ehea: Fix multicast registration on semi-promiscuous mode
authorBreno Leitao <leitao@linux.vnet.ibm.com>
Mon, 23 May 2011 03:36:35 +0000 (03:36 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 23 May 2011 20:33:31 +0000 (16:33 -0400)
Ehea will not register multicast groups in phyp if the physical
interface is in promiscuous mode. But it should register if the
logical port is in promiscuous mode, but the physical port is not.

Ehea physical promiscuous mode is defined by ehea_port->promisc,
while logical port is defined by IFF_PROMISC.

So currently, if the user set the interface in promiscuous mode,
IGMP will not be registred in PHYP, and PHYP will never pass
the multicast packet to the logical port, which is bad

So, this patch just fixes it, assuring that we register in phyp
if the physical port is not on promiscuous mode.

Signed-off-by: Breno Leitao <leitao@linux.vnet.ibm.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
drivers/net/ehea/ehea_main.c

index 6a0a8fca62bc58aeaa8354abc1f2c6d67175e659..3fd5a240034840d223cf43095bc76599811755e6 100644 (file)
@@ -2083,7 +2083,7 @@ static void ehea_set_multicast_list(struct net_device *dev)
        struct netdev_hw_addr *ha;
        int ret;
 
-       if (dev->flags & IFF_PROMISC) {
+       if (port->promisc) {
                ehea_promiscuous(dev, 1);
                return;
        }