On 9/21/16 10:13 PM, Mark Tomlinson wrote:
> The previous patch to ensure that the original iif was used when
> checking for forwarding also meant that this same interface was used to
> determine whether multicast packets should be received or not. This was
> incorrect, and would cause multicast packets to be dropped.
> The fix here is to use skb->dev when checking multicast addresses.
> skb->dev has been set to the l3mdev by this point, so the check will be
> against that, rather than the ingress interface.
l3mdev devices do not support IPv4 multicast so checking mcast against that
device should not be working at all. For that reason I was fine with the change
in the previous patch. ie., you want the real ingress device there not the vrf
What test are you running that says your previous patch broke something?