[ovs-dev] [PATCH v2 0/3] Support for OVS datapath on Windows platform.

Saurabh Shah ssaurabh at vmware.com
Sat Jul 26 00:32:16 UTC 2014


Hi Alessandro,

>Hi Saurabh,
>
>I¹d suggest that before doing a full review of the kernel driver we need
>to have:
>
>* Userspace Netlink interface support (which means that patches 1 and 2
>in this
>set can be skipped)

There is some dependence between patches 2 & 3. In my V3 change I will
drop out the entire userspace change & just post the kernel.

>
>* Kernel Netlink interface support (not provided by patch 3)
>
>* Support for multiple datapaths and vswitches

I would like to understand the use case a little better. Can you let me
know?

>
>* Split the kernel driver code in smaller patches to allow a proper
>review (I
>made a simple suggestion in a previous email, but any other solution is
>good)

Splitting the change into multiple smaller changes will likely be very
difficult and time consuming. Once the initial kernel support has been
checked in, lets do incremental changes/reviews from then on.

Thanks!
Saurabh




More information about the dev mailing list