[ovs-dev] [PATCH 4/4 v2] datapath-windows: remove extract flow in OvsDoRecirc()
Paul Boca
pboca at cloudbasesolutions.com
Mon May 23 06:02:34 UTC 2016
Looks good to me.
Acked-by: Paul-Daniel Boca <pboca at cloudbasesolutions.com>
> -----Original Message-----
> From: dev [mailto:dev-bounces at openvswitch.org] On Behalf Of Sairam
> Venugopal
> Sent: Tuesday, May 17, 2016 9:06 PM
> To: Nithin Raju; dev at openvswitch.org
> Subject: Re: [ovs-dev] [PATCH 4/4 v2] datapath-windows: remove extract flow
> in OvsDoRecirc()
>
> Acked-by: Sairam Venugopal <vsairam at vmware.com>
>
>
> On 5/17/16, 10:15 AM, "Nithin Raju" <nithin at vmware.com> wrote:
>
> >It is not necessary to do a flow extract in OvsDoRecirc().
> >In fact, doing it would overwrite the tunnel key within
> >'key'. So, let's remove the call.
> >
> >Signed-off-by: Nithin Raju <nithin at vmware.com>
> >Signed-off-by: Sairam Venugopal <vsairam at vmware.com>
> >Co-Authored-by: Sairam Venugopal <vsairam at vmware.com>
> >---
> > datapath-windows/ovsext/Actions.c | 9 ---------
> > 1 file changed, 9 deletions(-)
> >
> >diff --git a/datapath-windows/ovsext/Actions.c
> >b/datapath-windows/ovsext/Actions.c
> >index 5ad29ee..4edf7d0 100644
> >--- a/datapath-windows/ovsext/Actions.c
> >+++ b/datapath-windows/ovsext/Actions.c
> >@@ -1989,15 +1989,6 @@ OvsDoRecirc(POVS_SWITCH_CONTEXT
> switchContext,
> >
> >NET_BUFFER_LIST_SWITCH_FORWARDING_DETAIL(curNbl),
> > completionList, layers, TRUE);
> >
> >- status = OvsExtractFlow(ovsFwdCtx.curNbl, ovsFwdCtx.srcVportNo, key,
> >- &ovsFwdCtx.layers, NULL);
> >- if (status != NDIS_STATUS_SUCCESS) {
> >- OvsCompleteNBLForwardingCtx(&ovsFwdCtx,
> >- L"OVS-Dropped due to extract flow failure");
> >- ovsActionStats.failedFlowMiss++;
> >- return NDIS_STATUS_FAILURE;
> >- }
> >-
> > flow = OvsLookupFlow(&ovsFwdCtx.switchContext->datapath, key, &hash,
> >FALSE);
> > if (flow) {
> > UINT32 level = OvsDeferredActionsLevelGet();
> >--
> >2.7.1.windows.1
> >
>
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev
More information about the dev
mailing list