On Tue, Nov 10, 2015 at 02:31:13PM -0800, Ben Pfaff wrote:
> On Thu, Oct 22, 2015 at 03:29:01PM -0200, Thadeu Lima de Souza Cascardo wrote:
> > Add netlink attributes for IPv6 tunnel addresses. This enables IPv6 support
> > for tunnels.
> > 
> > [cascardo: Backport the key attribute values only.]
> > 
> > Signed-off-by: Thadeu Lima de Souza Cascardo <casca...@redhat.com>
> 
> This causes a new warning:
> 
> ../lib/odp-util.c:2116:17: error: enumeration values
>       'OVS_TUNNEL_KEY_ATTR_IPV6_SRC' and 'OVS_TUNNEL_KEY_ATTR_IPV6_DST' not
>       explicitly handled in switch [-Werror,-Wswitch-enum]
>         switch (type) {
>                 ^
> 
> (I'm the wrong person to review a datapath change anyway.)

I will look at the warning, didn't show up for me when I used sparse and clang.
I need to check why that have slipped. Nonetheless, it would be nice to hear
from Pravin and Joe if the backport like this is OK.

Thanks for your review.
Cascardo.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to