From patchwork Thu Aug 24 11:09:35 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 130709 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0B3E7430EF; Thu, 24 Aug 2023 13:11:49 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 04ED643279; Thu, 24 Aug 2023 13:11:02 +0200 (CEST) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2090.outbound.protection.outlook.com [40.107.220.90]) by mails.dpdk.org (Postfix) with ESMTP id 7F05C42D0E for ; Thu, 24 Aug 2023 13:10:58 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=CUpmBWRA/BXe+KFAXuPeSueiWE2pT0/S+3a0KI26yHEwndUNYnFWEZlIAYmDSL2rRCUxTPb+L6UEJ9FsDhfzksUnwQ2/+QIQNNjcjj5Ub24uI6CFPyTfOQPeAI7zJ3gBDJ+Lj6ysBaib34m9qeOVtxtres8evO0bPsTowND1SQ+mG8jBUOaTJyRWovlu3OTZWjl+0wKTBDD3SdeqKqRjQ7uXiZ70tvCrfvGswchgk/FpkKnBiwDO5k7a/ETpDka1WloeIkDb5pmLuRvfZAXSXWQXTvvfFc1u/1qYq7ZDRNKVu+0G6NmJDF1F9Kc0JC0qM94C+mWmi46Xhhyf3VVrrQ== 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=jdg17YUFbFpct+O4mFXynmJyTZP5gnhgctwPmgtL9EE=; b=UEtI+p2l8nUuLf25kb0YVeapQMOHN82KXHkXE4ZtLDloTlZW1oANqp0PCV25g4BlO9JJyBd5TYhsgvaX86weTlH19kiLDyikZxWVdRrS1H8UUc8wFEuoqrSW3Eih50EFEAnhsmxdgSHGLCnMTrgO66gZV9s3DzYCru8uYMbTOu5urYyPFRbacotIXh+sr9f7sSN/ZdncJZI3JQpBUOxpvpRYG/L6zMhSuNYdlRhaf3S246c9leQyrtsC4z/Q0+kU94VJRvk7gYMMTg+skUpcB4qlVM98zIyOhpPSGOdN++k2zXcD98hiXf985XghFCvJkAcFpopDiB3N17PzPWE/MA== 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=jdg17YUFbFpct+O4mFXynmJyTZP5gnhgctwPmgtL9EE=; b=Xvy+woMoVqh3AQtryLO53/9uK1O95kIWFvh6GSeoQGFKrPJKhxK7Caut2BIXwM9R23qnj39F/F72MwvoLqiA2SlRzN3uGQZtqZQoIVPb9SU+QdA3zI0HJAc7UuqbZZ1qRu6Qgd0emS98Bb2Tl4PfFvp/Nmuu+x6ncdb5MkeXvlQ= 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 PH0PR13MB6134.namprd13.prod.outlook.com (2603:10b6:510:295::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.6678.26; Thu, 24 Aug 2023 11:10:57 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::51fe:5846:af8b:bace]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::51fe:5846:af8b:bace%3]) with mapi id 15.20.6699.020; Thu, 24 Aug 2023 11:10:57 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, niklas.soderlund@corigine.com, Chaoyong He Subject: [PATCH 06/27] net/nfp: adjust the log statement Date: Thu, 24 Aug 2023 19:09:35 +0800 Message-Id: <20230824110956.1943559-7-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230824110956.1943559-1-chaoyong.he@corigine.com> References: <20230824110956.1943559-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SJ0PR05CA0005.namprd05.prod.outlook.com (2603:10b6:a03:33b::10) To SJ0PR13MB5545.namprd13.prod.outlook.com (2603:10b6:a03:424::5) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ0PR13MB5545:EE_|PH0PR13MB6134:EE_ X-MS-Office365-Filtering-Correlation-Id: d0726490-8d20-4ece-e055-08dba492ca1c X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: vtVwMgTxHMwnfikxVCTpIoIYoLqux9sx9rybaKAARFOd8cFBLky1b0SNzuV2YfROBrEn5OOEprTJnREpZ4JrsAP+hTnknGii3Gj5A/OlLqlOoHRwG7OPTfzHblfvfPBqqJwWT8yEOsQnzB+lJ2OaE03ZQ0Hm3qAWiqw+juyYNbeiwPiPAUUQwcYdXPSupR5IGu7csmRfnZ3dhXQFw/bJ1kqUnt2JVPzMI5jM/S/IQalwLNmXwVZsDT0WsDJo1e9rY0ukTsO167ZvtzKIV/Hv9hXgUYGQJ4TNg6iw8Arw5+OES3TqhVKxX7qiLgdOJ/ooLizyiAje0KagRbFEL5ZMufP76QZE7Bq0M81OW27S73kTCWNrXwtAGl0ToQ6nWnlO1pocPqYFjvd3FaaK46Cl749kHOSnyvMtIZ1pEcnXkPIaDD15sNDitgcTz8QHx1AnZ3cRIKwOqdAhUpFiWGxVc4Y/nYchl1+DkkOC0ZLl6HvNbeSgKVs2zLlWgfQ18oLk6A4TUCoz40gByIFZETIWBPDWZdbodHGf3BSsE63JBNlr2/eHR0wUuaLIKHuzLMOHtIMHmcMy3FRwIXDcnZUcKHdHEjT+13vyT8TLqZGPCN08rofHcuBL4T086Uw3rqwiGOE4FMgFQxfUxHRAvF2ZRlKUCI7mvZXxTZlOFyN/E2n62XyZ7vHKmISCcKz5KaJT3SOFdo3ITWpYdF9B2Po5IA== 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:(13230031)(346002)(396003)(366004)(136003)(376002)(39830400003)(1800799009)(451199024)(186009)(316002)(66556008)(6916009)(66946007)(66476007)(12101799020)(41300700001)(44832011)(5660300002)(8936002)(8676002)(38100700002)(66574015)(38350700002)(4326008)(30864003)(2906002)(83380400001)(26005)(86362001)(478600001)(107886003)(52116002)(36756003)(6506007)(6666004)(6512007)(6486002)(2616005)(1076003); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?JoDgnozYZRaCtZI44F/YnLHA42if?= =?utf-8?q?bXLVHN9WMt0SUJvNJsG/9DG6uXI9f07WeKjg44hPeWLh3hIym/qMzgshLVm0bWodk?= =?utf-8?q?JAJSqEgBMY1ng2J0Tg9xQuG+GY8Y3g2Ao0IIgS12e8hkDF714WMHFtEst8LzNFE4y?= =?utf-8?q?vyOdPrMW2kh+GjHwOXqnDd8rVd/7OM/0lUIdY5azo31FF3fD85Aun3Lx+TDxkYwa+?= =?utf-8?q?PtdRuUKgD/hkr1wZjCFCxs8JapHF0JKElv7Tji6IRfjq7gQ1ZobklRcDZkx/3Ft8l?= =?utf-8?q?GgAe8gBVGS4l4XiVo2HzKoH2RK0UbyYcXcAdIKj+Pd7nMtb8OolM8h+25suSSMEF2?= =?utf-8?q?PO2yaPWNx0UwYXNLJTfgmiH683+wtJUZLZQe7LS9Bl96ucORseUyumreuArMA+4Bf?= =?utf-8?q?nX62j0xNTgGOX3DzOd0i02+r6BDYDscHB6wYOp9C99G2qmIAmuUBkVg8+e+5uGs3R?= =?utf-8?q?3hRFimX0kxC4ZRoGgW9ZUKB54kkpuwf/8+vJgAn/v4GJKN/7IX8y6f1DpLL1V3ucc?= =?utf-8?q?hwQDDOGt419R+u0JuNfatDl9ZZ8w8IT9GyVx7TPlmlSvtZiPtVRdOAd/dDnHIxuIS?= =?utf-8?q?u9VyhwrCQiZe6sdFGXQz/96dSx+DvLM10gMt6Tzjz6lEY4Xuj1hYnZdM0zZrZe2VU?= =?utf-8?q?RlY5c/vKI49ubEi3qn7VQ7Zl534aLelVYeaPv812IUqu73c4G4YdDIzCgEnZ7Xni+?= =?utf-8?q?sqq1uGDHt1a4yq/cZCgy84rDkbIx7y8piuV9EaHB69amd/1nvLrUoyrLAJ32qdoBY?= =?utf-8?q?haEkDi1oMFKkkYmCJonjrAjdsIBjmBHFeEkCLuJPcrp0FXI5mUE7QyG/6Roruec/v?= =?utf-8?q?aeqlyBGC5OhAWfREHAQ6Me8c/PSTqx4bhhrzowW3desvUygZ5Lm7yYS11KMaSCYvb?= =?utf-8?q?aioKLiFmqsRYx6Vhtc/ip+PUA0EJ2LRIfXrtfvIhXWFP67vabvTnHCAvujxAGkJ0f?= =?utf-8?q?H3wdl7/UHludptQgS5yK/VQ0qX1aWOH6T3WHae793Q6gc/jb09f1VcrZmvmCLLWlR?= =?utf-8?q?eT3gfC0sOFx63VBad1PvUm4hT5PrYXheCGKkUbQSBzXEuj0qZ8XDdydNKjpd1ng3V?= =?utf-8?q?Jxqh6EK744CHHbQ8NmXFj6UGfSe/46cDcXSz975DCPNysrf39yBporQdRAV/uUY9Z?= =?utf-8?q?HWrtvLK7RDl+aPwN4tVi0QPkFNr8du1pCQIue8AIzupSrsh6Hc8iKmBKEzlXlI+GR?= =?utf-8?q?mWOWNIYWONfOXvwEc21AeDHQENsUi2oKOGomaGhvY/BDa0Od4BIw3wdJdOM1aFSdK?= =?utf-8?q?lO6sQnBMpSfOjnrrTcNaBcTN7+/RDAR9ejXhnwavH9jfgp+PXax1VOhoHnxaocxbK?= =?utf-8?q?hKm0oG1f22egIgC+EHoUt7qHiE1QmHDT1a14NIJb6oY4/0kjQGr8tY1xaci6iITW+?= =?utf-8?q?Ww53eYgr1N48qxIKqAJfiGOsIAv6VdvEs4cbBm3Ob9wPLEjNxwbxDmkmA/d5YU8Sa?= =?utf-8?q?pdWW0MCPuflKBHJ6lwy++V3tCj8pmzoYuHCY7XgtP183UErDAWrnz4nm+2Y10k1cU?= =?utf-8?q?DYQZWCg4zmIVDyEKb1oU8n5HBqiu44tHYg=3D=3D?= X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: d0726490-8d20-4ece-e055-08dba492ca1c X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 24 Aug 2023 11:10:56.9866 (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: GIKZHIrAz9TsDRypGN7zd91VtbEtWk4PvU0le65WEtZFTnPJuJMIhHS2dDTGmWtzW+lMufG1LmER/r5m/kDlCHNKNJgwgxX1H7DMLQz1SK0= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR13MB6134 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 Add log statement to the important control logic, which means more strict check of the return value of function call. Also remove some verbose info log statement. Signed-off-by: Chaoyong He Reviewed-by: Niklas Söderlund --- drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c | 37 ++++++---------------- drivers/net/nfp/nfpcore/nfp_cppcore.c | 20 +++++++++--- drivers/net/nfp/nfpcore/nfp_hwinfo.c | 28 ++++++++-------- drivers/net/nfp/nfpcore/nfp_mip.c | 7 ++-- drivers/net/nfp/nfpcore/nfp_mutex.c | 4 +-- drivers/net/nfp/nfpcore/nfp_nffw.c | 9 ++++-- drivers/net/nfp/nfpcore/nfp_nsp.c | 25 +++++++++++---- drivers/net/nfp/nfpcore/nfp_nsp_eth.c | 10 +++--- drivers/net/nfp/nfpcore/nfp_resource.c | 17 +++++++--- drivers/net/nfp/nfpcore/nfp_rtsym.c | 14 +++----- 10 files changed, 92 insertions(+), 79 deletions(-) diff --git a/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c b/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c index 78beee07ef..bdf4a658f5 100644 --- a/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c +++ b/drivers/net/nfp/nfpcore/nfp_cpp_pcie_ops.c @@ -174,18 +174,10 @@ nfp_compute_bar(const struct nfp_bar *bar, newcfg |= NFP_PCIE_BAR_PCIE2CPP_ACTION_BASEADDRESS(act); newcfg |= NFP_PCIE_BAR_PCIE2CPP_TOKEN_BASEADDRESS(tok); - if ((offset & mask) != ((offset + size - 1) & mask)) { - PMD_DRV_LOG(ERR, "BAR%d: Won't use for Fixed mapping <%#llx,%#llx>, action=%d BAR too small (0x%llx)", - bar->index, (unsigned long long)offset, - (unsigned long long)(offset + size), act, - (unsigned long long)mask); + if ((offset & mask) != ((offset + size - 1) & mask)) return -EINVAL; - } - offset &= mask; - PMD_DRV_LOG(DEBUG, "BAR%d: Created Fixed mapping %d:%d:%d:0x%#llx-0x%#llx>", - bar->index, tgt, act, tok, (unsigned long long)offset, - (unsigned long long)(offset + mask)); + offset &= mask; bitsize = 40 - 16; } else { @@ -198,30 +190,16 @@ nfp_compute_bar(const struct nfp_bar *bar, newcfg |= NFP_PCIE_BAR_PCIE2CPP_TARGET_BASEADDRESS(tgt); newcfg |= NFP_PCIE_BAR_PCIE2CPP_TOKEN_BASEADDRESS(tok); - if ((offset & mask) != ((offset + size - 1) & mask)) { - PMD_DRV_LOG(ERR, "BAR%d: Won't use for bulk mapping <%#llx,%#llx> target=%d, token=%d BAR too small (%#llx) - (%#llx != %#llx).", - bar->index, (unsigned long long)offset, - (unsigned long long)(offset + size), - tgt, tok, (unsigned long long)mask, - (unsigned long long)(offset & mask), - (unsigned long long)(offset + size - 1) & mask); + if ((offset & mask) != ((offset + size - 1) & mask)) return -EINVAL; - } offset &= mask; - PMD_DRV_LOG(DEBUG, "BAR%d: Created bulk mapping %d:x:%d:%#llx-%#llx", - bar->index, tgt, tok, (unsigned long long)offset, - (unsigned long long)(offset + ~mask)); - bitsize = 40 - 21; } - if (bar->bitsize < bitsize) { - PMD_DRV_LOG(ERR, "BAR%d: Too small for %d:%d:%d", bar->index, - tgt, tok, act); + if (bar->bitsize < bitsize) return -EINVAL; - } newcfg |= offset >> bitsize; @@ -254,7 +232,6 @@ nfp_bar_write(struct nfp_pcie_user *nfp, *(uint32_t *)(bar->csr) = newcfg; bar->barcfg = newcfg; - PMD_DRV_LOG(DEBUG, "BAR%d: updated to 0x%08x", bar->index, newcfg); return 0; } @@ -795,7 +772,11 @@ nfp6000_init(struct nfp_cpp *cpp, desc->cfg = dev->mem_resource[0].addr; desc->dev_id = dev->addr.function & 0x7; - nfp_enable_bars(desc); + ret = nfp_enable_bars(desc); + if (ret != 0) { + PMD_DRV_LOG(ERR, "Enable bars failed"); + return -1; + } nfp_cpp_priv_set(cpp, desc); diff --git a/drivers/net/nfp/nfpcore/nfp_cppcore.c b/drivers/net/nfp/nfpcore/nfp_cppcore.c index 776842bdf6..66f4ddaab7 100644 --- a/drivers/net/nfp/nfpcore/nfp_cppcore.c +++ b/drivers/net/nfp/nfpcore/nfp_cppcore.c @@ -200,6 +200,7 @@ nfp_cpp_area_alloc_with_name(struct nfp_cpp *cpp, err = cpp->op->area_init(area, dest, address, size); if (err < 0) { + PMD_DRV_LOG(ERR, "Area init op failed"); free(area); return NULL; } @@ -243,10 +244,13 @@ nfp_cpp_area_alloc_acquire(struct nfp_cpp *cpp, struct nfp_cpp_area *area; area = nfp_cpp_area_alloc(cpp, destination, address, size); - if (area == NULL) + if (area == NULL) { + PMD_DRV_LOG(ERR, "Failed to allocate CPP area"); return NULL; + } if (nfp_cpp_area_acquire(area) != 0) { + PMD_DRV_LOG(ERR, "Failed to acquire CPP area"); nfp_cpp_area_free(area); return NULL; } @@ -294,8 +298,10 @@ nfp_cpp_area_acquire(struct nfp_cpp_area *area) if (area->cpp->op->area_acquire != NULL) { int err = area->cpp->op->area_acquire(area); - if (err < 0) + if (err < 0) { + PMD_DRV_LOG(ERR, "Area acquire op failed"); return -1; + } } return 0; @@ -662,7 +668,7 @@ nfp_cpp_read(struct nfp_cpp *cpp, area = nfp_cpp_area_alloc_acquire(cpp, destination, address, length); if (area == NULL) { - PMD_DRV_LOG(ERR, "Area allocation/acquire failed"); + PMD_DRV_LOG(ERR, "Area allocation/acquire failed for read"); return -1; } @@ -691,8 +697,10 @@ nfp_cpp_write(struct nfp_cpp *cpp, struct nfp_cpp_area *area; area = nfp_cpp_area_alloc_acquire(cpp, destination, address, length); - if (area == NULL) + if (area == NULL) { + PMD_DRV_LOG(ERR, "Area allocation/acquire failed for write"); return -1; + } err = nfp_cpp_area_write(area, 0, kernel_vaddr, length); @@ -746,8 +754,10 @@ nfp_cpp_map_area(struct nfp_cpp *cpp, uint8_t *res; *area = nfp_cpp_area_alloc_acquire(cpp, cpp_id, addr, size); - if (*area == NULL) + if (*area == NULL) { + PMD_DRV_LOG(ERR, "Area allocation/acquire failed for map"); goto err_eio; + } res = nfp_cpp_area_iomem(*area); if (res == NULL) diff --git a/drivers/net/nfp/nfpcore/nfp_hwinfo.c b/drivers/net/nfp/nfpcore/nfp_hwinfo.c index 819761eda0..b658b5e900 100644 --- a/drivers/net/nfp/nfpcore/nfp_hwinfo.c +++ b/drivers/net/nfp/nfpcore/nfp_hwinfo.c @@ -74,7 +74,7 @@ nfp_hwinfo_db_validate(struct nfp_hwinfo *db, new_crc = nfp_crc32_posix((char *)db, size); crc = (uint32_t *)(db->start + size); if (new_crc != *crc) { - PMD_DRV_LOG(ERR, "Corrupt hwinfo table (CRC mismatch) calculated 0x%x, expected 0x%x", + PMD_DRV_LOG(ERR, "CRC mismatch, calculated %#x, expected %#x", new_crc, *crc); return -EINVAL; } @@ -94,34 +94,36 @@ nfp_hwinfo_try_fetch(struct nfp_cpp *cpp, struct nfp_hwinfo *header; res = nfp_resource_acquire(cpp, NFP_RESOURCE_NFP_HWINFO); - if (res) { - cpp_id = nfp_resource_cpp_id(res); - cpp_addr = nfp_resource_address(res); - *cpp_size = nfp_resource_size(res); + if (res == NULL) { + PMD_DRV_LOG(ERR, "HWInfo - acquire resource failed"); + return NULL; + } - nfp_resource_release(res); + cpp_id = nfp_resource_cpp_id(res); + cpp_addr = nfp_resource_address(res); + *cpp_size = nfp_resource_size(res); - if (*cpp_size < HWINFO_SIZE_MIN) - return NULL; - } else { + nfp_resource_release(res); + + if (*cpp_size < HWINFO_SIZE_MIN) return NULL; - } db = malloc(*cpp_size + 1); if (db == NULL) return NULL; err = nfp_cpp_read(cpp, cpp_id, cpp_addr, db, *cpp_size); - if (err != (int)*cpp_size) + if (err != (int)*cpp_size) { + PMD_DRV_LOG(ERR, "HWInfo - CPP read error %d", err); goto exit_free; + } header = (void *)db; - PMD_DRV_LOG(DEBUG, "NFP HWINFO header: %#08x", *(uint32_t *)header); if (nfp_hwinfo_is_updating(header)) goto exit_free; if (header->version != NFP_HWINFO_VERSION_2) { - PMD_DRV_LOG(DEBUG, "Unknown HWInfo version: 0x%08x", + PMD_DRV_LOG(ERR, "Unknown HWInfo version: %#08x", header->version); goto exit_free; } diff --git a/drivers/net/nfp/nfpcore/nfp_mip.c b/drivers/net/nfp/nfpcore/nfp_mip.c index 1e601313b4..086e82db70 100644 --- a/drivers/net/nfp/nfpcore/nfp_mip.c +++ b/drivers/net/nfp/nfpcore/nfp_mip.c @@ -46,16 +46,16 @@ nfp_mip_try_read(struct nfp_cpp *cpp, ret = nfp_cpp_read(cpp, cpp_id, addr, mip, sizeof(*mip)); if (ret != sizeof(*mip)) { - PMD_DRV_LOG(ERR, "Failed to read MIP data (%d, %zu)", ret, sizeof(*mip)); + PMD_DRV_LOG(ERR, "Failed to read MIP data"); return -EIO; } if (mip->signature != NFP_MIP_SIGNATURE) { - PMD_DRV_LOG(ERR, "Incorrect MIP signature (0x%08x)", + PMD_DRV_LOG(ERR, "Incorrect MIP signature %#08x", rte_le_to_cpu_32(mip->signature)); return -EINVAL; } if (mip->mip_version != NFP_MIP_VERSION) { - PMD_DRV_LOG(ERR, "Unsupported MIP version (%d)", + PMD_DRV_LOG(ERR, "Unsupported MIP version %d", rte_le_to_cpu_32(mip->mip_version)); return -EINVAL; } @@ -109,6 +109,7 @@ nfp_mip_open(struct nfp_cpp *cpp) err = nfp_mip_read_resource(cpp, mip); if (err != 0) { + PMD_DRV_LOG(ERR, "Failed to read MIP resource"); free(mip); return NULL; } diff --git a/drivers/net/nfp/nfpcore/nfp_mutex.c b/drivers/net/nfp/nfpcore/nfp_mutex.c index 05e0ff46e5..82919d8270 100644 --- a/drivers/net/nfp/nfpcore/nfp_mutex.c +++ b/drivers/net/nfp/nfpcore/nfp_mutex.c @@ -221,9 +221,7 @@ nfp_cpp_mutex_lock(struct nfp_cpp_mutex *mutex) if (err < 0 && err != -EBUSY) return err; if (time(NULL) >= warn_at) { - PMD_DRV_LOG(ERR, "Warning: waiting for NFP mutex usage:%u depth:%hd] target:%d addr:%lx key:%08x]", - mutex->usage, mutex->depth, mutex->target, - mutex->address, mutex->key); + PMD_DRV_LOG(WARNING, "Waiting for NFP mutex..."); warn_at = time(NULL) + 60; } sched_yield(); diff --git a/drivers/net/nfp/nfpcore/nfp_nffw.c b/drivers/net/nfp/nfpcore/nfp_nffw.c index 32e0fc94bb..6ba40cd085 100644 --- a/drivers/net/nfp/nfpcore/nfp_nffw.c +++ b/drivers/net/nfp/nfpcore/nfp_nffw.c @@ -3,6 +3,7 @@ * All rights reserved. */ +#include "../nfp_logs.h" #include "nfp_cpp.h" #include "nfp_nffw.h" #include "nfp_mip.h" @@ -131,8 +132,10 @@ nfp_nffw_info_open(struct nfp_cpp *cpp) memset(state, 0, sizeof(*state)); state->res = nfp_resource_acquire(cpp, NFP_RESOURCE_NFP_NFFW); - if (state->res == NULL) + if (state->res == NULL) { + PMD_DRV_LOG(ERR, "NFFW - acquire resource failed"); goto err_free; + } fwinf = &state->fwinf; @@ -142,8 +145,10 @@ nfp_nffw_info_open(struct nfp_cpp *cpp) err = nfp_cpp_read(cpp, nfp_resource_cpp_id(state->res), nfp_resource_address(state->res), fwinf, sizeof(*fwinf)); - if (err < (int)sizeof(*fwinf)) + if (err < (int)sizeof(*fwinf)) { + PMD_DRV_LOG(ERR, "NFFW - CPP read error %d", err); goto err_release; + } if (nffw_res_flg_init_get(fwinf) == 0) goto err_release; diff --git a/drivers/net/nfp/nfpcore/nfp_nsp.c b/drivers/net/nfp/nfpcore/nfp_nsp.c index a00bd5870d..76d418d478 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp.c +++ b/drivers/net/nfp/nfpcore/nfp_nsp.c @@ -82,8 +82,10 @@ nfp_nsp_check(struct nfp_nsp *state) nsp_status = nfp_resource_address(state->res) + NSP_STATUS; err = nfp_cpp_readq(cpp, nsp_cpp, nsp_status, ®); - if (err < 0) + if (err < 0) { + PMD_DRV_LOG(ERR, "NSP - CPP readq failed %d", err); return err; + } if (FIELD_GET(NSP_STATUS_MAGIC, reg) != NSP_MAGIC) { PMD_DRV_LOG(ERR, "Cannot detect NFP Service Processor"); @@ -119,8 +121,10 @@ nfp_nsp_open(struct nfp_cpp *cpp) struct nfp_resource *res; res = nfp_resource_acquire(cpp, NFP_RESOURCE_NSP); - if (res == NULL) + if (res == NULL) { + PMD_DRV_LOG(ERR, "NSP - resource acquire failed"); return NULL; + } state = malloc(sizeof(*state)); if (state == NULL) { @@ -133,6 +137,7 @@ nfp_nsp_open(struct nfp_cpp *cpp) err = nfp_nsp_check(state); if (err != 0) { + PMD_DRV_LOG(ERR, "NSP - check failed"); nfp_nsp_close(state); return NULL; } @@ -180,8 +185,10 @@ nfp_nsp_wait_reg(struct nfp_cpp *cpp, for (;;) { err = nfp_cpp_readq(cpp, nsp_cpp, addr, reg); - if (err < 0) + if (err < 0) { + PMD_DRV_LOG(ERR, "NSP - CPP readq failed"); return err; + } if ((*reg & mask) == val) return 0; @@ -234,8 +241,10 @@ nfp_nsp_command(struct nfp_nsp *state, nsp_buffer = nsp_base + NSP_BUFFER; err = nfp_nsp_check(state); - if (err != 0) + if (err != 0) { + PMD_DRV_LOG(ERR, "Check NSP command failed"); return err; + } if (!FIELD_FIT(NSP_BUFFER_CPP, buff_cpp >> 8) || !FIELD_FIT(NSP_BUFFER_ADDRESS, buff_addr)) { @@ -261,7 +270,7 @@ nfp_nsp_command(struct nfp_nsp *state, err = nfp_nsp_wait_reg(cpp, ®, nsp_cpp, nsp_command, NSP_COMMAND_START, 0); if (err != 0) { - PMD_DRV_LOG(ERR, "Error %d waiting for code 0x%04x to start", + PMD_DRV_LOG(ERR, "Error %d waiting for code %#04x to start", err, code); return err; } @@ -270,7 +279,7 @@ nfp_nsp_command(struct nfp_nsp *state, err = nfp_nsp_wait_reg(cpp, ®, nsp_cpp, nsp_status, NSP_STATUS_BUSY, 0); if (err != 0) { - PMD_DRV_LOG(ERR, "Error %d waiting for code 0x%04x to start", + PMD_DRV_LOG(ERR, "Error %d waiting for code %#04x to complete", err, code); return err; } @@ -352,8 +361,10 @@ nfp_nsp_command_buf(struct nfp_nsp *nsp, } ret = nfp_nsp_command(nsp, code, option, cpp_id, cpp_buf); - if (ret < 0) + if (ret < 0) { + PMD_DRV_LOG(ERR, "NSP command failed"); return ret; + } if (out_buf != NULL && out_size > 0) { err = nfp_cpp_read(cpp, cpp_id, cpp_buf, out_buf, out_size); diff --git a/drivers/net/nfp/nfpcore/nfp_nsp_eth.c b/drivers/net/nfp/nfpcore/nfp_nsp_eth.c index 9e8a247e5c..51dcf24f5f 100644 --- a/drivers/net/nfp/nfpcore/nfp_nsp_eth.c +++ b/drivers/net/nfp/nfpcore/nfp_nsp_eth.c @@ -259,7 +259,7 @@ __nfp_eth_read_ports(struct nfp_nsp *nsp) memset(entries, 0, NSP_ETH_TABLE_SIZE); ret = nfp_nsp_read_eth_table(nsp, entries, NSP_ETH_TABLE_SIZE); if (ret < 0) { - PMD_DRV_LOG(ERR, "reading port table failed %d", ret); + PMD_DRV_LOG(ERR, "Reading port table failed %d", ret); goto err; } @@ -278,7 +278,7 @@ __nfp_eth_read_ports(struct nfp_nsp *nsp) * above. */ if (ret != 0 && ret != cnt) { - PMD_DRV_LOG(ERR, "table entry count (%d) unmatch entries present (%d)", + PMD_DRV_LOG(ERR, "Table entry count (%d) unmatch entries present (%d)", ret, cnt); goto err; } @@ -357,12 +357,12 @@ nfp_eth_config_start(struct nfp_cpp *cpp, ret = nfp_nsp_read_eth_table(nsp, entries, NSP_ETH_TABLE_SIZE); if (ret < 0) { - PMD_DRV_LOG(ERR, "reading port table failed %d", ret); + PMD_DRV_LOG(ERR, "Reading port table failed %d", ret); goto err; } if ((entries[idx].port & NSP_ETH_PORT_LANES_MASK) == 0) { - PMD_DRV_LOG(ERR, "trying to set port state on disabled port %d", idx); + PMD_DRV_LOG(ERR, "Trying to set port state on disabled port %d", idx); goto err; } @@ -642,7 +642,7 @@ __nfp_eth_set_speed(struct nfp_nsp *nsp, rate = nfp_eth_speed2rate(speed); if (rate == RATE_INVALID) { - PMD_DRV_LOG(ERR, "could not find matching lane rate for speed %u", speed); + PMD_DRV_LOG(ERR, "Could not find matching lane rate for speed %u", speed); return -EINVAL; } diff --git a/drivers/net/nfp/nfpcore/nfp_resource.c b/drivers/net/nfp/nfpcore/nfp_resource.c index fa92f2762e..363f7d6198 100644 --- a/drivers/net/nfp/nfpcore/nfp_resource.c +++ b/drivers/net/nfp/nfpcore/nfp_resource.c @@ -118,16 +118,22 @@ nfp_resource_try_acquire(struct nfp_cpp *cpp, { int err; - if (nfp_cpp_mutex_lock(dev_mutex) != 0) + if (nfp_cpp_mutex_lock(dev_mutex) != 0) { + PMD_DRV_LOG(ERR, "RESOURCE - CPP mutex lock failed"); return -EINVAL; + } err = nfp_cpp_resource_find(cpp, res); - if (err != 0) + if (err != 0) { + PMD_DRV_LOG(ERR, "RESOURCE - CPP resource find failed"); goto err_unlock_dev; + } err = nfp_cpp_mutex_trylock(res->mutex); - if (err != 0) + if (err != 0) { + PMD_DRV_LOG(ERR, "RESOURCE - CPP mutex trylock failed"); goto err_res_mutex_free; + } nfp_cpp_mutex_unlock(dev_mutex); @@ -171,6 +177,7 @@ nfp_resource_acquire(struct nfp_cpp *cpp, dev_mutex = nfp_cpp_mutex_alloc(cpp, NFP_RESOURCE_TBL_TARGET, NFP_RESOURCE_TBL_BASE, NFP_RESOURCE_TBL_KEY); if (dev_mutex == NULL) { + PMD_DRV_LOG(ERR, "RESOURCE - CPP mutex alloc failed"); free(res); return NULL; } @@ -182,8 +189,10 @@ nfp_resource_acquire(struct nfp_cpp *cpp, err = nfp_resource_try_acquire(cpp, res, dev_mutex); if (err == 0) break; - if (err != -EBUSY) + if (err != -EBUSY) { + PMD_DRV_LOG(ERR, "RESOURCE - try acquire failed"); goto err_free; + } if (count++ > 1000) { PMD_DRV_LOG(ERR, "Error: resource %s timed out", name); diff --git a/drivers/net/nfp/nfpcore/nfp_rtsym.c b/drivers/net/nfp/nfpcore/nfp_rtsym.c index a34278beca..d15a920752 100644 --- a/drivers/net/nfp/nfpcore/nfp_rtsym.c +++ b/drivers/net/nfp/nfpcore/nfp_rtsym.c @@ -60,8 +60,6 @@ nfp_rtsym_sw_entry_init(struct nfp_rtsym_table *cache, sw->size = ((uint64_t)fw->size_hi << 32) | rte_le_to_cpu_32(fw->size_lo); - PMD_INIT_LOG(DEBUG, "rtsym_entry_init name=%s, addr=%" PRIx64 ", size=%" PRIu64 ", target=%d", - sw->name, sw->addr, sw->size, sw->target); switch (fw->target) { case SYM_TGT_LMEM: sw->target = NFP_RTSYM_TARGET_LMEM; @@ -227,7 +225,7 @@ nfp_rtsym_size(const struct nfp_rtsym *sym) { switch (sym->type) { case NFP_RTSYM_TYPE_NONE: - PMD_DRV_LOG(ERR, "rtsym '%s': type NONE", sym->name); + PMD_DRV_LOG(ERR, "The type of rtsym '%s' is NONE", sym->name); return 0; case NFP_RTSYM_TYPE_OBJECT: /* Fall through */ case NFP_RTSYM_TYPE_FUNCTION: @@ -235,7 +233,7 @@ nfp_rtsym_size(const struct nfp_rtsym *sym) case NFP_RTSYM_TYPE_ABS: return sizeof(uint64_t); default: - PMD_DRV_LOG(ERR, "rtsym '%s': unknown type: %d", sym->name, sym->type); + PMD_DRV_LOG(ERR, "Unknown RTSYM type %u", sym->type); return 0; } } @@ -366,7 +364,7 @@ nfp_rtsym_read_le(struct nfp_rtsym_table *rtbl, err = nfp_rtsym_readq(rtbl->cpp, sym, NFP_CPP_ACTION_RW, 0, 0, &val); break; default: - PMD_DRV_LOG(ERR, "rtsym '%s' unsupported size: %" PRId64, + PMD_DRV_LOG(ERR, "rtsym '%s' unsupported size: %#lx", name, sym->size); err = -EINVAL; break; @@ -396,10 +394,9 @@ nfp_rtsym_map(struct nfp_rtsym_table *rtbl, uint32_t cpp_id; const struct nfp_rtsym *sym; - PMD_DRV_LOG(DEBUG, "mapping symbol %s", name); sym = nfp_rtsym_lookup(rtbl, name); if (sym == NULL) { - PMD_INIT_LOG(ERR, "symbol lookup fails for %s", name); + PMD_DRV_LOG(ERR, "Symbol lookup fails for %s", name); return NULL; } @@ -418,10 +415,9 @@ nfp_rtsym_map(struct nfp_rtsym_table *rtbl, mem = nfp_cpp_map_area(rtbl->cpp, cpp_id, addr, sym->size, area); if (mem == NULL) { - PMD_INIT_LOG(ERR, "Failed to map symbol %s", name); + PMD_DRV_LOG(ERR, "Failed to map symbol %s", name); return NULL; } - PMD_DRV_LOG(DEBUG, "symbol %s with address %p", name, mem); return mem; }