net/ice/base: reduce adding advanced rule time
While calling ice_find_recp we were calling ice_get_recp_to_prof_map every time we called ice_find_recp. ice_get_recp_to_prof_map is a very expensive operation and we should try to reduce the number of times we call this function. So moved it into ice_get_recp_frm_fw since we only need to have fresh recp_to_profile mapping when we we check FW to see if the recipe we are trying to add already exists in FW. Signed-off-by: Shivanshu Shukla <shivanshu.shukla@intel.com> Signed-off-by: Paul M Stillwell Jr <paul.m.stillwell.jr@intel.com> Signed-off-by: Leyi Rong <leyi.rong@intel.com> Acked-by: Qi Zhang <qi.z.zhang@intel.com>
This commit is contained in:
parent
776b48c112
commit
95d3619b09
@ -248,6 +248,8 @@ static ice_declare_bitmap(recipe_to_profile[ICE_MAX_NUM_RECIPES],
|
||||
ICE_MAX_NUM_PROFILES);
|
||||
static ice_declare_bitmap(available_result_ids, ICE_CHAIN_FV_INDEX_START + 1);
|
||||
|
||||
static void ice_get_recp_to_prof_map(struct ice_hw *hw);
|
||||
|
||||
/**
|
||||
* ice_get_recp_frm_fw - update SW bookkeeping from FW recipe entries
|
||||
* @hw: pointer to hardware structure
|
||||
@ -269,6 +271,10 @@ ice_get_recp_frm_fw(struct ice_hw *hw, struct ice_sw_recipe *recps, u8 rid)
|
||||
struct ice_prot_lkup_ext *lkup_exts;
|
||||
enum ice_status status;
|
||||
|
||||
/* Get recipe to profile map so that we can get the fv from
|
||||
* lkups that we read for a recipe from FW.
|
||||
*/
|
||||
ice_get_recp_to_prof_map(hw);
|
||||
/* we need a buffer big enough to accommodate all the recipes */
|
||||
tmp = (struct ice_aqc_recipe_data_elem *)ice_calloc(hw,
|
||||
ICE_MAX_NUM_RECIPES, sizeof(*tmp));
|
||||
@ -4435,7 +4441,6 @@ static u16 ice_find_recp(struct ice_hw *hw, struct ice_prot_lkup_ext *lkup_exts)
|
||||
struct ice_sw_recipe *recp;
|
||||
u16 i;
|
||||
|
||||
ice_get_recp_to_prof_map(hw);
|
||||
/* Initialize available_result_ids which tracks available result idx */
|
||||
for (i = 0; i <= ICE_CHAIN_FV_INDEX_START; i++)
|
||||
ice_set_bit(ICE_CHAIN_FV_INDEX_START - i,
|
||||
|
Loading…
Reference in New Issue
Block a user