e9ef434dd11f89c3316d9ec3a653ca80fada8a95
[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     struct ovs_refcount 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     ovs_refcount_init(&ds->ref_cnt);
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         ovs_refcount_ref(&ds->ref_cnt);
341     }
342     return ds;
343 }
344
345 /* 32-bit fraction of packets to sample with.  A value of 0 samples no packets,
346  * a value of %UINT32_MAX samples all packets and intermediate values sample
347  * intermediate fractions of packets. */
348 uint32_t
349 dpif_sflow_get_probability(const struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
350 {
351     uint32_t probability;
352     ovs_mutex_lock(&mutex);
353     probability = ds->probability;
354     ovs_mutex_unlock(&mutex);
355     return probability;
356 }
357
358 void
359 dpif_sflow_unref(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
360 {
361     if (ds && ovs_refcount_unref(&ds->ref_cnt) == 1) {
362         struct dpif_sflow_port *dsp, *next;
363
364         route_table_unregister();
365         dpif_sflow_clear(ds);
366         HMAP_FOR_EACH_SAFE (dsp, next, hmap_node, &ds->ports) {
367             dpif_sflow_del_port__(ds, dsp);
368         }
369         hmap_destroy(&ds->ports);
370         ovs_refcount_destroy(&ds->ref_cnt);
371         free(ds);
372     }
373 }
374
375 static void
376 dpif_sflow_add_poller(struct dpif_sflow *ds, struct dpif_sflow_port *dsp)
377     OVS_REQUIRES(mutex)
378 {
379     SFLPoller *poller = sfl_agent_addPoller(ds->sflow_agent, &dsp->dsi, ds,
380                                             sflow_agent_get_counters);
381     sfl_poller_set_sFlowCpInterval(poller, ds->options->polling_interval);
382     sfl_poller_set_sFlowCpReceiver(poller, RECEIVER_INDEX);
383     sfl_poller_set_bridgePort(poller, odp_to_u32(dsp->odp_port));
384 }
385
386 void
387 dpif_sflow_add_port(struct dpif_sflow *ds, struct ofport *ofport,
388                     odp_port_t odp_port) OVS_EXCLUDED(mutex)
389 {
390     struct dpif_sflow_port *dsp;
391     int ifindex;
392
393     ovs_mutex_lock(&mutex);
394     dpif_sflow_del_port(ds, odp_port);
395
396     ifindex = netdev_get_ifindex(ofport->netdev);
397
398     if (ifindex <= 0) {
399         /* Not an ifindex port, so do not add a cross-reference to it here */
400         goto out;
401     }
402
403     /* Add to table of ports. */
404     dsp = xmalloc(sizeof *dsp);
405     dsp->ofport = ofport;
406     dsp->odp_port = odp_port;
407     SFL_DS_SET(dsp->dsi, SFL_DSCLASS_IFINDEX, ifindex, 0);
408     hmap_insert(&ds->ports, &dsp->hmap_node, hash_odp_port(odp_port));
409
410     /* Add poller. */
411     if (ds->sflow_agent) {
412         dpif_sflow_add_poller(ds, dsp);
413     }
414
415 out:
416     ovs_mutex_unlock(&mutex);
417 }
418
419 static void
420 dpif_sflow_del_port__(struct dpif_sflow *ds, struct dpif_sflow_port *dsp)
421     OVS_REQUIRES(mutex)
422 {
423     if (ds->sflow_agent) {
424         sfl_agent_removePoller(ds->sflow_agent, &dsp->dsi);
425         sfl_agent_removeSampler(ds->sflow_agent, &dsp->dsi);
426     }
427     hmap_remove(&ds->ports, &dsp->hmap_node);
428     free(dsp);
429 }
430
431 void
432 dpif_sflow_del_port(struct dpif_sflow *ds, odp_port_t odp_port)
433     OVS_EXCLUDED(mutex)
434 {
435     struct dpif_sflow_port *dsp;
436
437     ovs_mutex_lock(&mutex);
438     dsp = dpif_sflow_find_port(ds, odp_port);
439     if (dsp) {
440         dpif_sflow_del_port__(ds, dsp);
441     }
442     ovs_mutex_unlock(&mutex);
443 }
444
445 void
446 dpif_sflow_set_options(struct dpif_sflow *ds,
447                        const struct ofproto_sflow_options *options)
448     OVS_EXCLUDED(mutex)
449 {
450     struct dpif_sflow_port *dsp;
451     bool options_changed;
452     SFLReceiver *receiver;
453     SFLAddress agentIP;
454     time_t now;
455     SFLDataSource_instance dsi;
456     uint32_t dsIndex;
457     SFLSampler *sampler;
458
459     ovs_mutex_lock(&mutex);
460     if (sset_is_empty(&options->targets) || !options->sampling_rate) {
461         /* No point in doing any work if there are no targets or nothing to
462          * sample. */
463         dpif_sflow_clear__(ds);
464         goto out;
465     }
466
467     options_changed = (!ds->options
468                        || !ofproto_sflow_options_equal(options, ds->options));
469
470     /* Configure collectors if options have changed or if we're shortchanged in
471      * collectors (which indicates that opening one or more of the configured
472      * collectors failed, so that we should retry). */
473     if (options_changed
474         || collectors_count(ds->collectors) < sset_count(&options->targets)) {
475         collectors_destroy(ds->collectors);
476         collectors_create(&options->targets, SFL_DEFAULT_COLLECTOR_PORT,
477                           &ds->collectors);
478         if (ds->collectors == NULL) {
479             VLOG_WARN_RL(&rl, "no collectors could be initialized, "
480                          "sFlow disabled");
481             dpif_sflow_clear__(ds);
482             goto out;
483         }
484     }
485
486     /* Choose agent IP address and agent device (if not yet setup) */
487     if (!sflow_choose_agent_address(options->agent_device,
488                                     &options->targets,
489                                     options->control_ip, &agentIP)) {
490         dpif_sflow_clear__(ds);
491         goto out;
492     }
493
494     /* Avoid reconfiguring if options didn't change. */
495     if (!options_changed) {
496         goto out;
497     }
498     ofproto_sflow_options_destroy(ds->options);
499     ds->options = ofproto_sflow_options_clone(options);
500
501     /* Create agent. */
502     VLOG_INFO("creating sFlow agent %d", options->sub_id);
503     if (ds->sflow_agent) {
504         sfl_agent_release(ds->sflow_agent);
505     }
506     ds->sflow_agent = xcalloc(1, sizeof *ds->sflow_agent);
507     now = time_wall();
508     sfl_agent_init(ds->sflow_agent,
509                    &agentIP,
510                    options->sub_id,
511                    now,         /* Boot time. */
512                    now,         /* Current time. */
513                    ds,          /* Pointer supplied to callbacks. */
514                    sflow_agent_alloc_cb,
515                    sflow_agent_free_cb,
516                    sflow_agent_error_cb,
517                    sflow_agent_send_packet_cb);
518
519     receiver = sfl_agent_addReceiver(ds->sflow_agent);
520     sfl_receiver_set_sFlowRcvrOwner(receiver, "Open vSwitch sFlow");
521     sfl_receiver_set_sFlowRcvrTimeout(receiver, 0xffffffff);
522
523     /* Set the sampling_rate down in the datapath. */
524     ds->probability = MAX(1, UINT32_MAX / ds->options->sampling_rate);
525
526     /* Add a single sampler for the bridge. This appears as a PHYSICAL_ENTITY
527        because it is associated with the hypervisor, and interacts with the server
528        hardware directly.  The sub_id is used to distinguish this sampler from
529        others on other bridges within the same agent. */
530     dsIndex = 1000 + options->sub_id;
531     SFL_DS_SET(dsi, SFL_DSCLASS_PHYSICAL_ENTITY, dsIndex, 0);
532     sampler = sfl_agent_addSampler(ds->sflow_agent, &dsi);
533     sfl_sampler_set_sFlowFsPacketSamplingRate(sampler, ds->options->sampling_rate);
534     sfl_sampler_set_sFlowFsMaximumHeaderSize(sampler, ds->options->header_len);
535     sfl_sampler_set_sFlowFsReceiver(sampler, RECEIVER_INDEX);
536
537     /* Add pollers for the currently known ifindex-ports */
538     HMAP_FOR_EACH (dsp, hmap_node, &ds->ports) {
539         dpif_sflow_add_poller(ds, dsp);
540     }
541
542
543 out:
544     ovs_mutex_unlock(&mutex);
545 }
546
547 int
548 dpif_sflow_odp_port_to_ifindex(const struct dpif_sflow *ds,
549                                odp_port_t odp_port) OVS_EXCLUDED(mutex)
550 {
551     struct dpif_sflow_port *dsp;
552     int ret;
553
554     ovs_mutex_lock(&mutex);
555     dsp = dpif_sflow_find_port(ds, odp_port);
556     ret = dsp ? SFL_DS_INDEX(dsp->dsi) : 0;
557     ovs_mutex_unlock(&mutex);
558     return ret;
559 }
560
561 void
562 dpif_sflow_received(struct dpif_sflow *ds, struct ofpbuf *packet,
563                     const struct flow *flow, odp_port_t odp_in_port,
564                     const union user_action_cookie *cookie)
565     OVS_EXCLUDED(mutex)
566 {
567     SFL_FLOW_SAMPLE_TYPE fs;
568     SFLFlow_sample_element hdrElem;
569     SFLSampled_header *header;
570     SFLFlow_sample_element switchElem;
571     SFLSampler *sampler;
572     struct dpif_sflow_port *in_dsp;
573     ovs_be16 vlan_tci;
574
575     ovs_mutex_lock(&mutex);
576     sampler = ds->sflow_agent->samplers;
577     if (!sampler) {
578         goto out;
579     }
580
581     /* Build a flow sample. */
582     memset(&fs, 0, sizeof fs);
583
584     /* Look up the input ifIndex if this port has one. Otherwise just
585      * leave it as 0 (meaning 'unknown') and continue. */
586     in_dsp = dpif_sflow_find_port(ds, odp_in_port);
587     if (in_dsp) {
588         fs.input = SFL_DS_INDEX(in_dsp->dsi);
589     }
590
591     /* Make the assumption that the random number generator in the datapath converges
592      * to the configured mean, and just increment the samplePool by the configured
593      * sampling rate every time. */
594     sampler->samplePool += sfl_sampler_get_sFlowFsPacketSamplingRate(sampler);
595
596     /* Sampled header. */
597     memset(&hdrElem, 0, sizeof hdrElem);
598     hdrElem.tag = SFLFLOW_HEADER;
599     header = &hdrElem.flowType.header;
600     header->header_protocol = SFLHEADER_ETHERNET_ISO8023;
601     /* The frame_length should include the Ethernet FCS (4 bytes),
602      * but it has already been stripped,  so we need to add 4 here. */
603     header->frame_length = packet->size + 4;
604     /* Ethernet FCS stripped off. */
605     header->stripped = 4;
606     header->header_length = MIN(packet->size,
607                                 sampler->sFlowFsMaximumHeaderSize);
608     header->header_bytes = packet->data;
609
610     /* Add extended switch element. */
611     memset(&switchElem, 0, sizeof(switchElem));
612     switchElem.tag = SFLFLOW_EX_SWITCH;
613     switchElem.flowType.sw.src_vlan = vlan_tci_to_vid(flow->vlan_tci);
614     switchElem.flowType.sw.src_priority = vlan_tci_to_pcp(flow->vlan_tci);
615
616     /* Retrieve data from user_action_cookie. */
617     vlan_tci = cookie->sflow.vlan_tci;
618     switchElem.flowType.sw.dst_vlan = vlan_tci_to_vid(vlan_tci);
619     switchElem.flowType.sw.dst_priority = vlan_tci_to_pcp(vlan_tci);
620
621     fs.output = cookie->sflow.output;
622
623     /* Submit the flow sample to be encoded into the next datagram. */
624     SFLADD_ELEMENT(&fs, &hdrElem);
625     SFLADD_ELEMENT(&fs, &switchElem);
626     sfl_sampler_writeFlowSample(sampler, &fs);
627
628 out:
629     ovs_mutex_unlock(&mutex);
630 }
631
632 void
633 dpif_sflow_run(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
634 {
635     ovs_mutex_lock(&mutex);
636     if (ds->collectors != NULL) {
637         time_t now = time_now();
638         route_table_run();
639         if (now >= ds->next_tick) {
640             sfl_agent_tick(ds->sflow_agent, time_wall());
641             ds->next_tick = now + 1;
642         }
643     }
644     ovs_mutex_unlock(&mutex);
645 }
646
647 void
648 dpif_sflow_wait(struct dpif_sflow *ds) OVS_EXCLUDED(mutex)
649 {
650     ovs_mutex_lock(&mutex);
651     if (ds->collectors != NULL) {
652         poll_timer_wait_until(ds->next_tick * 1000LL);
653     }
654     ovs_mutex_unlock(&mutex);
655 }