On 12/10/2016 12:37, "Ben Pfaff" <b...@ovn.org> wrote:
>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.
Good point, done
>
>Acked-by: Ben Pfaff <b...@ovn.org>
Thanks, I applied this to master and branch-2.{6,5,4}
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev
