[ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

Sorin Vinturis svinturis at cloudbasesolutions.com
Fri Jun 12 12:27:27 UTC 2015


Nithin,

The previous patch you are referring was one of a series I had pushed when working at the "Multiple NBLs" issue. At that time I had some trouble when generating packets large enough for the a NBL with multiple NBs to be generated. Because Eitan and Ankur, that also worked on this issue, pushed me to commit my changes so they can see how things were handled, I committed the patch series without thoroughly testing it.

The current patch solves a bug I have found when managing to generate large enough packets. I hope this satisfies your curiosity.

-Sorin

-----Original Message-----
From: Nithin Raju [mailto:nithin at vmware.com] 
Sent: Thursday, 11 June, 2015 18:56
To: Sorin Vinturis
Cc: dev at openvswitch.org
Subject: Re: [ovs-dev] [PATCH] datapath-windows: Correctly complete the original NBL with multiple NBs

> On Jun 11, 2015, at 5:52 AM, Sorin Vinturis <svinturis at cloudbasesolutions.com> wrote:
> 
> OvsCreateNewNBLsFromMultipleNBs function failed to correctly complete 
> the original NBL with multiple NBs after creating multiple NBLs with 
> single NB.
> 
> Signed-off-by: Sorin Vinturis <svinturis at cloudbasesolutions.com>

Sorin,
Thanks for the fix. I am curious how this was not caught during testing for the previous patch.

Acked-by: Nithin Raju <nithin at vmware.com>

thanks,
-- Nithin



More information about the dev mailing list