[ovs-dev] [ovs-dev 0/2] Add offloading of multiple outputs

Chris Mi chrism at mellanox.com
Thu Apr 12 09:37:05 UTC 2018

Hi Simon,

> -----Original Message-----
> From: Simon Horman [mailto:simon.horman at netronome.com]
> Sent: Thursday, April 12, 2018 5:13 PM
> To: Chris Mi <chrism at mellanox.com>
> Cc: dev at openvswitch.org; Roi Dayan <roid at mellanox.com>; Paul Blakey
> <paulb at 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.


More information about the dev mailing list