On Tue, Oct 04, 2016 at 04:25:35PM -0700, Daniele Di Proietto wrote:
> dp_netdev_get_pmd() is allowed to return NULL (even if we call it with
> NON_PMD_CORE_ID) for different reasons:
> 
> * Since we use RCU to protect pmd threads, it is possible that
>   ovs_refcount_try_ref_rcu() has failed.
> * During reconfiguration we destroy every thread.

I'd recommend making the comment on dp_netdev_get_pmd() clearer, by
mentioning that it can always return NULL, even with NON_PMD_CORE_ID as
an argument.

Acked-by: Ben Pfaff <b...@ovn.org>
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to