Commit ea71b967 authored by Martyna Szapar-Mudlaw's avatar Martyna Szapar-Mudlaw Committed by Tony Nguyen

ice: Add support for VLAN TPID filters in switchdev

Enable support for adding TC rules that filter on the VLAN tag type
in switchdev mode.
Signed-off-by: default avatarMartyna Szapar-Mudlaw <martyna.szapar-mudlaw@intel.com>
Tested-by: default avatarSandeep Penigalapati <sandeep.penigalapati@intel.com>
Signed-off-by: default avatarTony Nguyen <anthony.l.nguyen@intel.com>
parent 06bca7c2
...@@ -111,13 +111,18 @@ enum ice_prot_id { ...@@ -111,13 +111,18 @@ enum ice_prot_id {
#define ICE_GRE_OF_HW 64 #define ICE_GRE_OF_HW 64
#define ICE_UDP_OF_HW 52 /* UDP Tunnels */ #define ICE_UDP_OF_HW 52 /* UDP Tunnels */
#define ICE_META_DATA_ID_HW 255 /* this is used for tunnel type */ #define ICE_META_DATA_ID_HW 255 /* this is used for tunnel and VLAN type */
#define ICE_MDID_SIZE 2 #define ICE_MDID_SIZE 2
#define ICE_TUN_FLAG_MDID 21 #define ICE_TUN_FLAG_MDID 21
#define ICE_TUN_FLAG_MDID_OFF (ICE_MDID_SIZE * ICE_TUN_FLAG_MDID) #define ICE_TUN_FLAG_MDID_OFF (ICE_MDID_SIZE * ICE_TUN_FLAG_MDID)
#define ICE_TUN_FLAG_MASK 0xFF #define ICE_TUN_FLAG_MASK 0xFF
#define ICE_VLAN_FLAG_MDID 20
#define ICE_VLAN_FLAG_MDID_OFF (ICE_MDID_SIZE * ICE_VLAN_FLAG_MDID)
#define ICE_PKT_FLAGS_0_TO_15_VLAN_FLAGS_MASK 0xD000
#define ICE_TUN_FLAG_FV_IND 2 #define ICE_TUN_FLAG_FV_IND 2
/* Mapping of software defined protocol ID to hardware defined protocol ID */ /* Mapping of software defined protocol ID to hardware defined protocol ID */
......
...@@ -5536,10 +5536,11 @@ static bool ice_tun_type_match_word(enum ice_sw_tunnel_type tun_type, u16 *mask) ...@@ -5536,10 +5536,11 @@ static bool ice_tun_type_match_word(enum ice_sw_tunnel_type tun_type, u16 *mask)
* ice_add_special_words - Add words that are not protocols, such as metadata * ice_add_special_words - Add words that are not protocols, such as metadata
* @rinfo: other information regarding the rule e.g. priority and action info * @rinfo: other information regarding the rule e.g. priority and action info
* @lkup_exts: lookup word structure * @lkup_exts: lookup word structure
* @dvm_ena: is double VLAN mode enabled
*/ */
static int static int
ice_add_special_words(struct ice_adv_rule_info *rinfo, ice_add_special_words(struct ice_adv_rule_info *rinfo,
struct ice_prot_lkup_ext *lkup_exts) struct ice_prot_lkup_ext *lkup_exts, bool dvm_ena)
{ {
u16 mask; u16 mask;
...@@ -5558,6 +5559,19 @@ ice_add_special_words(struct ice_adv_rule_info *rinfo, ...@@ -5558,6 +5559,19 @@ ice_add_special_words(struct ice_adv_rule_info *rinfo,
} }
} }
if (rinfo->vlan_type != 0 && dvm_ena) {
if (lkup_exts->n_val_words < ICE_MAX_CHAIN_WORDS) {
u8 word = lkup_exts->n_val_words++;
lkup_exts->fv_words[word].prot_id = ICE_META_DATA_ID_HW;
lkup_exts->fv_words[word].off = ICE_VLAN_FLAG_MDID_OFF;
lkup_exts->field_mask[word] =
ICE_PKT_FLAGS_0_TO_15_VLAN_FLAGS_MASK;
} else {
return -ENOSPC;
}
}
return 0; return 0;
} }
...@@ -5677,7 +5691,7 @@ ice_add_adv_recipe(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups, ...@@ -5677,7 +5691,7 @@ ice_add_adv_recipe(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups,
/* Create any special protocol/offset pairs, such as looking at tunnel /* Create any special protocol/offset pairs, such as looking at tunnel
* bits by extracting metadata * bits by extracting metadata
*/ */
status = ice_add_special_words(rinfo, lkup_exts); status = ice_add_special_words(rinfo, lkup_exts, ice_is_dvm_ena(hw));
if (status) if (status)
goto err_free_lkup_exts; goto err_free_lkup_exts;
...@@ -6010,6 +6024,36 @@ ice_fill_adv_packet_tun(struct ice_hw *hw, enum ice_sw_tunnel_type tun_type, ...@@ -6010,6 +6024,36 @@ ice_fill_adv_packet_tun(struct ice_hw *hw, enum ice_sw_tunnel_type tun_type,
return -EIO; return -EIO;
} }
/**
* ice_fill_adv_packet_vlan - fill dummy packet with VLAN tag type
* @vlan_type: VLAN tag type
* @pkt: dummy packet to fill in
* @offsets: offset info for the dummy packet
*/
static int
ice_fill_adv_packet_vlan(u16 vlan_type, u8 *pkt,
const struct ice_dummy_pkt_offsets *offsets)
{
u16 i;
/* Find VLAN header and insert VLAN TPID */
for (i = 0; offsets[i].type != ICE_PROTOCOL_LAST; i++) {
if (offsets[i].type == ICE_VLAN_OFOS ||
offsets[i].type == ICE_VLAN_EX) {
struct ice_vlan_hdr *hdr;
u16 offset;
offset = offsets[i].offset;
hdr = (struct ice_vlan_hdr *)&pkt[offset];
hdr->type = cpu_to_be16(vlan_type);
return 0;
}
}
return -EIO;
}
/** /**
* ice_find_adv_rule_entry - Search a rule entry * ice_find_adv_rule_entry - Search a rule entry
* @hw: pointer to the hardware structure * @hw: pointer to the hardware structure
...@@ -6045,6 +6089,7 @@ ice_find_adv_rule_entry(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups, ...@@ -6045,6 +6089,7 @@ ice_find_adv_rule_entry(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups,
} }
if (rinfo->sw_act.flag == list_itr->rule_info.sw_act.flag && if (rinfo->sw_act.flag == list_itr->rule_info.sw_act.flag &&
rinfo->tun_type == list_itr->rule_info.tun_type && rinfo->tun_type == list_itr->rule_info.tun_type &&
rinfo->vlan_type == list_itr->rule_info.vlan_type &&
lkups_matched) lkups_matched)
return list_itr; return list_itr;
} }
...@@ -6333,6 +6378,14 @@ ice_add_adv_rule(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups, ...@@ -6333,6 +6378,14 @@ ice_add_adv_rule(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups,
goto err_ice_add_adv_rule; goto err_ice_add_adv_rule;
} }
if (rinfo->vlan_type != 0 && ice_is_dvm_ena(hw)) {
status = ice_fill_adv_packet_vlan(rinfo->vlan_type,
s_rule->hdr_data,
profile->offsets);
if (status)
goto err_ice_add_adv_rule;
}
status = ice_aq_sw_rules(hw, (struct ice_aqc_sw_rules *)s_rule, status = ice_aq_sw_rules(hw, (struct ice_aqc_sw_rules *)s_rule,
rule_buf_sz, 1, ice_aqc_opc_add_sw_rules, rule_buf_sz, 1, ice_aqc_opc_add_sw_rules,
NULL); NULL);
...@@ -6570,7 +6623,7 @@ ice_rem_adv_rule(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups, ...@@ -6570,7 +6623,7 @@ ice_rem_adv_rule(struct ice_hw *hw, struct ice_adv_lkup_elem *lkups,
/* Create any special protocol/offset pairs, such as looking at tunnel /* Create any special protocol/offset pairs, such as looking at tunnel
* bits by extracting metadata * bits by extracting metadata
*/ */
status = ice_add_special_words(rinfo, &lkup_exts); status = ice_add_special_words(rinfo, &lkup_exts, ice_is_dvm_ena(hw));
if (status) if (status)
return status; return status;
......
...@@ -192,6 +192,7 @@ struct ice_adv_rule_info { ...@@ -192,6 +192,7 @@ struct ice_adv_rule_info {
u32 priority; u32 priority;
u8 rx; /* true means LOOKUP_RX otherwise LOOKUP_TX */ u8 rx; /* true means LOOKUP_RX otherwise LOOKUP_TX */
u16 fltr_rule_id; u16 fltr_rule_id;
u16 vlan_type;
struct ice_adv_rule_flags_info flags_info; struct ice_adv_rule_flags_info flags_info;
}; };
......
...@@ -138,6 +138,18 @@ ice_sw_type_from_tunnel(enum ice_tunnel_type type) ...@@ -138,6 +138,18 @@ ice_sw_type_from_tunnel(enum ice_tunnel_type type)
} }
} }
static u16 ice_check_supported_vlan_tpid(u16 vlan_tpid)
{
switch (vlan_tpid) {
case ETH_P_8021Q:
case ETH_P_8021AD:
case ETH_P_QINQ1:
return vlan_tpid;
default:
return 0;
}
}
static int static int
ice_tc_fill_tunnel_outer(u32 flags, struct ice_tc_flower_fltr *fltr, ice_tc_fill_tunnel_outer(u32 flags, struct ice_tc_flower_fltr *fltr,
struct ice_adv_lkup_elem *list) struct ice_adv_lkup_elem *list)
...@@ -273,8 +285,11 @@ ice_tc_fill_rules(struct ice_hw *hw, u32 flags, ...@@ -273,8 +285,11 @@ ice_tc_fill_rules(struct ice_hw *hw, u32 flags,
{ {
struct ice_tc_flower_lyr_2_4_hdrs *headers = &tc_fltr->outer_headers; struct ice_tc_flower_lyr_2_4_hdrs *headers = &tc_fltr->outer_headers;
bool inner = false; bool inner = false;
u16 vlan_tpid = 0;
int i = 0; int i = 0;
rule_info->vlan_type = vlan_tpid;
rule_info->tun_type = ice_sw_type_from_tunnel(tc_fltr->tunnel_type); rule_info->tun_type = ice_sw_type_from_tunnel(tc_fltr->tunnel_type);
if (tc_fltr->tunnel_type != TNL_LAST) { if (tc_fltr->tunnel_type != TNL_LAST) {
i = ice_tc_fill_tunnel_outer(flags, tc_fltr, list); i = ice_tc_fill_tunnel_outer(flags, tc_fltr, list);
...@@ -315,6 +330,10 @@ ice_tc_fill_rules(struct ice_hw *hw, u32 flags, ...@@ -315,6 +330,10 @@ ice_tc_fill_rules(struct ice_hw *hw, u32 flags,
/* copy VLAN info */ /* copy VLAN info */
if (flags & ICE_TC_FLWR_FIELD_VLAN) { if (flags & ICE_TC_FLWR_FIELD_VLAN) {
vlan_tpid = be16_to_cpu(headers->vlan_hdr.vlan_tpid);
rule_info->vlan_type =
ice_check_supported_vlan_tpid(vlan_tpid);
if (flags & ICE_TC_FLWR_FIELD_CVLAN) if (flags & ICE_TC_FLWR_FIELD_CVLAN)
list[i].type = ICE_VLAN_EX; list[i].type = ICE_VLAN_EX;
else else
...@@ -1075,6 +1094,8 @@ ice_parse_cls_flower(struct net_device *filter_dev, struct ice_vsi *vsi, ...@@ -1075,6 +1094,8 @@ ice_parse_cls_flower(struct net_device *filter_dev, struct ice_vsi *vsi,
cpu_to_be16(match.key->vlan_id & VLAN_VID_MASK); cpu_to_be16(match.key->vlan_id & VLAN_VID_MASK);
if (match.mask->vlan_priority) if (match.mask->vlan_priority)
headers->vlan_hdr.vlan_prio = match.key->vlan_priority; headers->vlan_hdr.vlan_prio = match.key->vlan_priority;
if (match.mask->vlan_tpid)
headers->vlan_hdr.vlan_tpid = match.key->vlan_tpid;
} }
if (flow_rule_match_key(rule, FLOW_DISSECTOR_KEY_CVLAN)) { if (flow_rule_match_key(rule, FLOW_DISSECTOR_KEY_CVLAN)) {
......
...@@ -41,6 +41,7 @@ struct ice_tc_flower_action { ...@@ -41,6 +41,7 @@ struct ice_tc_flower_action {
struct ice_tc_vlan_hdr { struct ice_tc_vlan_hdr {
__be16 vlan_id; /* Only last 12 bits valid */ __be16 vlan_id; /* Only last 12 bits valid */
u16 vlan_prio; /* Only last 3 bits valid (valid values: 0..7) */ u16 vlan_prio; /* Only last 3 bits valid (valid values: 0..7) */
__be16 vlan_tpid;
}; };
struct ice_tc_l2_hdr { struct ice_tc_l2_hdr {
......
...@@ -199,7 +199,6 @@ static bool ice_is_dvm_supported(struct ice_hw *hw) ...@@ -199,7 +199,6 @@ static bool ice_is_dvm_supported(struct ice_hw *hw)
#define ICE_SW_LKUP_VLAN_PKT_FLAGS_LKUP_IDX 2 #define ICE_SW_LKUP_VLAN_PKT_FLAGS_LKUP_IDX 2
#define ICE_SW_LKUP_PROMISC_VLAN_LOC_LKUP_IDX 2 #define ICE_SW_LKUP_PROMISC_VLAN_LOC_LKUP_IDX 2
#define ICE_PKT_FLAGS_0_TO_15_FV_IDX 1 #define ICE_PKT_FLAGS_0_TO_15_FV_IDX 1
#define ICE_PKT_FLAGS_0_TO_15_VLAN_FLAGS_MASK 0xD000
static struct ice_update_recipe_lkup_idx_params ice_dvm_dflt_recipes[] = { static struct ice_update_recipe_lkup_idx_params ice_dvm_dflt_recipes[] = {
{ {
/* Update recipe ICE_SW_LKUP_VLAN to filter based on the /* Update recipe ICE_SW_LKUP_VLAN to filter based on the
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment