<div dir="ltr">Just sent v2 patch using OVS_CB for review. <br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 25, 2013 at 9:57 AM, Andy Zhou <span dir="ltr"><<a href="mailto:azhou@nicira.com" target="_blank">azhou@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">No, the patch was prepared before your email. <br></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Tue, Jun 25, 2013 at 9:53 AM, Jesse Gross <span dir="ltr"><<a href="mailto:jesse@nicira.com" target="_blank">jesse@nicira.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>On Tue, Jun 25, 2013 at 8:22 AM, Andy Zhou <<a href="mailto:azhou@nicira.com" target="_blank">azhou@nicira.com</a>> wrote:<br>
> OVS_ACTION_ATTR_USERSPACE action was sending the key from the matching<br>
> flow. This works for exact match flows because flow keys are the<br>
> same as packet keys. However, it does not work with wildcarded flows as<br>
> the packet keys may be different than the flow keys. This patch preserves<br>
> the packet keys and send it when calling output_userspace().<br>
><br>
> Bug #18163<br>
><br>
> Signed-off-by: Andy Zhou <<a href="mailto:azhou@nicira.com" target="_blank">azhou@nicira.com</a>><br>
<br>
</div>Is there a reason to not put the extracted flow in OVS_CB as I had<br>
mentioned? I think it would make the patch much smaller and cleaner.<br>
</blockquote></div><br></div>
</div></div></blockquote></div><br></div>