From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Mody, Rasesh" Subject: [PATCH 11/18] net/qede/base: add periodic Doorbell Recovery support Date: Sat, 29 Sep 2018 08:14:33 +0000 Message-ID: <1538208822-9726-12-git-send-email-rasesh.mody@cavium.com> References: <1538208822-9726-1-git-send-email-rasesh.mody@cavium.com> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Cc: "Mody, Rasesh" , "ferruh.yigit@intel.com" , Dept-Eng DPDK Dev To: "dev@dpdk.org" Return-path: Received: from NAM02-CY1-obe.outbound.protection.outlook.com (mail-cys01nam02on0063.outbound.protection.outlook.com [104.47.37.63]) by dpdk.org (Postfix) with ESMTP id 323852C37 for ; Sat, 29 Sep 2018 10:14:36 +0200 (CEST) In-Reply-To: <1538208822-9726-1-git-send-email-rasesh.mody@cavium.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" Add support for periodic Doorbell Recovery. Signed-off-by: Rasesh Mody --- drivers/net/qede/base/bcm_osal.h | 1 + drivers/net/qede/base/ecore_int.c | 37 +++++++++++++++++------------= ---- drivers/net/qede/base/ecore_int_api.h | 11 ++++++++++ 3 files changed, 31 insertions(+), 18 deletions(-) diff --git a/drivers/net/qede/base/bcm_osal.h b/drivers/net/qede/base/bcm_o= sal.h index 70805f6..1abf44f 100644 --- a/drivers/net/qede/base/bcm_osal.h +++ b/drivers/net/qede/base/bcm_osal.h @@ -454,5 +454,6 @@ void qede_get_mcp_proto_stats(struct ecore_dev *, enum = ecore_mcp_protocol_type, #define OSAL_DIV_S64(a, b) ((a) / (b)) #define OSAL_LLDP_RX_TLVS(p_hwfn, tlv_buf, tlv_size) nothing #define OSAL_DBG_ALLOC_USER_DATA(p_hwfn, user_data_ptr) (0) +#define OSAL_DB_REC_OCCURRED(p_hwfn) nothing =20 #endif /* __BCM_OSAL_H */ diff --git a/drivers/net/qede/base/ecore_int.c b/drivers/net/qede/base/ecor= e_int.c index c9acc72..fd8f657 100644 --- a/drivers/net/qede/base/ecore_int.c +++ b/drivers/net/qede/base/ecore_int.c @@ -428,13 +428,13 @@ static enum _ecore_status_t ecore_fw_assertion(struct= ecore_hwfn *p_hwfn) #define ECORE_DORQ_ATTENTION_SIZE_MASK (0x7f) #define ECORE_DORQ_ATTENTION_SIZE_SHIFT (16) =20 -#define ECORE_DB_REC_COUNT 10 +#define ECORE_DB_REC_COUNT 1000 #define ECORE_DB_REC_INTERVAL 100 =20 static enum _ecore_status_t ecore_db_rec_flush_queue(struct ecore_hwfn *p_= hwfn, struct ecore_ptt *p_ptt) { - u8 count =3D ECORE_DB_REC_COUNT; + u32 count =3D ECORE_DB_REC_COUNT; u32 usage =3D 1; =20 /* wait for usage to zero or count to run out. This is necessary since @@ -463,12 +463,19 @@ static enum _ecore_status_t ecore_db_rec_flush_queue(= struct ecore_hwfn *p_hwfn, return ECORE_SUCCESS; } =20 -/* assumes sticky overflow indication was set for this PF */ -static enum _ecore_status_t ecore_db_rec_attn(struct ecore_hwfn *p_hwfn, - struct ecore_ptt *p_ptt) +enum _ecore_status_t ecore_db_rec_handler(struct ecore_hwfn *p_hwfn, + struct ecore_ptt *p_ptt) { + u32 overflow; enum _ecore_status_t rc; =20 + overflow =3D ecore_rd(p_hwfn, p_ptt, DORQ_REG_PF_OVFL_STICKY); + DP_NOTICE(p_hwfn, false, "PF Overflow sticky 0x%x\n", overflow); + if (!overflow) { + ecore_db_recovery_execute(p_hwfn, DB_REC_ONCE); + return ECORE_SUCCESS; + } + if (ecore_edpm_enabled(p_hwfn)) { rc =3D ecore_db_rec_flush_queue(p_hwfn, p_ptt); if (rc !=3D ECORE_SUCCESS) @@ -491,8 +498,7 @@ static enum _ecore_status_t ecore_db_rec_attn(struct ec= ore_hwfn *p_hwfn, =20 static enum _ecore_status_t ecore_dorq_attn_cb(struct ecore_hwfn *p_hwfn) { - u32 int_sts, first_drop_reason, details, address, overflow, - all_drops_reason; + u32 int_sts, first_drop_reason, details, address, all_drops_reason; struct ecore_ptt *p_ptt =3D p_hwfn->p_dpc_ptt; enum _ecore_status_t rc; =20 @@ -518,8 +524,6 @@ static enum _ecore_status_t ecore_dorq_attn_cb(struct e= core_hwfn *p_hwfn) DORQ_REG_DB_DROP_DETAILS); address =3D ecore_rd(p_hwfn, p_ptt, DORQ_REG_DB_DROP_DETAILS_ADDRESS); - overflow =3D ecore_rd(p_hwfn, p_ptt, - DORQ_REG_PF_OVFL_STICKY); all_drops_reason =3D ecore_rd(p_hwfn, p_ptt, DORQ_REG_DB_DROP_DETAILS_REASON); =20 @@ -530,19 +534,16 @@ static enum _ecore_status_t ecore_dorq_attn_cb(struct= ecore_hwfn *p_hwfn) "FID\t\t0x%04x\t\t(Opaque FID)\n" "Size\t\t0x%04x\t\t(in bytes)\n" "1st drop reason\t0x%08x\t(details on first drop since last handling)= \n" - "Sticky reasons\t0x%08x\t(all drop reasons since last handling)\n" - "Overflow\t0x%x\t\t(a per PF indication)\n", + "Sticky reasons\t0x%08x\t(all drop reasons since last handling)\n", address, GET_FIELD(details, ECORE_DORQ_ATTENTION_OPAQUE), GET_FIELD(details, ECORE_DORQ_ATTENTION_SIZE) * 4, - first_drop_reason, all_drops_reason, overflow); + first_drop_reason, all_drops_reason); =20 - /* if this PF caused overflow, initiate recovery */ - if (overflow) { - rc =3D ecore_db_rec_attn(p_hwfn, p_ptt); - if (rc !=3D ECORE_SUCCESS) - return rc; - } + rc =3D ecore_db_rec_handler(p_hwfn, p_ptt); + OSAL_DB_REC_OCCURRED(p_hwfn); + if (rc !=3D ECORE_SUCCESS) + return rc; =20 /* clear the doorbell drop details and prepare for next drop */ ecore_wr(p_hwfn, p_ptt, DORQ_REG_DB_DROP_DETAILS_REL, 0); diff --git a/drivers/net/qede/base/ecore_int_api.h b/drivers/net/qede/base/= ecore_int_api.h index aeaf469..5b9c31d 100644 --- a/drivers/net/qede/base/ecore_int_api.h +++ b/drivers/net/qede/base/ecore_int_api.h @@ -343,4 +343,15 @@ enum _ecore_status_t ecore_int_get_sb_dbg(struct ecore= _hwfn *p_hwfn, enum _ecore_status_t ecore_int_igu_relocate_sb(struct ecore_hwfn *p_hwfn, struct ecore_ptt *p_p= tt, u16 sb_id, bool b_to_vf); + +/** + * @brief - Doorbell Recovery handler. + * Run DB_REAL_DEAL doorbell recovery in case of PF overflow + * (and flush DORQ if needed), otherwise run DB_REC_ONCE. + * + * @param p_hwfn + * @param p_ptt + */ +enum _ecore_status_t ecore_db_rec_handler(struct ecore_hwfn *p_hwfn, + struct ecore_ptt *p_ptt); #endif --=20 1.7.10.3