From 143610014ab91f7998014f4322c36c8b07f50d5f Mon Sep 17 00:00:00 2001 From: Damjan Marion Date: Thu, 11 Mar 2021 12:17:33 +0100 Subject: [PATCH] misc: fix gcc-11 build Type: fix Change-Id: Ia17c8255806a2575bb75ed37050b47ddb347050c Signed-off-by: Damjan Marion --- src/plugins/lisp/lisp-gpe/lisp_gpe_fwd_entry.h | 4 +-- src/plugins/mactime/builtins.c | 8 ++--- src/vlib/pci/pci.c | 29 +++++++--------- src/vlib/threads.c | 48 +++++++++++--------------- 4 files changed, 38 insertions(+), 51 deletions(-) diff --git a/src/plugins/lisp/lisp-gpe/lisp_gpe_fwd_entry.h b/src/plugins/lisp/lisp-gpe/lisp_gpe_fwd_entry.h index a170787f5b2..322580007c4 100644 --- a/src/plugins/lisp/lisp-gpe/lisp_gpe_fwd_entry.h +++ b/src/plugins/lisp/lisp-gpe/lisp_gpe_fwd_entry.h @@ -212,8 +212,8 @@ vnet_lisp_gpe_add_del_fwd_entry (vnet_lisp_gpe_add_del_fwd_entry_args_t * a, extern void vnet_lisp_gpe_fwd_entry_flush (void); -extern u32 lisp_l2_fib_lookup (lisp_gpe_main_t * lgm, - u16 bd_index, u8 src_mac[8], u8 dst_mac[8]); +extern u32 lisp_l2_fib_lookup (lisp_gpe_main_t *lgm, u16 bd_index, + u8 src_mac[6], u8 dst_mac[6]); extern const dpo_id_t *lisp_nsh_fib_lookup (lisp_gpe_main_t * lgm, u32 spi_si); diff --git a/src/plugins/mactime/builtins.c b/src/plugins/mactime/builtins.c index d7fd7a0b32b..321502454a2 100644 --- a/src/plugins/mactime/builtins.c +++ b/src/plugins/mactime/builtins.c @@ -42,12 +42,10 @@ handle_get_mactime (http_builtin_method_type_t reqtype, if (PREDICT_FALSE ((now - mm->sunday_midnight) > 86400.0 * 7.0)) mm->sunday_midnight = clib_timebase_find_sunday_midnight (now); - /* *INDENT-OFF* */ - pool_foreach (dp, mm->devices) - { - vec_add1 (pool_indices, dp - mm->devices); + pool_foreach (dp, mm->devices) + { + vec_add1 (pool_indices, dp - mm->devices); } - /* *INDENT-ON* */ s = format (s, "{%smactime%s: [\n", q, q); diff --git a/src/vlib/pci/pci.c b/src/vlib/pci/pci.c index 7e96e1f7c6e..1c1f4b636f5 100644 --- a/src/vlib/pci/pci.c +++ b/src/vlib/pci/pci.c @@ -88,7 +88,7 @@ show_pci_fn (vlib_main_t * vm, "Product Name", "Vital Product Data"); addrs = vlib_pci_get_all_dev_addrs (); - /* *INDENT-OFF* */ + vec_foreach (addr, addrs) { vlib_pci_device_info_t *d; @@ -98,22 +98,19 @@ show_pci_fn (vlib_main_t * vm, continue; if (d->device_class != PCI_CLASS_NETWORK_ETHERNET && !show_all) - continue; - - vec_reset_length (s); - if (d->numa_node >= 0) - s = format (s, " %d", d->numa_node); - - vlib_cli_output (vm, "%-13U%-5v%04x:%04x %-14U%-16s%-32v%U", - format_vlib_pci_addr, addr, s, - d->vendor_id, d->device_id, - format_vlib_pci_link_speed, d, - d->driver_name ? (char *) d->driver_name : "", - d->product_name, - format_vlib_pci_vpd, d->vpd_r, (u8 *) 0); - vlib_pci_free_device_info (d); + continue; + + vec_reset_length (s); + if (d->numa_node >= 0) + s = format (s, " %d", d->numa_node); + + vlib_cli_output ( + vm, "%-13U%-5v%04x:%04x %-14U%-16s%-32v%U", format_vlib_pci_addr, + addr, s, d->vendor_id, d->device_id, format_vlib_pci_link_speed, d, + d->driver_name ? (char *) d->driver_name : "", d->product_name, + format_vlib_pci_vpd, d->vpd_r, (u8 *) 0); + vlib_pci_free_device_info (d); } - /* *INDENT-ON* */ vec_free (s); vec_free (addrs); diff --git a/src/vlib/threads.c b/src/vlib/threads.c index ea63653c53c..48e1bab8a66 100644 --- a/src/vlib/threads.c +++ b/src/vlib/threads.c @@ -48,13 +48,11 @@ barrier_trace_sync (f64 t_entry, f64 t_open, f64 t_closed) if (!vlib_worker_threads->barrier_elog_enabled) return; - /* *INDENT-OFF* */ - ELOG_TYPE_DECLARE (e) = - { - .format = "bar-trace-%s-#%d", - .format_args = "T4i4", - }; - /* *INDENT-ON* */ + ELOG_TYPE_DECLARE (e) = { + .format = "bar-trace-%s-#%d", + .format_args = "T4i4", + }; + struct { u32 caller, count, t_entry, t_open, t_closed; @@ -75,13 +73,11 @@ barrier_trace_sync_rec (f64 t_entry) if (!vlib_worker_threads->barrier_elog_enabled) return; - /* *INDENT-OFF* */ - ELOG_TYPE_DECLARE (e) = - { - .format = "bar-syncrec-%s-#%d", - .format_args = "T4i4", - }; - /* *INDENT-ON* */ + ELOG_TYPE_DECLARE (e) = { + .format = "bar-syncrec-%s-#%d", + .format_args = "T4i4", + }; + struct { u32 caller, depth; @@ -99,13 +95,11 @@ barrier_trace_release_rec (f64 t_entry) if (!vlib_worker_threads->barrier_elog_enabled) return; - /* *INDENT-OFF* */ - ELOG_TYPE_DECLARE (e) = - { - .format = "bar-relrrec-#%d", - .format_args = "i4", - }; - /* *INDENT-ON* */ + ELOG_TYPE_DECLARE (e) = { + .format = "bar-relrrec-#%d", + .format_args = "i4", + }; + struct { u32 depth; @@ -121,13 +115,11 @@ barrier_trace_release (f64 t_entry, f64 t_closed_total, f64 t_update_main) if (!vlib_worker_threads->barrier_elog_enabled) return; - /* *INDENT-OFF* */ - ELOG_TYPE_DECLARE (e) = - { - .format = "bar-rel-#%d-e%d-u%d-t%d", - .format_args = "i4i4i4i4", - }; - /* *INDENT-ON* */ + ELOG_TYPE_DECLARE (e) = { + .format = "bar-rel-#%d-e%d-u%d-t%d", + .format_args = "i4i4i4i4", + }; + struct { u32 count, t_entry, t_update_main, t_closed_total; -- 2.16.6