Re-Acking - Acked-by: Sairam Venugopal <[email protected]>
On 1/26/17, 3:38 PM, "[email protected] on behalf of Alin Serdean" <[email protected] on behalf of [email protected]> wrote: >From: Alin Serdean <[email protected]> > >bufferStart can be used directly to access the data of the net buffer. >Add the MDL offset to save unnecessary additions. > >Signed-off-by: Alin Gabriel Serdean <[email protected]> >Acked-by: Sairam Venugopal <[email protected]> >--- >v3: add acked >v2: no change >--- > datapath-windows/ovsext/Actions.c | 8 ++++---- > 1 file changed, 4 insertions(+), 4 deletions(-) > >diff --git a/datapath-windows/ovsext/Actions.c >b/datapath-windows/ovsext/Actions.c >index b4a286b..760888e 100644 >--- a/datapath-windows/ovsext/Actions.c >+++ b/datapath-windows/ovsext/Actions.c >@@ -1390,13 +1390,13 @@ OvsUpdateIPv4Header(OvsForwardingContext *ovsFwdCtx, > mdlLen -= curMdlOffset; > ASSERT(mdlLen >= hdrSize); > } >- >- ipHdr = (IPHdr *)(bufferStart + curMdlOffset + layers->l3Offset); >+ bufferStart += curMdlOffset; >+ ipHdr = (IPHdr *)(bufferStart + layers->l3Offset); > > if (layers->isTcp) { >- tcpHdr = (TCPHdr *)(bufferStart + curMdlOffset + layers->l4Offset); >+ tcpHdr = (TCPHdr *)(bufferStart + layers->l4Offset); > } else if (layers->isUdp) { >- udpHdr = (UDPHdr *)(bufferStart + curMdlOffset + layers->l4Offset); >+ udpHdr = (UDPHdr *)(bufferStart + layers->l4Offset); > } > > /* >-- >2.10.2.windows.1 >_______________________________________________ >dev mailing list >[email protected] >https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
