On 2 December 2016 at 16:06, Sairam Venugopal <[email protected]> wrote:
> Some of the variables were getting re-defined in the function. This causes > compiler warning in Windows SDK 10. > > Signed-off-by: Sairam Venugopal <[email protected]> > Applied, thanks! > --- > datapath-windows/ovsext/BufferMgmt.c | 22 +++++++++++----------- > datapath-windows/ovsext/Offload.c | 8 ++++---- > 2 files changed, 15 insertions(+), 15 deletions(-) > > diff --git a/datapath-windows/ovsext/BufferMgmt.c > b/datapath-windows/ovsext/BufferMgmt.c > index 9c5b8fb..6027c35 100644 > --- a/datapath-windows/ovsext/BufferMgmt.c > +++ b/datapath-windows/ovsext/BufferMgmt.c > @@ -1541,27 +1541,27 @@ OvsCompleteNBL(POVS_SWITCH_CONTEXT context, > } > > if (flags & (OVS_BUFFER_PRIVATE_MDL | OVS_BUFFER_PRIVATE_DATA)) { > - PNET_BUFFER nb = NET_BUFFER_LIST_FIRST_NB(nbl); > - while (nb) { > - PMDL mdl = NET_BUFFER_FIRST_MDL(nb); > - NET_BUFFER_FIRST_MDL(nb) = NULL; > + PNET_BUFFER nbTemp = NET_BUFFER_LIST_FIRST_NB(nbl); > + while (nbTemp) { > + PMDL mdl = NET_BUFFER_FIRST_MDL(nbTemp); > + NET_BUFFER_FIRST_MDL(nbTemp) = NULL; > ASSERT(mdl->Next == NULL); > OvsFreeMDLAndData(mdl); > - nb = NET_BUFFER_NEXT_NB(nb); > + nbTemp = NET_BUFFER_NEXT_NB(nbTemp); > } > } > > if (flags & OVS_BUFFER_PRIVATE_NET_BUFFER) { > - PNET_BUFFER nb, nextNb; > + PNET_BUFFER nbTemp, nextNb; > > - nb = NET_BUFFER_LIST_FIRST_NB(nbl); > - while (nb) { > - nextNb = NET_BUFFER_NEXT_NB(nb); > - NdisFreeNetBuffer(nb); > + nbTemp = NET_BUFFER_LIST_FIRST_NB(nbl); > + while (nbTemp) { > + nextNb = NET_BUFFER_NEXT_NB(nbTemp); > + NdisFreeNetBuffer(nbTemp); > #ifdef DBG > InterlockedDecrement((LONG volatile *)&ovsPool->nbCount); > #endif > - nb = nextNb; > + nbTemp = nextNb; > } > NET_BUFFER_LIST_FIRST_NB(nbl) = NULL; > } > diff --git a/datapath-windows/ovsext/Offload.c b/datapath-windows/ovsext/ > Offload.c > index 3246f5e..f3ab0e1 100644 > --- a/datapath-windows/ovsext/Offload.c > +++ b/datapath-windows/ovsext/Offload.c > @@ -66,12 +66,12 @@ CalculateOnesComplement(UINT8 *start, > UINT64 initial, > BOOLEAN isEvenStart) > { > - UINT64 sum = 0, val; > + UINT64 sum = 0, val64; > UINT64 *src = (UINT64 *)start; > while (totalLength > 7) { > - val = *src; > - sum += val; > - if (sum < val) sum++; > + val64 = *src; > + sum += val64; > + if (sum < val64) sum++; > src++; > totalLength -= 8; > } > -- > 2.9.0.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
