[ovs-dev] [PATCH V5 5/5] ofproto-dpif-upcall: Remove the dispatcher thread.
Alex Wang
alexw at nicira.com
Thu Mar 20 22:39:05 UTC 2014
Thanks for the review, Ben
I am confused by upcall_destroy()'s free_upcall parameter: all of the
> callers pass 'false'.
I think I'll remove the input argument 'free_upcall' in upall_destroy(),
since
the array of upcalls is defined as local variable in udpif_upcall_handler().
>
>
It might not be worth it to destroy the upcalls one by one as one
> decides that they're no longer needed. I believe that
> udpif_upcall_handler() could just destroy all of them, if
> handle_upcalls() didn't destroy any.
>
>
I like the idea of destroying all upcalls together. I still want to do it
in the end of
handle_upalls(). Basically, I will mark ''is_valid = false" for invalid or
erroneous.
And in the end of handle_upalls() destroys all upcalls.
I like the simplification.
>
;D
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openvswitch.org/pipermail/ovs-dev/attachments/20140320/dff8bd2a/attachment-0005.html>
More information about the dev
mailing list