Re: [PATCH net 1/2] macvlan: forbid L2 fowarding offload formacvtap

From: David Miller
Date: Mon Jan 06 2014 - 15:47:47 EST


From: Jason Wang <jasowang@xxxxxxxxxx>
Date: Mon, 6 Jan 2014 11:21:06 +0800

> L2 fowarding offload will bypass the rx handler of real device. This will make
> the packet could not be forwarded to macvtap device. Another problem is the
> dev_hard_start_xmit() called for macvtap does not have any synchronization.
>
> Fix this by forbidding L2 forwarding for macvtap.
>
> Cc: John Fastabend <john.r.fastabend@xxxxxxxxx>
> Cc: Neil Horman <nhorman@xxxxxxxxxxxxx>
> Signed-off-by: Jason Wang <jasowang@xxxxxxxxxx>

I think I agree with Neil that the rx_handler change might be the best
way to fix this. That change seems to have a lot of nice unintended
side effects, no?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/