<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Hey Ben,<div><br></div><div>Sorry for the delayed response. Unfortunately, I've had some issues with setting up an NVP perf environment lately so I haven't gotten any numbers. When I do, I'll publish them on this thread.</div><div><br></div><div>Ethan's rationale is that it simplifies the code and we don't incur the memory overhead of allocating a hash map and this would outweigh the negative effect of possibly re-processing a packet from the same flow. Since userspace forwarding has gotten so much faster, this wouldn't be so bad.</div><div><br></div><div>Obviously, I need perf numbers to justify these claims; I will post them when I do.</div><div><br><div>
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: medium; font-variant: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><b>Ryan Wilson</b><br><i style="font-weight: normal; ">Member of Technical Staff</i><br><a href="mailto:wryan@vmware.com">wryan@vmware.com</a><br>3401 Hillview Avenue, Palo Alto, CA<br>650.427.1511 Office<br>916.588.7783 Mobile</div>
</div>
<br><div><div>On May 9, 2014, at 8:25 AM, Ben Pfaff &lt;<a href="mailto:blp@nicira.com">blp@nicira.com</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">On Wed, May 07, 2014 at 03:14:01PM -0700, Ryan Wilson wrote:<br><blockquote type="cite">The upcall hander keeps a hash table which hashes flow to a list of<br>corresponding packets. This used to be necessary as packets with the same flow<br>had similar actions and calculating actions used to be a performance bottleneck.<br>Now that userspace action calculation performance has improved, there is no need<br>for this hash map.<br><br>This patch removes this hash map and each packet has its own upcall.<br><br>Signed-off-by: Ryan Wilson &lt;<a href="mailto:wryan@nicira.com">wryan@nicira.com</a>&gt;<br></blockquote><br>Hmm. &nbsp;Do we have performance numbers to back this up? &nbsp;Ethan, I assume<br>that you suggested this project; do you have additional rationale?<br></blockquote></div><br></div></body></html>