[ovs-dev] [PATCH] datapath-windows: Fix alignment in actions

[Anand Kumar]

Acked-by: Anand Kumar <kumaranand at vmware.com>
 
Thanks,
Anand Kumar

On 5/17/17, 5:57 AM, "ovs-dev-bounces at openvswitch.org on behalf of Alin Serdean" <ovs-dev-bounces at openvswitch.org on behalf of aserdean at cloudbasesolutions.com> wrote:

    Found by inspection.
    
    Signed-off-by: Alin Gabriel Serdean <aserdean at cloudbasesolutions.com>
    ---
     datapath-windows/ovsext/Actions.c | 24 ++++++++++++------------
     1 file changed, 12 insertions(+), 12 deletions(-)
    
    diff --git a/datapath-windows/ovsext/Actions.c b/datapath-windows/ovsext/Actions.c
    index e2eae9a..ebfb8a3 100644
    --- a/datapath-windows/ovsext/Actions.c
    +++ b/datapath-windows/ovsext/Actions.c
    @@ -2035,18 +2035,18 @@ OvsDoExecuteActions(POVS_SWITCH_CONTEXT switchContext,
                     }
                     goto dropit;
                 } else if (oldNbl != ovsFwdCtx.curNbl) {
    -               /*
    -                * OvsIpv4Reassemble consumes the original NBL and creates a
    -                * new one and assigns it to the curNbl of ovsFwdCtx.
    -                */
    -               OvsInitForwardingCtx(&ovsFwdCtx,
    -                                    ovsFwdCtx.switchContext,
    -                                    ovsFwdCtx.curNbl,
    -                                    ovsFwdCtx.srcVportNo,
    -                                    ovsFwdCtx.sendFlags,
    -                                    NET_BUFFER_LIST_SWITCH_FORWARDING_DETAIL(ovsFwdCtx.curNbl),
    -                                    ovsFwdCtx.completionList,
    -                                    &ovsFwdCtx.layers, FALSE);
    +                /*
    +                 * OvsIpv4Reassemble consumes the original NBL and creates a
    +                 * new one and assigns it to the curNbl of ovsFwdCtx.
    +                 */
    +                OvsInitForwardingCtx(&ovsFwdCtx,
    +                                     ovsFwdCtx.switchContext,
    +                                     ovsFwdCtx.curNbl,
    +                                     ovsFwdCtx.srcVportNo,
    +                                     ovsFwdCtx.sendFlags,
    +                                     NET_BUFFER_LIST_SWITCH_FORWARDING_DETAIL(ovsFwdCtx.curNbl),
    +                                     ovsFwdCtx.completionList,
    +                                     &ovsFwdCtx.layers, FALSE);
                 }
                 break;
             }
    -- 
    2.10.2.windows.1
    _______________________________________________
    dev mailing list
    dev at openvswitch.org
    https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=Q5z9tBe-nAOpE7LIHSPV8uy5-437agMXvkeHHMkR8Us&m=UsxSsO9WSAnknHPl2cBu_yKunEVS7VT_t782bL9cOmA&s=MuqK1pQyzFQzIfS9GbpUMBaefUMDGLoH5YmQqmzIdyw&e=