aboutsummaryrefslogtreecommitdiffstats
path: root/src/vppinfra
diff options
context:
space:
mode:
authorDave Barach <dave@barachs.net>2020-05-19 09:56:22 -0400
committerDave Wallace <dwallacelf@gmail.com>2020-06-11 14:39:53 +0000
commited42469be2f1ab0ea7b69fe0770c9c1e6f267e18 (patch)
tree6180d3bb4fffeac68d9dbdc1cf220ddd08667e46 /src/vppinfra
parent615ba7af1a76b436502b7756ddf827d9d16f2a1d (diff)
vppinfra: refactor mpcap.h
vppinfra source files MUST NOT #include <vlib/vlib.h>, <vnet/vnet.h> or similar. Move mpcap_add_packet(...), mpcap_add_buffer(...) to a new file: src/vnet/mpcap.h. Type: refactor Signed-off-by: Dave Barach <dave@barachs.net> Change-Id: Id517aef6fe49b618f853ce32940b91ba45a1e60d (cherry picked from commit 2a41919e39d4672f76a654f30be9c2093cef4fad)
Diffstat (limited to 'src/vppinfra')
-rw-r--r--src/vppinfra/mpcap.h92
1 files changed, 4 insertions, 88 deletions
diff --git a/src/vppinfra/mpcap.h b/src/vppinfra/mpcap.h
index 1710702d8bd..84f3332d71f 100644
--- a/src/vppinfra/mpcap.h
+++ b/src/vppinfra/mpcap.h
@@ -17,14 +17,14 @@
* @file
* @brief MPCAP utility definitions
*/
-#ifndef included_vnet_mpcap_h
-#define included_vnet_mpcap_h
+#ifndef included_vppinfra_mpcap_h
+#define included_vppinfra_mpcap_h
-#include <vlib/vlib.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <unistd.h>
#include <vppinfra/time_range.h>
+#include <vppinfra/lock.h>
/**
* @brief Packet types supported by MPCAP
@@ -162,91 +162,7 @@ clib_error_t *mpcap_close (mpcap_main_t * pm);
/** mmap a mpcap data file. */
clib_error_t *mpcap_map (mpcap_main_t * pm);
-/**
- * @brief Add packet
- *
- * @param *pm - mpcap_main_t
- * @param time_now - f64
- * @param n_bytes_in_trace - u32
- * @param n_bytes_in_packet - u32
- *
- * @return Packet Data
- *
- */
-static inline void *
-mpcap_add_packet (mpcap_main_t * pm,
- f64 time_now, u32 n_bytes_in_trace, u32 n_bytes_in_packet)
-{
- mpcap_packet_header_t *h;
- u8 *d;
-
- /* File already closed? */
- if (PREDICT_FALSE (pm->flags & MPCAP_FLAG_INIT_DONE) == 0)
- return 0;
-
- d = pm->current_va;
- pm->current_va += sizeof (h[0]) + n_bytes_in_trace;
-
- /* Out of space? */
- if (PREDICT_FALSE (pm->current_va >= pm->file_baseva + pm->max_file_size))
- return 0;
- h = (void *) (d);
- h->time_in_sec = time_now;
- h->time_in_usec = 1e6 * (time_now - h->time_in_sec);
- h->n_packet_bytes_stored_in_file = n_bytes_in_trace;
- h->n_bytes_in_packet = n_bytes_in_packet;
- pm->n_packets_captured++;
- return h->data;
-}
-
-/**
- * @brief Add buffer (vlib_buffer_t) to the trace
- *
- * @param *pm - mpcap_main_t
- * @param *vm - vlib_main_t
- * @param time_now - f64
- * @param buffer_index - u32
- * @param n_bytes_in_trace - u32
- *
- */
-static inline void
-mpcap_add_buffer (mpcap_main_t * pm,
- vlib_main_t * vm,
- f64 time_now, u32 buffer_index, u32 n_bytes_in_trace)
-{
- vlib_buffer_t *b = vlib_get_buffer (vm, buffer_index);
- u32 n = vlib_buffer_length_in_chain (vm, b);
- i32 n_left = clib_min (n_bytes_in_trace, n);
- void *d;
-
- clib_spinlock_lock_if_init (&pm->lock);
-
- d = mpcap_add_packet (pm, time_now, n_left, n);
- if (PREDICT_FALSE (d == 0))
- {
- mpcap_close (pm);
- clib_spinlock_unlock_if_init (&pm->lock);
- return;
- }
-
- while (1)
- {
- u32 copy_length = clib_min ((u32) n_left, b->current_length);
- clib_memcpy (d, b->data + b->current_data, copy_length);
- n_left -= b->current_length;
- if (n_left <= 0)
- break;
- d += b->current_length;
- ASSERT (b->flags & VLIB_BUFFER_NEXT_PRESENT);
- b = vlib_get_buffer (vm, b->next_buffer);
- }
- if (pm->n_packets_captured >= pm->n_packets_to_capture)
- mpcap_close (pm);
-
- clib_spinlock_unlock_if_init (&pm->lock);
-}
-
-#endif /* included_vnet_mpcap_h */
+#endif /* included_vppinfra_mpcap_h */
/*
* fd.io coding-style-patch-verification: ON