Hi Simon, > -----Original Message----- > From: Simon Horman [mailto:[email protected]] > Sent: Thursday, April 12, 2018 5:13 PM > To: Chris Mi <[email protected]> > Cc: [email protected]; Roi Dayan <[email protected]>; Paul Blakey > <[email protected]> > 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 [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
