Use "error-checking" mutexes in place of other kinds wherever possible.
[sliver-openvswitch.git] / ofproto / ofproto-dpif-sflow.c
1 /*
2  * Copyright (c) 2009, 2010, 2011, 2012, 2013 Nicira, Inc.
3  * Copyright (c) 2009 InMon Corp.
4  *
5  * Licensed under the Apache License, Version 2.0 (the "License");
6  * you may not use this file except in compliance with the License.
7  * You may obtain a copy of the License at:
8  *
9  *     http://www.apache.org/licenses/LICENSE-2.0
10  *
11  * Unless required by applicable law or agreed to in writing, software
12  * distributed under the License is distributed on an "AS IS" BASIS,
13  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
14  * See the License for the specific language governing permissions and
15  * limitations under the License.
16  */
17
18 #include <config.h>
19 #include "ofproto-dpif-sflow.h"
20 #include <inttypes.h>
21 #include <sys/socket.h>
22 #include <net/if.h>
23 #include <stdlib.h>
24 #include "collectors.h"
25 #include "compiler.h"
26 #include "dpif.h"
27 #include "hash.h"
28 #include "hmap.h"
29 #include "netdev.h"
30 #include "netlink.h"
31 #include "ofpbuf.h"
32 #include "ofproto.h"
33 #include "packets.h"
34 #include "poll-loop.h"
35 #include "route-table.h"
36 #include "sflow_api.h"
37 #include "socket-util.h"
38 #include "timeval.h"
39 #include "vlog.h"
40 #include "lib/odp-util.h"
41 #include "ofproto-provider.h"
42
43 VLOG_DEFINE_THIS_MODULE(sflow);
44
45 static struct ovs_mutex mutex;
46
47 struct dpif_sflow_port {
48     struct hmap_node hmap_node; /* In struct dpif_sflow's "ports" hmap. */
49     SFLDataSource_instance dsi; /* sFlow library's notion of port number. */
50     struct ofport *ofport;      /* To retrive port stats. */
51     odp_port_t odp_port;
52 };
53
54 struct dpif_sflow {
55     struct collectors *collectors;
56     SFLAgent *sflow_agent;
57     struct ofproto_sflow_options *options;
58     time_t next_tick;
59     size_t n_flood, n_all;
60     struct hmap ports;          /* Contains "struct dpif_sflow_port"s. */
61     uint32_t probability;
62     atomic_int ref_cnt;
63 };
64
65 static void dpif_sflow_del_port__(struct dpif_sflow *,
66                                   struct dpif_sflow_port *);
67
68 #define RECEIVER_INDEX 1
69
70 static struct vlog_rate_limit rl = VLOG_RATE_LIMIT_INIT(1, 5);
71
72 static bool
73 nullable_string_is_equal(const char *a, const char *b)
74 {
75     return a ? b && !strcmp(a, b) : !b;
76 }
77
78 static bool
79 ofproto_sflow_options_equal(const struct ofproto_sflow_options *a,
80                          const struct ofproto_sflow_options *b)
81 {
82     return (sset_equals(&a->targets, &b->targets)
83             && a->sampling_rate == b->sampling_rate
84             && a->polling_interval == b->polling_interval
85             && a->header_len == b->header_len
86             && a->sub_id == b->sub_id
87             && nullable_string_is_equal(a->agent_device, b->agent_device)
88             && nullable_string_is_equal(a->control_ip, b->control_ip));
89 }
90
91 static struct ofproto_sflow_options *
92 ofproto_sflow_options_clone(const struct ofproto_sflow_options *old)
93 {
94     struct ofproto_sflow_options *new = xmemdup(old, sizeof *old);
95     sset_clone(&new->targets, &old->targets);
96     new->agent_device = old->agent_device ? xstrdup(old->agent_device) : NULL;
97     new->control_ip = old->control_ip ? xstrdup(old->control_ip) : NULL;
98     return new;
99 }
100
101 static void
102 ofproto_sflow_options_destroy(struct ofproto_sflow_options *options)
103 {
104     if (options) {
105         sset_destroy(&options->targets);
106         free(options->agent_device);
107         free(options->control_ip);
108         free(options);
109     }
110 }
111
112 /* sFlow library callback to allocate memory. */
113 static void *
114 sflow_agent_alloc_cb(void *magic OVS_UNUSED, SFLAgent *agent OVS_UNUSED,
115                      size_t bytes)
116 {
117     return calloc(1, bytes);
118 }
119
120 /* sFlow library callback to free memory. */
121 static int
122 sflow_agent_free_cb(void *magic OVS_UNUSED, SFLAgent *agent OVS_UNUSED,
123                     void *obj)
124 {
125     free(obj);
126     return 0;
127 }
128
129 /* sFlow library callback to report error. */
130 static void
131 sflow_agent_error_cb(void *magic OVS_UNUSED, SFLAgent *agent OVS_UNUSED,
132                      char *msg)
133 {
134     VLOG_WARN("sFlow agent error: %s", msg);
135 }
136
137 /* sFlow library callback to send datagram. */
138 static void
139 sflow_agent_send_packet_cb(void *ds_, SFLAgent *agent OVS_UNUSED,
140                            SFLReceiver *receiver OVS_UNUSED, u_char *pkt,
141                            uint32_t pktLen)
142 {
143     struct dpif_sflow *ds = ds_;
144     collectors_send(ds->collectors, pkt, pktLen);
145 }
146
147 static struct dpif_sflow_port *
148 dpif_sflow_find_port(const struct dpif_sflow *ds, odp_port_t odp_port)
149     OVS_REQUIRES(mutex)
150 {
151     struct dpif_sflow_port *dsp;
152
153     HMAP_FOR_EACH_IN_BUCKET (dsp, hmap_node, hash_odp_port(odp_port),
154                              &ds->ports) {
155         if (dsp->odp_port == odp_port) {
156             return dsp;
157         }
158     }
159     return NULL;
160 }
161
162 static void
163 sflow_agent_get_counters(void *ds_, SFLPoller *poller,
164                          SFL_COUNTERS_SAMPLE_TYPE *cs)
165     OVS_REQUIRES(mutex)
166 {
167     struct dpif_sflow *ds = ds_;
168     SFLCounters_sample_element elem;
169     enum netdev_features current;
170     struct dpif_sflow_port *dsp;
171     SFLIf_counters *counters;
172     struct netdev_stats stats;
173     enum netdev_flags flags;
174
175     dsp = dpif_sflow_find_port(ds, u32_to_odp(poller->bridgePort));
176     if (!dsp) {
177         return;
178     }
179
180     elem.tag = SFLCOUNTERS_GENERIC;
181     counters = &elem.counterBlock.generic;
182     counters->ifIndex = SFL_DS_INDEX(poller->dsi);
183     counters->ifType = 6;
184     if (!netdev_get_features(dsp->ofport->netdev, &current, NULL, NULL, NULL)) {
185         /* The values of ifDirection come from MAU MIB (RFC 2668): 0 = unknown,
186            1 = full-duplex, 2 = half-duplex, 3 = in, 4=out */
187         counters->ifSpeed = netdev_features_to_bps(current, 0);
188         counters->ifDirection = (netdev_features_is_full_duplex(current)
189                                  ? 1 : 2);
190     } else {
191         counters->ifSpeed = 100000000;
192         counters->ifDirection = 0;
193     }
194     if (!netdev_get_flags(dsp->ofport->netdev, &flags) && flags & NETDEV_UP) {
195         counters->ifStatus = 1; /* ifAdminStatus up. */
196         if (netdev_get_carrier(dsp->ofport->netdev)) {
197             counters->ifStatus |= 2; /* ifOperStatus us. */
198         }
199     } else {
200         counters->ifStatus = 0;  /* Down. */
201     }
202
203     /* XXX
204        1. Is the multicast counter filled in?
205        2. Does the multicast counter include broadcasts?
206        3. Does the rx_packets counter include multicasts/broadcasts?
207     */
208     ofproto_port_get_stats(dsp->ofport, &stats);
209     counters->ifInOctets = stats.rx_bytes;
210     counters->ifInUcastPkts = stats.rx_packets;
211     counters->ifInMulticastPkts = stats.multicast;
212     counters->ifInBroadcastPkts = -1;
213     counters->ifInDiscards = stats.rx_dropped;
214     counters->ifInErrors = stats.rx_errors;
215     counters->ifInUnknownProtos = -1;
216     counters->ifOutOctets = stats.tx_bytes;
217     counters->ifOutUcastPkts = stats.tx_packets;
218     counters->ifOutMulticastPkts = -1;
219     counters->ifOutBroadcastPkts = -1;
220     counters->ifOutDiscards = stats.tx_dropped;
221     counters->ifOutErrors = stats.tx_errors;
222     counters->ifPromiscuousMode = 0;
223
224     SFLADD_ELEMENT(cs, &elem);
225     sfl_poller_writeCountersSample(poller, cs);
226 }
227
228 /* Obtains an address to use for the local sFlow agent and stores it into
229  * '*agent_addr'.  Returns true if successful, false on failure.
230  *
231  * The sFlow agent address should be a local IP address that is persistent and
232  * reachable over the network, if possible.  The IP address associated with
233  * 'agent_device' is used if it has one, and otherwise 'control_ip', the IP
234  * address used to talk to the controller.  If the agent device is not
235  * specified then it is figured out by taking a look at the routing table based
236  * on 'targets'. */
237 static bool
238 sflow_choose_agent_address(const char *agent_device,
239                            const struct sset *targets,
240                            const char *control_ip,
241                            SFLAddress *agent_addr)
242 {
243     const char *target;
244     struct in_addr in4;
245
246     memset(agent_addr, 0, sizeof *agent_addr);
247     agent_addr->type = SFLADDRESSTYPE_IP_V4;
248
249     if (agent_device) {
250         if (!netdev_get_in4_by_name(agent_device, &in4)) {
251             goto success;
252         }
253     }
254
255     SSET_FOR_EACH (target, targets) {
256         struct sockaddr_in sin;
257         char name[IFNAMSIZ];
258
259         if (inet_parse_active(target, SFL_DEFAULT_COLLECTOR_PORT, &sin)
260             && route_table_get_name(sin.sin_addr.s_addr, name)
261             && !netdev_get_in4_by_name(name, &in4)) {
262             goto success;
263         }
264     }
265
266     if (control_ip && !lookup_ip(control_ip, &in4)) {
267         goto success;
268     }
269
270     VLOG_ERR("could not determine IP address for sFlow agent");
271     return false;
272
273 success:
274     agent_addr->address.ip_v4.addr = (OVS_FORCE uint32_t) in4.s_addr;
275     return true;
276 }
277
278 static void
279 dpif_sflow_clear__(struct dpif_sflow *ds) OVS_REQUIRES(mutex)
280 {
281     if (ds->sflow_agent) {
282         sfl_agent_release(ds->sflow_agent);
283         free(ds->sflow_agent);
284         ds->sflow_agent = NULL;
285     }
286     collectors_destroy(ds->collectors);
287     ds->collectors = NULL;
288     ofproto_sflow_options_destroy(ds->options);
289     ds->options = NULL;
290
291     /* Turn off sampling to save CPU cycles. */
292     ds->probability = 0;
293 }
294
295 void
296 dpif_sflow_clear(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
297 {
298     ovs_mutex_lock(&mutex);
299     dpif_sflow_clear__(ds);
300     ovs_mutex_unlock(&mutex);
301 }
302
303 bool
304 dpif_sflow_is_enabled(const struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
305 {
306     bool enabled;
307
308     ovs_mutex_lock(&mutex);
309     enabled = ds->collectors != NULL;
310     ovs_mutex_unlock(&mutex);
311     return enabled;
312 }
313
314 struct dpif_sflow *
315 dpif_sflow_create(void)
316 {
317     static struct ovsthread_once once = OVSTHREAD_ONCE_INITIALIZER;
318     struct dpif_sflow *ds;
319
320     if (ovsthread_once_start(&once)) {
321         ovs_mutex_init_recursive(&mutex);
322         ovsthread_once_done(&once);
323     }
324
325     ds = xcalloc(1, sizeof *ds);
326     ds->next_tick = time_now() + 1;
327     hmap_init(&ds->ports);
328     ds->probability = 0;
329     route_table_register();
330     atomic_init(&ds->ref_cnt, 1);
331
332     return ds;
333 }
334
335 struct dpif_sflow *
336 dpif_sflow_ref(const struct dpif_sflow *ds_)
337 {
338     struct dpif_sflow *ds = CONST_CAST(struct dpif_sflow *, ds_);
339     if (ds) {
340         int orig;
341         atomic_add(&ds->ref_cnt, 1, &orig);
342         ovs_assert(orig > 0);
343     }
344     return ds;
345 }
346
347 /* 32-bit fraction of packets to sample with.  A value of 0 samples no packets,
348  * a value of %UINT32_MAX samples all packets and intermediate values sample
349  * intermediate fractions of packets. */
350 uint32_t
351 dpif_sflow_get_probability(const struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
352 {
353     uint32_t probability;
354     ovs_mutex_lock(&mutex);
355     probability = ds->probability;
356     ovs_mutex_unlock(&mutex);
357     return probability;
358 }
359
360 void
361 dpif_sflow_unref(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
362 {
363     int orig;
364
365     if (!ds) {
366         return;
367     }
368
369     atomic_sub(&ds->ref_cnt, 1, &orig);
370     ovs_assert(orig > 0);
371     if (orig == 1) {
372         struct dpif_sflow_port *dsp, *next;
373
374         route_table_unregister();
375         dpif_sflow_clear(ds);
376         HMAP_FOR_EACH_SAFE (dsp, next, hmap_node, &ds->ports) {
377             dpif_sflow_del_port__(ds, dsp);
378         }
379         hmap_destroy(&ds->ports);
380         free(ds);
381     }
382 }
383
384 static void
385 dpif_sflow_add_poller(struct dpif_sflow *ds, struct dpif_sflow_port *dsp)
386     OVS_REQUIRES(mutex)
387 {
388     SFLPoller *poller = sfl_agent_addPoller(ds->sflow_agent, &dsp->dsi, ds,
389                                             sflow_agent_get_counters);
390     sfl_poller_set_sFlowCpInterval(poller, ds->options->polling_interval);
391     sfl_poller_set_sFlowCpReceiver(poller, RECEIVER_INDEX);
392     sfl_poller_set_bridgePort(poller, odp_to_u32(dsp->odp_port));
393 }
394
395 void
396 dpif_sflow_add_port(struct dpif_sflow *ds, struct ofport *ofport,
397                     odp_port_t odp_port) OVS_EXCLUDED(mutex)
398 {
399     struct dpif_sflow_port *dsp;
400     int ifindex;
401
402     ovs_mutex_lock(&mutex);
403     dpif_sflow_del_port(ds, odp_port);
404
405     ifindex = netdev_get_ifindex(ofport->netdev);
406
407     if (ifindex <= 0) {
408         /* Not an ifindex port, so do not add a cross-reference to it here */
409         goto out;
410     }
411
412     /* Add to table of ports. */
413     dsp = xmalloc(sizeof *dsp);
414     dsp->ofport = ofport;
415     dsp->odp_port = odp_port;
416     SFL_DS_SET(dsp->dsi, SFL_DSCLASS_IFINDEX, ifindex, 0);
417     hmap_insert(&ds->ports, &dsp->hmap_node, hash_odp_port(odp_port));
418
419     /* Add poller. */
420     if (ds->sflow_agent) {
421         dpif_sflow_add_poller(ds, dsp);
422     }
423
424 out:
425     ovs_mutex_unlock(&mutex);
426 }
427
428 static void
429 dpif_sflow_del_port__(struct dpif_sflow *ds, struct dpif_sflow_port *dsp)
430     OVS_REQUIRES(mutex)
431 {
432     if (ds->sflow_agent) {
433         sfl_agent_removePoller(ds->sflow_agent, &dsp->dsi);
434         sfl_agent_removeSampler(ds->sflow_agent, &dsp->dsi);
435     }
436     hmap_remove(&ds->ports, &dsp->hmap_node);
437     free(dsp);
438 }
439
440 void
441 dpif_sflow_del_port(struct dpif_sflow *ds, odp_port_t odp_port)
442     OVS_EXCLUDED(mutex)
443 {
444     struct dpif_sflow_port *dsp;
445
446     ovs_mutex_lock(&mutex);
447     dsp = dpif_sflow_find_port(ds, odp_port);
448     if (dsp) {
449         dpif_sflow_del_port__(ds, dsp);
450     }
451     ovs_mutex_unlock(&mutex);
452 }
453
454 void
455 dpif_sflow_set_options(struct dpif_sflow *ds,
456                        const struct ofproto_sflow_options *options)
457     OVS_EXCLUDED(mutex)
458 {
459     struct dpif_sflow_port *dsp;
460     bool options_changed;
461     SFLReceiver *receiver;
462     SFLAddress agentIP;
463     time_t now;
464     SFLDataSource_instance dsi;
465     uint32_t dsIndex;
466     SFLSampler *sampler;
467
468     ovs_mutex_lock(&mutex);
469     if (sset_is_empty(&options->targets) || !options->sampling_rate) {
470         /* No point in doing any work if there are no targets or nothing to
471          * sample. */
472         dpif_sflow_clear__(ds);
473         goto out;
474     }
475
476     options_changed = (!ds->options
477                        || !ofproto_sflow_options_equal(options, ds->options));
478
479     /* Configure collectors if options have changed or if we're shortchanged in
480      * collectors (which indicates that opening one or more of the configured
481      * collectors failed, so that we should retry). */
482     if (options_changed
483         || collectors_count(ds->collectors) < sset_count(&options->targets)) {
484         collectors_destroy(ds->collectors);
485         collectors_create(&options->targets, SFL_DEFAULT_COLLECTOR_PORT,
486                           &ds->collectors);
487         if (ds->collectors == NULL) {
488             VLOG_WARN_RL(&rl, "no collectors could be initialized, "
489                          "sFlow disabled");
490             dpif_sflow_clear__(ds);
491             goto out;
492         }
493     }
494
495     /* Choose agent IP address and agent device (if not yet setup) */
496     if (!sflow_choose_agent_address(options->agent_device,
497                                     &options->targets,
498                                     options->control_ip, &agentIP)) {
499         dpif_sflow_clear__(ds);
500         goto out;
501     }
502
503     /* Avoid reconfiguring if options didn't change. */
504     if (!options_changed) {
505         goto out;
506     }
507     ofproto_sflow_options_destroy(ds->options);
508     ds->options = ofproto_sflow_options_clone(options);
509
510     /* Create agent. */
511     VLOG_INFO("creating sFlow agent %d", options->sub_id);
512     if (ds->sflow_agent) {
513         sfl_agent_release(ds->sflow_agent);
514     }
515     ds->sflow_agent = xcalloc(1, sizeof *ds->sflow_agent);
516     now = time_wall();
517     sfl_agent_init(ds->sflow_agent,
518                    &agentIP,
519                    options->sub_id,
520                    now,         /* Boot time. */
521                    now,         /* Current time. */
522                    ds,          /* Pointer supplied to callbacks. */
523                    sflow_agent_alloc_cb,
524                    sflow_agent_free_cb,
525                    sflow_agent_error_cb,
526                    sflow_agent_send_packet_cb);
527
528     receiver = sfl_agent_addReceiver(ds->sflow_agent);
529     sfl_receiver_set_sFlowRcvrOwner(receiver, "Open vSwitch sFlow");
530     sfl_receiver_set_sFlowRcvrTimeout(receiver, 0xffffffff);
531
532     /* Set the sampling_rate down in the datapath. */
533     ds->probability = MAX(1, UINT32_MAX / ds->options->sampling_rate);
534
535     /* Add a single sampler for the bridge. This appears as a PHYSICAL_ENTITY
536        because it is associated with the hypervisor, and interacts with the server
537        hardware directly.  The sub_id is used to distinguish this sampler from
538        others on other bridges within the same agent. */
539     dsIndex = 1000 + options->sub_id;
540     SFL_DS_SET(dsi, SFL_DSCLASS_PHYSICAL_ENTITY, dsIndex, 0);
541     sampler = sfl_agent_addSampler(ds->sflow_agent, &dsi);
542     sfl_sampler_set_sFlowFsPacketSamplingRate(sampler, ds->options->sampling_rate);
543     sfl_sampler_set_sFlowFsMaximumHeaderSize(sampler, ds->options->header_len);
544     sfl_sampler_set_sFlowFsReceiver(sampler, RECEIVER_INDEX);
545
546     /* Add pollers for the currently known ifindex-ports */
547     HMAP_FOR_EACH (dsp, hmap_node, &ds->ports) {
548         dpif_sflow_add_poller(ds, dsp);
549     }
550
551
552 out:
553     ovs_mutex_unlock(&mutex);
554 }
555
556 int
557 dpif_sflow_odp_port_to_ifindex(const struct dpif_sflow *ds,
558                                odp_port_t odp_port) OVS_EXCLUDED(mutex)
559 {
560     struct dpif_sflow_port *dsp;
561     int ret;
562
563     ovs_mutex_lock(&mutex);
564     dsp = dpif_sflow_find_port(ds, odp_port);
565     ret = dsp ? SFL_DS_INDEX(dsp->dsi) : 0;
566     ovs_mutex_unlock(&mutex);
567     return ret;
568 }
569
570 void
571 dpif_sflow_received(struct dpif_sflow *ds, struct ofpbuf *packet,
572                     const struct flow *flow, odp_port_t odp_in_port,
573                     const union user_action_cookie *cookie)
574     OVS_EXCLUDED(mutex)
575 {
576     SFL_FLOW_SAMPLE_TYPE fs;
577     SFLFlow_sample_element hdrElem;
578     SFLSampled_header *header;
579     SFLFlow_sample_element switchElem;
580     SFLSampler *sampler;
581     struct dpif_sflow_port *in_dsp;
582     ovs_be16 vlan_tci;
583
584     ovs_mutex_lock(&mutex);
585     sampler = ds->sflow_agent->samplers;
586     if (!sampler) {
587         goto out;
588     }
589
590     /* Build a flow sample. */
591     memset(&fs, 0, sizeof fs);
592
593     /* Look up the input ifIndex if this port has one. Otherwise just
594      * leave it as 0 (meaning 'unknown') and continue. */
595     in_dsp = dpif_sflow_find_port(ds, odp_in_port);
596     if (in_dsp) {
597         fs.input = SFL_DS_INDEX(in_dsp->dsi);
598     }
599
600     /* Make the assumption that the random number generator in the datapath converges
601      * to the configured mean, and just increment the samplePool by the configured
602      * sampling rate every time. */
603     sampler->samplePool += sfl_sampler_get_sFlowFsPacketSamplingRate(sampler);
604
605     /* Sampled header. */
606     memset(&hdrElem, 0, sizeof hdrElem);
607     hdrElem.tag = SFLFLOW_HEADER;
608     header = &hdrElem.flowType.header;
609     header->header_protocol = SFLHEADER_ETHERNET_ISO8023;
610     /* The frame_length should include the Ethernet FCS (4 bytes),
611      * but it has already been stripped,  so we need to add 4 here. */
612     header->frame_length = packet->size + 4;
613     /* Ethernet FCS stripped off. */
614     header->stripped = 4;
615     header->header_length = MIN(packet->size,
616                                 sampler->sFlowFsMaximumHeaderSize);
617     header->header_bytes = packet->data;
618
619     /* Add extended switch element. */
620     memset(&switchElem, 0, sizeof(switchElem));
621     switchElem.tag = SFLFLOW_EX_SWITCH;
622     switchElem.flowType.sw.src_vlan = vlan_tci_to_vid(flow->vlan_tci);
623     switchElem.flowType.sw.src_priority = vlan_tci_to_pcp(flow->vlan_tci);
624
625     /* Retrieve data from user_action_cookie. */
626     vlan_tci = cookie->sflow.vlan_tci;
627     switchElem.flowType.sw.dst_vlan = vlan_tci_to_vid(vlan_tci);
628     switchElem.flowType.sw.dst_priority = vlan_tci_to_pcp(vlan_tci);
629
630     fs.output = cookie->sflow.output;
631
632     /* Submit the flow sample to be encoded into the next datagram. */
633     SFLADD_ELEMENT(&fs, &hdrElem);
634     SFLADD_ELEMENT(&fs, &switchElem);
635     sfl_sampler_writeFlowSample(sampler, &fs);
636
637 out:
638     ovs_mutex_unlock(&mutex);
639 }
640
641 void
642 dpif_sflow_run(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
643 {
644     ovs_mutex_lock(&mutex);
645     if (ds->collectors != NULL) {
646         time_t now = time_now();
647         route_table_run();
648         if (now >= ds->next_tick) {
649             sfl_agent_tick(ds->sflow_agent, time_wall());
650             ds->next_tick = now + 1;
651         }
652     }
653     ovs_mutex_unlock(&mutex);
654 }
655
656 void
657 dpif_sflow_wait(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
658 {
659     ovs_mutex_lock(&mutex);
660     if (ds->collectors != NULL) {
661         poll_timer_wait_until(ds->next_tick * 1000LL);
662     }
663     ovs_mutex_unlock(&mutex);
664 }