X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=vlib%2Fvlib%2Fbuffer.c;h=84cecc27a81c1d29644de682a145186b7d767354;hb=9e6ed6e2e1b8af46a317f9448da2bca8d8c5395d;hp=f727e67a55a8d1534ac385a8376b219a02f4bc12;hpb=b5adaeab2fe89844db678a7a2d61d45b5d4cc4c8;p=vpp.git diff --git a/vlib/vlib/buffer.c b/vlib/vlib/buffer.c index f727e67a55a..84cecc27a81 100644 --- a/vlib/vlib/buffer.c +++ b/vlib/vlib/buffer.c @@ -39,9 +39,11 @@ #include -uword vlib_buffer_length_in_chain_slow_path (vlib_main_t * vm, vlib_buffer_t * b_first) +uword +vlib_buffer_length_in_chain_slow_path (vlib_main_t * vm, + vlib_buffer_t * b_first) { - vlib_buffer_t * b = b_first; + vlib_buffer_t *b = b_first; uword l_first = b_first->current_length; uword l = 0; while (b->flags & VLIB_BUFFER_NEXT_PRESENT) @@ -54,13 +56,13 @@ uword vlib_buffer_length_in_chain_slow_path (vlib_main_t * vm, vlib_buffer_t * b return l + l_first; } -u8 * format_vlib_buffer (u8 * s, va_list * args) +u8 * +format_vlib_buffer (u8 * s, va_list * args) { - vlib_buffer_t * b = va_arg (*args, vlib_buffer_t *); - + vlib_buffer_t *b = va_arg (*args, vlib_buffer_t *); + s = format (s, "current data %d, length %d, free-list %d", - b->current_data, b->current_length, - b->free_list_index); + b->current_data, b->current_length, b->free_list_index); if (b->flags & VLIB_BUFFER_IS_TRACED) s = format (s, ", trace 0x%x", b->trace_index); @@ -71,10 +73,11 @@ u8 * format_vlib_buffer (u8 * s, va_list * args) return s; } -u8 * format_vlib_buffer_and_data (u8 * s, va_list * args) +u8 * +format_vlib_buffer_and_data (u8 * s, va_list * args) { - vlib_buffer_t * b = va_arg (*args, vlib_buffer_t *); - + vlib_buffer_t *b = va_arg (*args, vlib_buffer_t *); + s = format (s, "%U, %U", format_vlib_buffer, b, format_hex_bytes, vlib_buffer_get_current (b), 64); @@ -82,10 +85,11 @@ u8 * format_vlib_buffer_and_data (u8 * s, va_list * args) return s; } -static u8 * format_vlib_buffer_known_state (u8 * s, va_list * args) +static u8 * +format_vlib_buffer_known_state (u8 * s, va_list * args) { vlib_buffer_known_state_t state = va_arg (*args, vlib_buffer_known_state_t); - char * t; + char *t; switch (state) { @@ -109,16 +113,16 @@ static u8 * format_vlib_buffer_known_state (u8 * s, va_list * args) return format (s, "%s", t); } -u8 * format_vlib_buffer_contents (u8 * s, va_list * va) +u8 * +format_vlib_buffer_contents (u8 * s, va_list * va) { - vlib_main_t * vm = va_arg (*va, vlib_main_t *); - vlib_buffer_t * b = va_arg (*va, vlib_buffer_t *); - + vlib_main_t *vm = va_arg (*va, vlib_main_t *); + vlib_buffer_t *b = va_arg (*va, vlib_buffer_t *); + while (1) { - vec_add (s, vlib_buffer_get_current (b), - b->current_length); - if (! (b->flags & VLIB_BUFFER_NEXT_PRESENT)) + vec_add (s, vlib_buffer_get_current (b), b->current_length); + if (!(b->flags & VLIB_BUFFER_NEXT_PRESENT)) break; b = vlib_get_buffer (vm, b->next_buffer); } @@ -129,41 +133,34 @@ u8 * format_vlib_buffer_contents (u8 * s, va_list * va) static u8 * vlib_validate_buffer_helper (vlib_main_t * vm, u32 bi, - uword follow_buffer_next, - uword ** unique_hash) - + uword follow_buffer_next, uword ** unique_hash) { - vlib_buffer_t * b = vlib_get_buffer (vm, bi); - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * fl; + vlib_buffer_t *b = vlib_get_buffer (vm, bi); + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *fl; - if (pool_is_free_index (bm->buffer_free_list_pool, - b->free_list_index)) + if (pool_is_free_index (bm->buffer_free_list_pool, b->free_list_index)) return format (0, "unknown free list 0x%x", b->free_list_index); - fl = pool_elt_at_index (bm->buffer_free_list_pool, - b->free_list_index); + fl = pool_elt_at_index (bm->buffer_free_list_pool, b->free_list_index); - if ((signed) b->current_data < (signed) - VLIB_BUFFER_PRE_DATA_SIZE) + if ((signed) b->current_data < (signed) -VLIB_BUFFER_PRE_DATA_SIZE) return format (0, "current data %d before pre-data", b->current_data); #if DPDK == 0 if (b->current_data + b->current_length > fl->n_data_bytes) return format (0, "%d-%d beyond end of buffer %d", - b->current_data, b->current_length, - fl->n_data_bytes); + b->current_data, b->current_length, fl->n_data_bytes); #endif - if (follow_buffer_next - && (b->flags & VLIB_BUFFER_NEXT_PRESENT)) + if (follow_buffer_next && (b->flags & VLIB_BUFFER_NEXT_PRESENT)) { vlib_buffer_known_state_t k; - u8 * msg, * result; + u8 *msg, *result; k = vlib_buffer_is_known (vm, b->next_buffer); if (k != VLIB_BUFFER_KNOWN_ALLOCATED) return format (0, "next 0x%x: %U", - b->next_buffer, - format_vlib_buffer_known_state, k); + b->next_buffer, format_vlib_buffer_known_state, k); if (unique_hash) { @@ -187,7 +184,10 @@ vlib_validate_buffer_helper (vlib_main_t * vm, u8 * vlib_validate_buffer (vlib_main_t * vm, u32 bi, uword follow_buffer_next) -{ return vlib_validate_buffer_helper (vm, bi, follow_buffer_next, /* unique_hash */ 0); } +{ + return vlib_validate_buffer_helper (vm, bi, follow_buffer_next, + /* unique_hash */ 0); +} u8 * vlib_validate_buffers (vlib_main_t * vm, @@ -197,10 +197,10 @@ vlib_validate_buffers (vlib_main_t * vm, vlib_buffer_known_state_t known_state, uword follow_buffer_next) { - uword i, * hash; - u32 bi, * b = buffers; + uword i, *hash; + u32 bi, *b = buffers; vlib_buffer_known_state_t k; - u8 * msg = 0, * result = 0; + u8 *msg = 0, *result = 0; hash = hash_create (0, 0); for (i = 0; i < n_buffers; i++) @@ -231,7 +231,7 @@ vlib_validate_buffers (vlib_main_t * vm, hash_set1 (hash, bi); } - done: +done: if (msg) { result = format (0, "0x%x: %v", bi, msg); @@ -251,24 +251,24 @@ vlib_buffer_validate_alloc_free (vlib_main_t * vm, uword n_buffers, vlib_buffer_known_state_t expected_state) { - u32 * b; + u32 *b; uword i, bi, is_free; if (CLIB_DEBUG == 0) return; - ASSERT(os_get_cpu_number() == 0); + ASSERT (os_get_cpu_number () == 0); /* smp disaster check */ if (vlib_mains) - ASSERT(vm == vlib_mains[0]); + ASSERT (vm == vlib_mains[0]); is_free = expected_state == VLIB_BUFFER_KNOWN_ALLOCATED; b = buffers; for (i = 0; i < n_buffers; i++) { vlib_buffer_known_state_t known; - + bi = b[0]; b += 1; known = vlib_buffer_is_known (vm, bi); @@ -278,8 +278,7 @@ vlib_buffer_validate_alloc_free (vlib_main_t * vm, vlib_panic_with_msg (vm, "%s %U buffer 0x%x", is_free ? "freeing" : "allocating", - format_vlib_buffer_known_state, known, - bi); + format_vlib_buffer_known_state, known, bi); } vlib_buffer_set_known_state @@ -290,10 +289,10 @@ vlib_buffer_validate_alloc_free (vlib_main_t * vm, /* Aligned copy routine. */ void -vlib_aligned_memcpy (void * _dst, void * _src, int n_bytes) +vlib_aligned_memcpy (void *_dst, void *_src, int n_bytes) { - vlib_copy_unit_t * dst = _dst; - vlib_copy_unit_t * src = _src; + vlib_copy_unit_t *dst = _dst; + vlib_copy_unit_t *src = _src; /* Arguments must be naturally aligned. */ ASSERT (pointer_to_uword (dst) % sizeof (dst[0]) == 0); @@ -341,7 +340,7 @@ vlib_aligned_memcpy (void * _dst, void * _src, int n_bytes) } } else - /* Cache line size unknown: fall back to slow version. */; + /* Cache line size unknown: fall back to slow version. */ ; while (n_bytes > 0) { @@ -412,8 +411,8 @@ merge_free_lists (vlib_buffer_free_list_t * dst, vlib_buffer_free_list_t * src) { uword l; - u32 * d; - + u32 *d; + trim_aligned (src); trim_aligned (dst); @@ -437,10 +436,10 @@ merge_free_lists (vlib_buffer_free_list_t * dst, always_inline u32 vlib_buffer_get_free_list_with_size (vlib_main_t * vm, u32 size) { - vlib_buffer_main_t * bm = vm->buffer_main; + vlib_buffer_main_t *bm = vm->buffer_main; size = vlib_buffer_round_size (size); - uword * p = hash_get (bm->free_list_by_size, size); + uword *p = hash_get (bm->free_list_by_size, size); return p ? p[0] : ~0; } @@ -448,24 +447,27 @@ vlib_buffer_get_free_list_with_size (vlib_main_t * vm, u32 size) static u32 vlib_buffer_create_free_list_helper (vlib_main_t * vm, u32 n_data_bytes, - u32 is_public, - u32 is_default, - u8 * name) + u32 is_public, u32 is_default, u8 * name) { - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * f; + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *f; - if (! is_default && pool_elts (bm->buffer_free_list_pool) == 0) + if (!is_default && pool_elts (bm->buffer_free_list_pool) == 0) { u32 default_free_free_list_index; - default_free_free_list_index = - vlib_buffer_create_free_list_helper (vm, - /* default buffer size */ VLIB_BUFFER_DEFAULT_FREE_LIST_BYTES, - /* is_public */ 1, - /* is_default */ 1, - (u8 *) "default"); - ASSERT (default_free_free_list_index == VLIB_BUFFER_DEFAULT_FREE_LIST_INDEX); + default_free_free_list_index = vlib_buffer_create_free_list_helper (vm, + /* default buffer size */ + VLIB_BUFFER_DEFAULT_FREE_LIST_BYTES, + /* is_public */ + 1, + /* is_default */ + 1, + (u8 + *) + "default"); + ASSERT (default_free_free_list_index == + VLIB_BUFFER_DEFAULT_FREE_LIST_INDEX); if (n_data_bytes == VLIB_BUFFER_DEFAULT_FREE_LIST_BYTES && is_public) return default_free_free_list_index; @@ -484,19 +486,20 @@ vlib_buffer_create_free_list_helper (vlib_main_t * vm, if (is_public) { - uword * p = hash_get (bm->free_list_by_size, f->n_data_bytes); - if (! p) + uword *p = hash_get (bm->free_list_by_size, f->n_data_bytes); + if (!p) hash_set (bm->free_list_by_size, f->n_data_bytes, f->index); } return f->index; } -u32 vlib_buffer_create_free_list (vlib_main_t * vm, u32 n_data_bytes, - char * fmt, ...) +u32 +vlib_buffer_create_free_list (vlib_main_t * vm, u32 n_data_bytes, + char *fmt, ...) { va_list va; - u8 * name; + u8 *name; va_start (va, fmt); name = va_format (0, fmt, &va); @@ -508,15 +511,16 @@ u32 vlib_buffer_create_free_list (vlib_main_t * vm, u32 n_data_bytes, name); } -u32 vlib_buffer_get_or_create_free_list (vlib_main_t * vm, u32 n_data_bytes, - char * fmt, ...) +u32 +vlib_buffer_get_or_create_free_list (vlib_main_t * vm, u32 n_data_bytes, + char *fmt, ...) { u32 i = vlib_buffer_get_free_list_with_size (vm, n_data_bytes); if (i == ~0) { va_list va; - u8 * name; + u8 *name; va_start (va, fmt); name = va_format (0, fmt, &va); @@ -527,7 +531,7 @@ u32 vlib_buffer_get_or_create_free_list (vlib_main_t * vm, u32 n_data_bytes, /* is_default */ 0, name); } - + return i; } @@ -545,15 +549,17 @@ del_free_list (vlib_main_t * vm, vlib_buffer_free_list_t * f) } /* Add buffer free list. */ -void vlib_buffer_delete_free_list (vlib_main_t * vm, u32 free_list_index) +void +vlib_buffer_delete_free_list (vlib_main_t * vm, u32 free_list_index) { - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * f; + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *f; u32 merge_index; f = vlib_buffer_get_free_list (vm, free_list_index); - ASSERT (vec_len (f->unaligned_buffers) + vec_len (f->aligned_buffers) == f->n_alloc); + ASSERT (vec_len (f->unaligned_buffers) + vec_len (f->aligned_buffers) == + f->n_alloc); merge_index = vlib_buffer_get_free_list_with_size (vm, f->n_data_bytes); if (merge_index != ~0 && merge_index != free_list_index) { @@ -572,12 +578,11 @@ void vlib_buffer_delete_free_list (vlib_main_t * vm, u32 free_list_index) /* Make sure free list has at least given number of free buffers. */ static uword fill_free_list (vlib_main_t * vm, - vlib_buffer_free_list_t * fl, - uword min_free_buffers) + vlib_buffer_free_list_t * fl, uword min_free_buffers) { - vlib_buffer_t * buffers, * b; + vlib_buffer_t *buffers, *b; int n, n_bytes, i; - u32 * bi; + u32 *bi; u32 n_remaining, n_alloc, n_this_chunk; trim_aligned (fl); @@ -598,14 +603,15 @@ fill_free_list (vlib_main_t * vm, while (n_remaining > 0) { n_this_chunk = clib_min (n_remaining, 16); - + n_bytes = n_this_chunk * (sizeof (b[0]) + fl->n_data_bytes); /* drb: removed power-of-2 ASSERT */ buffers = vm->os_physmem_alloc_aligned (&vm->physmem_main, - n_bytes, sizeof (vlib_buffer_t)); - if (! buffers) - return n_alloc; + n_bytes, + sizeof (vlib_buffer_t)); + if (!buffers) + return n_alloc; /* Record chunk as being allocated so we can free it later. */ vec_add1 (fl->buffer_memory_allocated, buffers); @@ -615,48 +621,49 @@ fill_free_list (vlib_main_t * vm, n_remaining -= n_this_chunk; b = buffers; - vec_add2_aligned (fl->aligned_buffers, bi, n_this_chunk, - sizeof (vlib_copy_unit_t)); + vec_add2_aligned (fl->aligned_buffers, bi, n_this_chunk, + sizeof (vlib_copy_unit_t)); for (i = 0; i < n_this_chunk; i++) - { - bi[i] = vlib_get_buffer_index (vm, b); + { + bi[i] = vlib_get_buffer_index (vm, b); - if (CLIB_DEBUG > 0) - vlib_buffer_set_known_state (vm, bi[i], VLIB_BUFFER_KNOWN_FREE); - b = vlib_buffer_next_contiguous (b, fl->n_data_bytes); - } + if (CLIB_DEBUG > 0) + vlib_buffer_set_known_state (vm, bi[i], VLIB_BUFFER_KNOWN_FREE); + b = vlib_buffer_next_contiguous (b, fl->n_data_bytes); + } memset (buffers, 0, n_bytes); /* Initialize all new buffers. */ b = buffers; for (i = 0; i < n_this_chunk; i++) - { - vlib_buffer_init_for_free_list (b, fl); - b = vlib_buffer_next_contiguous (b, fl->n_data_bytes); - } - + { + vlib_buffer_init_for_free_list (b, fl); + b = vlib_buffer_next_contiguous (b, fl->n_data_bytes); + } + if (fl->buffer_init_function) - fl->buffer_init_function (vm, fl, bi, n_this_chunk); + fl->buffer_init_function (vm, fl, bi, n_this_chunk); } return n_alloc; } always_inline uword copy_alignment (u32 * x) -{ return (pointer_to_uword (x) / sizeof (x[0])) % BUFFERS_PER_COPY; } +{ + return (pointer_to_uword (x) / sizeof (x[0])) % BUFFERS_PER_COPY; +} static u32 alloc_from_free_list (vlib_main_t * vm, vlib_buffer_free_list_t * free_list, - u32 * alloc_buffers, - u32 n_alloc_buffers) + u32 * alloc_buffers, u32 n_alloc_buffers) { - u32 * dst, * u_src; + u32 *dst, *u_src; uword u_len, n_left; uword n_unaligned_start, n_unaligned_end, n_filled; - ASSERT(os_get_cpu_number() == 0); + ASSERT (os_get_cpu_number () == 0); n_left = n_alloc_buffers; dst = alloc_buffers; @@ -666,7 +673,7 @@ alloc_from_free_list (vlib_main_t * vm, n_filled = fill_free_list (vm, free_list, n_alloc_buffers); if (n_filled == 0) return 0; - + n_left = n_filled < n_left ? n_filled : n_left; n_alloc_buffers = n_left; @@ -704,11 +711,12 @@ alloc_from_free_list (vlib_main_t * vm, /* Aligned copy. */ { - vlib_copy_unit_t * d, * s; + vlib_copy_unit_t *d, *s; uword n_copy; - if (vec_len(free_list->aligned_buffers) < ((n_left/BUFFERS_PER_COPY)*BUFFERS_PER_COPY)) - abort(); + if (vec_len (free_list->aligned_buffers) < + ((n_left / BUFFERS_PER_COPY) * BUFFERS_PER_COPY)) + abort (); n_copy = n_left / BUFFERS_PER_COPY; n_left = n_left % BUFFERS_PER_COPY; @@ -751,15 +759,14 @@ alloc_from_free_list (vlib_main_t * vm, u_len--; } - if (! free_list->unaligned_buffers) + if (!free_list->unaligned_buffers) ASSERT (u_len == 0); else _vec_len (free_list->unaligned_buffers) = u_len; /* Verify that buffers are known free. */ vlib_buffer_validate_alloc_free (vm, alloc_buffers, - n_alloc_buffers, - VLIB_BUFFER_KNOWN_FREE); + n_alloc_buffers, VLIB_BUFFER_KNOWN_FREE); return n_alloc_buffers; } @@ -767,10 +774,11 @@ alloc_from_free_list (vlib_main_t * vm, /* Allocate a given number of buffers into given array. Returns number actually allocated which will be either zero or number requested. */ -u32 vlib_buffer_alloc (vlib_main_t * vm, u32 * buffers, u32 n_buffers) +u32 +vlib_buffer_alloc (vlib_main_t * vm, u32 * buffers, u32 n_buffers) { - vlib_buffer_main_t * bm = vm->buffer_main; - ASSERT(os_get_cpu_number() == 0); + vlib_buffer_main_t *bm = vm->buffer_main; + ASSERT (os_get_cpu_number () == 0); return alloc_from_free_list (vm, @@ -779,89 +787,92 @@ u32 vlib_buffer_alloc (vlib_main_t * vm, u32 * buffers, u32 n_buffers) buffers, n_buffers); } -u32 vlib_buffer_alloc_from_free_list (vlib_main_t * vm, - u32 * buffers, - u32 n_buffers, - u32 free_list_index) +u32 +vlib_buffer_alloc_from_free_list (vlib_main_t * vm, + u32 * buffers, + u32 n_buffers, u32 free_list_index) { - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * f; + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *f; f = pool_elt_at_index (bm->buffer_free_list_pool, free_list_index); return alloc_from_free_list (vm, f, buffers, n_buffers); } always_inline void add_buffer_to_free_list (vlib_main_t * vm, - vlib_buffer_free_list_t * f, + vlib_buffer_free_list_t * f, u32 buffer_index, u8 do_init) { - vlib_buffer_t * b; + vlib_buffer_t *b; b = vlib_get_buffer (vm, buffer_index); - if (PREDICT_TRUE(do_init)) - vlib_buffer_init_for_free_list (b, f); - vec_add1_aligned (f->aligned_buffers, buffer_index, sizeof (vlib_copy_unit_t)); + if (PREDICT_TRUE (do_init)) + vlib_buffer_init_for_free_list (b, f); + vec_add1_aligned (f->aligned_buffers, buffer_index, + sizeof (vlib_copy_unit_t)); } always_inline vlib_buffer_free_list_t * buffer_get_free_list (vlib_main_t * vm, vlib_buffer_t * b, u32 * index) { - vlib_buffer_main_t * bm = vm->buffer_main; + vlib_buffer_main_t *bm = vm->buffer_main; u32 i; *index = i = b->free_list_index; return pool_elt_at_index (bm->buffer_free_list_pool, i); } -void *vlib_set_buffer_free_callback (vlib_main_t *vm, void *fp) +void * +vlib_set_buffer_free_callback (vlib_main_t * vm, void *fp) { - vlib_buffer_main_t * bm = vm->buffer_main; - void * rv = bm->buffer_free_callback; + vlib_buffer_main_t *bm = vm->buffer_main; + void *rv = bm->buffer_free_callback; bm->buffer_free_callback = fp; return rv; } void vnet_buffer_free_dpdk_mb (vlib_buffer_t * b) __attribute__ ((weak)); -void vnet_buffer_free_dpdk_mb (vlib_buffer_t * b) { } +void +vnet_buffer_free_dpdk_mb (vlib_buffer_t * b) +{ +} static_always_inline void vlib_buffer_free_inline (vlib_main_t * vm, - u32 * buffers, - u32 n_buffers, - u32 follow_buffer_next) + u32 * buffers, u32 n_buffers, u32 follow_buffer_next) { - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * fl; - static u32 * next_to_free[2]; /* smp bad */ - u32 i_next_to_free, * b, * n, * f, fi; + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *fl; + static u32 *next_to_free[2]; /* smp bad */ + u32 i_next_to_free, *b, *n, *f, fi; uword n_left; int i; - static vlib_buffer_free_list_t ** announce_list; - vlib_buffer_free_list_t * fl0 = 0, * fl1 = 0; - u32 bi0=(u32)~0, bi1=(u32)~0, fi0, fi1 = (u32)~0; - u8 free0, free1=0, free_next0, free_next1; - u32 (*cb)(vlib_main_t * vm, u32 * buffers, u32 n_buffers, - u32 follow_buffer_next); - - ASSERT(os_get_cpu_number() == 0); - + static vlib_buffer_free_list_t **announce_list; + vlib_buffer_free_list_t *fl0 = 0, *fl1 = 0; + u32 bi0 = (u32) ~ 0, bi1 = (u32) ~ 0, fi0, fi1 = (u32) ~ 0; + u8 free0, free1 = 0, free_next0, free_next1; + u32 (*cb) (vlib_main_t * vm, u32 * buffers, u32 n_buffers, + u32 follow_buffer_next); + + ASSERT (os_get_cpu_number () == 0); + cb = bm->buffer_free_callback; if (PREDICT_FALSE (cb != 0)) - n_buffers = (*cb)(vm, buffers, n_buffers, follow_buffer_next); + n_buffers = (*cb) (vm, buffers, n_buffers, follow_buffer_next); - if (! n_buffers) + if (!n_buffers) return; /* Use first buffer to get default free list. */ { u32 bi0 = buffers[0]; - vlib_buffer_t * b0; + vlib_buffer_t *b0; b0 = vlib_get_buffer (vm, bi0); fl = buffer_get_free_list (vm, b0, &fi); if (fl->buffers_added_to_freelist_function) - vec_add1 (announce_list, fl); + vec_add1 (announce_list, fl); } vec_validate (next_to_free[0], n_buffers - 1); @@ -871,11 +882,10 @@ vlib_buffer_free_inline (vlib_main_t * vm, n_left = n_buffers; b = buffers; - again: +again: /* Verify that buffers are known allocated. */ vlib_buffer_validate_alloc_free (vm, b, - n_left, - VLIB_BUFFER_KNOWN_ALLOCATED); + n_left, VLIB_BUFFER_KNOWN_ALLOCATED); vec_add2_aligned (fl->aligned_buffers, f, n_left, /* align */ sizeof (vlib_copy_unit_t)); @@ -883,7 +893,7 @@ vlib_buffer_free_inline (vlib_main_t * vm, n = next_to_free[i_next_to_free]; while (n_left >= 4) { - vlib_buffer_t * b0, * b1, * binit0, * binit1, dummy_buffers[2]; + vlib_buffer_t *b0, *b1, *binit0, *binit1, dummy_buffers[2]; bi0 = b[0]; bi1 = b[1]; @@ -942,23 +952,23 @@ vlib_buffer_free_inline (vlib_main_t * vm, fl1 = pool_elt_at_index (bm->buffer_free_list_pool, fi1); add_buffer_to_free_list (vm, fl0, bi0, free0); - if (PREDICT_FALSE(fl0->buffers_added_to_freelist_function != 0)) - { - int i; - for (i = 0; i < vec_len (announce_list); i++) - if (fl0 == announce_list[i]) - goto no_fl0; - vec_add1(announce_list, fl0); - } + if (PREDICT_FALSE (fl0->buffers_added_to_freelist_function != 0)) + { + int i; + for (i = 0; i < vec_len (announce_list); i++) + if (fl0 == announce_list[i]) + goto no_fl0; + vec_add1 (announce_list, fl0); + } no_fl0: - if (PREDICT_FALSE(fl1->buffers_added_to_freelist_function != 0)) - { - int i; - for (i = 0; i < vec_len (announce_list); i++) - if (fl1 == announce_list[i]) - goto no_fl1; - vec_add1(announce_list, fl1); - } + if (PREDICT_FALSE (fl1->buffers_added_to_freelist_function != 0)) + { + int i; + for (i = 0; i < vec_len (announce_list); i++) + if (fl1 == announce_list[i]) + goto no_fl1; + vec_add1 (announce_list, fl1); + } no_fl1: add_buffer_to_free_list (vm, fl1, bi1, free1); @@ -976,7 +986,7 @@ vlib_buffer_free_inline (vlib_main_t * vm, while (n_left >= 1) { - vlib_buffer_t * b0, * binit0, dummy_buffers[1]; + vlib_buffer_t *b0, *binit0, dummy_buffers[1]; bi0 = b[0]; f[0] = bi0; @@ -1019,15 +1029,15 @@ vlib_buffer_free_inline (vlib_main_t * vm, fl0 = pool_elt_at_index (bm->buffer_free_list_pool, fi0); add_buffer_to_free_list (vm, fl0, bi0, free0); - if (PREDICT_FALSE(fl0->buffers_added_to_freelist_function != 0)) - { - int i; - for (i = 0; i < vec_len (announce_list); i++) - if (fl0 == announce_list[i]) - goto no_fl00; - vec_add1(announce_list, fl0); - } - + if (PREDICT_FALSE (fl0->buffers_added_to_freelist_function != 0)) + { + int i; + for (i = 0; i < vec_len (announce_list); i++) + if (fl0 == announce_list[i]) + goto no_fl00; + vec_add1 (announce_list, fl0); + } + no_fl00: fi = fi0; fl = pool_elt_at_index (bm->buffer_free_list_pool, fi); @@ -1045,62 +1055,63 @@ vlib_buffer_free_inline (vlib_main_t * vm, _vec_len (fl->aligned_buffers) = f - fl->aligned_buffers; - if (vec_len(announce_list)) + if (vec_len (announce_list)) { - vlib_buffer_free_list_t * fl; + vlib_buffer_free_list_t *fl; for (i = 0; i < vec_len (announce_list); i++) - { - fl = announce_list[i]; - fl->buffers_added_to_freelist_function (vm, fl); - } - _vec_len(announce_list) = 0; + { + fl = announce_list[i]; + fl->buffers_added_to_freelist_function (vm, fl); + } + _vec_len (announce_list) = 0; } } -void vlib_buffer_free (vlib_main_t * vm, - u32 * buffers, - u32 n_buffers) +void +vlib_buffer_free (vlib_main_t * vm, u32 * buffers, u32 n_buffers) { - vlib_buffer_free_inline (vm, buffers, n_buffers, /* follow_buffer_next */ 1); + vlib_buffer_free_inline (vm, buffers, n_buffers, /* follow_buffer_next */ + 1); } -void vlib_buffer_free_no_next (vlib_main_t * vm, - u32 * buffers, - u32 n_buffers) +void +vlib_buffer_free_no_next (vlib_main_t * vm, u32 * buffers, u32 n_buffers) { - vlib_buffer_free_inline (vm, buffers, n_buffers, /* follow_buffer_next */ 0); + vlib_buffer_free_inline (vm, buffers, n_buffers, /* follow_buffer_next */ + 0); } /* Copy template packet data into buffers as they are allocated. */ static void vlib_packet_template_buffer_init (vlib_main_t * vm, vlib_buffer_free_list_t * fl, - u32 * buffers, - u32 n_buffers) + u32 * buffers, u32 n_buffers) { - vlib_packet_template_t * t = uword_to_pointer (fl->buffer_init_function_opaque, - vlib_packet_template_t *); + vlib_packet_template_t *t = + uword_to_pointer (fl->buffer_init_function_opaque, + vlib_packet_template_t *); uword i; for (i = 0; i < n_buffers; i++) { - vlib_buffer_t * b = vlib_get_buffer (vm, buffers[i]); + vlib_buffer_t *b = vlib_get_buffer (vm, buffers[i]); ASSERT (b->current_length == vec_len (t->packet_data)); - clib_memcpy (vlib_buffer_get_current (b), t->packet_data, b->current_length); + clib_memcpy (vlib_buffer_get_current (b), t->packet_data, + b->current_length); } } -void vlib_packet_template_init (vlib_main_t * vm, - vlib_packet_template_t * t, - void * packet_data, - uword n_packet_data_bytes, - uword min_n_buffers_each_physmem_alloc, - char * fmt, - ...) +void +vlib_packet_template_init (vlib_main_t * vm, + vlib_packet_template_t * t, + void *packet_data, + uword n_packet_data_bytes, + uword min_n_buffers_each_physmem_alloc, + char *fmt, ...) { - vlib_buffer_free_list_t * fl; + vlib_buffer_free_list_t *fl; va_list va; - u8 * name; + u8 *name; va_start (va, fmt); name = va_format (0, fmt, &va); @@ -1130,12 +1141,11 @@ void vlib_packet_template_init (vlib_main_t * vm, } void * -vlib_packet_template_get_packet (vlib_main_t * vm, - vlib_packet_template_t * t, - u32 * bi_result) +vlib_packet_template_get_packet (vlib_main_t * vm, + vlib_packet_template_t * t, u32 * bi_result) { u32 bi; - vlib_buffer_t * b; + vlib_buffer_t *b; if (vlib_buffer_alloc (vm, &bi, 1) != 1) return 0; @@ -1144,13 +1154,15 @@ vlib_packet_template_get_packet (vlib_main_t * vm, b = vlib_get_buffer (vm, bi); clib_memcpy (vlib_buffer_get_current (b), - t->packet_data, vec_len(t->packet_data)); - b->current_length = vec_len(t->packet_data); + t->packet_data, vec_len (t->packet_data)); + b->current_length = vec_len (t->packet_data); return b->data; } -void vlib_packet_template_get_packet_helper (vlib_main_t * vm, vlib_packet_template_t * t) +void +vlib_packet_template_get_packet_helper (vlib_main_t * vm, + vlib_packet_template_t * t) { word n = t->min_n_buffers_each_physmem_alloc; word l = vec_len (t->packet_data); @@ -1166,14 +1178,14 @@ void vlib_packet_template_get_packet_helper (vlib_main_t * vm, vlib_packet_templ } /* Append given data to end of buffer, possibly allocating new buffers. */ -u32 vlib_buffer_add_data (vlib_main_t * vm, - u32 free_list_index, - u32 buffer_index, - void * data, u32 n_data_bytes) +u32 +vlib_buffer_add_data (vlib_main_t * vm, + u32 free_list_index, + u32 buffer_index, void *data, u32 n_data_bytes) { u32 n_buffer_bytes, n_left, n_left_this_buffer, bi; - vlib_buffer_t * b; - void * d; + vlib_buffer_t *b; + void *d; bi = buffer_index; if (bi == 0 @@ -1183,11 +1195,11 @@ u32 vlib_buffer_add_data (vlib_main_t * vm, d = data; n_left = n_data_bytes; n_buffer_bytes = vlib_buffer_free_list_buffer_size (vm, free_list_index); - + b = vlib_get_buffer (vm, bi); b->flags &= ~VLIB_BUFFER_TOTAL_LENGTH_VALID; - /* Get to the end of the chain before we try to append data...*/ + /* Get to the end of the chain before we try to append data... */ while (b->flags & VLIB_BUFFER_NEXT_PRESENT) b = vlib_get_buffer (vm, b->next_buffer); @@ -1196,7 +1208,8 @@ u32 vlib_buffer_add_data (vlib_main_t * vm, u32 n; ASSERT (n_buffer_bytes >= b->current_length); - n_left_this_buffer = n_buffer_bytes - (b->current_data + b->current_length); + n_left_this_buffer = + n_buffer_bytes - (b->current_data + b->current_length); n = clib_min (n_left_this_buffer, n_left); clib_memcpy (vlib_buffer_get_current (b) + b->current_length, d, n); b->current_length += n; @@ -1205,7 +1218,9 @@ u32 vlib_buffer_add_data (vlib_main_t * vm, break; d += n; - if (1 != vlib_buffer_alloc_from_free_list (vm, &b->next_buffer, 1, free_list_index)) + if (1 != + vlib_buffer_alloc_from_free_list (vm, &b->next_buffer, 1, + free_list_index)) goto out_of_buffers; b->flags |= VLIB_BUFFER_NEXT_PRESENT; @@ -1215,68 +1230,81 @@ u32 vlib_buffer_add_data (vlib_main_t * vm, return bi; - out_of_buffers: +out_of_buffers: clib_error ("out of buffers"); return bi; } u16 -vlib_buffer_chain_append_data_with_alloc(vlib_main_t *vm, - u32 free_list_index, - vlib_buffer_t *first, - vlib_buffer_t **last, - void * data, u16 data_len) { +vlib_buffer_chain_append_data_with_alloc (vlib_main_t * vm, + u32 free_list_index, + vlib_buffer_t * first, + vlib_buffer_t ** last, + void *data, u16 data_len) +{ vlib_buffer_t *l = *last; - u32 n_buffer_bytes = vlib_buffer_free_list_buffer_size (vm, free_list_index); + u32 n_buffer_bytes = + vlib_buffer_free_list_buffer_size (vm, free_list_index); u16 copied = 0; - ASSERT(n_buffer_bytes >= l->current_length + l->current_data); - while (data_len) { - u16 max = n_buffer_bytes - l->current_length - l->current_data; - if (max == 0) { - if (1 != vlib_buffer_alloc_from_free_list (vm, &l->next_buffer, 1, free_list_index)) - return copied; - *last = l = vlib_buffer_chain_buffer(vm, first, l, l->next_buffer); - max = n_buffer_bytes - l->current_length - l->current_data; - } + ASSERT (n_buffer_bytes >= l->current_length + l->current_data); + while (data_len) + { + u16 max = n_buffer_bytes - l->current_length - l->current_data; + if (max == 0) + { + if (1 != + vlib_buffer_alloc_from_free_list (vm, &l->next_buffer, 1, + free_list_index)) + return copied; + *last = l = vlib_buffer_chain_buffer (vm, first, l, l->next_buffer); + max = n_buffer_bytes - l->current_length - l->current_data; + } - u16 len = (data_len > max)?max:data_len; - clib_memcpy(vlib_buffer_get_current (l) + l->current_length, data + copied, len); - vlib_buffer_chain_increase_length(first, l, len); - data_len -= len; - copied += len; - } + u16 len = (data_len > max) ? max : data_len; + clib_memcpy (vlib_buffer_get_current (l) + l->current_length, + data + copied, len); + vlib_buffer_chain_increase_length (first, l, len); + data_len -= len; + copied += len; + } return copied; } /* * Fills in the required rte_mbuf fields for chained buffers given a VLIB chain. */ -void vlib_buffer_chain_validate (vlib_main_t * vm, vlib_buffer_t * b_first) +void +vlib_buffer_chain_validate (vlib_main_t * vm, vlib_buffer_t * b_first) { return; } -static void vlib_serialize_tx (serialize_main_header_t * m, serialize_stream_t * s) +static void +vlib_serialize_tx (serialize_main_header_t * m, serialize_stream_t * s) { - vlib_main_t * vm; - vlib_serialize_buffer_main_t * sm; + vlib_main_t *vm; + vlib_serialize_buffer_main_t *sm; uword n, n_bytes_to_write; - vlib_buffer_t * last; + vlib_buffer_t *last; n_bytes_to_write = s->current_buffer_index; - sm = uword_to_pointer (s->data_function_opaque, vlib_serialize_buffer_main_t *); + sm = + uword_to_pointer (s->data_function_opaque, + vlib_serialize_buffer_main_t *); vm = sm->vlib_main; ASSERT (sm->tx.max_n_data_bytes_per_chain > 0); if (serialize_stream_is_end_of_stream (s) - || sm->tx.n_total_data_bytes + n_bytes_to_write > sm->tx.max_n_data_bytes_per_chain) + || sm->tx.n_total_data_bytes + n_bytes_to_write > + sm->tx.max_n_data_bytes_per_chain) { - vlib_process_t * p = vlib_get_current_process (vm); + vlib_process_t *p = vlib_get_current_process (vm); last = vlib_get_buffer (vm, sm->last_buffer); last->current_length = n_bytes_to_write; - vlib_set_next_frame_buffer (vm, &p->node_runtime, sm->tx.next_index, sm->first_buffer); + vlib_set_next_frame_buffer (vm, &p->node_runtime, sm->tx.next_index, + sm->first_buffer); sm->first_buffer = sm->last_buffer = ~0; sm->tx.n_total_data_bytes = 0; @@ -1286,19 +1314,28 @@ static void vlib_serialize_tx (serialize_main_header_t * m, serialize_stream_t * { ASSERT (sm->first_buffer == ~0); ASSERT (sm->last_buffer == ~0); - n = vlib_buffer_alloc_from_free_list (vm, &sm->first_buffer, 1, sm->tx.free_list_index); + n = + vlib_buffer_alloc_from_free_list (vm, &sm->first_buffer, 1, + sm->tx.free_list_index); if (n != 1) - serialize_error (m, clib_error_create ("vlib_buffer_alloc_from_free_list fails")); + serialize_error (m, + clib_error_create + ("vlib_buffer_alloc_from_free_list fails")); sm->last_buffer = sm->first_buffer; - s->n_buffer_bytes = vlib_buffer_free_list_buffer_size (vm, sm->tx.free_list_index); + s->n_buffer_bytes = + vlib_buffer_free_list_buffer_size (vm, sm->tx.free_list_index); } if (n_bytes_to_write > 0) { - vlib_buffer_t * prev = vlib_get_buffer (vm, sm->last_buffer); - n = vlib_buffer_alloc_from_free_list (vm, &sm->last_buffer, 1, sm->tx.free_list_index); + vlib_buffer_t *prev = vlib_get_buffer (vm, sm->last_buffer); + n = + vlib_buffer_alloc_from_free_list (vm, &sm->last_buffer, 1, + sm->tx.free_list_index); if (n != 1) - serialize_error (m, clib_error_create ("vlib_buffer_alloc_from_free_list fails")); + serialize_error (m, + clib_error_create + ("vlib_buffer_alloc_from_free_list fails")); sm->tx.n_total_data_bytes += n_bytes_to_write; prev->current_length = n_bytes_to_write; prev->next_buffer = sm->last_buffer; @@ -1314,13 +1351,16 @@ static void vlib_serialize_tx (serialize_main_header_t * m, serialize_stream_t * } } -static void vlib_serialize_rx (serialize_main_header_t * m, serialize_stream_t * s) +static void +vlib_serialize_rx (serialize_main_header_t * m, serialize_stream_t * s) { - vlib_main_t * vm; - vlib_serialize_buffer_main_t * sm; - vlib_buffer_t * last; + vlib_main_t *vm; + vlib_serialize_buffer_main_t *sm; + vlib_buffer_t *last; - sm = uword_to_pointer (s->data_function_opaque, vlib_serialize_buffer_main_t *); + sm = + uword_to_pointer (s->data_function_opaque, + vlib_serialize_buffer_main_t *); vm = sm->vlib_main; if (serialize_stream_is_end_of_stream (s)) @@ -1343,8 +1383,11 @@ static void vlib_serialize_rx (serialize_main_header_t * m, serialize_stream_t * { while (clib_fifo_elts (sm->rx.buffer_fifo) == 0) { - sm->rx.ready_one_time_event = vlib_process_create_one_time_event (vm, vlib_current_process (vm), ~0); - vlib_process_wait_for_one_time_event (vm, /* no event data */ 0, sm->rx.ready_one_time_event); + sm->rx.ready_one_time_event = + vlib_process_create_one_time_event (vm, vlib_current_process (vm), + ~0); + vlib_process_wait_for_one_time_event (vm, /* no event data */ 0, + sm->rx.ready_one_time_event); } clib_fifo_sub1 (sm->rx.buffer_fifo, sm->first_buffer); @@ -1362,12 +1405,11 @@ static void vlib_serialize_rx (serialize_main_header_t * m, serialize_stream_t * static void serialize_open_vlib_helper (serialize_main_t * m, vlib_main_t * vm, - vlib_serialize_buffer_main_t * sm, - uword is_read) + vlib_serialize_buffer_main_t * sm, uword is_read) { /* Initialize serialize main but save overflow buffer for re-use between calls. */ { - u8 * save = m->stream.overflow_buffer; + u8 *save = m->stream.overflow_buffer; memset (m, 0, sizeof (m[0])); m->stream.overflow_buffer = save; if (save) @@ -1384,18 +1426,28 @@ serialize_open_vlib_helper (serialize_main_t * m, m->stream.data_function_opaque = pointer_to_uword (sm); } -void serialize_open_vlib_buffer (serialize_main_t * m, vlib_main_t * vm, vlib_serialize_buffer_main_t * sm) -{ serialize_open_vlib_helper (m, vm, sm, /* is_read */ 0); } +void +serialize_open_vlib_buffer (serialize_main_t * m, vlib_main_t * vm, + vlib_serialize_buffer_main_t * sm) +{ + serialize_open_vlib_helper (m, vm, sm, /* is_read */ 0); +} -void unserialize_open_vlib_buffer (serialize_main_t * m, vlib_main_t * vm, vlib_serialize_buffer_main_t * sm) -{ serialize_open_vlib_helper (m, vm, sm, /* is_read */ 1); } +void +unserialize_open_vlib_buffer (serialize_main_t * m, vlib_main_t * vm, + vlib_serialize_buffer_main_t * sm) +{ + serialize_open_vlib_helper (m, vm, sm, /* is_read */ 1); +} -u32 serialize_close_vlib_buffer (serialize_main_t * m) +u32 +serialize_close_vlib_buffer (serialize_main_t * m) { - vlib_serialize_buffer_main_t * sm - = uword_to_pointer (m->stream.data_function_opaque, vlib_serialize_buffer_main_t *); - vlib_buffer_t * last; - serialize_stream_t * s = &m->stream; + vlib_serialize_buffer_main_t *sm + = uword_to_pointer (m->stream.data_function_opaque, + vlib_serialize_buffer_main_t *); + vlib_buffer_t *last; + serialize_stream_t *s = &m->stream; last = vlib_get_buffer (sm->vlib_main, sm->last_buffer); last->current_length = s->current_buffer_index; @@ -1413,10 +1465,12 @@ u32 serialize_close_vlib_buffer (serialize_main_t * m) return sm->first_buffer; } -void unserialize_close_vlib_buffer (serialize_main_t * m) +void +unserialize_close_vlib_buffer (serialize_main_t * m) { - vlib_serialize_buffer_main_t * sm - = uword_to_pointer (m->stream.data_function_opaque, vlib_serialize_buffer_main_t *); + vlib_serialize_buffer_main_t *sm + = uword_to_pointer (m->stream.data_function_opaque, + vlib_serialize_buffer_main_t *); if (sm->first_buffer != ~0) vlib_buffer_free_one (sm->vlib_main, sm->first_buffer); clib_fifo_reset (sm->rx.buffer_fifo); @@ -1424,14 +1478,16 @@ void unserialize_close_vlib_buffer (serialize_main_t * m) _vec_len (m->stream.overflow_buffer) = 0; } -static u8 * format_vlib_buffer_free_list (u8 * s, va_list * va) +static u8 * +format_vlib_buffer_free_list (u8 * s, va_list * va) { - vlib_buffer_free_list_t * f = va_arg (*va, vlib_buffer_free_list_t *); + vlib_buffer_free_list_t *f = va_arg (*va, vlib_buffer_free_list_t *); uword bytes_alloc, bytes_free, n_free, size; - if (! f) + if (!f) return format (s, "%=30s%=12s%=12s%=12s%=12s%=12s%=12s", - "Name", "Index", "Size", "Alloc", "Free", "#Alloc", "#Free"); + "Name", "Index", "Size", "Alloc", "Free", "#Alloc", + "#Free"); size = sizeof (vlib_buffer_t) + f->n_data_bytes; n_free = vec_len (f->aligned_buffers) + vec_len (f->unaligned_buffers); @@ -1441,30 +1497,40 @@ static u8 * format_vlib_buffer_free_list (u8 * s, va_list * va) s = format (s, "%30s%12d%12d%=12U%=12U%=12d%=12d", f->name, f->index, f->n_data_bytes, format_memory_size, bytes_alloc, - format_memory_size, bytes_free, - f->n_alloc, n_free); + format_memory_size, bytes_free, f->n_alloc, n_free); return s; } static clib_error_t * show_buffers (vlib_main_t * vm, - unformat_input_t * input, - vlib_cli_command_t * cmd) + unformat_input_t * input, vlib_cli_command_t * cmd) { - vlib_buffer_main_t * bm = vm->buffer_main; - vlib_buffer_free_list_t * f; + vlib_buffer_main_t *bm = vm->buffer_main; + vlib_buffer_free_list_t *f; vlib_cli_output (vm, "%U", format_vlib_buffer_free_list, 0); + /* *INDENT-OFF* */ pool_foreach (f, bm->buffer_free_list_pool, ({ vlib_cli_output (vm, "%U", format_vlib_buffer_free_list, f); })); +/* *INDENT-ON* */ return 0; } +/* *INDENT-OFF* */ VLIB_CLI_COMMAND (show_buffers_command, static) = { .path = "show buffers", .short_help = "Show packet buffer allocation", .function = show_buffers, }; +/* *INDENT-ON* */ + +/* + * fd.io coding-style-patch-verification: ON + * + * Local Variables: + * eval: (c-set-style "gnu") + * End: + */