On Thu, Dec 21, 2017 at 02:25:11PM -0800, Justin Pettit wrote:
>     - This reduces the number of times upcall cookies are processed.
>     - It separate true miss calls from slow-path actions.
> 
> The reorganization will also be useful for a future commit.
> 
> Signed-off-by: Justin Pettit <jpet...@ovn.org>

I took a second look at this patch, giving it more scrutiny.  I can't
yet explain why this line is deleted.  It seems like ->ufid is still
used in the same way as it was before:

> @@ -813,7 +816,6 @@ recv_upcalls(struct handler *handler)
>  
>          upcall->key = dupcall->key;
>          upcall->key_len = dupcall->key_len;
> -        upcall->ufid = &dupcall->ufid;
>  
>          upcall->out_tun_key = dupcall->out_tun_key;
>          upcall->actions = dupcall->actions;
_______________________________________________
dev mailing list
d...@openvswitch.org
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to