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

_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to