From patchwork Fri Apr 19 03:26:29 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 139524 X-Patchwork-Delegate: thomas@monjalon.net 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 A374343EA9; Fri, 19 Apr 2024 05:26:57 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 80E9440633; Fri, 19 Apr 2024 05:26:55 +0200 (CEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2108.outbound.protection.outlook.com [40.107.236.108]) by mails.dpdk.org (Postfix) with ESMTP id 16BAB40273; Fri, 19 Apr 2024 05:26:52 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=NhAG8zlR6op048oW+NevbhJ4kO/1606v6xzYD57Q1S4jlXJprae2QnDUZPEI96krRYMDZzZYhThzmYygZkuJnBw6YBg0nSm3w/ZDc4yDz3px0GltvD+gZRHj429s3wG8SoK3OX29joaU/EI0MlQ3Os4NookVQu6L8bYqHS02xDftshwS7NihkgqvlGfYa13csRpnnGn63C0wOVEEr5STpOZ9M/uhla9mlt7THWrzsOhjPp+gMqmWr75k3oPmsS5G1Qp7/LNjnhyRgthcd2FYRUhKcz0luL1XYw4dx6YeF2E7B0eqZQIeup+ryi83UVdw6rgM3obrHNPkxD2LnTTUqQ== 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=xLhataWRZeS2l3E9kPIQQk5JFrqIwPjJx2R0fzhpRnw=; b=X8e6fmcT5/iNBjVCCZKExB2SpWYg0ipyylt/qrv350gylmOR0U2w4GNBecz00hRfTS4Mej+71YJGnUT8nPiU7PlXgBVfU29wLHEkZ1eEnG+MPY/diXzukLceYXyavIzr8PH7iOF4ymLSMlHa/Uiz1ns1zfB9ppH4WrwFU7TlJRXx35cIDWAaTxEJSgpDqUvWZQ7ppEwIX8zJ2AcmVw3Q7gdgHdKySRVKBvmBGmO8sh0WuYeUeei2Dzr5QvUoCK2dBSzQeXQ4qAAi6K39n16MzCtWevHtKD2McS/sR7GVBVxCjW3XqrMSgKoJdS90AIbTS2DEocvqBdzHOQkiYQC3cA== 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=xLhataWRZeS2l3E9kPIQQk5JFrqIwPjJx2R0fzhpRnw=; b=W+pFi9mcHvgC5zy/99RRMd2pKf+0O4fdVPo+EVaj+IcVKHczyga+uCtdieZ/mvOX17wCramtmOoekAbb2zTK2iEK7Fzc3CRVC7Ivwsrdfx6shLpZJLgYEP/EFpAzwSQf+AqoAKmI3VD6vheI5oKNRjH9n7z0JdcZx+4U5z8KGwA= 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 PH7PR13MB6170.namprd13.prod.outlook.com (2603:10b6:510:241::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7472.41; Fri, 19 Apr 2024 03:26:50 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::ec12:7411:559a:850e]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::ec12:7411:559a:850e%5]) with mapi id 15.20.7472.042; Fri, 19 Apr 2024 03:26:50 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, Zerun Fu , mukawa@igel.co.jp, stable@dpdk.org, Chaoyong He , Long Wu , Peng Zhang Subject: [PATCH v3 1/2] bus/pci: fix secondary process PCI uio resource map problem Date: Fri, 19 Apr 2024 11:26:29 +0800 Message-Id: <20240419032630.1215256-2-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20240419032630.1215256-1-chaoyong.he@corigine.com> References: <20240129092231.3531217-1-chaoyong.he@corigine.com> <20240419032630.1215256-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR01CA0008.apcprd01.prod.exchangelabs.com (2603:1096:4:191::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_|PH7PR13MB6170:EE_ X-MS-Office365-Filtering-Correlation-Id: 918d5302-030d-4a17-fd93-08dc60208d42 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: AVd96sVMNnHSCq0LJ8M0LwQge9fmLqyyCTgZHSV0XYn6frath2dcqsLZWvuftwisEbCVM90IMNfAG7WKwYSFxZ1Ty8+lwZI/J2fbbNo7hYdUuFe1YgrATqM7qP/o1dPJjxTKxYxXZAmOzEZcjWiSW0Q1tEHJ22A3iRbx7mqYkI7tqgYPcNghsnQiqQwJm6TXbxQyZv0Ilfw3VdRzNrQMAO57gUOoCt6TetY9MnT4vHrdJSKm7OLhwRkhw2X7Z4+1aLzgRgexm/852iDLR66NlPFkM+PcDQ0ilKPSFDOu07J3+QmKipEuy+LjFidV3naY+EC0iFLE/Ig/exy2AZATTJaYOR2lus8kfVeby/LOkq0YyfPoQtReK0dm5Fu7lifAjrY8M8RjoVIjEakiYslEk0Rd/22ZKL2eOQnHH1udEozH044cGXxiNBmvcFxpuhHGZdTHygp41jkU65nGpvGcNHq7NRW7yZ35TvKZSJ2/q/n9mrKmVoknNW2zyu13XULmMIC2yvI3sdu+9fEh2oHyofaypQpo8ZkuQ1rt8RbF3p84hGlBG2r2tJusn+l9ChBltKjz3XRkQcOGWuh8mADMnJT5HOHtq14BA11On1iThRqwRvPK0ludnPyjvpeRclIfo0J8gnF6+gfpSdjjV9BKaZlJpb86yWGEmycKt14ZY84jj8ehgWz+Cz3zw19oEE6eYOdqD8Z1JuswxZSPvUiV6fKlnlFtsFJie8zpJ6r7iDdPFcNaX2VO1uk360m3tAa2CQSwK8oEWwnz2NSaCCyRa9cO3EvsaiVbXs/D26fo8biMFF0SG3cJwE7J0dQx7wdUENF5s+yNRQrPPQWELmaNJ9qv6xi9Wm7r2btGEAHemxoYOMPi7pPlzn5Ddp0c8TzhQI34ebAt1AgvdfEIALRzisGVkyCoPdJCEg/bxZu8oR2m15MnyQOh/IkU+8nI3I6Ng5kIBH0xH4yJFpSNfRXUqLMDNcqkOTSgQToWfbPrnS6jMY06QF7arbv7XQN+yrbtw41UmBatF1KDAkKUtJEzZODQKpgwQ0USfiQTJ/6DPwwXwuUkA4qN0dT62HohZYRMzlKAH65+FEzIWfgi/kZRd2K/19zdFtEOCzhbt7JmEyzIsCUEiuAFXgudkIkXScy+Gn3TF3AlnYAczASYupxh2yjCmu98Hu0gbWu4Ts1n60IPW2eedoPYFflAHOtTmWCchw0LJt0jWsu405hAFfRG5RrOQPeBjiG+bGbUpN1QRBggrCKxrImXAJel9bVV3wQpnnkpDFCt6bemQaeGBduYkBgwOxMFHOWqGi5M+Mcdyt8oGOZ6lZaO46nljmYAwjYbplyMpWU8sZXyhdrnnhNXng== 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)(1800799015)(366007)(376005)(52116005)(38350700005); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: 0XP8RBVEeuSbD2KhmquUoFf7fNsN4v61XrY2cvlUx551gf62ZnivMjmMTjdpvY/SVIWrlCJw42NU0nt+5IdAeeAg/SShPxC3TV1QStK2NIh6f/K+1qoKabuYcVbXyig1FIQj9Jgw0WmQS2BYc4MrminwU0OLvLdEUq7LZ3Ki9opdLXf1plXGf4KVAvj0Z6ycVw+1nDJMu9dNosT0rOErZJqGYayOL9N8syEM3sN9Z8x2P4Goly63L/NZkVqmV84WSKbv7/eD1Nc9q6LNALWT0fpCEX/kLgb5jF7SW3iINi9PZRy7GcZPJ5YIuzHmJg26Zr0v9BOZpS03TfUz5SkUKOZ4ywjMsv8BaATPSvGdogyGxIls6TJ4/ltwbCE9+daEcPh1LEMfxcufhULUJX/AENZY4vfljn6EXslFYfeNAPmRb/5/541/r3CLU9j2pQyMmtsyeP5HWLUPWgsj9dYGxP4hQ7QvCQyYBd7l0WrqAKptcKaYtIncHewJ+Wrb92I9gFyOkLzxWBr/no95sQSFzaqobXLamKgyXo3+johNJeeYzH2FPqhEzKYPFYUpol3LC8Z4BXT5plZQi6G6ceLq1fqty343Ceji6YdOXC9klSX/9t7mNSmEUux9xgjYg/sNxw+7RpxzL9XxzjNqAG3QKIn0jWdX4pfLrcEdgPxuWY5PGwt9IU1Lqh/rT672bpclcXbJWPpv2V3XTHKvlV1+tFijUpNlLer7Dam/4Jv+DD9Nz0kse0PVSdzKRAiHq6SWxf8qqI/HnH8vLarGGKcTPi4ldLAlY4embiPcxFKWmAF100M0XAegYfwVaCuvQylzb3LYLtU323Khr/KXuu8W8o8solAeQiM+moR6Fhs8rQ4I5YUhAWcbmnsfaYjP4GyYFJh9HdyRTLiPvJ/bA07XXlXaBC41EPwbtZ5TTXLnRCc8rGxZcGCAWLsDHYYo8g/ipe2Wa4E9p16iIhU3ibCez1rBA9ongK4ex9DXZAGLZiult14uN51hES4lcyuVA7HrVTZLI1b4LQZmzSS3B11QNCf936L0K0TM+Qwcel/9U0xaw6YoFYjxrll4vUndvQvoNrlxTguIyqXZKVl9kx7XiKy0O8ADzCMyf/nhmWiRfrFKPZ0xxqZsuet+Vj9T544aOLP9GvhALce4Jwi2rG0mHfk4jGTUCFmAe9QNER0trskvmbSeqRVCGRya9hH3bp3vzUtMX1UKswjMiRLnD0NNFCzIeyUO6AcJLRZ8q4IrRwsXog5QivqD2Yk+bZW2novoGghfSwdlLgNij4fm6bwXgU6g+vVgBDyO9SrROlb5b3XnTcOxEyzGdxbpGvPMKTFmxRZzS2zNemGOPblfMDlhk9caGYpyZSRRDmj4VvRwhAp0B8AaN3O1Z2GvA68iVi8KiXO8ln2z6tk56gyXRS+AyewnhxfKa2PkDlZjNJeUgFxh5KmrUZ18ka4sWL/cJFHqOTinqQ0dtFHzMa6nlmR+bT/6ALqLbZyelwJCScdCvN23b66TU2YREpKof+fF/KbBZEN6r3cPY4JOFjpN6WyUZcfLEDkdhKrrW2wMp2CUKVBLHOczG8dkYL5NVXtGRp1uBnee4Dv2HLfLiBMDOhGR+w== X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 918d5302-030d-4a17-fd93-08dc60208d42 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2024 03:26:50.6358 (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: Bev7dUSxs/eJI/NpEEHFEXR32x37LMhiteGpNDVp6u9y/SVes6Rv/ZHkN1C2I5rSlCVtnCmJA/okx/3+uKs7xINQ1NJJWjwn1TQIs9ojMu0= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR13MB6170 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 From: Zerun Fu For the primary process, the logic loops all BARs and will skip the map of BAR with an invalid physical address (0), also will assign 'uio_res->nb_maps' with the real mapped BARs number. But for the secondary process, instead of loops all BARs, the logic using the 'uio_res->nb_map' as index. If the device uses continuous BARs there will be no problem, whereas if it uses discrete BARs, it will lead to mapping errors. Fix this problem by also loops all BARs and skip the map of BAR with an invalid physical address in secondary process. Fixes: 9b957f378abf ("pci: merge uio functions for linux and bsd") Cc: mukawa@igel.co.jp Cc: stable@dpdk.org Signed-off-by: Zerun Fu Reviewed-by: Chaoyong He Reviewed-by: Long Wu Reviewed-by: Peng Zhang --- drivers/bus/pci/pci_common_uio.c | 40 ++++++++++++++++++++------------ 1 file changed, 25 insertions(+), 15 deletions(-) diff --git a/drivers/bus/pci/pci_common_uio.c b/drivers/bus/pci/pci_common_uio.c index 76c661f054..432d68b12c 100644 --- a/drivers/bus/pci/pci_common_uio.c +++ b/drivers/bus/pci/pci_common_uio.c @@ -26,7 +26,7 @@ EAL_REGISTER_TAILQ(rte_uio_tailq) static int pci_uio_map_secondary(struct rte_pci_device *dev) { - int fd, i, j; + int fd, i, map_idx = 0, res_idx; struct mapped_pci_resource *uio_res; struct mapped_pci_res_list *uio_res_list = RTE_TAILQ_CAST(rte_uio_tailq.head, mapped_pci_res_list); @@ -37,41 +37,51 @@ pci_uio_map_secondary(struct rte_pci_device *dev) if (rte_pci_addr_cmp(&uio_res->pci_addr, &dev->addr)) continue; - for (i = 0; i != uio_res->nb_maps; i++) { + /* Map all BARs */ + for (res_idx = 0; res_idx != PCI_MAX_RESOURCE; res_idx++) { + /* skip empty BAR */ + if (dev->mem_resource[res_idx].phys_addr == 0) + continue; + + if (map_idx >= uio_res->nb_maps) + return -1; + /* * open devname, to mmap it */ - fd = open(uio_res->maps[i].path, O_RDWR); + fd = open(uio_res->maps[map_idx].path, O_RDWR); if (fd < 0) { RTE_LOG(ERR, EAL, "Cannot open %s: %s\n", - uio_res->maps[i].path, strerror(errno)); + uio_res->maps[map_idx].path, strerror(errno)); return -1; } - void *mapaddr = pci_map_resource(uio_res->maps[i].addr, - fd, (off_t)uio_res->maps[i].offset, - (size_t)uio_res->maps[i].size, 0); + void *mapaddr = pci_map_resource(uio_res->maps[map_idx].addr, + fd, (off_t)uio_res->maps[map_idx].offset, + (size_t)uio_res->maps[map_idx].size, 0); /* fd is not needed in secondary process, close it */ close(fd); - if (mapaddr != uio_res->maps[i].addr) { + if (mapaddr != uio_res->maps[map_idx].addr) { RTE_LOG(ERR, EAL, "Cannot mmap device resource file %s to address: %p\n", - uio_res->maps[i].path, - uio_res->maps[i].addr); + uio_res->maps[map_idx].path, + uio_res->maps[map_idx].addr); if (mapaddr != NULL) { /* unmap addrs correctly mapped */ - for (j = 0; j < i; j++) + for (i = 0; i < map_idx; i++) pci_unmap_resource( - uio_res->maps[j].addr, - (size_t)uio_res->maps[j].size); + uio_res->maps[i].addr, + (size_t)uio_res->maps[i].size); /* unmap addr wrongly mapped */ pci_unmap_resource(mapaddr, - (size_t)uio_res->maps[i].size); + (size_t)uio_res->maps[map_idx].size); } return -1; } - dev->mem_resource[i].addr = mapaddr; + dev->mem_resource[res_idx].addr = mapaddr; + + map_idx++; } return 0; } From patchwork Fri Apr 19 03:26:30 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Chaoyong He X-Patchwork-Id: 139525 X-Patchwork-Delegate: thomas@monjalon.net 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 91A8A43EA9; Fri, 19 Apr 2024 05:27:04 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 0C52B40693; Fri, 19 Apr 2024 05:26:58 +0200 (CEST) Received: from NAM11-BN8-obe.outbound.protection.outlook.com (mail-bn8nam11on2112.outbound.protection.outlook.com [40.107.236.112]) by mails.dpdk.org (Postfix) with ESMTP id A8BAB40608; Fri, 19 Apr 2024 05:26:54 +0200 (CEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=DLqa2V47bPauE+0MXnwltAUjXpbcSqibtNurLOKS2XAxcQaBjNXopA39+yc7dQwuqCmMkE4ObN1j5KcTVlYjxeOjucMPCkdpzFieS0VzWfmMh0b0BXxoVQrIgeD1Q3B/FPqf3cjTTna7cjpEP8M5RKoUERTYKkrlFlRAGtkMxG4mqQIzPihIxIQ95ITtKV1RASYei6fUIfkrvKRGEKP4v6Wz4BXC3laCyhtn/33xjspeuntr/UtkFm2PT2LrHJ35t48Oq1Gz03Bw7W60pxChD2IOe8SZeRTIyb502WKVr3tI9dc5S9mfIN3RcE9WySc/ODBMoom8Olpjh8U3ZznV1A== 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=sIAqd443W9sHgM64xZS4RMueawsm4oE8C1S8pBZqJFk=; b=f6RbOYdnKmtmBfXHjtNTaItGihQmvVM/j4EJNhSEZHTfBVobL1wuTimoRsz1gT2O0s5DL4pKN2D3BfMEzzQ+BUbQvEFUZBxUm4KVzkuzrsRnpw5h6utdd8pBnmT8X/+xzXggJWaOuKPQRhiYpkwo/gNlOYTtKCYyUiTcFwvaKwBBOXsmsTd34eu+l0S3+sW66EuQsyJZ3ueX16gnCbY+RfCHP53zBZGKWD6PJBlNmydROtE9tcfdTiCKO6VsTkVtMtBFuoUMjOQRTsRbqWff2yCnNHu2SKv2IAfd3p00Impmyre/U1sLSfEiT3kz7IazG//MGpyR01Yrad2/Qp2IVg== 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=sIAqd443W9sHgM64xZS4RMueawsm4oE8C1S8pBZqJFk=; b=eQv7QBgttDneOM7Qa+RMjHrLaiJAQH37+8voKQj4SILKcK5qzRJroybWQ6jJvxhmtE6Oj4t/HNKZ6e1/7IL30bimbOu1LBO+TLFnAnLixJfDIPbHiMcYcWdzx7jue6TbxoqZTewYaPJE5UpjrUEKnd9mrGr+7ZsY2sLrM64jUB0= 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 PH7PR13MB6170.namprd13.prod.outlook.com (2603:10b6:510:241::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7472.41; Fri, 19 Apr 2024 03:26:53 +0000 Received: from SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::ec12:7411:559a:850e]) by SJ0PR13MB5545.namprd13.prod.outlook.com ([fe80::ec12:7411:559a:850e%5]) with mapi id 15.20.7472.042; Fri, 19 Apr 2024 03:26:53 +0000 From: Chaoyong He To: dev@dpdk.org Cc: oss-drivers@corigine.com, Zerun Fu , mukawa@igel.co.jp, stable@dpdk.org, Chaoyong He , Long Wu , Peng Zhang Subject: [PATCH v3 2/2] bus/pci: fix secondary process save 'FD' problem Date: Fri, 19 Apr 2024 11:26:30 +0800 Message-Id: <20240419032630.1215256-3-chaoyong.he@corigine.com> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20240419032630.1215256-1-chaoyong.he@corigine.com> References: <20240129092231.3531217-1-chaoyong.he@corigine.com> <20240419032630.1215256-1-chaoyong.he@corigine.com> X-ClientProxiedBy: SI2PR01CA0008.apcprd01.prod.exchangelabs.com (2603:1096:4:191::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_|PH7PR13MB6170:EE_ X-MS-Office365-Filtering-Correlation-Id: 4308a3d4-0e38-4515-3d48-08dc60208ef1 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: ug0pFK9zd29/hvuB+T0EaZtHXCykU23VxGsn/sI3VGErX4i6Gmv59xbNhZ9YidOtOD2LLUnrWbFPPiqp9JgXCfg3JmrbXlyhQVHe92jRzUAzEuzHihnMPpx/tXbT5nhzN+uNhKq6ZPrn6n+Qk9j48YdgfIzvtexvYOKRhqJS6oOsaV6GdIKX4txvxjJ9dWh9Aky8woNuTaOy8sTPkTx9VrSo4O7cWDj/wK7+xsAdg4OBMSLZqqBAvmHup4eoU2n9gWdgkhyMxVgMvamgujqQ2IxSy0rZoWlvzwWGZvVFxM3Bq24Ep9oVsPjyg3blp3Li0IyJC0HDiCteeQ92OYojgVOEimN4cqaPT2Pn0F7GNqFZZRCXl49/rhgt6hf7bP8IX4CEvPfIurX4Xv9pEgjPBUA1LfWPDcjj0VD9+QZVqH8g7Qt2XxrjMMAjcijeNc4yCIkN5LEUesA3HMQLKUIFy8E9OPpZxg2Mw3SUxBEcmNY2cRyzmrzXcsaNMMwcDFL2bPgL9slEKdDqPGI3BqBInifJkWcw29O2b2Ew2FZh5jv9xT6wOvbzN9ixXgaJba/AtuMutgU7AMWvnfmzFciY7NBzW/Jl6n15h6BV8nFPTDG9d7wZRkvZzPni+QBf4GuKn76K/5ZHB9Fx1viwboT0IOCXUKK+RPyiay1fcnBic9uR668gG72vQOT8o9lvMqrkcWvcuCrQojLw3WbufkbFRBh/eTcXwOzj4jGCo0/28qFumH3RnRA81+TmZsNC0/SaGtHftrD4W7HFRl9SxDJ/uZR8gIPr2Qgw3DD3dsJPACZBfM4IJG2iRX3prq2xiXi8m1Ihg5MkRwPJlijo2gs50bN0nswPdwaV5WOJGSMST16BHQLSbdsMP5g/FXTlddd7LKXdCrLtsOXq31OCWaNT/wOZ+xq0HyxRtZ6xY/MAUbgnSiJNw239x2z/PqJR0qvWSRWK+Lv6pOXRouMKdrxL3BPlBRvkh/5R4tRscvC4qhtyuWXE5gCrFtW3w4WqPoY9Txmg1K2rtJhqeFm/s3BhiPIevc2phuvDIVX1WweA5IwgSpd2QCZnkpfq6MU3ZyVcFJMu14QfQr0xx9te9agyb2659nlSUj/BtWzER1eTZdEMV0XY+W2Ytjdj8sd9iCV321DxHEyVLRqsJc/SaCJXEHQT+IvQJrKaoVAv5P8dzSw9JoWsqNeoqEB29MDjQFA1mFWrnAaCYcX59jTdcPQztVhn2hMf/vigHM/9p9ic8aIMGhmtLlthFfI9XSxcLhFBdxswPOFnLeaL9n+9fWQ8iXrxa6EZux27p1ZE4sPy+VULSPEEOYl2b5c1IWca0CBYoN3u2mKWKU1YNiwZXDFhtA== 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)(1800799015)(366007)(376005)(52116005)(38350700005); DIR:OUT; SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: x/mjA2tKbUs6wxWFZcP1vH+uJ1VNWumnQulzEeNEPuw8Y9cKsKHUJvmOASB2gtfJYPkvCvHQSntowaMNrYBn0+eWIU8fw/KlSpMDODm8hxTBHjUM1VhRaa0CPiBy5EHnPTKIV9KpUY8RVdofftji0brq6bpSxJrzteXcGr6eC2dUWg0odXy3eM8NLRwHfEXtUuIZUD7SkTNHALKfX/v8aUXklYKSNfnkGh7qCiAmVowiPNkCYSCgGcTA2EGKftAof7Fr74olhkrPdNHeot2WAqMLRK0mx2+bhPF9RlZCLeP7Y8PaY55LvUuGzUaVMVZ+1VR+FLkwBB5P/apwKvzWm3D2B5HRW2HA01MWQ5IwugtiuzObsJC2+bEVAA2Ekv5O8V748jDxAt+AJJFEj7BS6fNEXuZynrkHX6UGb3kXjusM2PX2uivYeQq6kF4etfVXBwxz4lbt//IeQH+WVf9l7CD6jKIq6J9vAQ+Sk6nuRz0CyAKbWcdlaZW6swo7vbW1HzfHibCnQXN8AkD+GNBvTHFcSCvZ25Jq2thFuDhUlpk0tceOyjCwLDohpFVd7M0SZNe9rx2XQklV2rVSyMoVIEocyfFDOomJbIzc7B1HLRtf21+dOw5AEHEwP3QoG3EvoBHsQJtIBuve3voAgm4oJ+ig6NToBJK2xQcmXcAB+BihTgAbz0lnCPlksYGVQ306Y2MRNIOXaS/FpFKgxBs3C2eiyJcX81wh5DP7PJg716byO1UqgoQmzm/skhDaVhhGioStjAf51l5wwLp/d6AzexU98codgca5ygA5CXdhq8Ge6zHJRYslzF/jwrdzA8WpGB+uKVLAFx2Ous6xSmtd18j0ntd4gT/EpEzI8C8DBeBUF1JzZSvomORoLePHoujbGL1T7ltOiHLKQMLFZiei2WB+Dp6X1upvecVwkgwU3Z6Jh2qhT7oLfaQvMBEro5KTxNZHcpYKggINLB1LXLxVXcrbIclNp+9edIVDzboa9Bl082+yzJutnwZAcAr2xkZiX1lJbG+PR6Bmyp7IBx6n3kYAN5lwtJSOeLdtIdOpOyMan2ejXsPiT6pzYPXTkx6l/GIZzef1s+keEOhD5k3+yv6Hez+nccTNxhWTq27clWIULjnYUZOzfm5vtTut/QgKhHXCbzMnCMTsSAkQs+o6FSXxS2DaFjiITkngemRTyfC08IV8FPqNl52ZOyxIum4U6e+Xk0Ozt2p+XcJdFtN4xL7e5AVQuk+/pWdyZ3Pd9Gdq1RcEhfznn9M7831xv6T19uZDvEe9ozJ3w+hSvi8SF2X7VF8fjmRV6ounayrAMVascZGT1g6aFL2KQ8fm0LBCGvSNwzu8gwrJfWiIDjfu5JNpmpnxbRvDzpZj3vLvkEb54m+GkjbJMESu2XGWXViHpzLiFAZURiSWWDB3dw5+2MypZMD3o4BMkDYf9MYjAN3oxpoYwHNHzAVGVqdCDTnA0snNReoo5lMLf+kJ048ZgKfpL5iJ4xdYANATI5sl0y2c857kNm0dEgmCSLdiEp1mMDJdklcr+4CdwvF3N6bsa4VCzn/ufexiRlqaKWvvDqpq11V55rWgUp42diz6w9GpyqxtFNcyRXYwhWavVl+dyA== X-OriginatorOrg: corigine.com X-MS-Exchange-CrossTenant-Network-Message-Id: 4308a3d4-0e38-4515-3d48-08dc60208ef1 X-MS-Exchange-CrossTenant-AuthSource: SJ0PR13MB5545.namprd13.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Apr 2024 03:26:53.4664 (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: j6MzpAPq1yLkNJ5ZyM7kSfBmp35u2HnFYfQ1c/fbRl65HEjjo5h+FgK8MqtEO15wNQNV/sIPgWJRdxH+qVSeiz5exb8fV1AVGnFkG56UOSc= X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH7PR13MB6170 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 From: Zerun Fu In the previous logic the 'fd' was only saved in the primary process, but for some devices this value is also used in the secondary logic. For example, the call of 'rte_pci_find_ext_capability()' will fail in the secondary process. Fix this problem by getting and saving the value of 'fd' also in the secondary process logic. Fixes: 9b957f378abf ("pci: merge uio functions for linux and bsd") Cc: mukawa@igel.co.jp Cc: stable@dpdk.org Signed-off-by: Zerun Fu Reviewed-by: Chaoyong He Reviewed-by: Long Wu Reviewed-by: Peng Zhang --- drivers/bus/pci/linux/pci_uio.c | 5 ++++- drivers/bus/pci/pci_common_uio.c | 8 ++++---- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/drivers/bus/pci/linux/pci_uio.c b/drivers/bus/pci/linux/pci_uio.c index 97d740dfe5..6680e42efb 100644 --- a/drivers/bus/pci/linux/pci_uio.c +++ b/drivers/bus/pci/linux/pci_uio.c @@ -237,7 +237,7 @@ pci_uio_alloc_resource(struct rte_pci_device *dev, } snprintf(devname, sizeof(devname), "/dev/uio%u", uio_num); - /* save fd if in primary process */ + /* save fd */ fd = open(devname, O_RDWR); if (fd < 0) { RTE_LOG(ERR, EAL, "Cannot open %s: %s\n", @@ -261,6 +261,9 @@ pci_uio_alloc_resource(struct rte_pci_device *dev, if (rte_intr_dev_fd_set(dev->intr_handle, uio_cfg_fd)) goto error; + if (rte_eal_process_type() != RTE_PROC_PRIMARY) + return 0; + if (dev->kdrv == RTE_PCI_KDRV_IGB_UIO) { if (rte_intr_type_set(dev->intr_handle, RTE_INTR_HANDLE_UIO)) goto error; diff --git a/drivers/bus/pci/pci_common_uio.c b/drivers/bus/pci/pci_common_uio.c index 432d68b12c..8688862738 100644 --- a/drivers/bus/pci/pci_common_uio.c +++ b/drivers/bus/pci/pci_common_uio.c @@ -106,15 +106,15 @@ pci_uio_map_resource(struct rte_pci_device *dev) if (rte_intr_dev_fd_set(dev->intr_handle, -1)) return -1; - /* secondary processes - use already recorded details */ - if (rte_eal_process_type() != RTE_PROC_PRIMARY) - return pci_uio_map_secondary(dev); - /* allocate uio resource */ ret = pci_uio_alloc_resource(dev, &uio_res); if (ret) return ret; + /* secondary processes - use already recorded details */ + if (rte_eal_process_type() != RTE_PROC_PRIMARY) + return pci_uio_map_secondary(dev); + /* Map all BARs */ for (i = 0; i != PCI_MAX_RESOURCE; i++) { /* skip empty BAR */