summaryrefslogtreecommitdiffstats
path: root/src/vnet/flow/flow_report.c
diff options
context:
space:
mode:
authorSwarup Nayak <swarupnpvt@gmail.com>2017-11-26 23:11:40 +0530
committerDamjan Marion <dmarion.lists@gmail.com>2017-11-26 19:36:13 +0000
commit6bcac065b82ef2e48fc23db18534d9dc9a7833a1 (patch)
tree13aae9f6feb438835afd7e632b25173a1511e8f7 /src/vnet/flow/flow_report.c
parent84517cfd1508f6da24937f310f7fffe752f22584 (diff)
VPP-252 Coding standards cleanup - vnet/vnet/flow
Change-Id: I288b0eeefd22f65e80634068e4b3ce0d83fb50fc Signed-off-by: Swarup Nayak <swarupnpvt@gmail.com>
Diffstat (limited to 'src/vnet/flow/flow_report.c')
-rw-r--r--src/vnet/flow/flow_report.c442
1 files changed, 232 insertions, 210 deletions
diff --git a/src/vnet/flow/flow_report.c b/src/vnet/flow/flow_report.c
index ccc84235695..f912618ec75 100644
--- a/src/vnet/flow/flow_report.c
+++ b/src/vnet/flow/flow_report.c
@@ -20,77 +20,84 @@
flow_report_main_t flow_report_main;
-static_always_inline u8 stream_index_valid (u32 index)
+static_always_inline u8
+stream_index_valid (u32 index)
{
- flow_report_main_t * frm = &flow_report_main;
- return index < vec_len(frm->streams) &&
- frm->streams[index].domain_id != ~0;
+ flow_report_main_t *frm = &flow_report_main;
+ return index < vec_len (frm->streams) &&
+ frm->streams[index].domain_id != ~0;
}
-static_always_inline flow_report_stream_t * add_stream (void)
+static_always_inline flow_report_stream_t *
+add_stream (void)
{
- flow_report_main_t * frm = &flow_report_main;
+ flow_report_main_t *frm = &flow_report_main;
u32 i;
- for (i = 0; i < vec_len(frm->streams); i++)
- if (!stream_index_valid(i))
+ for (i = 0; i < vec_len (frm->streams); i++)
+ if (!stream_index_valid (i))
return &frm->streams[i];
- u32 index = vec_len(frm->streams);
- vec_validate(frm->streams, index);
+ u32 index = vec_len (frm->streams);
+ vec_validate (frm->streams, index);
return &frm->streams[index];
}
-static_always_inline void delete_stream (u32 index)
+static_always_inline void
+delete_stream (u32 index)
{
- flow_report_main_t * frm = &flow_report_main;
- ASSERT (index < vec_len(frm->streams));
+ flow_report_main_t *frm = &flow_report_main;
+ ASSERT (index < vec_len (frm->streams));
ASSERT (frm->streams[index].domain_id != ~0);
frm->streams[index].domain_id = ~0;
}
-static i32 find_stream (u32 domain_id, u16 src_port)
+static i32
+find_stream (u32 domain_id, u16 src_port)
{
- flow_report_main_t * frm = &flow_report_main;
- flow_report_stream_t * stream;
+ flow_report_main_t *frm = &flow_report_main;
+ flow_report_stream_t *stream;
u32 i;
- for (i = 0; i < vec_len(frm->streams); i++)
- if (stream_index_valid(i)) {
- stream = &frm->streams[i];
- if (domain_id == stream->domain_id) {
- if (src_port != stream->src_port)
- return -2;
- return i;
- } else if (src_port == stream->src_port) {
- return -2;
+ for (i = 0; i < vec_len (frm->streams); i++)
+ if (stream_index_valid (i))
+ {
+ stream = &frm->streams[i];
+ if (domain_id == stream->domain_id)
+ {
+ if (src_port != stream->src_port)
+ return -2;
+ return i;
+ }
+ else if (src_port == stream->src_port)
+ {
+ return -2;
+ }
}
- }
return -1;
}
-int send_template_packet (flow_report_main_t *frm,
- flow_report_t *fr,
- u32 * buffer_indexp)
+int
+send_template_packet (flow_report_main_t * frm,
+ flow_report_t * fr, u32 * buffer_indexp)
{
u32 bi0;
- vlib_buffer_t * b0;
- ip4_ipfix_template_packet_t * tp;
- ipfix_message_header_t * h;
- ip4_header_t * ip;
- udp_header_t * udp;
- vlib_main_t * vm = frm->vlib_main;
- flow_report_stream_t * stream;
+ vlib_buffer_t *b0;
+ ip4_ipfix_template_packet_t *tp;
+ ipfix_message_header_t *h;
+ ip4_header_t *ip;
+ udp_header_t *udp;
+ vlib_main_t *vm = frm->vlib_main;
+ flow_report_stream_t *stream;
vlib_buffer_free_list_t *fl;
ASSERT (buffer_indexp);
if (fr->update_rewrite || fr->rewrite == 0)
{
- if (frm->ipfix_collector.as_u32 == 0
- || frm->src_address.as_u32 == 0)
- {
- vlib_node_set_state (frm->vlib_main, flow_report_process_node.index,
- VLIB_NODE_STATE_DISABLED);
- return -1;
- }
+ if (frm->ipfix_collector.as_u32 == 0 || frm->src_address.as_u32 == 0)
+ {
+ vlib_node_set_state (frm->vlib_main, flow_report_process_node.index,
+ VLIB_NODE_STATE_DISABLED);
+ return -1;
+ }
vec_free (fr->rewrite);
fr->update_rewrite = 1;
}
@@ -98,15 +105,15 @@ int send_template_packet (flow_report_main_t *frm,
if (fr->update_rewrite)
{
fr->rewrite = fr->rewrite_callback (frm, fr,
- &frm->ipfix_collector,
- &frm->src_address,
- frm->collector_port);
+ &frm->ipfix_collector,
+ &frm->src_address,
+ frm->collector_port);
fr->update_rewrite = 0;
}
if (vlib_buffer_alloc (vm, &bi0, 1) != 1)
return -1;
-
+
b0 = vlib_get_buffer (vm, bi0);
/* Initialize the buffer */
@@ -115,7 +122,7 @@ int send_template_packet (flow_report_main_t *frm,
VLIB_BUFFER_TRACE_TRAJECTORY_INIT (b0);
ASSERT (vec_len (fr->rewrite) < VLIB_BUFFER_DEFAULT_FREE_LIST_BYTES);
-
+
clib_memcpy (b0->data, fr->rewrite, vec_len (fr->rewrite));
b0->current_data = 0;
b0->current_length = vec_len (fr->rewrite);
@@ -124,20 +131,20 @@ int send_template_packet (flow_report_main_t *frm,
vnet_buffer (b0)->sw_if_index[VLIB_TX] = frm->fib_index;
tp = vlib_buffer_get_current (b0);
- ip = (ip4_header_t *) &tp->ip4;
- udp = (udp_header_t *) (ip+1);
- h = (ipfix_message_header_t *)(udp+1);
+ ip = (ip4_header_t *) & tp->ip4;
+ udp = (udp_header_t *) (ip + 1);
+ h = (ipfix_message_header_t *) (udp + 1);
- /* FIXUP: message header export_time */
- h->export_time = (u32)
- (((f64)frm->unix_time_0) +
- (vlib_time_now(frm->vlib_main) - frm->vlib_time_0));
- h->export_time = clib_host_to_net_u32(h->export_time);
+ /* FIXUP: message header export_time */
+ h->export_time = (u32)
+ (((f64) frm->unix_time_0) +
+ (vlib_time_now (frm->vlib_main) - frm->vlib_time_0));
+ h->export_time = clib_host_to_net_u32 (h->export_time);
stream = &frm->streams[fr->stream_index];
/* FIXUP: message header sequence_number. Templates do not increase it */
- h->sequence_number = clib_host_to_net_u32(stream->sequence_number);
+ h->sequence_number = clib_host_to_net_u32 (stream->sequence_number);
/* FIXUP: udp length */
udp->length = clib_host_to_net_u16 (b0->current_length - sizeof (*ip));
@@ -147,7 +154,7 @@ int send_template_packet (flow_report_main_t *frm,
/* RFC 7011 section 10.3.2. */
udp->checksum = ip4_tcp_udp_compute_checksum (vm, b0, ip);
if (udp->checksum == 0)
- udp->checksum = 0xffff;
+ udp->checksum = 0xffff;
}
*buffer_indexp = bi0;
@@ -159,16 +166,15 @@ int send_template_packet (flow_report_main_t *frm,
static uword
flow_report_process (vlib_main_t * vm,
- vlib_node_runtime_t * rt,
- vlib_frame_t * f)
+ vlib_node_runtime_t * rt, vlib_frame_t * f)
{
- flow_report_main_t * frm = &flow_report_main;
- flow_report_t * fr;
+ flow_report_main_t *frm = &flow_report_main;
+ flow_report_t *fr;
u32 ip4_lookup_node_index;
- vlib_node_t * ip4_lookup_node;
- vlib_frame_t * nf = 0;
+ vlib_node_t *ip4_lookup_node;
+ vlib_frame_t *nf = 0;
u32 template_bi;
- u32 * to_next;
+ u32 *to_next;
int send_template;
f64 now;
int rv;
@@ -179,104 +185,107 @@ flow_report_process (vlib_main_t * vm,
vlib_process_wait_for_event_or_clock (vm, 1e9);
event_type = vlib_process_get_events (vm, &event_data);
if (event_type != 1)
- clib_warning ("bogus kickoff event received, %d", event_type);
+ clib_warning ("bogus kickoff event received, %d", event_type);
vec_reset_length (event_data);
/* Enqueue pkts to ip4-lookup */
ip4_lookup_node = vlib_get_node_by_name (vm, (u8 *) "ip4-lookup");
ip4_lookup_node_index = ip4_lookup_node->index;
- while (1)
+ while (1)
{
vlib_process_wait_for_event_or_clock (vm, 5.0);
event_type = vlib_process_get_events (vm, &event_data);
vec_reset_length (event_data);
-
+
vec_foreach (fr, frm->reports)
- {
- now = vlib_time_now (vm);
-
- /* Need to send a template packet? */
- send_template =
- now > (fr->last_template_sent + frm->template_interval);
- send_template += fr->last_template_sent == 0;
- template_bi = ~0;
- rv = 0;
-
- if (send_template)
- rv = send_template_packet (frm, fr, &template_bi);
-
- if (rv < 0)
- continue;
-
- nf = vlib_get_frame_to_node (vm, ip4_lookup_node_index);
- nf->n_vectors = 0;
- to_next = vlib_frame_vector_args (nf);
-
- if (template_bi != ~0)
- {
- to_next[0] = template_bi;
- to_next++;
- nf->n_vectors++;
- }
-
- nf = fr->flow_data_callback (frm, fr,
- nf, to_next, ip4_lookup_node_index);
- if (nf)
- vlib_put_frame_to_node (vm, ip4_lookup_node_index, nf);
- }
+ {
+ now = vlib_time_now (vm);
+
+ /* Need to send a template packet? */
+ send_template =
+ now > (fr->last_template_sent + frm->template_interval);
+ send_template += fr->last_template_sent == 0;
+ template_bi = ~0;
+ rv = 0;
+
+ if (send_template)
+ rv = send_template_packet (frm, fr, &template_bi);
+
+ if (rv < 0)
+ continue;
+
+ nf = vlib_get_frame_to_node (vm, ip4_lookup_node_index);
+ nf->n_vectors = 0;
+ to_next = vlib_frame_vector_args (nf);
+
+ if (template_bi != ~0)
+ {
+ to_next[0] = template_bi;
+ to_next++;
+ nf->n_vectors++;
+ }
+
+ nf = fr->flow_data_callback (frm, fr,
+ nf, to_next, ip4_lookup_node_index);
+ if (nf)
+ vlib_put_frame_to_node (vm, ip4_lookup_node_index, nf);
+ }
}
- return 0; /* not so much */
+ return 0; /* not so much */
}
+/* *INDENT-OFF* */
VLIB_REGISTER_NODE (flow_report_process_node) = {
.function = flow_report_process,
.type = VLIB_NODE_TYPE_PROCESS,
.name = "flow-report-process",
};
+/* *INDENT-ON* */
-int vnet_flow_report_add_del (flow_report_main_t *frm,
- vnet_flow_report_add_del_args_t *a,
- u16 *template_id)
+int
+vnet_flow_report_add_del (flow_report_main_t * frm,
+ vnet_flow_report_add_del_args_t * a,
+ u16 * template_id)
{
int i;
int found_index = ~0;
flow_report_t *fr;
- flow_report_stream_t * stream;
+ flow_report_stream_t *stream;
u32 si;
-
- si = find_stream(a->domain_id, a->src_port);
+
+ si = find_stream (a->domain_id, a->src_port);
if (si == -2)
return VNET_API_ERROR_INVALID_VALUE;
if (si == -1 && a->is_add == 0)
return VNET_API_ERROR_NO_SUCH_ENTRY;
- for (i = 0; i < vec_len(frm->reports); i++)
+ for (i = 0; i < vec_len (frm->reports); i++)
{
fr = vec_elt_at_index (frm->reports, i);
if (fr->opaque.as_uword == a->opaque.as_uword
- && fr->rewrite_callback == a->rewrite_callback
- && fr->flow_data_callback == a->flow_data_callback)
- {
- found_index = i;
- if (template_id)
- *template_id = fr->template_id;
- break;
- }
+ && fr->rewrite_callback == a->rewrite_callback
+ && fr->flow_data_callback == a->flow_data_callback)
+ {
+ found_index = i;
+ if (template_id)
+ *template_id = fr->template_id;
+ break;
+ }
}
if (a->is_add == 0)
{
if (found_index != ~0)
- {
- vec_delete (frm->reports, 1, found_index);
- stream = &frm->streams[si];
- stream->n_reports--;
- if (stream->n_reports == 0)
- delete_stream(si);
- return 0;
- }
+ {
+ vec_delete (frm->reports, 1, found_index);
+ stream = &frm->streams[si];
+ stream->n_reports--;
+ if (stream->n_reports == 0)
+ delete_stream (si);
+ return 0;
+ }
return VNET_API_ERROR_NO_SUCH_ENTRY;
}
@@ -285,7 +294,7 @@ int vnet_flow_report_add_del (flow_report_main_t *frm,
if (si == -1)
{
- stream = add_stream();
+ stream = add_stream ();
stream->domain_id = a->domain_id;
stream->src_port = a->src_port;
stream->sequence_number = 0;
@@ -313,63 +322,68 @@ int vnet_flow_report_add_del (flow_report_main_t *frm,
return 0;
}
-clib_error_t * flow_report_add_del_error_to_clib_error (int error)
+clib_error_t *
+flow_report_add_del_error_to_clib_error (int error)
{
- switch (error)
- {
- case 0:
- return 0;
- case VNET_API_ERROR_NO_SUCH_ENTRY:
- return clib_error_return (0, "Flow report not found");
- case VNET_API_ERROR_VALUE_EXIST:
- return clib_error_return (0, "Flow report already exists");
- case VNET_API_ERROR_INVALID_VALUE:
- return clib_error_return (0, "Expecting either still unused values "
- "for both domain_id and src_port "
- "or already used values for both fields");
- default:
- return clib_error_return (0, "vnet_flow_report_add_del returned %d",
- error);
- }
+ switch (error)
+ {
+ case 0:
+ return 0;
+ case VNET_API_ERROR_NO_SUCH_ENTRY:
+ return clib_error_return (0, "Flow report not found");
+ case VNET_API_ERROR_VALUE_EXIST:
+ return clib_error_return (0, "Flow report already exists");
+ case VNET_API_ERROR_INVALID_VALUE:
+ return clib_error_return (0, "Expecting either still unused values "
+ "for both domain_id and src_port "
+ "or already used values for both fields");
+ default:
+ return clib_error_return (0, "vnet_flow_report_add_del returned %d",
+ error);
+ }
}
-void vnet_flow_reports_reset (flow_report_main_t * frm)
+void
+vnet_flow_reports_reset (flow_report_main_t * frm)
{
flow_report_t *fr;
u32 i;
- for (i = 0; i < vec_len(frm->streams); i++)
- if (stream_index_valid(i))
+ for (i = 0; i < vec_len (frm->streams); i++)
+ if (stream_index_valid (i))
frm->streams[i].sequence_number = 0;
vec_foreach (fr, frm->reports)
- {
- fr->update_rewrite = 1;
- fr->last_template_sent = 0;
- }
+ {
+ fr->update_rewrite = 1;
+ fr->last_template_sent = 0;
+ }
}
-void vnet_stream_reset (flow_report_main_t * frm, u32 stream_index)
+void
+vnet_stream_reset (flow_report_main_t * frm, u32 stream_index)
{
flow_report_t *fr;
frm->streams[stream_index].sequence_number = 0;
vec_foreach (fr, frm->reports)
- if (frm->reports->stream_index == stream_index) {
+ if (frm->reports->stream_index == stream_index)
+ {
fr->update_rewrite = 1;
fr->last_template_sent = 0;
}
}
-int vnet_stream_change (flow_report_main_t * frm,
- u32 old_domain_id, u16 old_src_port,
- u32 new_domain_id, u16 new_src_port)
+int
+vnet_stream_change (flow_report_main_t * frm,
+ u32 old_domain_id, u16 old_src_port,
+ u32 new_domain_id, u16 new_src_port)
{
i32 stream_index = find_stream (old_domain_id, old_src_port);
if (stream_index < 0)
return 1;
- flow_report_stream_t * stream = &frm->streams[stream_index];
+ flow_report_stream_t *stream = &frm->streams[stream_index];
stream->domain_id = new_domain_id;
stream->src_port = new_src_port;
if (old_domain_id != new_domain_id || old_src_port != new_src_port)
@@ -379,61 +393,61 @@ int vnet_stream_change (flow_report_main_t * frm,
static clib_error_t *
set_ipfix_exporter_command_fn (vlib_main_t * vm,
- unformat_input_t * input,
- vlib_cli_command_t * cmd)
+ unformat_input_t * input,
+ vlib_cli_command_t * cmd)
{
- flow_report_main_t * frm = &flow_report_main;
+ flow_report_main_t *frm = &flow_report_main;
ip4_address_t collector, src;
u16 collector_port = UDP_DST_PORT_ipfix;
u32 fib_id;
u32 fib_index = ~0;
-
+
collector.as_u32 = 0;
src.as_u32 = 0;
- u32 path_mtu = 512; // RFC 7011 section 10.3.3.
+ u32 path_mtu = 512; // RFC 7011 section 10.3.3.
u32 template_interval = 20;
u8 udp_checksum = 0;
- while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT) {
- if (unformat (input, "collector %U", unformat_ip4_address, &collector))
- ;
- else if (unformat (input, "port %u", &collector_port))
- ;
- else if (unformat (input, "src %U", unformat_ip4_address, &src))
- ;
- else if (unformat (input, "fib-id %u", &fib_id))
- {
- ip4_main_t * im = &ip4_main;
- uword * p = hash_get (im->fib_index_by_table_id, fib_id);
- if (! p)
- return clib_error_return (0, "fib ID %d doesn't exist\n",
- fib_id);
- fib_index = p[0];
- }
- else if (unformat (input, "path-mtu %u", &path_mtu))
- ;
- else if (unformat (input, "template-interval %u", &template_interval))
- ;
- else if (unformat (input, "udp-checksum"))
- udp_checksum = 1;
- else
- break;
- }
-
+ while (unformat_check_input (input) != UNFORMAT_END_OF_INPUT)
+ {
+ if (unformat (input, "collector %U", unformat_ip4_address, &collector))
+ ;
+ else if (unformat (input, "port %u", &collector_port))
+ ;
+ else if (unformat (input, "src %U", unformat_ip4_address, &src))
+ ;
+ else if (unformat (input, "fib-id %u", &fib_id))
+ {
+ ip4_main_t *im = &ip4_main;
+ uword *p = hash_get (im->fib_index_by_table_id, fib_id);
+ if (!p)
+ return clib_error_return (0, "fib ID %d doesn't exist\n", fib_id);
+ fib_index = p[0];
+ }
+ else if (unformat (input, "path-mtu %u", &path_mtu))
+ ;
+ else if (unformat (input, "template-interval %u", &template_interval))
+ ;
+ else if (unformat (input, "udp-checksum"))
+ udp_checksum = 1;
+ else
+ break;
+ }
+
if (collector.as_u32 != 0 && src.as_u32 == 0)
return clib_error_return (0, "src address required");
- if (path_mtu > 1450 /* vpp does not support fragmentation */)
- return clib_error_return (0, "too big path-mtu value, maximum is 1450");
+ if (path_mtu > 1450 /* vpp does not support fragmentation */ )
+ return clib_error_return (0, "too big path-mtu value, maximum is 1450");
if (path_mtu < 68)
- return clib_error_return (0, "too small path-mtu value, minimum is 68");
+ return clib_error_return (0, "too small path-mtu value, minimum is 68");
/* Reset report streams if we are reconfiguring IP addresses */
if (frm->ipfix_collector.as_u32 != collector.as_u32 ||
frm->src_address.as_u32 != src.as_u32 ||
frm->collector_port != collector_port)
- vnet_flow_reports_reset(frm);
+ vnet_flow_reports_reset (frm);
frm->ipfix_collector.as_u32 = collector.as_u32;
frm->collector_port = collector_port;
@@ -442,25 +456,25 @@ set_ipfix_exporter_command_fn (vlib_main_t * vm,
frm->path_mtu = path_mtu;
frm->template_interval = template_interval;
frm->udp_checksum = udp_checksum;
-
+
if (collector.as_u32)
vlib_cli_output (vm, "Collector %U, src address %U, "
- "fib index %d, path MTU %u, "
- "template resend interval %us, "
- "udp checksum %s",
- format_ip4_address, &frm->ipfix_collector,
- format_ip4_address, &frm->src_address,
- fib_index, path_mtu, template_interval,
- udp_checksum ? "enabled" : "disabled");
+ "fib index %d, path MTU %u, "
+ "template resend interval %us, "
+ "udp checksum %s",
+ format_ip4_address, &frm->ipfix_collector,
+ format_ip4_address, &frm->src_address,
+ fib_index, path_mtu, template_interval,
+ udp_checksum ? "enabled" : "disabled");
else
vlib_cli_output (vm, "IPFIX Collector is disabled");
/* Turn on the flow reporting process */
- vlib_process_signal_event (vm, flow_report_process_node.index,
- 1, 0);
+ vlib_process_signal_event (vm, flow_report_process_node.index, 1, 0);
return 0;
}
+/* *INDENT-OFF* */
VLIB_CLI_COMMAND (set_ipfix_exporter_command, static) = {
.path = "set ipfix exporter",
.short_help = "set ipfix exporter "
@@ -471,37 +485,45 @@ VLIB_CLI_COMMAND (set_ipfix_exporter_command, static) = {
"[udp-checksum]",
.function = set_ipfix_exporter_command_fn,
};
+/* *INDENT-ON* */
static clib_error_t *
ipfix_flush_command_fn (vlib_main_t * vm,
- unformat_input_t * input,
- vlib_cli_command_t * cmd)
+ unformat_input_t * input, vlib_cli_command_t * cmd)
{
/* poke the flow reporting process */
- vlib_process_signal_event (vm, flow_report_process_node.index,
- 1, 0);
+ vlib_process_signal_event (vm, flow_report_process_node.index, 1, 0);
return 0;
}
+/* *INDENT-OFF* */
VLIB_CLI_COMMAND (ipfix_flush_command, static) = {
.path = "ipfix flush",
.short_help = "flush the current ipfix data [for make test]",
.function = ipfix_flush_command_fn,
};
+/* *INDENT-ON* */
-static clib_error_t *
-flow_report_init (vlib_main_t *vm)
+static clib_error_t *
+flow_report_init (vlib_main_t * vm)
{
- flow_report_main_t * frm = &flow_report_main;
+ flow_report_main_t *frm = &flow_report_main;
frm->vlib_main = vm;
- frm->vnet_main = vnet_get_main();
- frm->unix_time_0 = time(0);
- frm->vlib_time_0 = vlib_time_now(frm->vlib_main);
+ frm->vnet_main = vnet_get_main ();
+ frm->unix_time_0 = time (0);
+ frm->vlib_time_0 = vlib_time_now (frm->vlib_main);
frm->fib_index = ~0;
return 0;
}
VLIB_INIT_FUNCTION (flow_report_init)
+/*
+ * fd.io coding-style-patch-verification: ON
+ *
+ * Local Variables:
+ * eval: (c-set-style "gnu")
+ * End:
+ */