From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 88302C3DA79 for ; Mon, 15 Jan 2024 18:38:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=1WWDGdQcCvp5bnCJCXuN2Sd52IZOjcP+UUApDAzko2w=; b=YR8Dvz0UG9W4gj 4jmaZWWFewxTq9kkr8oFyS835tgm1IQG2b0uDJAyws022hfjAC/fvxpeHA8EMShJLrNwrv4vZI9iO btvIoHFEAfZoEKdb7oeAvX5MLuW93uXNDPYt1V4Bwa6vt/sQPbHUAr5maic8o/fS/VLGzA0SUmF7n TBqmYGQhqLfp/T0Y2FDdowc5ZocIMbWaxZZQDvPZGoMpMvJ2zvpQ5yImhAlMk6gEUZw5Y77mxIeGM P/zxM9H0iccK1H8FDjH9+eAZ+TD1rTn0wRD8oxIpcjPyg5oYKszQMBfR1aTok9VxprdKTWzsHc9z1 It0lLqOVRTxCSy7pxztQ==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rPRqY-009xde-1l; Mon, 15 Jan 2024 18:38:06 +0000 Received: from mail-co1nam11on20601.outbound.protection.outlook.com ([2a01:111:f403:2416::601] helo=NAM11-CO1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rPRqU-009xbD-2m; Mon, 15 Jan 2024 18:38:04 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bNPypzcHqtwmlkR35+8+wG0HQBj4NHh/ggbbP5Ug4dxjcd+vQcUrW8vFhFymcGY7t7IIbDAy0ueN2qeVc4c8m3KMzQ9DFw9+IRb3g9AVhEcqKwgJEhhMq0kPoeaO/spbKR+xyN5qfBm2uLNbYcM2xYRksvk7eJQONIlzlT3Qm0hPOzFMyilPewVamsVlz3qqRBmR8S1+Xobkle2YhXbEEeX9pY3u3ZNypcdbSe/3ixFlcE0xhos7/0Nzleyv7aTvBOiZV+EfucY/iQagUqvjSECYAbqwKZXSjw2inV8HWxBrblob+N8YMc3WJIW3AopnyN4tI00VbcR/2CG6M/8Cew== 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=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=hW5orNKAXX+GGWH59gD80stcd3enkxxX6u1BYpv7LWa8eDsHca6oqGVmaf/lLDJopux6vVidjmY121HMXpbjT5KX9MQtpr/SMMLylPuhqESNqWZWKddM8fsMQBl6qcaZfGNvMhHqq3ySYFDlum0sTS+r8uZ8AbeHOArBpIO9JGdnLgRQ1HMOdu9glJrwlDN8AlRyBiTRLhThkBEJcwPo2MxC7qgWYVH5lsk3+bDU3XT2R3VKKnu8+gf0evdUyCLoqgauj/4+8xpJJpDAafOgU9xrSV/njvvQ0rlxXQtOULAGPKYZDZ4BQw5K9zdxgCFM/vxUoZ4XCwv5TvEk2v1A6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=ObHrXJ1zOxxxYhNPRVtjnyY/anGB4XXQHcUTWVYPKFWm8pSgA//fRjh8Deo4fy/RMswqQ/kVaSieTu+zuUYwbjBW0IV413VMvWblWN+7VTkxEPxCSpKwFEhPUYX65hcHmxlE3GFX9QSh99/lequ4SbCMu9O0yW8FT9KMtplzzkElxtVQlHHHdWxm0tim25SGEQ5JcY5vxydM45pUyJIvxXY70v0nfu1I1M0HXoF2TBJGfx9+jOU3Ww7qbNWlli6oaSsMyhP/k2ikmV4bR56gaQuG0wSS3cVacxox7JTFjSq9KvzknfnloRAKdxXK/9qRZkY3uq6dDNvZciAb0XmiPg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) by PH0PR12MB7792.namprd12.prod.outlook.com (2603:10b6:510:281::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.23; Mon, 15 Jan 2024 18:37:49 +0000 Received: from LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873]) by LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873%6]) with mapi id 15.20.7181.020; Mon, 15 Jan 2024 18:37:49 +0000 Date: Mon, 15 Jan 2024 14:37:48 -0400 From: Jason Gunthorpe To: peterx@redhat.com Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, James Houghton , David Hildenbrand , "Kirill A . Shutemov" , Yang Shi , linux-riscv@lists.infradead.org, Andrew Morton , "Aneesh Kumar K . V" , Rik van Riel , Andrea Arcangeli , Axel Rasmussen , Mike Rapoport , John Hubbard , Vlastimil Babka , Michael Ellerman , Christophe Leroy , Andrew Jones , linuxppc-dev@lists.ozlabs.org, Mike Kravetz , Muchun Song , linux-arm-kernel@lists.infradead.org, Christoph Hellwig , Lorenzo Stoakes , Matthew Wilcox Subject: Re: [PATCH v2 06/13] mm/gup: Drop folio_fast_pin_allowed() in hugepd processing Message-ID: <20240115183748.GR734935@nvidia.com> References: <20240103091423.400294-1-peterx@redhat.com> <20240103091423.400294-7-peterx@redhat.com> Content-Disposition: inline In-Reply-To: <20240103091423.400294-7-peterx@redhat.com> X-ClientProxiedBy: MN2PR19CA0012.namprd19.prod.outlook.com (2603:10b6:208:178::25) To LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: LV2PR12MB5869:EE_|PH0PR12MB7792:EE_ X-MS-Office365-Filtering-Correlation-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NMHcwfKS5YpOgwMuTPz84dvhmzhTBPwFQkMqFqxZNsg8m6LeRCTTa7yhE7dxhsz7RTGXTmjlFmX+CmDqQz3ovOTcLqFrPqaozCanARZgyFmgZC09x0PNLg2EQzt613xqWQFTuOi1X2TdNksq0D32RVt91oITouUSB+IR/hrAJ2BTuGWq31O+I181h7ZeG0d4S4yG+WeY2BrCy5UIDPYZ0hNgS48o757OXenu8elR61aY0r+idnrr9ob7EDSLa266pIsU1ewx8aOfGal2MY1VHtZ6/5h+hBKkrWb/lgvxNgHHiZwkB0FWQZ6/2+od93+J6E7s5weGx3TQ4mQdQqtcG9oUL1aAfPUzuSxWf0tRCE06bALQkqeajpp4y4iVqeeKtEeo3HZ1CW4AdkPPIADtJBbcWlPcbAVP2iH7rZFqXeTQatYiGEKJX194HFPEMfvsBvpLNnhRZAT0N2YSU0akUiSVM1XjH5zT19dUmieeHn/I+a53Mklwk3vRGrj+vw1+1ZBdIsuGq9kIjmD9BKUoEW/jeocziEPSG0TjU9uQmlYd/W8LtxCmMySnNyX7ED3JqQPevdzXvpimrBahey+cLof4MZuDcXp/887nI6JJFyJJe5S+ZwyPVsT8zV9v4r82 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:LV2PR12MB5869.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(64100799003)(451199024)(1800799012)(186009)(66946007)(83380400001)(6506007)(66556008)(6512007)(26005)(478600001)(1076003)(38100700002)(4326008)(2616005)(5660300002)(8936002)(7416002)(6916009)(54906003)(8676002)(66476007)(6486002)(2906002)(316002)(33656002)(36756003)(86362001)(41300700001)(27376004);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?m5QXqIddeL91NXMRxH4Uhbm4kQnyOX6HVsDIEbKpqiG+4PGO8IiaDLf0K5tR?= =?us-ascii?Q?IGSqxvFVSSmNerD1UaysnGHjSNoRWCr8Kv7B7FFmra2vUdsE4soiIkTbLc9Q?= =?us-ascii?Q?gpmkxGHY7Lp9xhviDJxGG0xJSl1uPWgdYW/MnAMRtJgNqj2yENxk2uhfHVZ2?= =?us-ascii?Q?CbIzE8JkfvR2WUKAVuzkV7cqJqtnAxNbM3CGr/PCspOoSot5EZ+iHeKJmhQp?= =?us-ascii?Q?ORg8dPKRLBh2dIxxU/VkuSAqarek5MOAyZk4oTK47FJ5KnY8TM9rzERNj699?= =?us-ascii?Q?+XMNN5KDe2JiKsvynSKqm1YGZUnJGsBF6T0wcDF/By2SRTT+/1K5n7qLiGN+?= =?us-ascii?Q?28XTLxtLlyr9EW7usdUJa8PMHiAyVRrin8H0tMx/AlZQiYwxK4DixfueEwI5?= =?us-ascii?Q?pTmIJbJhRsOZoVwd4KLVyh7Rufkw27UHjCMrBE6hnj+ZulPW/jxBdK/Prazv?= =?us-ascii?Q?+5TU/xGoyE6SYmC5tTOCisWC5ngKR/ThJUTwtS7eFpuiIJC5roJf+ObVY7N7?= =?us-ascii?Q?6kiROaIh1D/YanrIw2nlqhOFkU6gvPmszbYnIKiXKwSESobd+FZtz8nV4VGX?= =?us-ascii?Q?PseeE3ILoZktSbxFbhR/oJz+9jpnjsOK/P7dZH7eswCBJih9aAtIx1KX7dx8?= =?us-ascii?Q?AaBIjbA+T5TYBnAE9x/4xG+JgRnPUIux37/suInYHfd80+uwwxFRm6LE7LgF?= =?us-ascii?Q?usiCdtFDO5d7ZYSxTPbudO4zJqYCTXXnj8BEKTx5BxuPKe8SDTnghpCs25N0?= =?us-ascii?Q?20QdD6GvjvyEu1xDotg8iJcCMy3uMQ1nYXA1mOIeBUvmHhPiaYs9PV1czWV9?= =?us-ascii?Q?uQ1vZyg4X7217CaRyStcdLN82C+y/AkR1cXpW3Jz7g6PANvx2Pgbmz9avujV?= =?us-ascii?Q?TIm5guhgLjFGT3lwDgQtDvymt93hBmPqufpF9S0PtlHMmD0LGW81gk6PFC8s?= =?us-ascii?Q?35gBRMhVBWcEne4POgnI4Ajqk0vUULd67qo5ibzdXSvklH8IDlkLl4dwuFZh?= =?us-ascii?Q?ymsxdV64jY3+Fnd7pTZOkNbeGS60lIP0nldZrprp3qhBVjTypSWAAINSscVV?= =?us-ascii?Q?uHJBjHPIZRTnt5qEedcn2inFFcgWCWJvsd776ZTxQl81QX8x+NV2wiFnAXJn?= =?us-ascii?Q?ya0Eq8WmTQ908Kg1LZZcOKpJvTDxpTDWYreS6wAhVNJ6UjzQz4eKDt1fwEdH?= =?us-ascii?Q?5JfyJFM241llTZYVgZGXvj/AS3eoqoyhWKJB7V0CtlxwTUINydjb9fbNHmyk?= =?us-ascii?Q?i+f//7NADvkDzCfmBl/WIpGB8umB3kyLO04JuS7LBozcTKnK8BbxUwWulWaI?= =?us-ascii?Q?uydUWJ0tva2r8sppmC/Wzm5bF/iO/q6EiUK8Snbe1dKpBEbBZkU15CSQVsvj?= =?us-ascii?Q?JwHPIxypoCueN6GAUE7V6vnIyKjeVOMLLUCqwXsyKE71JuWWEcqZkD2+3erj?= =?us-ascii?Q?v/ejxhfI2a44inKW3Bb3aeOFmhCTXA8Bq7xoVBZjS0EA9UCUtAxNABFD7uKY?= =?us-ascii?Q?ZSNTWl0J6Kn3d+dNOTv8o6GXKP3KUFEm02/du27wp5bALPBkuWBCQX/uKtuW?= =?us-ascii?Q?ykvnDEHgC+YuIA8xnirfEUX8tD9vuVpsszImLq3H?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2024 18:37:49.4941 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: XQurqgZ/9Llavqp3HGEPNNUnZ+2dzwegzvxUdWvdyZgW8AYv2hfdhumQnB9vock/ X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7792 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240115_103802_949265_B9D28DF6 X-CRM114-Status: GOOD ( 17.26 ) X-BeenThere: linux-riscv@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-riscv" Errors-To: linux-riscv-bounces+linux-riscv=archiver.kernel.org@lists.infradead.org On Wed, Jan 03, 2024 at 05:14:16PM +0800, peterx@redhat.com wrote: > From: Peter Xu > > Hugepd format for GUP is only used in PowerPC with hugetlbfs. There are > some kernel usage of hugepd (can refer to hugepd_populate_kernel() for > PPC_8XX), however those pages are not candidates for GUP. > > Commit a6e79df92e4a ("mm/gup: disallow FOLL_LONGTERM GUP-fast writing to > file-backed mappings") added a check to fail gup-fast if there's potential > risk of violating GUP over writeback file systems. That should never apply > to hugepd. Considering that hugepd is an old format (and even > software-only), there's no plan to extend hugepd into other file typed > memories that is prone to the same issue. I didn't dig into the ppc stuff too deeply, but this looks to me like it is the same thing as ARM's contig bits? ie a chunk of PMD/etc entries are all managed together as though they are a virtual larger entry and we use the hugepte_addr_end() stuff to iterate over each sub entry. But WHY is GUP doing this or caring about this? GUP should have no problem handling the super-size entry (eg 8M on nohash) as a single thing. It seems we only lack an API to get this out of the arch code? It seems to me we should see ARM and PPC agree on what the API is for this and then get rid of hugepd by making both use the same page table walker API. Is that too hopeful? > Drop that check, not only because it'll never be true for hugepd per any > known plan, but also it paves way for reusing the function outside > fast-gup. I didn't see any other caller of this function in this series? When does this re-use happen?? Jason _______________________________________________ linux-riscv mailing list linux-riscv@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-riscv From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from lists.ozlabs.org (lists.ozlabs.org [112.213.38.117]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 0C0D7C3DA79 for ; Mon, 15 Jan 2024 18:39:05 +0000 (UTC) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=Nvidia.com header.i=@Nvidia.com header.a=rsa-sha256 header.s=selector2 header.b=ObHrXJ1z; dkim-atps=neutral Received: from boromir.ozlabs.org (localhost [IPv6:::1]) by lists.ozlabs.org (Postfix) with ESMTP id 4TDLWr23Bnz3vc7 for ; Tue, 16 Jan 2024 05:39:04 +1100 (AEDT) Authentication-Results: lists.ozlabs.org; dkim=pass (2048-bit key; unprotected) header.d=Nvidia.com header.i=@Nvidia.com header.a=rsa-sha256 header.s=selector2 header.b=ObHrXJ1z; dkim-atps=neutral Authentication-Results: lists.ozlabs.org; spf=pass (sender SPF authorized) smtp.mailfrom=nvidia.com (client-ip=2a01:111:f403:2416::601; helo=nam11-co1-obe.outbound.protection.outlook.com; envelope-from=jgg@nvidia.com; receiver=lists.ozlabs.org) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on20601.outbound.protection.outlook.com [IPv6:2a01:111:f403:2416::601]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by lists.ozlabs.org (Postfix) with ESMTPS id 4TDLVw4c5mz3cWV for ; Tue, 16 Jan 2024 05:38:13 +1100 (AEDT) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bNPypzcHqtwmlkR35+8+wG0HQBj4NHh/ggbbP5Ug4dxjcd+vQcUrW8vFhFymcGY7t7IIbDAy0ueN2qeVc4c8m3KMzQ9DFw9+IRb3g9AVhEcqKwgJEhhMq0kPoeaO/spbKR+xyN5qfBm2uLNbYcM2xYRksvk7eJQONIlzlT3Qm0hPOzFMyilPewVamsVlz3qqRBmR8S1+Xobkle2YhXbEEeX9pY3u3ZNypcdbSe/3ixFlcE0xhos7/0Nzleyv7aTvBOiZV+EfucY/iQagUqvjSECYAbqwKZXSjw2inV8HWxBrblob+N8YMc3WJIW3AopnyN4tI00VbcR/2CG6M/8Cew== 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=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=hW5orNKAXX+GGWH59gD80stcd3enkxxX6u1BYpv7LWa8eDsHca6oqGVmaf/lLDJopux6vVidjmY121HMXpbjT5KX9MQtpr/SMMLylPuhqESNqWZWKddM8fsMQBl6qcaZfGNvMhHqq3ySYFDlum0sTS+r8uZ8AbeHOArBpIO9JGdnLgRQ1HMOdu9glJrwlDN8AlRyBiTRLhThkBEJcwPo2MxC7qgWYVH5lsk3+bDU3XT2R3VKKnu8+gf0evdUyCLoqgauj/4+8xpJJpDAafOgU9xrSV/njvvQ0rlxXQtOULAGPKYZDZ4BQw5K9zdxgCFM/vxUoZ4XCwv5TvEk2v1A6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=ObHrXJ1zOxxxYhNPRVtjnyY/anGB4XXQHcUTWVYPKFWm8pSgA//fRjh8Deo4fy/RMswqQ/kVaSieTu+zuUYwbjBW0IV413VMvWblWN+7VTkxEPxCSpKwFEhPUYX65hcHmxlE3GFX9QSh99/lequ4SbCMu9O0yW8FT9KMtplzzkElxtVQlHHHdWxm0tim25SGEQ5JcY5vxydM45pUyJIvxXY70v0nfu1I1M0HXoF2TBJGfx9+jOU3Ww7qbNWlli6oaSsMyhP/k2ikmV4bR56gaQuG0wSS3cVacxox7JTFjSq9KvzknfnloRAKdxXK/9qRZkY3uq6dDNvZciAb0XmiPg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) by PH0PR12MB7792.namprd12.prod.outlook.com (2603:10b6:510:281::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.23; Mon, 15 Jan 2024 18:37:49 +0000 Received: from LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873]) by LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873%6]) with mapi id 15.20.7181.020; Mon, 15 Jan 2024 18:37:49 +0000 Date: Mon, 15 Jan 2024 14:37:48 -0400 From: Jason Gunthorpe To: peterx@redhat.com Subject: Re: [PATCH v2 06/13] mm/gup: Drop folio_fast_pin_allowed() in hugepd processing Message-ID: <20240115183748.GR734935@nvidia.com> References: <20240103091423.400294-1-peterx@redhat.com> <20240103091423.400294-7-peterx@redhat.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240103091423.400294-7-peterx@redhat.com> X-ClientProxiedBy: MN2PR19CA0012.namprd19.prod.outlook.com (2603:10b6:208:178::25) To LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: LV2PR12MB5869:EE_|PH0PR12MB7792:EE_ X-MS-Office365-Filtering-Correlation-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NMHcwfKS5YpOgwMuTPz84dvhmzhTBPwFQkMqFqxZNsg8m6LeRCTTa7yhE7dxhsz7RTGXTmjlFmX+CmDqQz3ovOTcLqFrPqaozCanARZgyFmgZC09x0PNLg2EQzt613xqWQFTuOi1X2TdNksq0D32RVt91oITouUSB+IR/hrAJ2BTuGWq31O+I181h7ZeG0d4S4yG+WeY2BrCy5UIDPYZ0hNgS48o757OXenu8elR61aY0r+idnrr9ob7EDSLa266pIsU1ewx8aOfGal2MY1VHtZ6/5h+hBKkrWb/lgvxNgHHiZwkB0FWQZ6/2+od93+J6E7s5weGx3TQ4mQdQqtcG9oUL1aAfPUzuSxWf0tRCE06bALQkqeajpp4y4iVqeeKtEeo3HZ1CW4AdkPPIADtJBbcWlPcbAVP2iH7rZFqXeTQatYiGEKJX194HFPEMfvsBvpLNnhRZAT0N2YSU0akUiSVM1XjH5zT19dUmieeHn/I+a53Mklwk3vRGrj+vw1+1ZBdIsuGq9kIjmD9BKUoEW/jeocziEPSG0TjU9uQmlYd/W8LtxCmMySnNyX7ED3JqQPevdzXvpimrBahey+cLof4MZuDcXp/887nI6JJFyJJe5S+ZwyPVsT8zV9v4r82 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:LV2PR12MB5869.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(64100799003)(451199024)(1800799012)(186009)(66946007)(83380400001)(6506007)(66556008)(6512007)(26005)(478600001)(1076003)(38100700002)(4326008)(2616005)(5660300002)(8936002)(7416002)(6916009)(54906003)(8676002)(66476007)(6486002)(2906002)(316002)(33656002)(36756003)(86362001)(41300700001)(27376004);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?m5QXqIddeL91NXMRxH4Uhbm4kQnyOX6HVsDIEbKpqiG+4PGO8IiaDLf0K5tR?= =?us-ascii?Q?IGSqxvFVSSmNerD1UaysnGHjSNoRWCr8Kv7B7FFmra2vUdsE4soiIkTbLc9Q?= =?us-ascii?Q?gpmkxGHY7Lp9xhviDJxGG0xJSl1uPWgdYW/MnAMRtJgNqj2yENxk2uhfHVZ2?= =?us-ascii?Q?CbIzE8JkfvR2WUKAVuzkV7cqJqtnAxNbM3CGr/PCspOoSot5EZ+iHeKJmhQp?= =?us-ascii?Q?ORg8dPKRLBh2dIxxU/VkuSAqarek5MOAyZk4oTK47FJ5KnY8TM9rzERNj699?= =?us-ascii?Q?+XMNN5KDe2JiKsvynSKqm1YGZUnJGsBF6T0wcDF/By2SRTT+/1K5n7qLiGN+?= =?us-ascii?Q?28XTLxtLlyr9EW7usdUJa8PMHiAyVRrin8H0tMx/AlZQiYwxK4DixfueEwI5?= =?us-ascii?Q?pTmIJbJhRsOZoVwd4KLVyh7Rufkw27UHjCMrBE6hnj+ZulPW/jxBdK/Prazv?= =?us-ascii?Q?+5TU/xGoyE6SYmC5tTOCisWC5ngKR/ThJUTwtS7eFpuiIJC5roJf+ObVY7N7?= =?us-ascii?Q?6kiROaIh1D/YanrIw2nlqhOFkU6gvPmszbYnIKiXKwSESobd+FZtz8nV4VGX?= =?us-ascii?Q?PseeE3ILoZktSbxFbhR/oJz+9jpnjsOK/P7dZH7eswCBJih9aAtIx1KX7dx8?= =?us-ascii?Q?AaBIjbA+T5TYBnAE9x/4xG+JgRnPUIux37/suInYHfd80+uwwxFRm6LE7LgF?= =?us-ascii?Q?usiCdtFDO5d7ZYSxTPbudO4zJqYCTXXnj8BEKTx5BxuPKe8SDTnghpCs25N0?= =?us-ascii?Q?20QdD6GvjvyEu1xDotg8iJcCMy3uMQ1nYXA1mOIeBUvmHhPiaYs9PV1czWV9?= =?us-ascii?Q?uQ1vZyg4X7217CaRyStcdLN82C+y/AkR1cXpW3Jz7g6PANvx2Pgbmz9avujV?= =?us-ascii?Q?TIm5guhgLjFGT3lwDgQtDvymt93hBmPqufpF9S0PtlHMmD0LGW81gk6PFC8s?= =?us-ascii?Q?35gBRMhVBWcEne4POgnI4Ajqk0vUULd67qo5ibzdXSvklH8IDlkLl4dwuFZh?= =?us-ascii?Q?ymsxdV64jY3+Fnd7pTZOkNbeGS60lIP0nldZrprp3qhBVjTypSWAAINSscVV?= =?us-ascii?Q?uHJBjHPIZRTnt5qEedcn2inFFcgWCWJvsd776ZTxQl81QX8x+NV2wiFnAXJn?= =?us-ascii?Q?ya0Eq8WmTQ908Kg1LZZcOKpJvTDxpTDWYreS6wAhVNJ6UjzQz4eKDt1fwEdH?= =?us-ascii?Q?5JfyJFM241llTZYVgZGXvj/AS3eoqoyhWKJB7V0CtlxwTUINydjb9fbNHmyk?= =?us-ascii?Q?i+f//7NADvkDzCfmBl/WIpGB8umB3kyLO04JuS7LBozcTKnK8BbxUwWulWaI?= =?us-ascii?Q?uydUWJ0tva2r8sppmC/Wzm5bF/iO/q6EiUK8Snbe1dKpBEbBZkU15CSQVsvj?= =?us-ascii?Q?JwHPIxypoCueN6GAUE7V6vnIyKjeVOMLLUCqwXsyKE71JuWWEcqZkD2+3erj?= =?us-ascii?Q?v/ejxhfI2a44inKW3Bb3aeOFmhCTXA8Bq7xoVBZjS0EA9UCUtAxNABFD7uKY?= =?us-ascii?Q?ZSNTWl0J6Kn3d+dNOTv8o6GXKP3KUFEm02/du27wp5bALPBkuWBCQX/uKtuW?= =?us-ascii?Q?ykvnDEHgC+YuIA8xnirfEUX8tD9vuVpsszImLq3H?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2024 18:37:49.4941 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: XQurqgZ/9Llavqp3HGEPNNUnZ+2dzwegzvxUdWvdyZgW8AYv2hfdhumQnB9vock/ X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7792 X-BeenThere: linuxppc-dev@lists.ozlabs.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: James Houghton , David Hildenbrand , Yang Shi , Andrew Jones , linux-mm@kvack.org, Matthew Wilcox , linux-riscv@lists.infradead.org, Andrea Arcangeli , Christoph Hellwig , "Aneesh Kumar K . V" , Vlastimil Babka , Axel Rasmussen , Rik van Riel , John Hubbard , "Kirill A . Shutemov" , linux-arm-kernel@lists.infradead.org, Lorenzo Stoakes , Muchun Song , linux-kernel@vger.kernel.org, Andrew Morton , linuxppc-dev@lists.ozlabs.org, Mike Rapoport , Mike Kravetz Errors-To: linuxppc-dev-bounces+linuxppc-dev=archiver.kernel.org@lists.ozlabs.org Sender: "Linuxppc-dev" On Wed, Jan 03, 2024 at 05:14:16PM +0800, peterx@redhat.com wrote: > From: Peter Xu > > Hugepd format for GUP is only used in PowerPC with hugetlbfs. There are > some kernel usage of hugepd (can refer to hugepd_populate_kernel() for > PPC_8XX), however those pages are not candidates for GUP. > > Commit a6e79df92e4a ("mm/gup: disallow FOLL_LONGTERM GUP-fast writing to > file-backed mappings") added a check to fail gup-fast if there's potential > risk of violating GUP over writeback file systems. That should never apply > to hugepd. Considering that hugepd is an old format (and even > software-only), there's no plan to extend hugepd into other file typed > memories that is prone to the same issue. I didn't dig into the ppc stuff too deeply, but this looks to me like it is the same thing as ARM's contig bits? ie a chunk of PMD/etc entries are all managed together as though they are a virtual larger entry and we use the hugepte_addr_end() stuff to iterate over each sub entry. But WHY is GUP doing this or caring about this? GUP should have no problem handling the super-size entry (eg 8M on nohash) as a single thing. It seems we only lack an API to get this out of the arch code? It seems to me we should see ARM and PPC agree on what the API is for this and then get rid of hugepd by making both use the same page table walker API. Is that too hopeful? > Drop that check, not only because it'll never be true for hugepd per any > known plan, but also it paves way for reusing the function outside > fast-gup. I didn't see any other caller of this function in this series? When does this re-use happen?? Jason From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id C9CF2C3DA79 for ; Mon, 15 Jan 2024 18:38:34 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=G9d6a19uXOVL34tVhs7Vb1WWt4QCwZgDvzirI71fN+A=; b=vnHXMFVxCMNUt6 1VkVaWRAEy0/zcuEdnm+Md4r3wyYevznw90Ah5XAuqZvHXzh5VmXf6uQFoe6RHmwoCrsb3SdquxDD Wudf9q6zhWujgpRFzDf01bhpRqnnCAiOvaWwzK8pfMUGrVto9YjVa4BhrXtj0rDCa6IFL4FbYKJm0 kgnCY7NBBxPxWIrAa8Re10DF2D9FqMc62p+nvS7odUrz12AzgPFA07iTfSWUnG0q72FSBsZH84U64 4b/ds6QoU1m/BifvMK1vCLNoMNhp+r8AcVROMsiYS5kTDPzWQQloI3wbTkPTVvuhe1vfFMfA8sh6e CVTDx8y4hXEw53Fy0vog==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rPRqX-009xdA-37; Mon, 15 Jan 2024 18:38:05 +0000 Received: from mail-co1nam11on20601.outbound.protection.outlook.com ([2a01:111:f403:2416::601] helo=NAM11-CO1-obe.outbound.protection.outlook.com) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rPRqU-009xbD-2m; Mon, 15 Jan 2024 18:38:04 +0000 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bNPypzcHqtwmlkR35+8+wG0HQBj4NHh/ggbbP5Ug4dxjcd+vQcUrW8vFhFymcGY7t7IIbDAy0ueN2qeVc4c8m3KMzQ9DFw9+IRb3g9AVhEcqKwgJEhhMq0kPoeaO/spbKR+xyN5qfBm2uLNbYcM2xYRksvk7eJQONIlzlT3Qm0hPOzFMyilPewVamsVlz3qqRBmR8S1+Xobkle2YhXbEEeX9pY3u3ZNypcdbSe/3ixFlcE0xhos7/0Nzleyv7aTvBOiZV+EfucY/iQagUqvjSECYAbqwKZXSjw2inV8HWxBrblob+N8YMc3WJIW3AopnyN4tI00VbcR/2CG6M/8Cew== 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=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=hW5orNKAXX+GGWH59gD80stcd3enkxxX6u1BYpv7LWa8eDsHca6oqGVmaf/lLDJopux6vVidjmY121HMXpbjT5KX9MQtpr/SMMLylPuhqESNqWZWKddM8fsMQBl6qcaZfGNvMhHqq3ySYFDlum0sTS+r8uZ8AbeHOArBpIO9JGdnLgRQ1HMOdu9glJrwlDN8AlRyBiTRLhThkBEJcwPo2MxC7qgWYVH5lsk3+bDU3XT2R3VKKnu8+gf0evdUyCLoqgauj/4+8xpJJpDAafOgU9xrSV/njvvQ0rlxXQtOULAGPKYZDZ4BQw5K9zdxgCFM/vxUoZ4XCwv5TvEk2v1A6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=ObHrXJ1zOxxxYhNPRVtjnyY/anGB4XXQHcUTWVYPKFWm8pSgA//fRjh8Deo4fy/RMswqQ/kVaSieTu+zuUYwbjBW0IV413VMvWblWN+7VTkxEPxCSpKwFEhPUYX65hcHmxlE3GFX9QSh99/lequ4SbCMu9O0yW8FT9KMtplzzkElxtVQlHHHdWxm0tim25SGEQ5JcY5vxydM45pUyJIvxXY70v0nfu1I1M0HXoF2TBJGfx9+jOU3Ww7qbNWlli6oaSsMyhP/k2ikmV4bR56gaQuG0wSS3cVacxox7JTFjSq9KvzknfnloRAKdxXK/9qRZkY3uq6dDNvZciAb0XmiPg== Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=nvidia.com; Received: from LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) by PH0PR12MB7792.namprd12.prod.outlook.com (2603:10b6:510:281::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.23; Mon, 15 Jan 2024 18:37:49 +0000 Received: from LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873]) by LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873%6]) with mapi id 15.20.7181.020; Mon, 15 Jan 2024 18:37:49 +0000 Date: Mon, 15 Jan 2024 14:37:48 -0400 From: Jason Gunthorpe To: peterx@redhat.com Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, James Houghton , David Hildenbrand , "Kirill A . Shutemov" , Yang Shi , linux-riscv@lists.infradead.org, Andrew Morton , "Aneesh Kumar K . V" , Rik van Riel , Andrea Arcangeli , Axel Rasmussen , Mike Rapoport , John Hubbard , Vlastimil Babka , Michael Ellerman , Christophe Leroy , Andrew Jones , linuxppc-dev@lists.ozlabs.org, Mike Kravetz , Muchun Song , linux-arm-kernel@lists.infradead.org, Christoph Hellwig , Lorenzo Stoakes , Matthew Wilcox Subject: Re: [PATCH v2 06/13] mm/gup: Drop folio_fast_pin_allowed() in hugepd processing Message-ID: <20240115183748.GR734935@nvidia.com> References: <20240103091423.400294-1-peterx@redhat.com> <20240103091423.400294-7-peterx@redhat.com> Content-Disposition: inline In-Reply-To: <20240103091423.400294-7-peterx@redhat.com> X-ClientProxiedBy: MN2PR19CA0012.namprd19.prod.outlook.com (2603:10b6:208:178::25) To LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: LV2PR12MB5869:EE_|PH0PR12MB7792:EE_ X-MS-Office365-Filtering-Correlation-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NMHcwfKS5YpOgwMuTPz84dvhmzhTBPwFQkMqFqxZNsg8m6LeRCTTa7yhE7dxhsz7RTGXTmjlFmX+CmDqQz3ovOTcLqFrPqaozCanARZgyFmgZC09x0PNLg2EQzt613xqWQFTuOi1X2TdNksq0D32RVt91oITouUSB+IR/hrAJ2BTuGWq31O+I181h7ZeG0d4S4yG+WeY2BrCy5UIDPYZ0hNgS48o757OXenu8elR61aY0r+idnrr9ob7EDSLa266pIsU1ewx8aOfGal2MY1VHtZ6/5h+hBKkrWb/lgvxNgHHiZwkB0FWQZ6/2+od93+J6E7s5weGx3TQ4mQdQqtcG9oUL1aAfPUzuSxWf0tRCE06bALQkqeajpp4y4iVqeeKtEeo3HZ1CW4AdkPPIADtJBbcWlPcbAVP2iH7rZFqXeTQatYiGEKJX194HFPEMfvsBvpLNnhRZAT0N2YSU0akUiSVM1XjH5zT19dUmieeHn/I+a53Mklwk3vRGrj+vw1+1ZBdIsuGq9kIjmD9BKUoEW/jeocziEPSG0TjU9uQmlYd/W8LtxCmMySnNyX7ED3JqQPevdzXvpimrBahey+cLof4MZuDcXp/887nI6JJFyJJe5S+ZwyPVsT8zV9v4r82 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:LV2PR12MB5869.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(64100799003)(451199024)(1800799012)(186009)(66946007)(83380400001)(6506007)(66556008)(6512007)(26005)(478600001)(1076003)(38100700002)(4326008)(2616005)(5660300002)(8936002)(7416002)(6916009)(54906003)(8676002)(66476007)(6486002)(2906002)(316002)(33656002)(36756003)(86362001)(41300700001)(27376004);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?m5QXqIddeL91NXMRxH4Uhbm4kQnyOX6HVsDIEbKpqiG+4PGO8IiaDLf0K5tR?= =?us-ascii?Q?IGSqxvFVSSmNerD1UaysnGHjSNoRWCr8Kv7B7FFmra2vUdsE4soiIkTbLc9Q?= =?us-ascii?Q?gpmkxGHY7Lp9xhviDJxGG0xJSl1uPWgdYW/MnAMRtJgNqj2yENxk2uhfHVZ2?= =?us-ascii?Q?CbIzE8JkfvR2WUKAVuzkV7cqJqtnAxNbM3CGr/PCspOoSot5EZ+iHeKJmhQp?= =?us-ascii?Q?ORg8dPKRLBh2dIxxU/VkuSAqarek5MOAyZk4oTK47FJ5KnY8TM9rzERNj699?= =?us-ascii?Q?+XMNN5KDe2JiKsvynSKqm1YGZUnJGsBF6T0wcDF/By2SRTT+/1K5n7qLiGN+?= =?us-ascii?Q?28XTLxtLlyr9EW7usdUJa8PMHiAyVRrin8H0tMx/AlZQiYwxK4DixfueEwI5?= =?us-ascii?Q?pTmIJbJhRsOZoVwd4KLVyh7Rufkw27UHjCMrBE6hnj+ZulPW/jxBdK/Prazv?= =?us-ascii?Q?+5TU/xGoyE6SYmC5tTOCisWC5ngKR/ThJUTwtS7eFpuiIJC5roJf+ObVY7N7?= =?us-ascii?Q?6kiROaIh1D/YanrIw2nlqhOFkU6gvPmszbYnIKiXKwSESobd+FZtz8nV4VGX?= =?us-ascii?Q?PseeE3ILoZktSbxFbhR/oJz+9jpnjsOK/P7dZH7eswCBJih9aAtIx1KX7dx8?= =?us-ascii?Q?AaBIjbA+T5TYBnAE9x/4xG+JgRnPUIux37/suInYHfd80+uwwxFRm6LE7LgF?= =?us-ascii?Q?usiCdtFDO5d7ZYSxTPbudO4zJqYCTXXnj8BEKTx5BxuPKe8SDTnghpCs25N0?= =?us-ascii?Q?20QdD6GvjvyEu1xDotg8iJcCMy3uMQ1nYXA1mOIeBUvmHhPiaYs9PV1czWV9?= =?us-ascii?Q?uQ1vZyg4X7217CaRyStcdLN82C+y/AkR1cXpW3Jz7g6PANvx2Pgbmz9avujV?= =?us-ascii?Q?TIm5guhgLjFGT3lwDgQtDvymt93hBmPqufpF9S0PtlHMmD0LGW81gk6PFC8s?= =?us-ascii?Q?35gBRMhVBWcEne4POgnI4Ajqk0vUULd67qo5ibzdXSvklH8IDlkLl4dwuFZh?= =?us-ascii?Q?ymsxdV64jY3+Fnd7pTZOkNbeGS60lIP0nldZrprp3qhBVjTypSWAAINSscVV?= =?us-ascii?Q?uHJBjHPIZRTnt5qEedcn2inFFcgWCWJvsd776ZTxQl81QX8x+NV2wiFnAXJn?= =?us-ascii?Q?ya0Eq8WmTQ908Kg1LZZcOKpJvTDxpTDWYreS6wAhVNJ6UjzQz4eKDt1fwEdH?= =?us-ascii?Q?5JfyJFM241llTZYVgZGXvj/AS3eoqoyhWKJB7V0CtlxwTUINydjb9fbNHmyk?= =?us-ascii?Q?i+f//7NADvkDzCfmBl/WIpGB8umB3kyLO04JuS7LBozcTKnK8BbxUwWulWaI?= =?us-ascii?Q?uydUWJ0tva2r8sppmC/Wzm5bF/iO/q6EiUK8Snbe1dKpBEbBZkU15CSQVsvj?= =?us-ascii?Q?JwHPIxypoCueN6GAUE7V6vnIyKjeVOMLLUCqwXsyKE71JuWWEcqZkD2+3erj?= =?us-ascii?Q?v/ejxhfI2a44inKW3Bb3aeOFmhCTXA8Bq7xoVBZjS0EA9UCUtAxNABFD7uKY?= =?us-ascii?Q?ZSNTWl0J6Kn3d+dNOTv8o6GXKP3KUFEm02/du27wp5bALPBkuWBCQX/uKtuW?= =?us-ascii?Q?ykvnDEHgC+YuIA8xnirfEUX8tD9vuVpsszImLq3H?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2024 18:37:49.4941 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: XQurqgZ/9Llavqp3HGEPNNUnZ+2dzwegzvxUdWvdyZgW8AYv2hfdhumQnB9vock/ X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7792 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240115_103802_949265_B9D28DF6 X-CRM114-Status: GOOD ( 17.26 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Wed, Jan 03, 2024 at 05:14:16PM +0800, peterx@redhat.com wrote: > From: Peter Xu > > Hugepd format for GUP is only used in PowerPC with hugetlbfs. There are > some kernel usage of hugepd (can refer to hugepd_populate_kernel() for > PPC_8XX), however those pages are not candidates for GUP. > > Commit a6e79df92e4a ("mm/gup: disallow FOLL_LONGTERM GUP-fast writing to > file-backed mappings") added a check to fail gup-fast if there's potential > risk of violating GUP over writeback file systems. That should never apply > to hugepd. Considering that hugepd is an old format (and even > software-only), there's no plan to extend hugepd into other file typed > memories that is prone to the same issue. I didn't dig into the ppc stuff too deeply, but this looks to me like it is the same thing as ARM's contig bits? ie a chunk of PMD/etc entries are all managed together as though they are a virtual larger entry and we use the hugepte_addr_end() stuff to iterate over each sub entry. But WHY is GUP doing this or caring about this? GUP should have no problem handling the super-size entry (eg 8M on nohash) as a single thing. It seems we only lack an API to get this out of the arch code? It seems to me we should see ARM and PPC agree on what the API is for this and then get rid of hugepd by making both use the same page table walker API. Is that too hopeful? > Drop that check, not only because it'll never be true for hugepd per any > known plan, but also it paves way for reusing the function outside > fast-gup. I didn't see any other caller of this function in this series? When does this re-use happen?? Jason _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from kanga.kvack.org (kanga.kvack.org [205.233.56.17]) by smtp.lore.kernel.org (Postfix) with ESMTP id B3E6CC3DA79 for ; Mon, 15 Jan 2024 18:37:59 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 02DF96B0078; Mon, 15 Jan 2024 13:37:59 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id F1F746B007D; Mon, 15 Jan 2024 13:37:58 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id DC17F6B0080; Mon, 15 Jan 2024 13:37:58 -0500 (EST) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0017.hostedemail.com [216.40.44.17]) by kanga.kvack.org (Postfix) with ESMTP id CA7816B0078 for ; Mon, 15 Jan 2024 13:37:58 -0500 (EST) Received: from smtpin28.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay05.hostedemail.com (Postfix) with ESMTP id 9A36740506 for ; Mon, 15 Jan 2024 18:37:58 +0000 (UTC) X-FDA: 81682404636.28.5D90A19 Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11on2079.outbound.protection.outlook.com [40.107.220.79]) by imf30.hostedemail.com (Postfix) with ESMTP id B42C880011 for ; Mon, 15 Jan 2024 18:37:55 +0000 (UTC) Authentication-Results: imf30.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=ObHrXJ1z; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf30.hostedemail.com: domain of jgg@nvidia.com designates 40.107.220.79 as permitted sender) smtp.mailfrom=jgg@nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1") ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1705343875; h=from:from:sender:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=aAO2iKgyfEx/dMdMaRirsXEgfWAKLEcz1TmQCyZ1Qr/6pL+We+NZ+hAbg7DWp8BbjmDOWT FaJr8TDCwcLEOE8vuwT5WScPoGGPxuV0sWD9mUgZ12rsyJsoFyAenZRu7p22QcvgrxDLwe fxc8RuS8IYm1Oi9TDo96OA7ivHZfQnU= ARC-Authentication-Results: i=2; imf30.hostedemail.com; dkim=pass header.d=Nvidia.com header.s=selector2 header.b=ObHrXJ1z; dmarc=pass (policy=reject) header.from=nvidia.com; spf=pass (imf30.hostedemail.com: domain of jgg@nvidia.com designates 40.107.220.79 as permitted sender) smtp.mailfrom=jgg@nvidia.com; arc=pass ("microsoft.com:s=arcselector9901:i=1") ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1705343875; a=rsa-sha256; cv=pass; b=tAGR3iD+YHtaN78TARUo6CdB0aaJbKLkxJYBsAYo91Yn/zI3RYZDNpUVf0C7v/NWK7xwmo BJU/IYKc7z73SZTDbGQX7+NqGOuZ2lcwDIvEPLxGSF+XVbM2iEzgVIUNqJZLwvo3Fmg48+ RlQYuLT4tvwzb9MdiFvzpU81szSBSfA= ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=bNPypzcHqtwmlkR35+8+wG0HQBj4NHh/ggbbP5Ug4dxjcd+vQcUrW8vFhFymcGY7t7IIbDAy0ueN2qeVc4c8m3KMzQ9DFw9+IRb3g9AVhEcqKwgJEhhMq0kPoeaO/spbKR+xyN5qfBm2uLNbYcM2xYRksvk7eJQONIlzlT3Qm0hPOzFMyilPewVamsVlz3qqRBmR8S1+Xobkle2YhXbEEeX9pY3u3ZNypcdbSe/3ixFlcE0xhos7/0Nzleyv7aTvBOiZV+EfucY/iQagUqvjSECYAbqwKZXSjw2inV8HWxBrblob+N8YMc3WJIW3AopnyN4tI00VbcR/2CG6M/8Cew== 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=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=hW5orNKAXX+GGWH59gD80stcd3enkxxX6u1BYpv7LWa8eDsHca6oqGVmaf/lLDJopux6vVidjmY121HMXpbjT5KX9MQtpr/SMMLylPuhqESNqWZWKddM8fsMQBl6qcaZfGNvMhHqq3ySYFDlum0sTS+r8uZ8AbeHOArBpIO9JGdnLgRQ1HMOdu9glJrwlDN8AlRyBiTRLhThkBEJcwPo2MxC7qgWYVH5lsk3+bDU3XT2R3VKKnu8+gf0evdUyCLoqgauj/4+8xpJJpDAafOgU9xrSV/njvvQ0rlxXQtOULAGPKYZDZ4BQw5K9zdxgCFM/vxUoZ4XCwv5TvEk2v1A6Q== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=nvidia.com; dmarc=pass action=none header.from=nvidia.com; dkim=pass header.d=nvidia.com; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=Nvidia.com; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=vYsO8ORNHs+BcqJyqksM2bOPMvnXEzfhiC4p8DKZRI4=; b=ObHrXJ1zOxxxYhNPRVtjnyY/anGB4XXQHcUTWVYPKFWm8pSgA//fRjh8Deo4fy/RMswqQ/kVaSieTu+zuUYwbjBW0IV413VMvWblWN+7VTkxEPxCSpKwFEhPUYX65hcHmxlE3GFX9QSh99/lequ4SbCMu9O0yW8FT9KMtplzzkElxtVQlHHHdWxm0tim25SGEQ5JcY5vxydM45pUyJIvxXY70v0nfu1I1M0HXoF2TBJGfx9+jOU3Ww7qbNWlli6oaSsMyhP/k2ikmV4bR56gaQuG0wSS3cVacxox7JTFjSq9KvzknfnloRAKdxXK/9qRZkY3uq6dDNvZciAb0XmiPg== Received: from LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) by PH0PR12MB7792.namprd12.prod.outlook.com (2603:10b6:510:281::21) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7159.23; Mon, 15 Jan 2024 18:37:49 +0000 Received: from LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873]) by LV2PR12MB5869.namprd12.prod.outlook.com ([fe80::96dd:1160:6472:9873%6]) with mapi id 15.20.7181.020; Mon, 15 Jan 2024 18:37:49 +0000 Date: Mon, 15 Jan 2024 14:37:48 -0400 From: Jason Gunthorpe To: peterx@redhat.com Cc: linux-mm@kvack.org, linux-kernel@vger.kernel.org, James Houghton , David Hildenbrand , "Kirill A . Shutemov" , Yang Shi , linux-riscv@lists.infradead.org, Andrew Morton , "Aneesh Kumar K . V" , Rik van Riel , Andrea Arcangeli , Axel Rasmussen , Mike Rapoport , John Hubbard , Vlastimil Babka , Michael Ellerman , Christophe Leroy , Andrew Jones , linuxppc-dev@lists.ozlabs.org, Mike Kravetz , Muchun Song , linux-arm-kernel@lists.infradead.org, Christoph Hellwig , Lorenzo Stoakes , Matthew Wilcox Subject: Re: [PATCH v2 06/13] mm/gup: Drop folio_fast_pin_allowed() in hugepd processing Message-ID: <20240115183748.GR734935@nvidia.com> References: <20240103091423.400294-1-peterx@redhat.com> <20240103091423.400294-7-peterx@redhat.com> Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20240103091423.400294-7-peterx@redhat.com> X-ClientProxiedBy: MN2PR19CA0012.namprd19.prod.outlook.com (2603:10b6:208:178::25) To LV2PR12MB5869.namprd12.prod.outlook.com (2603:10b6:408:176::16) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: LV2PR12MB5869:EE_|PH0PR12MB7792:EE_ X-MS-Office365-Filtering-Correlation-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: NMHcwfKS5YpOgwMuTPz84dvhmzhTBPwFQkMqFqxZNsg8m6LeRCTTa7yhE7dxhsz7RTGXTmjlFmX+CmDqQz3ovOTcLqFrPqaozCanARZgyFmgZC09x0PNLg2EQzt613xqWQFTuOi1X2TdNksq0D32RVt91oITouUSB+IR/hrAJ2BTuGWq31O+I181h7ZeG0d4S4yG+WeY2BrCy5UIDPYZ0hNgS48o757OXenu8elR61aY0r+idnrr9ob7EDSLa266pIsU1ewx8aOfGal2MY1VHtZ6/5h+hBKkrWb/lgvxNgHHiZwkB0FWQZ6/2+od93+J6E7s5weGx3TQ4mQdQqtcG9oUL1aAfPUzuSxWf0tRCE06bALQkqeajpp4y4iVqeeKtEeo3HZ1CW4AdkPPIADtJBbcWlPcbAVP2iH7rZFqXeTQatYiGEKJX194HFPEMfvsBvpLNnhRZAT0N2YSU0akUiSVM1XjH5zT19dUmieeHn/I+a53Mklwk3vRGrj+vw1+1ZBdIsuGq9kIjmD9BKUoEW/jeocziEPSG0TjU9uQmlYd/W8LtxCmMySnNyX7ED3JqQPevdzXvpimrBahey+cLof4MZuDcXp/887nI6JJFyJJe5S+ZwyPVsT8zV9v4r82 X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:LV2PR12MB5869.namprd12.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(39860400002)(366004)(346002)(136003)(376002)(396003)(230922051799003)(64100799003)(451199024)(1800799012)(186009)(66946007)(83380400001)(6506007)(66556008)(6512007)(26005)(478600001)(1076003)(38100700002)(4326008)(2616005)(5660300002)(8936002)(7416002)(6916009)(54906003)(8676002)(66476007)(6486002)(2906002)(316002)(33656002)(36756003)(86362001)(41300700001)(27376004);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?m5QXqIddeL91NXMRxH4Uhbm4kQnyOX6HVsDIEbKpqiG+4PGO8IiaDLf0K5tR?= =?us-ascii?Q?IGSqxvFVSSmNerD1UaysnGHjSNoRWCr8Kv7B7FFmra2vUdsE4soiIkTbLc9Q?= =?us-ascii?Q?gpmkxGHY7Lp9xhviDJxGG0xJSl1uPWgdYW/MnAMRtJgNqj2yENxk2uhfHVZ2?= =?us-ascii?Q?CbIzE8JkfvR2WUKAVuzkV7cqJqtnAxNbM3CGr/PCspOoSot5EZ+iHeKJmhQp?= =?us-ascii?Q?ORg8dPKRLBh2dIxxU/VkuSAqarek5MOAyZk4oTK47FJ5KnY8TM9rzERNj699?= =?us-ascii?Q?+XMNN5KDe2JiKsvynSKqm1YGZUnJGsBF6T0wcDF/By2SRTT+/1K5n7qLiGN+?= =?us-ascii?Q?28XTLxtLlyr9EW7usdUJa8PMHiAyVRrin8H0tMx/AlZQiYwxK4DixfueEwI5?= =?us-ascii?Q?pTmIJbJhRsOZoVwd4KLVyh7Rufkw27UHjCMrBE6hnj+ZulPW/jxBdK/Prazv?= =?us-ascii?Q?+5TU/xGoyE6SYmC5tTOCisWC5ngKR/ThJUTwtS7eFpuiIJC5roJf+ObVY7N7?= =?us-ascii?Q?6kiROaIh1D/YanrIw2nlqhOFkU6gvPmszbYnIKiXKwSESobd+FZtz8nV4VGX?= =?us-ascii?Q?PseeE3ILoZktSbxFbhR/oJz+9jpnjsOK/P7dZH7eswCBJih9aAtIx1KX7dx8?= =?us-ascii?Q?AaBIjbA+T5TYBnAE9x/4xG+JgRnPUIux37/suInYHfd80+uwwxFRm6LE7LgF?= =?us-ascii?Q?usiCdtFDO5d7ZYSxTPbudO4zJqYCTXXnj8BEKTx5BxuPKe8SDTnghpCs25N0?= =?us-ascii?Q?20QdD6GvjvyEu1xDotg8iJcCMy3uMQ1nYXA1mOIeBUvmHhPiaYs9PV1czWV9?= =?us-ascii?Q?uQ1vZyg4X7217CaRyStcdLN82C+y/AkR1cXpW3Jz7g6PANvx2Pgbmz9avujV?= =?us-ascii?Q?TIm5guhgLjFGT3lwDgQtDvymt93hBmPqufpF9S0PtlHMmD0LGW81gk6PFC8s?= =?us-ascii?Q?35gBRMhVBWcEne4POgnI4Ajqk0vUULd67qo5ibzdXSvklH8IDlkLl4dwuFZh?= =?us-ascii?Q?ymsxdV64jY3+Fnd7pTZOkNbeGS60lIP0nldZrprp3qhBVjTypSWAAINSscVV?= =?us-ascii?Q?uHJBjHPIZRTnt5qEedcn2inFFcgWCWJvsd776ZTxQl81QX8x+NV2wiFnAXJn?= =?us-ascii?Q?ya0Eq8WmTQ908Kg1LZZcOKpJvTDxpTDWYreS6wAhVNJ6UjzQz4eKDt1fwEdH?= =?us-ascii?Q?5JfyJFM241llTZYVgZGXvj/AS3eoqoyhWKJB7V0CtlxwTUINydjb9fbNHmyk?= =?us-ascii?Q?i+f//7NADvkDzCfmBl/WIpGB8umB3kyLO04JuS7LBozcTKnK8BbxUwWulWaI?= =?us-ascii?Q?uydUWJ0tva2r8sppmC/Wzm5bF/iO/q6EiUK8Snbe1dKpBEbBZkU15CSQVsvj?= =?us-ascii?Q?JwHPIxypoCueN6GAUE7V6vnIyKjeVOMLLUCqwXsyKE71JuWWEcqZkD2+3erj?= =?us-ascii?Q?v/ejxhfI2a44inKW3Bb3aeOFmhCTXA8Bq7xoVBZjS0EA9UCUtAxNABFD7uKY?= =?us-ascii?Q?ZSNTWl0J6Kn3d+dNOTv8o6GXKP3KUFEm02/du27wp5bALPBkuWBCQX/uKtuW?= =?us-ascii?Q?ykvnDEHgC+YuIA8xnirfEUX8tD9vuVpsszImLq3H?= X-OriginatorOrg: Nvidia.com X-MS-Exchange-CrossTenant-Network-Message-Id: 06ebd5d6-34bd-4179-4b7c-08dc15f9133a X-MS-Exchange-CrossTenant-AuthSource: LV2PR12MB5869.namprd12.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Jan 2024 18:37:49.4941 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: XQurqgZ/9Llavqp3HGEPNNUnZ+2dzwegzvxUdWvdyZgW8AYv2hfdhumQnB9vock/ X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR12MB7792 X-Rspamd-Queue-Id: B42C880011 X-Rspam-User: X-Rspamd-Server: rspam04 X-Stat-Signature: t55skao7p37xp71bpmke738qdm3rta5n X-HE-Tag: 1705343875-39642 X-HE-Meta: U2FsdGVkX1/yJli+jtMRj4LmVsplaS52q70xnr+Z81PLddcf+cv6QJbusijTqZC5aLybimJBt51zS/+9m95I2RuwP2zCSn/bRfUiqV8rrr6da8NnSJPUo/gzAmC38YeDZdmhuoFUrxHfxdmW+7EooKsxLOEINnK/YoWISVP6RfHVkg35hbcLQlmoNb8NEV0J5Oq/wLO6Syf6ulyyEsIp+tLQc10ZE2wIKi20O2DmEk8G5/mdEM64lyNv6/GDeVA5bj5YPLeu/ZOt5zgI7fNEt44H8fI4mMK1fr1l4FN3sl3pbMj9CtzIKaW1Ks+3J7i8F2fa1AmVzbcD2/Q7V3/vg0/bXFGeoSiC8VCdNyDyraUGwl+JkUxWxlcu8KGbLLm5qrR+zF22wTbwFlyMNx1XSj/Le3hgajSTayPJ57JQQWeDrXbppwAw+wOEpvwYhohO7J/q6g1xHu8DtCGJjudfGtyoz9xStipw4SRfTL7ZNQoSUNTN/bUfiOpdGnn7Q6LBtdcjacCd6x7CUrPxAzslMQMv2VtJQLy5Z70oodHuLzbtgxzTmEfwMP4EewlbS0/Lxabni+Ycevj+ke+rgoqTrMwL/bLj1rlQ20eQuKcaTbhrJMezvuuajNtZCtZbkRUBXPABQT+jtZ05a4wfTcLWQHxlowQqWhWHK2t0M5ZWrkGPdrgh3rjyfw9M5TAPV52MRlnVK5SlhmE1pUjaavM7/rLOeCKbqV12Nibf6pzp+7l31Ig6MAmQCM7+AMdJzK9VVEzEImgBLV0f18wSEWiVBiijjz4saI5P9yz36PG4c4kVu/UtYZ3/YMBJY1S9GC6Wr8TNmprnwP5RceotCbfgz1bBbL8b0n4c4Ipb1ygdPLV2tRg+N3lkpNPjQaHQrEphq51dEKM9rF68dDBPGNQL1OWO4v+FaHQVaxrJU0LrGRXSjK+0elIhrjC8X5XdDSEh1TNS4n26tmbsg+JfdJm ihsoS2or kxjeafIiHoBasVjqL0SAxqObljuR5YkMPr4Adkk/PVRWCwgbJUwsg/6jTzgO1Bmh9fd/tg9grMHpbnK0fwuyO3rMsmph2IFJGzqHiv7mzE6QCh3JtozqvOp1LcXjFoIRfncfa1c5eItiYsXomCcQDmqbrqY5O4kYqqbPS0z94PPqcL9bRMZt/a1rB56oH7YB2/LDeq8lRJRARUULgS7s3Suwm2wkqsVfamhb97s14H8t40JFC8LqbE7W4KmDhROjo9vJUyzPlc4siCS8cKbfVrGXuW7n43X1LVBIy X-Bogosity: Ham, tests=bogofilter, spamicity=0.000000, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: On Wed, Jan 03, 2024 at 05:14:16PM +0800, peterx@redhat.com wrote: > From: Peter Xu > > Hugepd format for GUP is only used in PowerPC with hugetlbfs. There are > some kernel usage of hugepd (can refer to hugepd_populate_kernel() for > PPC_8XX), however those pages are not candidates for GUP. > > Commit a6e79df92e4a ("mm/gup: disallow FOLL_LONGTERM GUP-fast writing to > file-backed mappings") added a check to fail gup-fast if there's potential > risk of violating GUP over writeback file systems. That should never apply > to hugepd. Considering that hugepd is an old format (and even > software-only), there's no plan to extend hugepd into other file typed > memories that is prone to the same issue. I didn't dig into the ppc stuff too deeply, but this looks to me like it is the same thing as ARM's contig bits? ie a chunk of PMD/etc entries are all managed together as though they are a virtual larger entry and we use the hugepte_addr_end() stuff to iterate over each sub entry. But WHY is GUP doing this or caring about this? GUP should have no problem handling the super-size entry (eg 8M on nohash) as a single thing. It seems we only lack an API to get this out of the arch code? It seems to me we should see ARM and PPC agree on what the API is for this and then get rid of hugepd by making both use the same page table walker API. Is that too hopeful? > Drop that check, not only because it'll never be true for hugepd per any > known plan, but also it paves way for reusing the function outside > fast-gup. I didn't see any other caller of this function in this series? When does this re-use happen?? Jason