From bb3768842e81c76c7cb390b71a0cc29d7098dcec Mon Sep 17 00:00:00 2001 From: Eyal Bari Date: Thu, 23 Aug 2018 12:16:41 +0300 Subject: l2:bvi-dont aceess sup hw int for broadcasts sup hw int is needed only for unicast validation Change-Id: I5e5753c09d1c16fdb2435b4db5628a2379fe6f96 Signed-off-by: Eyal Bari --- src/vnet/l2/l2_bvi.h | 31 +++++++++++-------------------- 1 file changed, 11 insertions(+), 20 deletions(-) (limited to 'src/vnet/l2/l2_bvi.h') diff --git a/src/vnet/l2/l2_bvi.h b/src/vnet/l2/l2_bvi.h index 662ec402a87..cc86072fab7 100644 --- a/src/vnet/l2/l2_bvi.h +++ b/src/vnet/l2/l2_bvi.h @@ -40,31 +40,25 @@ l2_to_bvi (vlib_main_t * vlib_main, vlib_buffer_t * b0, u32 bvi_sw_if_index, next_by_ethertype_t * l3_next, u32 * next0) { - u8 l2_len; - u16 ethertype; - u8 *l3h; - ethernet_header_t *e0; - vnet_hw_interface_t *hi; - - e0 = vlib_buffer_get_current (b0); - hi = vnet_get_sup_hw_interface (vnet_main, bvi_sw_if_index); - /* Perform L3 my-mac filter */ - if ((!ethernet_address_cast (e0->dst_address)) && - (!eth_mac_equal ((u8 *) e0, hi->hw_address))) + ethernet_header_t *e0 = vlib_buffer_get_current (b0); + if (!ethernet_address_cast (e0->dst_address)) { - return TO_BVI_ERR_BAD_MAC; + vnet_hw_interface_t *hi = + vnet_get_sup_hw_interface (vnet_main, bvi_sw_if_index); + if (!eth_mac_equal (e0->dst_address, hi->hw_address)) + return TO_BVI_ERR_BAD_MAC; } /* Save L2 header position which may be changed due to packet replication */ vnet_buffer (b0)->l2_hdr_offset = b0->current_data; /* Strip L2 header */ - l2_len = vnet_buffer (b0)->l2.l2_len; + u8 l2_len = vnet_buffer (b0)->l2.l2_len; vlib_buffer_advance (b0, l2_len); - l3h = vlib_buffer_get_current (b0); - ethertype = clib_net_to_host_u16 (*(u16 *) (l3h - 2)); + u8 *l3h = vlib_buffer_get_current (b0); + u16 ethertype = clib_net_to_host_u16 (*(u16 *) (l3h - 2)); /* Set the input interface to be the BVI interface */ vnet_buffer (b0)->sw_if_index[VLIB_RX] = bvi_sw_if_index; @@ -82,9 +76,7 @@ l2_to_bvi (vlib_main_t * vlib_main, else { /* uncommon ethertype, check table */ - u32 i0; - - i0 = sparse_vec_index (l3_next->input_next_by_type, ethertype); + u32 i0 = sparse_vec_index (l3_next->input_next_by_type, ethertype); *next0 = vec_elt (l3_next->input_next_by_type, i0); if (i0 == SPARSE_VEC_INVALID_INDEX) @@ -97,8 +89,7 @@ l2_to_bvi (vlib_main_t * vlib_main, vlib_increment_combined_counter (vnet_main->interface_main.combined_sw_if_counters + VNET_INTERFACE_COUNTER_RX, - vlib_main->thread_index, - vnet_buffer (b0)->sw_if_index[VLIB_RX], + vlib_main->thread_index, bvi_sw_if_index, 1, vlib_buffer_length_in_chain (vlib_main, b0)); return TO_BVI_ERR_OK; } -- cgit 1.2.3-korg