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 83026CE7D0B for ; Tue, 1 Oct 2024 10:29:43 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 0F20928006B; Tue, 1 Oct 2024 06:29:43 -0400 (EDT) Received: by kanga.kvack.org (Postfix, from userid 40) id 0A2A5280068; Tue, 1 Oct 2024 06:29:43 -0400 (EDT) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id EABE428006B; Tue, 1 Oct 2024 06:29:42 -0400 (EDT) X-Delivered-To: linux-mm@kvack.org Received: from relay.hostedemail.com (smtprelay0010.hostedemail.com [216.40.44.10]) by kanga.kvack.org (Postfix) with ESMTP id CC556280068 for ; Tue, 1 Oct 2024 06:29:42 -0400 (EDT) Received: from smtpin23.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay10.hostedemail.com (Postfix) with ESMTP id 5DA7FC0F05 for ; Tue, 1 Oct 2024 10:29:42 +0000 (UTC) X-FDA: 82624662204.23.069B97F Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by imf02.hostedemail.com (Postfix) with ESMTP id 1D42680015 for ; Tue, 1 Oct 2024 10:29:39 +0000 (UTC) Authentication-Results: imf02.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=odvpEnER; dmarc=pass (policy=none) header.from=ibm.com; spf=pass (imf02.hostedemail.com: domain of gerald.schaefer@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=gerald.schaefer@linux.ibm.com ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1727778440; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references:dkim-signature; bh=VG3OfmvRm7AEmTWv1uILD0B/40IGpSJhbrITCHAcZ5Q=; b=Mx1lwObh13BKfSw0anmFOtKDqRKpdGaC4YZydAdkIvR/+GkqpcYtU5tXUg3u5q68s02ZQe 3Kfg6tZJpCfjmb9ACaVW3taHTpCyF7BPA+7vtp3ydNEiN6uRxchozqVbRpz2zEypzcp7s5 SUu/SLY/HHvPRXFmq/ky761SjluRIVc= ARC-Seal: i=1; s=arc-20220608; d=hostedemail.com; t=1727778440; a=rsa-sha256; cv=none; b=07ij3IadjjiwNo+XCVa23YxbCl5+DMk+JKZ1QcszPHw1MBHaHnpHmasTySIeDXJUu6sTpj iFi16GvP7hO4WOm/AJoPmS2zQaKOo6NXtAPifWBfO40z1cMp7LhyxLLR48z1dcR8oOOYvN hQRZypLSiWS3WGkKkrAY3eCGTh4tLsA= ARC-Authentication-Results: i=1; imf02.hostedemail.com; dkim=pass header.d=ibm.com header.s=pp1 header.b=odvpEnER; dmarc=pass (policy=none) header.from=ibm.com; spf=pass (imf02.hostedemail.com: domain of gerald.schaefer@linux.ibm.com designates 148.163.158.5 as permitted sender) smtp.mailfrom=gerald.schaefer@linux.ibm.com Received: from pps.filterd (m0356516.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 4916LVWF011804; Tue, 1 Oct 2024 10:29:32 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=date :from:to:cc:subject:message-id:in-reply-to:references :content-type:content-transfer-encoding:mime-version; s=pp1; bh= VG3OfmvRm7AEmTWv1uILD0B/40IGpSJhbrITCHAcZ5Q=; b=odvpEnERntXHrHVk Yp1GAH3YkMbbayIY4Jb4p3dKxcTUAiEhoycBGIvzSQFOqtOAbnH4YFjma9TVTi6O zxPUmfmV64Sy5bhISku6Sp8hajV2WbUaC+8tnC1f+QcWQgrBxdLVwXQSZKAZilh7 h/c/tWvvx6B+COIIjNLraGi9JhlbzZEzKjoMJWJi5AeTwyTtAjg201sQgSdcuQde D91YmEU6lXSLbPhcUtSt2ahjR1kD62MS62clq8N1qN6mPJXwgAMTH0/sIZtAM/82 GrpLY9lH1jp2Wi7PSkr8eP7sqX9Ek8Uo45MlvTIhLIB6iHZdfXFeo3Pqo5AwUp/q FlXE/Q== Received: from ppma13.dal12v.mail.ibm.com (dd.9e.1632.ip4.static.sl-reverse.com [50.22.158.221]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 420bqs166g-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 01 Oct 2024 10:29:32 +0000 (GMT) Received: from pps.filterd (ppma13.dal12v.mail.ibm.com [127.0.0.1]) by ppma13.dal12v.mail.ibm.com (8.18.1.2/8.18.1.2) with ESMTP id 491AKLcN013047; Tue, 1 Oct 2024 10:29:31 GMT Received: from smtprelay02.fra02v.mail.ibm.com ([9.218.2.226]) by ppma13.dal12v.mail.ibm.com (PPS) with ESMTPS id 41xxbjbffg-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 01 Oct 2024 10:29:31 +0000 Received: from smtpav02.fra02v.mail.ibm.com (smtpav02.fra02v.mail.ibm.com [10.20.54.101]) by smtprelay02.fra02v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 491ATSSH56295798 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Tue, 1 Oct 2024 10:29:28 GMT Received: from smtpav02.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 4C7B420043; Tue, 1 Oct 2024 10:29:28 +0000 (GMT) Received: from smtpav02.fra02v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 4507E20040; Tue, 1 Oct 2024 10:29:27 +0000 (GMT) Received: from thinkpad-T15 (unknown [9.171.59.94]) by smtpav02.fra02v.mail.ibm.com (Postfix) with SMTP; Tue, 1 Oct 2024 10:29:27 +0000 (GMT) Date: Tue, 1 Oct 2024 12:29:25 +0200 From: Gerald Schaefer To: Dan Williams Cc: hca@linux.ibm.com, gor@linux.ibm.com, agordeev@linux.ibm.com, Christian Borntraeger , Sven Schnelle , Jan Kara , Matthew Wilcox , Christoph Hellwig , Alistair Popple , linux-s390@vger.kernel.org, linux-mm@kvack.org Subject: Re: [PATCH] dcssblk: Mark DAX broken Message-ID: <20241001122925.52558c08@thinkpad-T15> In-Reply-To: <172721874675.497781.3277495908107141898.stgit@dwillia2-xfh.jf.intel.com> References: <172721874675.497781.3277495908107141898.stgit@dwillia2-xfh.jf.intel.com> X-Mailer: Claws Mail 4.3.0 (GTK 3.24.43; x86_64-redhat-linux-gnu) Content-Type: text/plain; charset=US-ASCII X-TM-AS-GCONF: 00 X-Proofpoint-GUID: ruMcx1VKO3ZyLX4ztnq2rFmxFuw6PeE0 X-Proofpoint-ORIG-GUID: ruMcx1VKO3ZyLX4ztnq2rFmxFuw6PeE0 Content-Transfer-Encoding: 7bit X-Proofpoint-UnRewURL: 0 URL was un-rewritten MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1051,Hydra:6.0.680,FMLib:17.12.62.30 definitions=2024-10-01_07,2024-09-30_01,2024-09-30_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 priorityscore=1501 clxscore=1011 lowpriorityscore=0 suspectscore=0 mlxlogscore=999 bulkscore=0 phishscore=0 spamscore=0 adultscore=0 impostorscore=0 malwarescore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.19.0-2408220000 definitions=main-2410010066 X-Rspamd-Server: rspam07 X-Rspamd-Queue-Id: 1D42680015 X-Stat-Signature: xz8j516yqubwfpr4b18df81ssfp5nf9y X-Rspam-User: X-HE-Tag: 1727778579-916913 X-HE-Meta: U2FsdGVkX19N9Wcsjf00dxmTM6HRd1u4TOVEHyW36slg018l4fkKFxGPChtSm/boeiyVg3CE0SCjcH/9IKQd9ctZXQOReTQSrO81CvRHHspz/bnH+YeMC+W8A89iz7c/CHwz1mkcgqIm+jHu4pN06P5EiB/jZhvMRzQQ3Z9ihR+roiXR9PcCsJF3nzA9oun3JLvj9REmFLc8tesJnXgS6uTrgnqNt3xAvcOlAcSXEsn+oZbeUt/FWtrfNQ5/zeoPHH15to59lz2ObzwXIaD0BhwcNYdv5N4BmYj3cYZZQHAD5544nMcHVCnIsGAWaqWXg8Z7wsFMIsbKkXpJXIfclQxAX3B+wxF05NvXsGgtQjyJWi0k57M/IjjJ7mN1zefaBiFQtpvv6HY4oiLZW1TPecKWUxs17mscFHcTke3Lq7PeZPl2FjFrT8xKtXJU/KmJQvRl1IK7kTlXZfRc3aZN4NSayKZ/7OSf7SbMGoBaPfwlSgIwnkLlc9ZnfO71yDLNqGX8IL60dqPP2IVGWDQsacM9DN1uliNcszCakJvW3NOKaRohFD6Bg39Oh0J+unT8VmFV3YeUZ7a9HkpBPI6vOv7yHnG79/aoiXoa+Nh1rLOiedS3iYN0+VrS9JFwVKDxVGKpR7bS75xdC4M/7teObVuyuA66gStWOF+yTj10XcpjOtXOHLFKPdx/jABlA1ei7iwX9I3/1zxMx0ZbaHATVHId7OGUDQPbwsbByXhlIZpcxv0Mz52w8mGa9TBcaNYA7a/H6rbzwDgaK2tREl7O6vY9W9dSpJWkchkYy7uDSpaYEoBmmfVHiP4G+6/19XcsmolFC1E0OK4B7d3irtYPnaXcF/ArTDKHqrgCLjen7DTMxhWF/9aspUhhb9qlRMxITYAPwTeOXTHd23ySuuy3eX8ed5FcKp8oqtUbCG+t+aOQVuZOIb8ucwo7jGXuLt0LbrBqaikRcUaXILx0RPm tjk+JI5Q lS+bZwDb73EKkFlzP09N+m4/frFMeTdjgZA9tOCjIcVD5OnMrndqlKt9EfrYgnO5a7/vdUGZ7NXG4bPV36MeR0C8/MWDgd6mfGq5DpubYQmLCiIFATBrksAxOyLzgn/xsv5t2ZU3F4cafII9HPtDroG5WsTTZjV2m+bLhYjWK80xYXRo21XGw+1HgLBlrYavwRerN+Vc8/MqhKZEQqLTFCrMC+B4Vhkizf9uTsvrO7ljjAlFyjaqQR6l4426iiMCg1cAb4Sf3ZvWv8nkW8dFwp5STWKgGEBGmzg9p8f3YYj+imtm08RByCFIgAJQyqFA7VTYC2otDdTxK1dIESNUmMI7Gva2z15Wj3ZsdNEuIdyMicVDDkE7XsYgMo5HPxe8gFcXJAf3IE3GGKfhsur5W5oyoxowIN8QYGW1+NjbjCCTuB8vlsJ8X6Xx+piiAyPYpli4A+9GV12STXXtf/JQAkuzOY6SprtIp36y+ X-Bogosity: Ham, tests=bogofilter, spamicity=0.000001, 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 Tue, 24 Sep 2024 15:59:08 -0700 Dan Williams wrote: > The dcssblk driver has long needed special case supoprt to enable > limited dax operation, so called CONFIG_FS_DAX_LIMITED. This mode > works around the incomplete support for ZONE_DEVICE on s390 by forgoing > the ability of dax-mapped pages to support GUP. > > Now, pending cleanups to fsdax that fix its reference counting [1] depend on > the ability of all dax drivers to supply ZONE_DEVICE pages. > > To allow that work to move forward, dax support needs to be paused for > dcssblk until ZONE_DEVICE support arrives. That work has been known for > a few years [2], and the removal of "pte_devmap" requirements [3] makes the > conversion easier. Thanks, that's great news! Without requiring the extra PTE bit, it should now finally be possible to add struct pages and ZONE_DEVICE support for dcssblk. In the meantime, it is OK to pause the DAX support for dcssblk as you suggested, and finally remove that ugly CONFIG_FS_DAX_LIMITED. Thanks for bearing with us for so long! > > For now, place the support behind CONFIG_BROKEN, and remove PFN_SPECIAL > (dcssblk was the only user). Ok, I guess that PFN_SPECIAL was there because we had no struct pages for the DCSS memory. When we come back, with proper ZONE_DEVICE and struct pages, it should not be needed any more. And yes, the chance to completely remove pfn_t, after Alistair's series, is quite impressive and even more motivation than CONFIG_FS_DAX_LIMITED. > > Link: http://lore.kernel.org/cover.9f0e45d52f5cff58807831b6b867084d0b14b61c.1725941415.git-series.apopple@nvidia.com [1] > Link: http://lore.kernel.org/20210820210318.187742e8@thinkpad/ [2] > Link: http://lore.kernel.org/4511465a4f8429f45e2ac70d2e65dc5e1df1eb47.1725941415.git-series.apopple@nvidia.com [3] > Cc: Gerald Schaefer > Cc: Heiko Carstens > Cc: Vasily Gorbik > Cc: Alexander Gordeev > Cc: Christian Borntraeger > Cc: Sven Schnelle > Cc: Jan Kara > Cc: Matthew Wilcox > Cc: Christoph Hellwig > Cc: Alistair Popple > Signed-off-by: Dan Williams > --- > drivers/s390/block/Kconfig | 12 ++++++++++-- > drivers/s390/block/dcssblk.c | 26 +++++++++++++++++--------- > fs/Kconfig | 9 +-------- > fs/dax.c | 12 ------------ > include/linux/pfn_t.h | 15 --------------- > mm/memory.c | 2 -- > mm/memremap.c | 4 ---- > 7 files changed, 28 insertions(+), 52 deletions(-) When you also remove the now unused dax_dev definition at the top of dcssblk_add_store(), as noticed by kernel test robot: Reviewed-by: Gerald Schaefer