From patchwork Mon Aug 5 03:25:42 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Huan Yang X-Patchwork-Id: 99126 Received: from sy.mirrors.kernel.org ([147.75.48.161]) by linuxtv.org with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.96) (envelope-from ) id 1saoMZ-0003Ci-1R for patchwork@linuxtv.org; Mon, 05 Aug 2024 03:26:27 +0000 Received: from smtp.subspace.kernel.org (wormhole.subspace.kernel.org [52.25.139.140]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by sy.mirrors.kernel.org (Postfix) with ESMTPS id 388FFB20C13 for ; Mon, 5 Aug 2024 03:26:20 +0000 (UTC) Received: from localhost.localdomain (localhost.localdomain [127.0.0.1]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 0C6C713D50E; Mon, 5 Aug 2024 03:26:14 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b="ASq+hjIp" X-Original-To: linux-media@vger.kernel.org Received: from APC01-SG2-obe.outbound.protection.outlook.com (mail-sgaapc01on2040.outbound.protection.outlook.com [40.107.215.40]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id E9AB047F5D; Mon, 5 Aug 2024 03:26:10 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=40.107.215.40 ARC-Seal: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828373; cv=fail; b=i0QD+Sjr8ANJiVI7BPnUS61+2w/IZaahxXonQ5W5puatfzvLxSS0Xw/y9i3aWHONH7Lg4iTOhtmngnkPAbU001lUc/zS0K+tsC2U8XJmJIfOuXfauQ7rh3xuY9UmNFC3s1IMbMB+DPu3l0ng7cHGldSmAwfm8+iZbsLY8Jh/WBc= ARC-Message-Signature: i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1722828373; c=relaxed/simple; bh=9bDEQMr+wUuY5vgPFBJO2dlzeGW+XS2kirumWIpladY=; h=From:To:Cc:Subject:Date:Message-ID:Content-Type:MIME-Version; b=Xp+gSJoxcgJqpwdRot88OLNtZ8U/SmrczA2GNhiIfrPbhG4h6FSNW5uCTfdtMQJu+D/pMe0rl3SNsjMSkcUz5hnKjS92h97ginASYI7Zn8L959CdrJ1a/VCdmtVbskrL3A+WRntjcIIRPS5Iqk1QkC6/APFbb9PG/zxuq+PeiX8= ARC-Authentication-Results: i=2; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com; spf=pass smtp.mailfrom=vivo.com; dkim=pass (2048-bit key) header.d=vivo.com header.i=@vivo.com header.b=ASq+hjIp; arc=fail smtp.client-ip=40.107.215.40 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=vivo.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=vivo.com ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=qYmtZzd0X3XzutAnE4/PsHOwLpZEvW+VH2Wj+xanttWGISA51vclZP2w5b7nKz24qxOSFUn55wlhuHHw1YspiudxbZLvgQQDDihB1XLwWFS1u3UYckGfGKVCqmTK37CE8oT72bVvsEAxCOuAllvAWq0x9dukvSQPuR/OKSr9m3QoJ9xsXRbtnCecNQw70UFZt7ESa0e6ry/go9YSZnfYKFg9IQKZs+hU3LQJ14SQz/QDHROdxhKkJGGT6FIxG4cBFuTYEuV6Q/PX6/3vGSxfjplr3emqlEae08YuwA53cdD7ZvTbFrpJhHv1LWIni/tLpQRZpkOr/Nypqak68GgUJA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=bMMgKldtfDr3QFcMmOCqX4/gPmI6D2ZORAU1RuBPZM4=; b=dt8NUC5h+qxgMOByZ2SPl94eUogQv3+z5v1mZjsScxodygC0H20NE3JGnQkUjmzFpnxKjSWehyZogwC01YS1685f7Nfx878IYIjB/BXuZ8NA2vO8rJdNKBx0LIk+F/2nqNcDeYXc0gn1DY17aqYqnAyUQeky75Qj0Dd3j3JxuYsgTAxrrrThSeLpG77P0L+WISaYUgPQRG6BR/hl7NBHQZq7awnx5DFjHyzk7yTPbV1wepHw99Wv0lZ0mW/Al//7iPKNXnqd9Bei0AesDPFhnoXGt/fNNspy37x74bh5m1vvYH/9Abgm/A0r8AboHwE+lmdrAGDp23WaIBEBwYRliQ== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=vivo.com; dmarc=pass action=none header.from=vivo.com; dkim=pass header.d=vivo.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=vivo.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=bMMgKldtfDr3QFcMmOCqX4/gPmI6D2ZORAU1RuBPZM4=; b=ASq+hjIpweQ6E69ZcnFI5iqbaAL4e87nVDstv0dxJ2qG1DVAo1v3BtKG/ZA0kt5dMpgHSNgzB6Ivaq3tetDt8NutS3Xwy0eNqyyHeOmHCx5F/O7iC/m3VOXvEm3+ACil3+ATOcjcIZBIlLHnD5ZqyMiA/LT+1p69KDmCpJoOolL2uekTc1S+mYWgPsF28FjYwkqyq3yYfC72sE5lWOduhPlTO/bxfa5spP8cJSWuIJxLSqIjBVbVu6/uvn963pDtdsmM+DlAbJSwoHL9OV103+Z3z24nu0ZNrpS3M1gBlKQZ5bvd0aTEzj6u0GHp4JV1cQAFxFbsMVUH48QN6whraQ== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=vivo.com; Received: from PUZPR06MB5676.apcprd06.prod.outlook.com (2603:1096:301:f8::10) by JH0PR06MB7349.apcprd06.prod.outlook.com (2603:1096:990:a5::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7828.26; Mon, 5 Aug 2024 03:26:06 +0000 Received: from PUZPR06MB5676.apcprd06.prod.outlook.com ([fe80::a00b:f422:ac44:636f]) by PUZPR06MB5676.apcprd06.prod.outlook.com ([fe80::a00b:f422:ac44:636f%4]) with mapi id 15.20.7828.023; Mon, 5 Aug 2024 03:26:06 +0000 From: Huan Yang To: Gerd Hoffmann , Sumit Semwal , =?utf-8?q?Christian_K=C3=B6nig?= , dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, linaro-mm-sig@lists.linaro.org, linux-kernel@vger.kernel.org Cc: opensource.kernel@vivo.com, Huan Yang Subject: [PATCH v2 0/4] udmbuf bug fix and some improvements Date: Mon, 5 Aug 2024 11:25:42 +0800 Message-ID: <20240805032550.3912454-1-link@vivo.com> X-Mailer: git-send-email 2.45.2 X-ClientProxiedBy: SI2P153CA0025.APCP153.PROD.OUTLOOK.COM (2603:1096:4:190::12) To PUZPR06MB5676.apcprd06.prod.outlook.com (2603:1096:301:f8::10) Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PUZPR06MB5676:EE_|JH0PR06MB7349:EE_ X-MS-Office365-Filtering-Correlation-Id: 3db27d5e-de4e-4177-59ec-08dcb4fe5772 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|52116014|376014|366016|1800799024|38350700014; X-Microsoft-Antispam-Message-Info: =?utf-8?q?dS8fKQFENEKXtJc/6E+gH24w6cpaeZQ?= =?utf-8?q?gBYue8A4rEl45yU03qaPaqjIIRGtfJP/O9vK+/Z7imIgLI0wBFr5Xv07khCaF6O1g?= =?utf-8?q?gIingtOaeEwyZClpbmeFZMM39eF1bwTvh68qGOcEHY9FUdbQB3+3Jc/v4T8IcebgI?= =?utf-8?q?6WFwM7rzTwdBZmcEhJaMeRysmiCXLOHsbhfwAgsfJHWzlXDy0TMoq/Z6l6B+QTP9y?= =?utf-8?q?pEzBtOC1p2VytG4cQfec7Ld440yg4zlzUtyWvO8jtl8BeXChK9ne8FeWf7XbXTMbA?= =?utf-8?q?ufTyPP6v0ebeDTPZSnTnkdi2R7tqXXwUmW27jlnvNMc+B0ZnbHJT0SFq7hay5rbxn?= =?utf-8?q?TtjoUnP2W40RF9Gd4UwgTWTBofNzOh80ajo7J/Zfi7wl6aHNREAOix7S3CEKwE9nb?= =?utf-8?q?jWdZ3MjfpSIlbYaVdDVzouqo39LjvhnujwWf5lqEvEV8Rad/mKYRYndmYIkJOs4vu?= =?utf-8?q?2U70jrtc+m0kJ8Nn2eecXan3k2ucxhxMrwIbdggNIzbwJ9qOTX1PJ9AYK3AA+17H9?= =?utf-8?q?TeKY6O841WynLsFeEm2I3wwW7SXhARCozmnOMpPZug62nSphgcNEf767fKhDahTeD?= =?utf-8?q?kNa985/vEUB9KtkquyFQt1WGLlhuZ2hYCks5CYUfhg7TCXs7x/ZDLOawiCyxIFJft?= =?utf-8?q?DMP4Hk061aPx3mJkwkkaxvZOGMAyx8BK2GhGe3YeIcs77rHRNUMknXHdijqWBTwFA?= =?utf-8?q?8pqmeJOd7rhETUuLEXhKfFICCoenrd4Oz9hRAQ3SflAJe4xLUbzyKOGM1pbgD22tD?= =?utf-8?q?xzOoq5AkU7t0IKBPb9mbEwW/hS4OixnWhauzeVXWmoahyVoTG49vsH425zwLC7JWI?= =?utf-8?q?qw5v44Ud+1uCQso7WZqjuXF6VTKVOPE+aVhT7zUrlS1rAsKZOGoEnTepmNHG2MPG1?= =?utf-8?q?VFU2d2GC1tnOLaqsEtvEWtCyOfBVLZGk4Cq2GSuj7SfIIZBXKC6WPH1zTzPHl2Xsv?= =?utf-8?q?jLORdcYOLYZeIDFu3AdpoHOLjQHXm0fopnpv3ZmN96a4jQTFTqozO8Tg5txuh1EDI?= =?utf-8?q?XR9nbB2y/vwrl4sxwSehVQTFV8NTQMhdK5omGlUgMJbCTS6Ot6C5cfZxsCG+mWN1G?= =?utf-8?q?OZod1IbpLwzRfx0uVickzaudjNl3xPZreniqlsQoWPLImdkJNbK9U3Pf5xp3z4Cem?= =?utf-8?q?oclITEi1yOWIyu3jY3EqB4DCeBvJrFuJIqAJPTmwwbHp22P402Y43Oq+JX3WGNXBL?= =?utf-8?q?nmOvSjDQAqcmQmCzs4OrBR4/12Z/maOwnY33zYKMvDY6ysmFubF4hoqnIDnvVS4EY?= =?utf-8?q?k4rNyiFzuZ467IbSC0KkKb1EtlQIvWfYbSbHzQConBd4tBrry+wtLPUI=3D?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PUZPR06MB5676.apcprd06.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(52116014)(376014)(366016)(1800799024)(38350700014);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?q?ay1DHPvIAgZAE4uXyH4iFozG3u4V?= =?utf-8?q?MdN/LkVPQRox2C049oomq2VXyLdcjsPk94baY3GlzJXpt2/CMIwOWeNGR17NEuvhS?= =?utf-8?q?IeNAwjyeDoeJYbdXUcPamZQTabUHnHnKpSHxs3pOaxpkzu9Khr+lTuS9rk3bFAf1Z?= =?utf-8?q?fWn2DNn20kMoVFnZniWSwkOLgkH56QMhCi1iit/A+GgT9aQVIK+j8aY6r5Q5vM/yn?= =?utf-8?q?jSubaSEk0QkifUv7kzYqFdZlCvwOsobXRRh1bq5nS2ETtw9Dnzc5SAh93DiWJu102?= =?utf-8?q?y2HXnE9SQi2izn5GnqLlZIkfKRgYIpnr/qPCEKqszR31Tq4qNLDq45J3iHgpwMnMF?= =?utf-8?q?PlHY5XEmSThMEFb+ll020X6dU5oqbz+frKnh59xTkGRHLW9tHQVdCK0ZGRNKjvUpT?= =?utf-8?q?m92i1pJB9i0bOfcPeHQpT07MjtFxkdtMxLEb/5Iqs88cRSF9QOmsMtucfcMM/McGu?= =?utf-8?q?rpIyw/A34Ao8gPtGh5kgC7AY2tPU49LUJHOmr4zaoXNqSHy0U2fLJI5NmaDp8DmgI?= =?utf-8?q?W+GdwJxqhdK7hjzEiWGdp7ZlY74XX3uh1Sg1Egy1QS4mb9qhHptp5Wq1Fwhmyv/ga?= =?utf-8?q?59q/4RnqKjnRl7UznQfJVLZFhO9mJtAeNCqlJEl3rowXwcMYEgxPCZwCz8+BICxZU?= =?utf-8?q?XZaEqfvTvPmRqsVPgHFDDMKdyogZK5170gd4juQbUx0ywL/Pe7vcHPueiXug6jZHz?= =?utf-8?q?/qpA3jyf3mAoaV73hHeg8RYWzXAlMK3VH1sxaHJItonwMmauDFC0ISJ6xkzcvNomO?= =?utf-8?q?AvFC4dT9ilQrtHQQ9PE7eD9OGbT/1ZXNS23Qj2IIhP/zOl6bQsU/NcgvBLxbCxmUc?= =?utf-8?q?5H4P53W2hnOHbULEW0vQNOBG35Qc0xzrOqaghMCncCIzqAM3i4pQpu6XNH9aO+b3h?= =?utf-8?q?LD/0dYd6pg4DZNSrFhLL81GKbHHim5lFGOYYlS6nftQ6sbMY3z5aGS4f6opGzVmnO?= =?utf-8?q?C7Q1wfVtn7bPpzvQErVvLbDMbnQ4uDH+FSES1JJldQ9TXPdIU3QgNBfFAwEv8eSbJ?= =?utf-8?q?6mMAlY7FLjgsz9NfhkavrC7AShdj8wAkpmGVvAs3NF1gMxEK9kmWyniHeqOjQ5i4l?= =?utf-8?q?p2VAnTa+TK9GxA6etmgzTMjMtgo9FSesnxaJZ+gJSxdlvJIjOa6yzgU8BBHepjfio?= =?utf-8?q?oh/CUiAVP83HkdJ88Fchy9tO+ZoH3Uuqykpgl0+CmlUYfnutPbsggsCTrG1fajSmv?= =?utf-8?q?E9h+BvWhApjgRuWPH+ub8ZXGTW428HmbEiaV8Yuj5hSK+GFBrnGMRGqWIx/BOWsP5?= =?utf-8?q?prbZKziMVcR/QYRuM7mBVrSaucl3nZg7U7bGuvX5JQJ9I0qJum0oq/A4Pmo6C+wmT?= =?utf-8?q?N9BHwJ0Hwx6+hMApsGIXCrr9vf6G4aI5mln4C+k1X3d/AICA1D6Ro0clvGQrNhuNN?= =?utf-8?q?mJFGiylthpxI4jpzzm8YlriQM/lvVKxJFtQ90PUkxLTn2uAV39SXEbHl3szaFo1tJ?= =?utf-8?q?pwa1GfoErtOOSRV6S7vMV/jEi9ZG4iAalk3T8UvIv4XebV1Rp9EE+7B7OWY0O/61K?= =?utf-8?q?+9sCsbqBA3e3?= X-OriginatorOrg: vivo.com X-MS-Exchange-CrossTenant-Network-Message-Id: 3db27d5e-de4e-4177-59ec-08dcb4fe5772 X-MS-Exchange-CrossTenant-AuthSource: PUZPR06MB5676.apcprd06.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Aug 2024 03:26:06.3227 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 923e42dc-48d5-4cbe-b582-1a797a6412ed X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 7+IPE1coSyszF+EWbd1lQ8fv3LT+JrzjwbfMraW/SYo4OGnFdKKHo2JWFDTktCatpymmfRM3vW/gEfNpnJP0WA== X-MS-Exchange-Transport-CrossTenantHeadersStamped: JH0PR06MB7349 X-LSpam-Score: -6.3 (------) X-LSpam-Report: No, score=-6.3 required=5.0 tests=ARC_SIGNED=0.001,ARC_VALID=-0.1,BAYES_00=-1.9,DKIM_SIGNED=0.1,DKIM_VALID=-0.1,DKIM_VALID_AU=-0.1,DMARC_PASS=-0.001,HEADER_FROM_DIFFERENT_DOMAINS=0.5,MAILING_LIST_MULTI=-1,RCVD_IN_VALIDITY_CERTIFIED=-3,RCVD_IN_VALIDITY_RPBL=1.31,RCVD_IN_VALIDITY_SAFE=-2,SPF_HELO_NONE=0.001,SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no This patchset attempts to fix some errors in udmabuf and remove the upin_list structure. Some of this fix just gather the patches which I upload before. Patch1,2,4 has passed the udmabuf self-test suite's tests. Suggested by Kasireddy, Vivek Test item 6 maybe requires running the command: echo 1024 > /sys/kernel/mm/hugepages/hugepages-2048kB/nr_hugepages Patch3 changed vmap, which can't simple test by testsuit. But Patch4 restore vmap to the implementation before adding folio. Patch1 === Try to remove page fault mmap and direct map it. Due to current udmabuf has already obtained and pinned the folio upon completion of the creation.This means that the physical memory has already been acquired, rather than being accessed dynamically. The current page fault method only saves some page table memory. As a result, the page fault mechanism has lost its purpose as a demanding page. Due to the fact that page fault requires trapping into kernel mode and filling in when accessing the corresponding virtual address in mmap, this means that user mode access to virtual addresses needs to trap into kernel mode. Therefore, when creating a large size udmabuf, this represents a considerable overhead. Patch2 === This is the same to patch: https://lore.kernel.org/all/20240725021349.580574-1-link@vivo.com/ Patch3 === The current implementation of udmabuf's vmap has issues. It does not correctly set each page of the folio to the page structure, so that when vmap is called, all pages are the head page of the folio. This implementation is same as this patch: https://lore.kernel.org/all/20240731090233.1343559-1-link@vivo.com/ Patch4 === Attempt to remove unpin_list and other related data structures. In order to adapt to Folio, we established the unpin_list data structure to unpin all folios and maintain the page mapping relationship. However, this data structure requires 24 bytes for each page and has low traversal performance for the list. And maintaining the offset structure also consumes a portion of memory. This patch attempts to remove these data structures and modify the semantics of some existing data structures. udmabuf: add folios -> folio array, which only contain's the folio, org contains duplicate. add foliocount -> folios array number add pages -> page array, which contains pages which in folios, offset, size determined by the offset of the memfd when udmabuf created. This patch also remove single folios' create on each create item, use it be the ubuf->folios arrays' pointer, slide to fill the corresponding folio under the item into the array. Changelog === v2 -> v1: Patch1, 3 Rectify the improper use of the sg table. suggested-by Christian König Patch2 add acked-by Christian K�nig which marked in v1 Patch4 Modify the data structure to restore the use of pages and correct the misunderstanding of loop conditions such as "pgcnt". make sure pass self test. remove v1's patch4 v1 https://lore.kernel.org/all/20240801104512.4056860-1-link@vivo.com/ Huan Yang (4): udmabuf: cancel mmap page fault, direct map it udmabuf: change folios array from kmalloc to kvmalloc fix vmap_udmabuf error page set udmabuf: remove folio unpin list drivers/dma-buf/udmabuf.c | 207 ++++++++++++++------------------------ 1 file changed, 78 insertions(+), 129 deletions(-) base-commit: 048d8cb65cde9fe7534eb4440bcfddcf406bb49c