Isn't this here to cover the case where the datapath disappears when it is returned to the caller?
(In which case, perhaps we could explain this in the comment) On 15 July 2014 09:55, Andy Zhou <az...@nicira.com> wrote: > Signed-off-by: Andy Zhou <az...@nicira.com> > --- > datapath/datapath.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/datapath/datapath.c b/datapath/datapath.c > index 065356f..1fd7bf3 100644 > --- a/datapath/datapath.c > +++ b/datapath/datapath.c > @@ -140,7 +140,6 @@ static int queue_gso_packets(struct datapath *dp, > struct sk_buff *, > static int queue_userspace_packet(struct datapath *dp, struct sk_buff *, > const struct dp_upcall_info *); > > -/* Must be called with rcu_read_lock or ovs_mutex. */ > static struct datapath *get_dp(struct net *net, int dp_ifindex) > { > struct datapath *dp = NULL; > -- > 1.9.1 > > _______________________________________________ > dev mailing list > dev@openvswitch.org > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev