From patchwork Thu Dec 3 13:30:23 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Devendra Singh Rawat X-Patchwork-Id: 84734 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from dpdk.org (dpdk.org [92.243.14.124]) by inbox.dpdk.org (Postfix) with ESMTP id 18EFEA04B0; Thu, 3 Dec 2020 14:31:09 +0100 (CET) Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id B49C4C954; Thu, 3 Dec 2020 14:31:07 +0100 (CET) Received: from mx0b-0016f401.pphosted.com (mx0a-0016f401.pphosted.com [67.231.148.174]) by dpdk.org (Postfix) with ESMTP id 89516C910; Thu, 3 Dec 2020 14:31:05 +0100 (CET) Received: from pps.filterd (m0045849.ppops.net [127.0.0.1]) by mx0a-0016f401.pphosted.com (8.16.0.43/8.16.0.43) with SMTP id 0B3DPJvd017767; Thu, 3 Dec 2020 05:31:02 -0800 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=marvell.com; h=from : to : cc : subject : date : message-id : mime-version : content-type; s=pfpt0220; bh=RbLP1+2lWsPXPPQ/y8RJhkruNlfW9v1VGLokQcI/zI0=; b=h064JMWDBbNGGumRbCmaEtPH+Ght7WCzkMnEEsaLKa93q/o8m/B6B/MyIuZBSdgdWcSn CHkf5NamT9uRYK3ijMRbDF3oZT+YMPqC9APpnnd2ruR2L1h6KSZf374l6GuxLWst98ej pdb/KJvJ8ywdG8qsw/EIrgYTGbms1z4EJ0YmNOd0af7cj6T/evwA+IZtuETVV5O+bobJ QRtwzFio6kwCkqohqHDCUx61CdzpaJ15yjkyEIC6mrgHRc11EpKShBrnN3HjKZALWegy azLQJg1or+9pEY1GY3JnvXh1buqWp69S9QH7CsRRirT+dBoUpwSKeOm8LEjvg4gSl0WG Aw== Received: from sc-exch03.marvell.com ([199.233.58.183]) by mx0a-0016f401.pphosted.com with ESMTP id 3568jfc863-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-SHA384 bits=256 verify=NOT); Thu, 03 Dec 2020 05:31:01 -0800 Received: from DC5-EXCH02.marvell.com (10.69.176.39) by SC-EXCH03.marvell.com (10.93.176.83) with Microsoft SMTP Server (TLS) id 15.0.1497.2; Thu, 3 Dec 2020 05:31:00 -0800 Received: from maili.marvell.com (10.69.176.80) by DC5-EXCH02.marvell.com (10.69.176.39) with Microsoft SMTP Server id 15.0.1497.2 via Frontend Transport; Thu, 3 Dec 2020 05:31:00 -0800 Received: from ahp-smb-share.punelab.qlogic.com (unknown [10.30.45.239]) by maili.marvell.com (Postfix) with ESMTP id 00B923F7043; Thu, 3 Dec 2020 05:30:57 -0800 (PST) From: Devendra Singh Rawat To: , , CC: , , , , Devendra Singh Rawat , Date: Thu, 3 Dec 2020 19:00:23 +0530 Message-ID: <20201203133023.4060250-1-dsinghrawat@marvell.com> X-Mailer: git-send-email 2.18.2 MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.312, 18.0.737 definitions=2020-12-03_07:2020-12-03, 2020-12-03 signatures=0 Subject: [dpdk-dev] [PATCH] net/qede: change driver's default load parameter 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" Loading ecore clients (PMD or linux kernel driver) having different ecore versions concurrently, is problematic from device's init/config perspective. This fix changes driver's default load parameter to have ecore version validated by MFW. All ecore clients having matching ecore version should be allowed to load, otherwise MFW should fail the load request. Fixes: c5e1108920 ("alter driver's force load behavior") Cc: stable@dpdk.org Signed-off-by: Devendra Singh Rawat Signed-off-by: Manish Chopra Signed-off-by: Igor Russkikh --- drivers/net/qede/qede_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/net/qede/qede_main.c b/drivers/net/qede/qede_main.c index caa9d1d4f..d208d276a 100644 --- a/drivers/net/qede/qede_main.c +++ b/drivers/net/qede/qede_main.c @@ -300,7 +300,7 @@ static int qed_slowpath_start(struct ecore_dev *edev, memset(&drv_load_params, 0, sizeof(drv_load_params)); drv_load_params.mfw_timeout_val = ECORE_LOAD_REQ_LOCK_TO_DEFAULT; drv_load_params.avoid_eng_reset = false; - drv_load_params.override_force_load = ECORE_OVERRIDE_FORCE_LOAD_ALWAYS; + drv_load_params.override_force_load = ECORE_OVERRIDE_FORCE_LOAD_NONE; hw_init_params.avoid_eng_affin = false; hw_init_params.p_drv_load_params = &drv_load_params;