From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id A2B9CA04AB; Fri, 8 Nov 2019 07:22:00 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 6AF2E1BED1; Fri, 8 Nov 2019 07:22:00 +0100 (CET) Received: from mga05.intel.com (mga05.intel.com [192.55.52.43]) by dpdk.org (Postfix) with ESMTP id 6B1802142 for ; Fri, 8 Nov 2019 07:21:58 +0100 (CET) X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga002.jf.intel.com ([10.7.209.21]) by fmsmga105.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 07 Nov 2019 22:21:57 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.68,280,1569308400"; d="scan'208";a="214838307" Received: from yexl-server.sh.intel.com (HELO localhost) ([10.67.117.17]) by orsmga002.jf.intel.com with ESMTP; 07 Nov 2019 22:21:56 -0800 Date: Fri, 8 Nov 2019 14:17:49 +0800 From: Ye Xiaolong To: Qi Zhang Cc: qiming.yang@intel.com, dev@dpdk.org Message-ID: <20191108061749.GK100445@intel.com> References: <20191107095910.29854-1-qi.z.zhang@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20191107095910.29854-1-qi.z.zhang@intel.com> User-Agent: Mutt/1.9.4 (2018-02-28) Subject: Re: [dpdk-dev] [PATCH] net/ice: clear the HW tables when close device X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Hi, Qi On 11/07, Qi Zhang wrote: >Clear the HW tables during dev_close. Otherwise HW tables will not >be initialized correctly after device reset. > >Fixes: a4c8c48fe3f4 ("net/ice: load OS default package") > >Signed-off-by: Qi Zhang >--- > drivers/net/ice/base/ice_flex_pipe.c | 55 ++++++++++++++++++++++++++++++++++++ > drivers/net/ice/base/ice_flex_pipe.h | 1 + > drivers/net/ice/ice_ethdev.c | 1 + > 3 files changed, 57 insertions(+) > >diff --git a/drivers/net/ice/base/ice_flex_pipe.c b/drivers/net/ice/base/ice_flex_pipe.c >index dd098f529..b816ca67f 100644 >--- a/drivers/net/ice/base/ice_flex_pipe.c >+++ b/drivers/net/ice/base/ice_flex_pipe.c >@@ -3618,6 +3618,61 @@ static void ice_init_flow_profs(struct ice_hw *hw, u8 blk_idx) > } > > /** >+ * ice_clear_hw_tbls - clear HW tables and flow profiles >+ * @hw: pointer to the hardware structure >+ */ >+void ice_clear_hw_tbls(struct ice_hw *hw) What is the difference between this one and existing ice_free_hw_tbls, can we use ice_free_hw_tbls directly? Thanks, Xiaolong >+{ >+ u8 i; >+ >+ for (i = 0; i < ICE_BLK_COUNT; i++) { >+ struct ice_prof_redir *prof_redir = &hw->blk[i].prof_redir; >+ struct ice_prof_tcam *prof = &hw->blk[i].prof; >+ struct ice_xlt1 *xlt1 = &hw->blk[i].xlt1; >+ struct ice_xlt2 *xlt2 = &hw->blk[i].xlt2; >+ struct ice_es *es = &hw->blk[i].es; >+ >+ if (hw->blk[i].is_list_init) { >+ ice_free_prof_map(hw, i); >+ ice_free_flow_profs(hw, i); >+ } >+ >+ ice_free_vsig_tbl(hw, (enum ice_block)i); >+ >+ ice_memset(xlt1->ptypes, 0, xlt1->count * sizeof(*xlt1->ptypes), >+ ICE_NONDMA_MEM); >+ ice_memset(xlt1->ptg_tbl, 0, >+ ICE_MAX_PTGS * sizeof(*xlt1->ptg_tbl), >+ ICE_NONDMA_MEM); >+ ice_memset(xlt1->t, 0, xlt1->count * sizeof(*xlt1->t), >+ ICE_NONDMA_MEM); >+ >+ ice_memset(xlt2->vsis, 0, xlt2->count * sizeof(*xlt2->vsis), >+ ICE_NONDMA_MEM); >+ ice_memset(xlt2->vsig_tbl, 0, >+ xlt2->count * sizeof(*xlt2->vsig_tbl), >+ ICE_NONDMA_MEM); >+ ice_memset(xlt2->t, 0, xlt2->count * sizeof(*xlt2->t), >+ ICE_NONDMA_MEM); >+ >+ ice_memset(prof->t, 0, prof->count * sizeof(*prof->t), >+ ICE_NONDMA_MEM); >+ ice_memset(prof_redir->t, 0, >+ prof_redir->count * sizeof(*prof_redir->t), >+ ICE_NONDMA_MEM); >+ >+ ice_memset(es->t, 0, es->count * sizeof(*es->t), >+ ICE_NONDMA_MEM); >+ ice_memset(es->ref_count, 0, es->count * sizeof(*es->ref_count), >+ ICE_NONDMA_MEM); >+ ice_memset(es->written, 0, es->count * sizeof(*es->written), >+ ICE_NONDMA_MEM); >+ ice_memset(es->mask_ena, 0, es->count * sizeof(*es->mask_ena), >+ ICE_NONDMA_MEM); >+ } >+} >+ >+/** > * ice_init_hw_tbls - init hardware table memory > * @hw: pointer to the hardware structure > */ >diff --git a/drivers/net/ice/base/ice_flex_pipe.h b/drivers/net/ice/base/ice_flex_pipe.h >index ee606af15..fa72e386d 100644 >--- a/drivers/net/ice/base/ice_flex_pipe.h >+++ b/drivers/net/ice/base/ice_flex_pipe.h >@@ -71,6 +71,7 @@ ice_copy_and_init_pkg(struct ice_hw *hw, const u8 *buf, u32 len); > enum ice_status ice_init_hw_tbls(struct ice_hw *hw); > void ice_free_seg(struct ice_hw *hw); > void ice_fill_blk_tbls(struct ice_hw *hw); >+void ice_clear_hw_tbls(struct ice_hw *hw); > void ice_free_hw_tbls(struct ice_hw *hw); > enum ice_status > ice_rem_prof(struct ice_hw *hw, enum ice_block blk, u64 id); >diff --git a/drivers/net/ice/ice_ethdev.c b/drivers/net/ice/ice_ethdev.c >index d81eb5eeb..0b439cc0b 100644 >--- a/drivers/net/ice/ice_ethdev.c >+++ b/drivers/net/ice/ice_ethdev.c >@@ -2322,6 +2322,7 @@ ice_dev_close(struct rte_eth_dev *dev) > rte_free(hw->port_info); > hw->port_info = NULL; > ice_shutdown_all_ctrlq(hw); >+ ice_clear_hw_tbls(hw); > rte_free(pf->proto_xtr); > pf->proto_xtr = NULL; > >-- >2.13.6 >