Hi Simon,

> -----Original Message-----
> From: Simon Horman [mailto:simon.hor...@netronome.com]
> Sent: Thursday, April 12, 2018 5:13 PM
> To: Chris Mi <chr...@mellanox.com>
> Cc: d...@openvswitch.org; Roi Dayan <r...@mellanox.com>; Paul Blakey
> <pa...@mellanox.com>
> Subject: Re: [ovs-dev 0/2] Add offloading of multiple outputs
> 
> On Tue, Apr 10, 2018 at 02:18:07PM +0900, Chris Mi wrote:
> > This patchset adds the offloading support of multiple outputs.
> >
> > The first patch makes the actions order consistent. In previous
> > implementation, the actions order is lost when offloading. If there is
> > only one output, there is on problem. But if there are multiple
> > outputs, some ports should see the packets before some actions and
> > some ports should see the packets after some actions. So we should
> > keep the actions order that OVS DP passes to TC library to make sure
> > the right ports see the right packets.
> >
> > The second patch is straighforward that we remove the one output
> > limitation. If the output is the last action, we use TC mirred
> > redirect. Otherwise, we use TC mirred mirror.
> 
> Thanks, I'm very pleased to see this. And it was doubly pleasing to see the
> simplicity of the second patch, which adds the feature, with the first patch 
> in
> place.
> 
> I have applied this series to the master branch after removing the Issue and
> Change-Id tags.

I should remove the issue and Change-Id tags before review. Sorry about that.
Thanks for your help to remove them and the quick review.

Thanks,
Chris
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to