On Mon, Sep 9, 2013 at 12:20 AM, Simon Horman <ho...@verge.net.au> wrote: > diff --git a/datapath/actions.c b/datapath/actions.c > index 30ea1d2..6741d81 100644 > --- a/datapath/actions.c > +++ b/datapath/actions.c > @@ -105,22 +105,29 @@ static int pop_vlan(struct sk_buff *skb) > return 0; > } > > -static int push_vlan(struct sk_buff *skb, const struct ovs_action_push_vlan > *vlan) > +/* push down current VLAN tag */ > +static struct sk_buff *put_vlan(struct sk_buff *skb)
This never changes the skb, right? Can we simplify things and just return an error code? _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev
