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 3293AC02180 for ; Tue, 14 Jan 2025 02:18:48 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:MIME-Version:In-Reply-To: Content-Type:References:Message-ID:Subject:CC:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=uvPQ+fPxu9/8Mwfj6PzqbgmO7XfngvRA8FwjuabHsJ8=; b=KLAj+fwy08XjJF3Qq5NfrJorR5 6M7+qU1geqjvEdGpadPD70BNf3uHhQrfNN+uvZH8z5PAwReyUolqJzfm3eJdjgronfQ9C/1jacrOQ wKx2pDMFogGs+x3MbYrRrkFD5ia5sIYEbYubLe4ww8KM9rZTOXl1UaiftzGjtosntPwrt8qFZ8FgL PyEOF9uqr2lrwBJmWoo7dwADZ2riDDJWFPfn9mf5YqTw90h1a6Z8lZJ5ipLtQFQfzLlbU+wuggsn/ mM2Jvu6W75Yc/cpX3d+N9e+nF2HsaoOky7fgKoAUPOMHMffxczJdYmGVkR0pdfHRxQZVRIIxC/buq XOpKof9A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1tXWVo-00000007Dsu-10tl; Tue, 14 Jan 2025 02:18:36 +0000 Received: from mgamail.intel.com ([198.175.65.11]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1tXWUX-00000007DfP-3RQJ for linux-arm-kernel@lists.infradead.org; Tue, 14 Jan 2025 02:17:19 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1736821038; x=1768357038; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=dEYTSAlt9VO2iDDuVtLevqxTBnrZktPoMKb3hUUP/4I=; b=DRMHYDF+JFHBntlsyWumu1+pH/P4GgawFpX7Z9q2Y9I6i0ZI5gQM7Pt8 r8YCTw6sdvd8T/IW3nkDUL6mLmAucf5N5P9Nzn2GVi9UUxyiFyhF6M38d cxjGIFRq+j8lqaMsuOz12PzT3sOWs0pE/HBpxJCNZ/JgLFpfGpseS+XCa GO+uN0ONM3TnCWRuVoEJbmZG+FBTBG7anpWX/9QkgpWBiNDdg/xf1VeRM iKSpebNGKlJqLYVH5IabbcgYSfpW1AxHZIXCxUnHDMMK2msg18ke++U6I 15OmL32C1i7HzzuhXKQWnBgcX/SRuFhReLOk7Y99qImSW9vJJgsqcrV5P w==; X-CSE-ConnectionGUID: NJ0Pf0MHROaRTESQCe3JEg== X-CSE-MsgGUID: 6vDf73aSTYWx4L2CnrNFUw== X-IronPort-AV: E=McAfee;i="6700,10204,11314"; a="47595550" X-IronPort-AV: E=Sophos;i="6.12,312,1728975600"; d="scan'208";a="47595550" Received: from orviesa009.jf.intel.com ([10.64.159.149]) by orvoesa103.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 13 Jan 2025 18:17:11 -0800 X-CSE-ConnectionGUID: RSHIJIpHStmfkH66EClzjg== X-CSE-MsgGUID: wQTJnQSPRXa68Na9ievzSQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.12,312,1728975600"; d="scan'208";a="104433169" Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by orviesa009.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 13 Jan 2025 18:17:11 -0800 Received: from orsmsx601.amr.corp.intel.com (10.22.229.14) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.44; Mon, 13 Jan 2025 18:17:10 -0800 Received: from orsedg603.ED.cps.intel.com (10.7.248.4) by orsmsx601.amr.corp.intel.com (10.22.229.14) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.44 via Frontend Transport; Mon, 13 Jan 2025 18:17:10 -0800 Received: from NAM10-DM6-obe.outbound.protection.outlook.com (104.47.58.42) by edgegateway.intel.com (134.134.137.100) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.44; Mon, 13 Jan 2025 18:17:09 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=rA55TWnscC/3BAQPpe0tbfhSdTnQwke71hjz4+U+L28wGQ7rHgoa4THoU1HnwdxhVaFZXEGoKXtKeYd6DypADBvpRna3fF1IZ9imrl6YqrIrFO0Uec36jEOLjgRKE8pEovTohiXJnfcv87I4UEPp4qPB0RwX7p96+7HcrjgAuLdq3v6MMITJXg+NguIBoUmtvLnOC9Glw75sKUel0b9kEtrsEvX3qigJwaBMCqTz4jNQpSaqneELWahbruGGeCKU5nGuuZm9T3+dZX5ccuzhADfarZtqZ94JVFN4vBTBbJjIqYrO82aGMXrxSHon4yNzpJovSd3E/XTyTwAKBHWlxg== 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=uvPQ+fPxu9/8Mwfj6PzqbgmO7XfngvRA8FwjuabHsJ8=; b=gJXT4fdtJooRMzt1bgIjGzrH6eLCx1WsbM6rOhZVI7W5vHQISi0FjoZw1EsZ6lk68v6O9Hm2yhyDsaZR+xr0A4EXL6cvEi67e8FfhRyOIqHTmMI+g5wxqyXx2uuGirmzREKp55RziuwrldTJ6A63xENc+5cbPs8nU1Ia9ZcTj2jsgw4oJuBDBoQBT+n5+vYkkRE3Ph5TTKAC1paPEKxBF7tClWVSAmoT+/m1jXmlNm1AZvLx1w6F3Wegwl36+TvbEot+orjCN2CE9BHa3jM9k37gzH+rHNxqiiAQN7I//ASo2ALzOtB6T0cBP6siT6pkxeDAHsdZoo0dhtIFpyXEHg== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=intel.com; dmarc=pass action=none header.from=intel.com; dkim=pass header.d=intel.com; arc=none Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=intel.com; Received: from PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) by CH3PR11MB7865.namprd11.prod.outlook.com (2603:10b6:610:128::10) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8335.18; Tue, 14 Jan 2025 02:16:26 +0000 Received: from PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6b05:74cf:a304:ecd8]) by PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6b05:74cf:a304:ecd8%5]) with mapi id 15.20.8335.011; Tue, 14 Jan 2025 02:16:26 +0000 Date: Mon, 13 Jan 2025 18:16:22 -0800 From: Dan Williams To: Alistair Popple , , , CC: , Alistair Popple , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: Re: [PATCH v6 18/26] mm/gup: Don't allow FOLL_LONGTERM pinning of FS DAX pages Message-ID: <6785c8f6511d4_20fa2944a@dwillia2-xfh.jf.intel.com.notmuch> References: Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: X-ClientProxiedBy: MW3PR05CA0003.namprd05.prod.outlook.com (2603:10b6:303:2b::8) To PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH8PR11MB8107:EE_|CH3PR11MB7865:EE_ X-MS-Office365-Filtering-Correlation-Id: b2e59534-ca3d-4491-7936-08dd34417305 X-LD-Processed: 46c98d88-e344-4ed4-8496-4ed7712e255d,ExtAddr X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|1800799024|366016|7053199007; X-Microsoft-Antispam-Message-Info: =?us-ascii?Q?wcsZxmCZGnxfcSGmt4tpobTD58Xxrs/zcrI32/9ojrsVxkKBrZr4OaunUnoR?= =?us-ascii?Q?yjKkX2cmGrwCTc3rJxZVdbIVcDpgFh1HjYVxBszhfn7x3SqVpZSOOIFU8/tw?= =?us-ascii?Q?KN4yh8IeVunIuzp7CCKVGTPRacxXqXawuBrPBGhh+rYQcsvuXEwe5EHOHMqk?= =?us-ascii?Q?DK0SP+LpI8kptV+TlNQrTLuHtH5PGBiX8uACkPrSYqVDPfoam8p+kjOeuvw8?= =?us-ascii?Q?6Ih0jLRpUZIHMrrpdCq5bN6pi2+XTGjyQw0pvR314eR9taQIHqoOV1QZx3oA?= =?us-ascii?Q?AnWYIZlsd46BPPZq/H3Pqu4mJdeU5EkC94+VcYBUb2lC82AvXAVFGWLvWr0c?= =?us-ascii?Q?/vkHo2c3jz2SUd7h5OSZDemW4CXsWp79QXACa4hH+eFMnUJs/Q7n8QZ1RUQE?= =?us-ascii?Q?KmlAhDC6nHA++K+2JS4tivBcYVHPqDPdqQP8aJjvEz2fukwY82LsoExHN3pK?= =?us-ascii?Q?5mhiFWuar/TA4oNkZ8E9kL7QGQ8JYhOnzUlTamioCx7bVcGv3JqPVN148REg?= =?us-ascii?Q?9soj6G7rNuWE1zAqG+90druOLv/bjeKISb/3K+T9n+6kTAj9o+3eIln1NWoy?= =?us-ascii?Q?+urr8mq6mE8hS1tOgWL3l+illscbM74JBmMqZI9eggUZ8wXFibwdHRKFN77m?= =?us-ascii?Q?6HR60fhO5QtbDW02hrrC/6ud5eEOmhRmyIQybCAUPkfR/hf61W3kEoaZjdHp?= =?us-ascii?Q?of29dnW559unvFVC8EVpJN2i+01+z3GSUFxffqevFMls/PPq1Ra0bI+7n9Gl?= =?us-ascii?Q?o+4AE5+bYI0w8ZXZxAMWC6JJBXb8R4Atahh+Mr7sj/PFAzD93VifjXvDm2+J?= =?us-ascii?Q?gXwhpO6UmExoo1Hc5zwQOwOy61mPAS/0VqQEUyukoaHheMQP2Nctw4ciX029?= =?us-ascii?Q?XpbhN7WPVYzcEo57ClHK2wns3/8xihbppmi+paJCSSHzKEJ4c8WMu6vdE5Xz?= =?us-ascii?Q?zxiEK9buNvLDIkbtV5X8wwn+bKivhQJQUJ+AlWUqS+qfwCkRDudv5pqdUs/1?= =?us-ascii?Q?tCmKzAkFPrYM+iQgTjtBARcheaGzm3q0csPIPKbx995CdiRA3aP3PQh1fLG0?= =?us-ascii?Q?WJZZjK9RzpynK0j210QF7VmYjizFI4muWnUpszPnpa3tR+bevGWs8eBZPyP5?= =?us-ascii?Q?FHxKsujqXAUKUHMWL/OrinRV7jsWeNc6aYilDNMSO6wuauX9FUdpUG2LUdqS?= =?us-ascii?Q?+Aghu135wU3CfPKcOeZ/IDzeR8xug99u90MhOhI4fUqpau9cemKbxTKtdKMe?= =?us-ascii?Q?t2xabMocux95xAvAPR8OaRzoQXSBWYaNtWZffAs4SOOj91USzBuBg7iAkPT9?= =?us-ascii?Q?K3KACHF+1d9NZkWXh9UZV2Pd5n97r06TC+JwiYvXNPbjLkSxDJ5Z86iW0Ja4?= =?us-ascii?Q?wFS1AyAkiBIrVQ2QkWte1uP46nzL?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:PH8PR11MB8107.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(1800799024)(366016)(7053199007);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?uvN1dFEb5YmRV0O5W0NMjHPNVhC4NfqVVDzOOvn0lhz3uOG9dNHyZdXo7XUP?= =?us-ascii?Q?Lh6cKfpannElRLEfmtLe8sk16YuanEfK5dXOVeL1iymXU9G5dL1Atoiunhgf?= =?us-ascii?Q?+YnrRYksShIZasPXY/5dsIHLjdsdUEWTaIMgqcsUhTm7BSI10+5RVXtubMPm?= =?us-ascii?Q?JIz97A7hUJs0JCSl5SHykd7bv7mIt2RFub6payPy47T4jQmMbYaDF6lLSoPQ?= =?us-ascii?Q?91ZGNGz/XENS7DsoWIPBxDexacgWFLGxvDo+3vOHsmsLfZvRiuGDXdbD+HzD?= =?us-ascii?Q?oSZTDW5R7SkPRjK0kNwUrqdexkFVo/q83zis8AjQaL36Yk49fwgmajV5fluE?= =?us-ascii?Q?zov9Gb7ToIuXJDPuSfChtzgqzylIXv+pkB2ekUrxanmsuugKNzvtpAD1ZAw3?= =?us-ascii?Q?92WxYpJLcnzLF2Q/N6giaOhOuCi+61tjfBuF1ij3DCMtkvX1RqgN+Zu+0iuH?= =?us-ascii?Q?rC7Ce2yJ4CTRCQum9GGKDfanv4pKuyCw7GC5eKkXTkJyD0I0+fj/nGhVl2cs?= =?us-ascii?Q?Vv8szQJ+v16pX7QhGOFVikX/DmkrgcmjlQD1jtnk1jllkUcXJ2ZqayJw92+r?= =?us-ascii?Q?rUqumVM+XPuqGYfE/usaSmYfrYU9eDFR/eWN6E8ivAyH+XV6ytCOSKv3OZEj?= =?us-ascii?Q?AWpoIFTzgRi6+8E/sLnjX2JI5EjyyZEvN/wbRhUCYpMjqP03t/fxz5l7iNuq?= =?us-ascii?Q?hnYfoiGMR3MBNbFWcmhlZ3IDsRuxvxU3vqMEoXdu8p5ZPdAUWP5BexY9g6MT?= =?us-ascii?Q?h2h+Y/V3wjYjmG54AK8kIgwBCD1nC9dHJuObGp54JBSNeMcXqHlE5v2KMEA9?= =?us-ascii?Q?2cF7v9O6TQ2GALirHgpT22o8o9CPfik02bvDR6qLBh1J0fQxDjWg3jXZGV72?= =?us-ascii?Q?EeMeij3wMjO9OgU77UuNJymLPPwwScGu1JUqpcjf2c+9ryyGL767dVffRXnt?= =?us-ascii?Q?jfdF1Ig5Z7wrW2X5d4RJNWaLokvLY77c5424quI5yb5I829wJgu3ftHtPAGT?= =?us-ascii?Q?cytwBi0JWYrpr1t/nWIrUHtwBhShVpWglD8btKFkEpIGJOEZWTyohaNzaNba?= =?us-ascii?Q?fUqAySyfNOBxtrDJUYIGiiTHLWR4CJjsNFflLhjVUmBvdMxsm5X2fjMTS4vl?= =?us-ascii?Q?GmyW8FvJ08h/Nvmgn1xsPy+nwMCWZ4gh7uwx63R5Vbs2bgbJyPlUNXH6jYOR?= =?us-ascii?Q?F0DBFJRtierCd3XhxjNke0H+6BJa7ewJPB/cYHWwT7H9x1VDcpQiZ0Uvy7O0?= =?us-ascii?Q?oclcAjpVPt/oPGvyJm+b83HHgoYzZ4ibQTeEFK9Oc6ZBAnHd+bDNTsI4tf6J?= =?us-ascii?Q?rNHPa80BbJfxD66Hc7mzTJyV35q8W5XZI5/DXMaFxLi+ez84kLGu1qAbIA1b?= =?us-ascii?Q?YFKGtvCck3RX/8Xo5K7Yya0c3kXXbSRzkC13mXCfaeJ26kMuqlPbV3Kuy0Lr?= =?us-ascii?Q?zO0miAMeMGwnUOw69p3vmZsQsifMoQZgy1ie0lSBOV0FJpNoqSp54+8M2rOi?= =?us-ascii?Q?9xJjp9xoDtWmGXHhgZeWsYrYnNHq2f6j/QiPkatsAz89h3XvkWnyTaveffBu?= =?us-ascii?Q?VCmz+XBmvBsXLaQWwGJ1TvVrLZsLZhgkjET+Eww09FhnPcQWulLQhARJCLWW?= =?us-ascii?Q?Jw=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: b2e59534-ca3d-4491-7936-08dd34417305 X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 14 Jan 2025 02:16:26.4940 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: 46c98d88-e344-4ed4-8496-4ed7712e255d X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: o5WEyZ8bKbuOI/oYbgAuP3i5rLgiS+vEpDMq9/hOrZQXT8EFAWEI1ptC8WrNRXBXte4DflGcRdaogirqRQI952gVXpC6z51y3kJnpJiSfX4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CH3PR11MB7865 X-OriginatorOrg: intel.com X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20250113_181717_910319_6341A273 X-CRM114-Status: GOOD ( 15.66 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Alistair Popple wrote: > Longterm pinning of FS DAX pages should already be disallowed by > various pXX_devmap checks. However a future change will cause these > checks to be invalid for FS DAX pages so make > folio_is_longterm_pinnable() return false for FS DAX pages. > > Signed-off-by: Alistair Popple > Reviewed-by: John Hubbard > Acked-by: David Hildenbrand > --- > include/linux/mm.h | 4 ++++ > 1 file changed, 4 insertions(+) > > diff --git a/include/linux/mm.h b/include/linux/mm.h > index f267b06..01edca9 100644 > --- a/include/linux/mm.h > +++ b/include/linux/mm.h > @@ -2078,6 +2078,10 @@ static inline bool folio_is_longterm_pinnable(struct folio *folio) > if (folio_is_device_coherent(folio)) > return false; > > + /* DAX must also always allow eviction. */ This 'eviction' terminology seems like it was copied from the device-memory comment, but with fsdax it does not fit. How about: /* * Filesystems can only tolerate transient delays to truncate and * hole-punch operations */ > + if (folio_is_fsdax(folio)) > + return false; > + After the comment fixup you can add: Reviewed-by: Dan Williams