diff options
| author |   Dave Barach <dave@barachs.net> | 2016-08-15 11:12:27 -0400 |
|---|---|---|
| committer | Dave Barach <dave@barachs.net> | 2016-08-15 11:12:40 -0400 |
| commit | c379999665febd12ec55bfb3a7545224f2b39d3d (patch) | |
| tree | 8bf0c11e52c2162e1239b6c3f4a5f74b62a07409 /vppinfra/vppinfra/smp_fifo.c | |
| parent | b3d93dacfde8ab21bbce171fff2971b2ed7bce6a (diff) | |
VPP-327 Coding standards cleanup for vppinfra
Fix additional a few additional deviations reported elsewhere by
checkstyle
Change-Id: I026a8ae1c5b1856bbe3c4a555e1b690e7501b045
Signed-off-by: Dave Barach <dave@barachs.net>
Diffstat (limited to 'vppinfra/vppinfra/smp_fifo.c')
| -rw-r--r-- | vppinfra/vppinfra/smp_fifo.c | 34 |
1 files changed, 24 insertions, 10 deletions
diff --git a/vppinfra/vppinfra/smp_fifo.c b/vppinfra/vppinfra/smp_fifo.c index 863c2fd8068..bb74064d8f3 100644 --- a/vppinfra/vppinfra/smp_fifo.c +++ b/vppinfra/vppinfra/smp_fifo.c @@ -38,9 +38,10 @@ #include <vppinfra/smp_fifo.h> #include <vppinfra/mem.h> -clib_smp_fifo_t * clib_smp_fifo_init (uword max_n_elts, uword n_bytes_per_elt) +clib_smp_fifo_t * +clib_smp_fifo_init (uword max_n_elts, uword n_bytes_per_elt) { - clib_smp_fifo_t * f; + clib_smp_fifo_t *f; uword n_bytes_per_elt_cache_aligned; f = clib_mem_alloc_aligned (sizeof (f[0]), CLIB_CACHE_LINE_BYTES); @@ -51,27 +52,40 @@ clib_smp_fifo_t * clib_smp_fifo_init (uword max_n_elts, uword n_bytes_per_elt) f->log2_max_n_elts = max_log2 (max_n_elts); f->max_n_elts_less_one = (1 << f->log2_max_n_elts) - 1; - n_bytes_per_elt_cache_aligned = clib_smp_fifo_round_elt_bytes (n_bytes_per_elt); - clib_exec_on_global_heap ({ - f->data = clib_mem_alloc_aligned (n_bytes_per_elt_cache_aligned << f->log2_max_n_elts, - CLIB_CACHE_LINE_BYTES); - }); + n_bytes_per_elt_cache_aligned = + clib_smp_fifo_round_elt_bytes (n_bytes_per_elt); + clib_exec_on_global_heap ( + { + f->data = + clib_mem_alloc_aligned + (n_bytes_per_elt_cache_aligned << + f->log2_max_n_elts, CLIB_CACHE_LINE_BYTES);} + ); /* Zero all data and mark all elements free. */ { uword i; for (i = 0; i <= f->max_n_elts_less_one; i++) { - void * d = clib_smp_fifo_elt_at_index (f, n_bytes_per_elt, i); - clib_smp_fifo_data_footer_t * t; + void *d = clib_smp_fifo_elt_at_index (f, n_bytes_per_elt, i); + clib_smp_fifo_data_footer_t *t; memset (d, 0, n_bytes_per_elt_cache_aligned); t = clib_smp_fifo_get_data_footer (d, n_bytes_per_elt); - clib_smp_fifo_data_footer_set_state (t, CLIB_SMP_FIFO_DATA_STATE_free); + clib_smp_fifo_data_footer_set_state (t, + CLIB_SMP_FIFO_DATA_STATE_free); } } return f; } + +/* + * fd.io coding-style-patch-verification: ON + * + * Local Variables: + * eval: (c-set-style "gnu") + * End: + */ |