aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorEyal Bari <ebari@cisco.com>2018-08-23 12:16:41 +0300
committerJohn Lo <loj@cisco.com>2018-08-24 13:36:09 +0000
commitbb3768842e81c76c7cb390b71a0cc29d7098dcec (patch)
tree98258204987fa97650d569ef1ffb90b54aef7738 /src
parentaf7dd5b32eca5c924695637bd3b922e6793b1d8a (diff)
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 <ebari@cisco.com>
Diffstat (limited to 'src')
-rw-r--r--src/vnet/l2/l2_bvi.h31
1 files changed, 11 insertions, 20 deletions
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;
}