[ovs-dev] Agenda for IRC neeting for 8/13

Nithin Raju nithin at vmware.com
Thu Aug 14 15:28:28 UTC 2014

I just wanted to send out the meeting minutes for any future reference:

Attendees: Alin Serdean, Samuel Ghinet, Ankur Sharma, Saurabh Shah, Ben Pfaff, Nithin Raju.

1. Netlink implementation discussion - design discussions,  status.
- Synced up on where things are. VMware folks to send out kernel patch, and try to integrate with the patch that Alin sent out for porting dpif-linux.c.
- No licensing issues around defining data structures to handle netlink commands. Licensing issues can arise if we start reusing data structure as is from the Linux datapath.
- We might need a mapping structure to map from nl_attr() to a more 'convenient' and contained data structure that can be consumed by the handler functions. We'll look at Cloudbase's mapping structures as well as VMware's data structures defined in OvsPub.h.
- Ankur will be posting a patch soon for the netlink parsing. Ben clarified that we should use the Apache2 licensed code from userspace.
- Expectedly, there are things that need to be ironed out regarding the IOCP. Evidently, Alin needs to know this to make the userspace changes. We'll discuss this over the ML next week.
- Another area of discussion was around the implementation of flow dump. It was explained that the kernel need not take a snapshot of the kernel flowtable to implement flowdump. This provides clarity, and we can make progress on where to maintain the index during flow dump. Further discussions on the ML.
- #Action-item: VMware to send out the patch.
- #Action-item: VMware to review the patch sent by Alin to port dpif-linux.c to Windows posted the same day earlier.

2. Discussion about the persistent ports patch that Cloudbase is tasked with.
- Sam pointed out that a review is already out.

3. Discussion about P0 issues.
- We duped one of the issues, and tagged the Netlink implementation (enhancement) one as P0.
- We'll be looking at Sam's changes to one of the P0 issues.

4. Discussion of other issues.
- None.

5. Queue packets inside the kernel and only send information needed to the userspace, instead of sending the whole packet.
- This is a topic that has been discussed earlier internally in VMware, perhaps publicly as well.
- At a high level, the fraction of packets that make it to userspace should be very small. So it is not worth the optimization.
- There may be some usecases specific to Windows and we'll discuss on the ML.
- #Action-item: VMware to explain the current rationale on the ML or on the issue (ovs-issues/issues/14)

6.. Spooky hash.
- It was agreed that Cloudbase folks would get some performance numbers and if it is better than the existing hash, we'll take it up.

7. Coding styles.
8. Packet buffer management.
9. Fixed sized array
- Reviews to follow up on the ML.

10. Reference counting
- This was seen as useful for flowtable if not for vport. That said, it would be a good idea to lay out the usecase in pseudo code and then take it up. Reviews will follow on the ML.


On Aug 13, 2014, at 8:52 AM, Alin Serdean <aserdean at cloudbasesolutions.com> wrote:

> Hi Nithin,
> Thank you for the feedback!
> We would like to add the following topics:
> Thanks,
> Alin.
> -----Mesaj original-----
> De la: dev [mailto:dev-bounces at openvswitch.org] În numele Nithin Raju
> Trimis: Wednesday, August 13, 2014 6:29 PM
> Către: dev at openvswitch.org
> Subiect: Re: [ovs-dev] Agenda for IRC neeting for 8/13
> Sam/Alin,
> I saw a bunch of reviews go out this morning. This is great!
> Ben was mentioning that as much as possible, we should keep review comments on ovs-dev ML. We can certainly discuss stuff about reviews that are better discussed in an interactive medium, but otherwise, I plan to give out comments and have discussions on the ML itself.
> Just wanted to communicate this.
> thanks,
> Nithin
> On Aug 11, 2014, at 9:19 AM, Nithin Raju <nithin at vmware.com<mailto:nithin at vmware.com>>
> wrote:
> hi folks,
> Here are the items to discuss. Pls. feel free to fill in more items.
> 1. Netlink implementation discussion - design discussions, status.
> 2. Discussion about the persistent ports patch that Cloudbase is tasked with.
> 3. Discussion about P0 issues.
> 4. Discussion of other issues.
> 5. Samuel's reviews:
>   * I'll let Sam and Alin fill this in.
> Like we discussed last week, the meeting is going to be Wednesday 10 AM PST.
> thanks,
> Nithin
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> https://urldefense.proofpoint.com/v1/url?u=http://openvswitch.org/mailman/listinfo/dev&k=oIvRg1%2BdGAgOoM1BIlLLqw%3D%3D%0A&r=ubrOpIWavCMqX4l4j1LEVpTfDj%2FD5Qyn8KCoJIBGvzo%3D%0A&m=mUGzMb%2FM3XWurYXCmU44IW7Lm%2BsYd%2Bpc6lAL3OL1Aek%3D%0A&s=d0bf79a79b8f38522f2e35b040e79d6a95c4c6a3f4c931181e56cc175a1db889

More information about the dev mailing list