On Thu, Aug 17, 2017 at 12:21:43PM +0300, Paul Blakey wrote: > > > On 14/08/2017 16:00, Simon Horman wrote: > >On Tue, Aug 08, 2017 at 05:21:53PM +0300, Roi Dayan wrote: > >>From: Paul Blakey <pa...@mellanox.com> > >> > >>To be later used to implement ovs action set offloading. > >> > >>Signed-off-by: Paul Blakey <pa...@mellanox.com> > >>Reviewed-by: Roi Dayan <r...@mellanox.com> > >>--- > >> lib/tc.c | 372 > >> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++- > >> lib/tc.h | 12 +++ > >> 2 files changed, 381 insertions(+), 3 deletions(-)
... > >>+static void > >> nl_msg_put_act_push_vlan(struct ofpbuf *request, uint16_t vid, uint8_t > >> prio) > >> { > >> size_t offset; > > Hi and thanks for the review, > > The above style corresponds with the rest of the file (scope and > nl_msg_put_act_* funcs) > Maybe we do a style patch that remove all this extra scopes in a later > patch? Sure, that is fine by me. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev