From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 7B133A0093; Fri, 1 Jul 2022 04:21:17 +0200 (CEST) Received: from [217.70.189.124] (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 7281042905; Fri, 1 Jul 2022 04:21:12 +0200 (CEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2041.outbound.protection.outlook.com [40.107.236.41]) by mails.dpdk.org (Postfix) with ESMTP id 1DF0C40A80 for ; Fri, 1 Jul 2022 04:21:10 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Sou0GYo5HkWapvGIJNBxm9FbBMnWZ9iBfIyen67rZUnqTMzn/g3sSosFJxA2qJYHGD3/ISl4fqg+7iYQrtn+ODslcndpUA3GNa74/b/aHT6kfRxgO3sf40g98utgGEDwoIZU/eo+6jC79xNJU+18xjh78wfBMlLgH27iXWumcnl+kvs44TBvlgfNQfGkOCFkOZUwP0j7dnTNbOFWnWNNAqh37a9Ic1SLqankFEfhspYBrGVn6BxMfhs3AvcHnUB5+/CB9RX/5DVS6cyc1KGJaoGGIawmDLjy7NPi3h8HWvmLWEm451pNnBNgQfOB22xf1P1t7JcwPQkoJTWtPyPo1w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=Ft/XwXnwub++Lq508dydo4EZa+ijmdHevGF9s4Hph+s=; b=KSAQXUuZOCy73cO5kSSnW4+BMf02Xyn1imb9OZe7dBog78WFGD59Z6ewNdk+B2Rq/e1J0svVQCZUyOERtJXYBvXzMIJKsKDkX/PYtmr6x1GWeVko/RMaUz+J7XjYOlkDoGULh3kYHTLnlhQb8z9mKuvck0IcMbZHsGTtiRGYuwKII0zBzyTBeVxUKf8RcdRpqfOf19RHxNYapA1Hc8Bx6WAQ+9vor39DVjFRty7u4kMrYTci19Eblj8UEz2iAbDnE8t+ogamDIiviqQ3Eh74SQMXW9ld6AyJehkgEjQjZB9zuai8iYCgCE2zOs5ijAXMHaqmV2wGGp8PbC9ledu5Vw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=corigine.com; dmarc=pass action=none header.from=corigine.com; dkim=pass header.d=corigine.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=corigine.onmicrosoft.com; s=selector2-corigine-onmicrosoft-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=Ft/XwXnwub++Lq508dydo4EZa+ijmdHevGF9s4Hph+s=; b=lUIY4t5F6sL1rCzaawR+wzpmwUsZGDdvi39Qj7+K10RTYq9LoeS78qqzrmWK4uzrL6OXam6G1tF1nJL8oLTfkl8uOudvWc3kDDNWUMpmIULwEqQFM6Tn5QZS1hqIL6zEX33RC0N5ZjFE2hmzeeLalH8vGGktw372uCgE+GjgbkI= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=corigine.com; Received: from SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) by CH2PR13MB4459.namprd13.prod.outlook.com (2603:10b6:610:35::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5395.11; Fri, 1 Jul 2022 02:21:09 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::5d9e:9ce5:51d1:2021]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::5d9e:9ce5:51d1:2021%9]) with mapi id 15.20.5395.014; Fri, 1 Jul 2022 02:21:09 +0000 From: Chaoyong He To: dev@dpdk.org Cc: niklas.soderlund@corigine.com, Chaoyong He Subject: [PATCH v4 02/12] net/nfp: simplify initialization and remove dead code Date: Fri, 1 Jul 2022 10:20:07 +0800 Message-Id: <1656642017-8992-3-git-send-email-chaoyong.he@corigine.com> X-Mailer: git-send-email 1.8.3.1 In-Reply-To: <1656642017-8992-1-git-send-email-chaoyong.he@corigine.com> References: <1656642017-8992-1-git-send-email-chaoyong.he@corigine.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-ClientProxiedBy: SG2PR02CA0079.apcprd02.prod.outlook.com (2603:1096:4:90::19) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 445c4bf7-22ac-4168-bcf0-08da5b085c08 X-MS-TrafficTypeDiagnostic: CH2PR13MB4459:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: IB/qBXjfrLyoxHXo0y30pfhG930pDFmqRbm4JCPtHYEijJaEp3cSurObHEOTOUsgfXefw+Ca/l4hm7dDKq+TNIOxapwnSbzS2FeRR+VP8kFluZArGcNi4ZxuwcaGusJj3ePyuXlaZ2E7ksSzeVCzAid99zcuDzFhWRCHsnxrE8FZQ6rhaKdT4yR1GdYtMA7hC1RY62aNyBiRFgEups79o/wWWnwnJD3wfHXK65icdYRNu8zISowvON5KCdb8568xA7oR2iDRxm9M2004N+XnFfPs8UykwW5O/4RhhmfkHPZyg82Qep8AuUBp7WzSbPmWmdHL1LXRXUhKNEbN/sxWebEL2FSenEPoJ1mLlb4ZU6QEW7FI67BFgr/MdnjBO62VFuDHQPz45nOkWlzg0RKdBe5BI5c/FDGikDgcN+yTn/diwsXxuZoGWacicmxf9pz+B+xKtdtjNjjZxfGu53qXfBCzYygXMkeuWbEmXNpITx7mPIpaAdy+tknmdB7rq/X6RBIw5GfF9JJGvi8IK7ccq3/ZH18q/6sb/kySRB+hkcOxMrBS4rsSbRQity25gNLzogu8s+C+RFDU1+OM8wmt8L2/LVk3a4fJed3kd17BvRA/kU2VTKJBBHCGlgpNZqJPef+PTeruOD26fKQG1/3S2FslneHhs90aC9LYGg8GFPqBZp99ywmsMIgYKGPotS6Yb0t/iKpjiPqaL2BXUdXNwnkbR58h4sXyI9HtBzT8OLje2jIti+9B5uSFZw6flCc8xoye2pJ4z+lmJRssjU2ug8m99EI7DnLzSyD/J+gJwy6+OqTg+m3TU7QHslRzJr7ONpXKedqxlIx/e5g4na2o3Ay4/mdl/mbbgpHldDBzoMqrlJ3L6PaeStPPcv+BhAgT X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:SJ0PR13MB5545.namprd13.prod.outlook.com; PTR:; CAT:NONE; SFS:(13230016)(4636009)(39840400004)(346002)(376002)(366004)(396003)(136003)(6486002)(4326008)(186003)(36756003)(26005)(66476007)(66556008)(2616005)(8676002)(83380400001)(6512007)(107886003)(38350700002)(5660300002)(66946007)(8936002)(38100700002)(2906002)(478600001)(44832011)(6916009)(6506007)(316002)(52116002)(41300700001)(86362001); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?cmtGU21GZ0U4aFVQd09oSEQ3bXVFVjJpT2J3N2xSeVYxWG9EV2pXVVkxZlk3?= =?utf-8?B?NjV4cGdPK3pSSklvV2hkUG1QRWd0QUwxRGRHbm9mT29BMGxkTllrVVlKUlFv?= =?utf-8?B?WkRSc0hRK084aGZnWUZ0M3dvREVtcXJxYjBrR0hiNnZFVU5wNnlJb24xcDg1?= =?utf-8?B?L3pWS0ttUkVnUkkvQ29BNjN3L1NadEJ5SWdJZFBtL0hYVHN3N3ZKQWdQL1hv?= =?utf-8?B?Q3AyNVdPSEJQK3llTDRaVDFMQTMrTnY2Y0RQWERqVEdMUVFTaTJJUldZYjBK?= =?utf-8?B?TzhIdGpiWnVOK3BKN2IveEJvQ1ZZdVYxODFhT21ENnJDa2NvT2JVeWs3UE9E?= =?utf-8?B?RXlKc3Ywc1dJdjJCSXpWYzJPamhQdllFT3RVZzhubytYa1pYaFBXSW1XM3I5?= =?utf-8?B?dkZ2OVZ4RnViOGYwNXRvOHlYUzBOWk9MM1R3eWkvaWR6c295Vmx0UStMUXhM?= =?utf-8?B?YzFIZ1JuUy94SHEySE1aWnQvd1pCMERkNmp3UGFkajB0dGRTRU4wODhYa2ZI?= =?utf-8?B?akN2WTZOd0JkSDMyaEhXNUZSQURrVXVNcEUzbXlOb0xVaVpOM3JmSmRESGxF?= =?utf-8?B?M2ViUWdvVkN6b3lTSnIySXR1WWlGb0VPdXE3M1o0dW5SNlYxWmZFSkV0ZldT?= =?utf-8?B?SmpBdHBDSUxaWTVET3NUdUZwZDEyWWx3Z29xQyswK1BnL3NrenZpVlBOU0lN?= =?utf-8?B?MC80OVd0WUdmMVA4aWVOS0FucjRhZG1NMHp5Z1FscE9Sdi9tWjAxSG9pVHYy?= =?utf-8?B?OGo0M2JCN1Znd0g5Qk9QWHc5QUU3L3d6ZWdVeXpMYW1veDR0V2d4WFJWWVlO?= =?utf-8?B?WlBLWjB1Nm1EZWNQZG53VTZReGxsTFFkd0pxMzVwVGtZTm5EWGtMSGM3aytq?= =?utf-8?B?NTRXTlZMZ3Vka3diQkVEYkN1L25IL0xsemF6ZlRBNnEyZDdrRURYcTZ6Wnpl?= =?utf-8?B?aG1ZUHJ2bys1cXNYK3RFZ0VZMjFIcUpJUmRPL1loYjB3d0x6NTJWTUU0Ymps?= =?utf-8?B?RU82SXZTRnlmVU1NdGxIRUVQamg1K2M3UGt5VElwVFo2T25lN0Y0cXFRamhu?= =?utf-8?B?dDNvMzlTSnNvRmNRLzZYbDRINDRRNHIwZFhnWVBMOUFIdnJCR3JUdGdQYXBN?= =?utf-8?B?YktVMm56TS9LVVNvVUJCcHZZd0IyRUZFQjJWV0xacU1FZjduSTUwQlZEOGhO?= =?utf-8?B?NVB2QTV4NFAzeGFrbXhIdlBnL1g4RExkTlpPczUrb1l2TmNmeDVuWjBPTkp6?= =?utf-8?B?allqVXdJclpuTENndnYxZkhyVU1paG9uRVRIR1dYUjZtcFJBcXJzWTl6a0x1?= =?utf-8?B?VFRZZ2drZEliV2UvS2s4VEZmbjJ4bkJNOFNKbzFjTGlzTFhvUFh3TlZ2N1U3?= =?utf-8?B?ZnBjekt0Z0VtN1NrYmJYem1LZ0kvdkxNRUFGYTk5akxxRWtDOFpwdnkyWnFt?= =?utf-8?B?ZytVN3JHSGx4d1p3YkFKTk5mRU9ncDRzeDlza1VZVFpZeUFGQjVOWVJGUXhK?= =?utf-8?B?Z20vNWtVUWpmUWhOOFpTWmVxUmZRVWpMSVNSOXR4azAvS3VKdHRMTlJ4ckZ4?= =?utf-8?B?bTI2RzlnK3MwcTVrb3c0WFlmRXBKdW1wUDJ1Z3pXSVpDNGR5MCsrZzJDRG1u?= =?utf-8?B?TGJTOXc5YkJ6VnRFdzRxK2V4UmVCbGRPcEgwaTVNZmVkV3AvWTJUenlkSGxB?= =?utf-8?B?ck1rdmYzbVFPdmtOcWRvYUhqekdvaHZkZld5Rmp3aTZtL1kxMkZkOVIrWUtl?= =?utf-8?B?TUlpVktCdjRDR0RIV0VKT3RsT09rcnRzeSt5Q1FoU2NIL2Q1bk9zbC9EZnQ3?= =?utf-8?B?WUdRVGxBdmxCckgzUnFDdXMyRFgyeU5JR2J2aG5LOWhTQ3UwbkdvdWgrLzI4?= =?utf-8?B?WWlvV2htRmxtY0g1ZVR5dnZwT1laY2tlL1pUU2EvaS9XN2U0WklqU1JMSnVj?= =?utf-8?B?Q1dnWVBRRzkvdXVaNzZmdjhNK2hxZmtySHNkMXdEcjRNM29SVGlzSHd3a1h1?= =?utf-8?B?T0VhL25EaDZNTFEwbytVNHBORlJIMEFsVnFGTTRKZ28vRDdnSXhwaU9jWFR0?= =?utf-8?B?RnJ0b3h5QUQvUmpXN0hqQ3FtaWhhai94WURJZ1IreU1pRmpUNFpOVlZqSUow?= =?utf-8?B?NUxRNWkzeEpaRTJLUXZSSnNMNW1IMkY0SkVVekppSVBjMnZjK2YzczhnV0Nr?= =?utf-8?B?U0E9PQ==?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 445c4bf7-22ac-4168-bcf0-08da5b085c08 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 01 Jul 2022 02:21:09.0944 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: fe128f2c-073b-4c20-818e-7246a585940c X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: N/LQVUzCTnXtNPXjn4O3Udf0ZZsvd/oDgiZwiJvX/1/acI1G47neToi2xABLp+qC1bJ0JoEAkLpZ41wCFm4rBzb9GUrkET0PfWu0iMJPS2k= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH2PR13MB4459 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Calling nfp_net_init() is only done for the corenic firmware flavor and it is guaranteed to always be called from the primary process, so the explicit check for RTE_PROC_PRIMARY can be dropped. The calling graph of nfp_net_init() already guaranteed the free of resources when it fail, so remove the necessary free logics inside it. While at it remove the unused member is_phyport from struct nfp_net_hw. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfp_common.h | 1 - drivers/net/nfp/nfp_ethdev.c | 40 +++++++++++----------------------------- 2 files changed, 11 insertions(+), 30 deletions(-) diff --git a/drivers/net/nfp/nfp_common.h b/drivers/net/nfp/nfp_common.h index 2aaf1d6..b28ebc9 100644 --- a/drivers/net/nfp/nfp_common.h +++ b/drivers/net/nfp/nfp_common.h @@ -238,7 +238,6 @@ struct nfp_net_hw { uint8_t idx; /* Internal port number as seen from NFP */ uint8_t nfp_idx; - bool is_phyport; union eth_table_entry *eth_table; diff --git a/drivers/net/nfp/nfp_ethdev.c b/drivers/net/nfp/nfp_ethdev.c index 3c4b0ac..2c5607c 100644 --- a/drivers/net/nfp/nfp_ethdev.c +++ b/drivers/net/nfp/nfp_ethdev.c @@ -417,7 +417,6 @@ uint32_t start_q; int stride = 4; int port = 0; - int err; PMD_INIT_FUNC_TRACE(); @@ -452,10 +451,6 @@ PMD_INIT_LOG(DEBUG, "Working with physical port number: %d, " "NFP internal port number: %d", port, hw->nfp_idx); - /* For secondary processes, the primary has done all the work */ - if (rte_eal_process_type() != RTE_PROC_PRIMARY) - return 0; - rte_eth_copy_pci_info(eth_dev, pci_dev); hw->device_id = pci_dev->id.device_id; @@ -506,8 +501,7 @@ break; default: PMD_DRV_LOG(ERR, "nfp_net: no device ID matching"); - err = -ENODEV; - goto dev_err_ctrl_map; + return -ENODEV; } PMD_INIT_LOG(DEBUG, "tx_bar_off: 0x%" PRIx64 "", tx_bar_off); @@ -573,8 +567,7 @@ RTE_ETHER_ADDR_LEN, 0); if (eth_dev->data->mac_addrs == NULL) { PMD_INIT_LOG(ERR, "Failed to space for MAC address"); - err = -ENOMEM; - goto dev_err_queues_map; + return -ENOMEM; } nfp_net_pf_read_mac(app_nic, port); @@ -604,24 +597,15 @@ hw->mac_addr[0], hw->mac_addr[1], hw->mac_addr[2], hw->mac_addr[3], hw->mac_addr[4], hw->mac_addr[5]); - if (rte_eal_process_type() == RTE_PROC_PRIMARY) { - /* Registering LSC interrupt handler */ - rte_intr_callback_register(pci_dev->intr_handle, - nfp_net_dev_interrupt_handler, (void *)eth_dev); - /* Telling the firmware about the LSC interrupt entry */ - nn_cfg_writeb(hw, NFP_NET_CFG_LSC, NFP_NET_IRQ_LSC_IDX); - /* Recording current stats counters values */ - nfp_net_stats_reset(eth_dev); - } + /* Registering LSC interrupt handler */ + rte_intr_callback_register(pci_dev->intr_handle, + nfp_net_dev_interrupt_handler, (void *)eth_dev); + /* Telling the firmware about the LSC interrupt entry */ + nn_cfg_writeb(hw, NFP_NET_CFG_LSC, NFP_NET_IRQ_LSC_IDX); + /* Recording current stats counters values */ + nfp_net_stats_reset(eth_dev); return 0; - -dev_err_queues_map: - nfp_cpp_area_free(hw->hwqueues_area); -dev_err_ctrl_map: - nfp_cpp_area_free(hw->ctrl_area); - - return err; } #define DEFAULT_FW_PATH "/lib/firmware/netronome" @@ -818,7 +802,6 @@ hw->eth_dev = eth_dev; hw->idx = i; hw->nfp_idx = nfp_eth_table->ports[i].index; - hw->is_phyport = true; eth_dev->device = &pf_dev->pci_dev->device; @@ -884,8 +867,7 @@ if (cpp == NULL) { PMD_INIT_LOG(ERR, "A CPP handle can not be obtained"); - ret = -EIO; - goto error; + return -EIO; } hwinfo = nfp_hwinfo_read(cpp); @@ -1005,7 +987,7 @@ free(hwinfo); cpp_cleanup: nfp_cpp_free(cpp); -error: + return ret; } -- 1.8.3.1