On Tue, Feb 16, 2016 at 10:43 AM, Joe Stringer <j...@ovn.org> wrote:
> diff --git a/datapath/linux/compat/lisp.c b/datapath/linux/compat/lisp.c
> index e5a6a7fe00a4..0c81dd52ecab 100644
> --- a/datapath/linux/compat/lisp.c
> +++ b/datapath/linux/compat/lisp.c
> @@ -114,6 +114,7 @@ struct lisphdr {
> };
>
> #define LISP_HLEN (sizeof(struct udphdr) + sizeof(struct lisphdr))
> +#define LISP_MAX_MTU (LISP_HLEN + sizeof(struct iphdr) + sizeof(struct
> ethhdr))
This doesn't look right to me - isn't this a very small value? Also,
LISP doesn't have an inner Ethernet header, unlike the other tunnels.
Based on the upstream discussion, it seems like some additional
patches may be coming in this area as well that we might want to roll
in to make all tunnels consistent.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
