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 B52A8C4167B for ; Wed, 6 Dec 2023 19:38:52 +0000 (UTC) Received: by kanga.kvack.org (Postfix) id 4CB016B0074; Wed, 6 Dec 2023 14:38:52 -0500 (EST) Received: by kanga.kvack.org (Postfix, from userid 40) id 454176B0075; Wed, 6 Dec 2023 14:38:52 -0500 (EST) X-Delivered-To: int-list-linux-mm@kvack.org Received: by kanga.kvack.org (Postfix, from userid 63042) id 27FCB6B0078; Wed, 6 Dec 2023 14:38:52 -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 1098B6B0074 for ; Wed, 6 Dec 2023 14:38:52 -0500 (EST) Received: from smtpin12.hostedemail.com (a10.router.float.18 [10.200.18.1]) by unirelay03.hostedemail.com (Postfix) with ESMTP id DCC75A0249 for ; Wed, 6 Dec 2023 19:38:51 +0000 (UTC) X-FDA: 81537406062.12.F6D7CDE Received: from mgamail.intel.com (mgamail.intel.com [192.55.52.151]) by imf29.hostedemail.com (Postfix) with ESMTP id F2407120020 for ; Wed, 6 Dec 2023 19:38:47 +0000 (UTC) Authentication-Results: imf29.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=mH4wjAHj; dmarc=pass (policy=none) header.from=intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); spf=pass (imf29.hostedemail.com: domain of dan.j.williams@intel.com designates 192.55.52.151 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com ARC-Message-Signature: i=2; a=rsa-sha256; c=relaxed/relaxed; d=hostedemail.com; s=arc-20220608; t=1701891528; 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=tj1YchFAsScFiiLpGf5TnCq7+6YmgsDB6V6n4hBkjwA=; b=0LJuAx3+6uso5t/YN9tRn92blar+73pfUa4mq7KevuDSIb5e8b/sTHvJvJYjOFxkuxeebj +ieOiH/GteB9tafLzZfq3f7w76pLjcMFmfMBkS/K9Rsn6bIoEWL/md/RD0tbQIkviq8NEL beJDUxRD3vv2g8C9T8OIaph16P11x8Q= ARC-Authentication-Results: i=2; imf29.hostedemail.com; dkim=pass header.d=intel.com header.s=Intel header.b=mH4wjAHj; dmarc=pass (policy=none) header.from=intel.com; arc=reject ("signature check failed: fail, {[1] = sig:microsoft.com:reject}"); spf=pass (imf29.hostedemail.com: domain of dan.j.williams@intel.com designates 192.55.52.151 as permitted sender) smtp.mailfrom=dan.j.williams@intel.com ARC-Seal: i=2; s=arc-20220608; d=hostedemail.com; t=1701891528; a=rsa-sha256; cv=fail; b=mWSG3vExkDZhg8ZcXuxZDhxHbNJ+c7f/DFfRuvfpc1BqcK12JFRhG3yIxeGzkmdfAXFR1O ysKyu/Hja5mhEplAqBgt3fJ6ngjjux64GeeZZrkyQwR+TyQO5yM8nwk6QoYG8EE5a+oroO 5qn+iJZ9Qnsz2rkqDEr8gtvaTb6UcSU= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1701891528; x=1733427528; h=date:from:to:cc:subject:message-id:references: in-reply-to:mime-version; bh=9epjVGXDJBbZLrQ55qfQ+bEj530zLB18lLWwNRfo08s=; b=mH4wjAHjSER5DWIWlr3GDNLVIw0AWah5c02isIcX3AaGKsmBdY01ajE+ 9aGotMkh8MrqSWOv2mk66LQb1e/pqBnSTozHsArRlwMbaMiKD4/Rc4g/A 4dLX6jWfglmabvmevKouofBhk7OZamGCNDZX/wVlQY+FyNxGdQhPjmXkw BuiV2cl8qWkVyFYPkWiwIY3UkQBQ6+Wxd06xfN16QJHGCpYj86E5LR4PD s51pXfvIT/BQ2y07hNHv+mmTkSqEFncqOW7qNWsmkuUeQ4YzcqHLh9U/A Nngg+mYKbNpPu+tzIMUGTP6HmzJn35e7VLgqp9lGlwG9rLIHFoRXji4Rp g==; X-IronPort-AV: E=McAfee;i="6600,9927,10916"; a="374289488" X-IronPort-AV: E=Sophos;i="6.04,256,1695711600"; d="scan'208";a="374289488" Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 06 Dec 2023 11:38:46 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10916"; a="721197179" X-IronPort-AV: E=Sophos;i="6.04,256,1695711600"; d="scan'208";a="721197179" Received: from fmsmsx602.amr.corp.intel.com ([10.18.126.82]) by orsmga003.jf.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 06 Dec 2023 11:38:44 -0800 Received: from fmsmsx612.amr.corp.intel.com (10.18.126.92) by fmsmsx602.amr.corp.intel.com (10.18.126.82) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Wed, 6 Dec 2023 11:38:44 -0800 Received: from fmsmsx610.amr.corp.intel.com (10.18.126.90) by fmsmsx612.amr.corp.intel.com (10.18.126.92) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35; Wed, 6 Dec 2023 11:38:43 -0800 Received: from FMSEDG603.ED.cps.intel.com (10.1.192.133) by fmsmsx610.amr.corp.intel.com (10.18.126.90) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.35 via Frontend Transport; Wed, 6 Dec 2023 11:38:43 -0800 Received: from NAM12-DM6-obe.outbound.protection.outlook.com (104.47.59.169) by edgegateway.intel.com (192.55.55.68) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.35; Wed, 6 Dec 2023 11:38:43 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=SiQ5URHDCjhUdAtRzrAsQlv5jfwc3V27ZEu+NuO2OcLjneC1iM7BSSMgDhYQlDFrqxtyWE3GFkMvuUAhaobiXdj2cmgOLhuZlFf4/fRdJGI7VNcMTQ4BtHZBBX0dHwjrpozkfJ6dAfCnHjFIQzrvBOvZUZ2tCx9wzBCZe8jvvdADBAghWA/2b3j+3ZRJy9KFLnIZtOcitOyLYPPkXdwbXKw0w8GeaeOB+OZD2fSqkoATqqaKMrwTGKBK3o+qkG/SxlMZkmWD4vmykX4Zq8IfPnWryYVHMdt7Fotkec/HWt8nBiEHAwS8oEMsGst7xNTiDcH/Nt8NfJXpp+YAuPzEVQ== 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=tj1YchFAsScFiiLpGf5TnCq7+6YmgsDB6V6n4hBkjwA=; b=SbinyWAgAIhqVPYd9nBL7gZDKzSAt61grUBxQngZ9mZMeoGYT89n4fhQDAX/dPlly0UJoQAMZYPW5YyOaC6NfWuof2oVcmCNRvBfx+kBbFlOxeR+4U8gj72e+5BBrJyN3JdEmhNcTZwaPIm6Pl3curI//duicDLv1hOg27ML80EyR9cq462DSPN+2bQkgKT/6K9WBPYCk9X68KyY2iPXWbQn3CrkWCSb2guOSmjUtD1oreiRtgwxMB1CNyz5O13d93Hf1fPV3VDB+pKEj/rs2y3B2erZzWPmrDZ55Ej1LpnEJjVIDSbwRNGp4HuwrHHe66aJjpJAae9RYj9bzv+TGg== 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 Received: from PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) by MN2PR11MB4661.namprd11.prod.outlook.com (2603:10b6:208:26b::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7068.25; Wed, 6 Dec 2023 19:38:40 +0000 Received: from PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6362:763e:f84b:4169]) by PH8PR11MB8107.namprd11.prod.outlook.com ([fe80::6362:763e:f84b:4169%5]) with mapi id 15.20.7068.025; Wed, 6 Dec 2023 19:38:40 +0000 Date: Wed, 6 Dec 2023 11:38:34 -0800 From: Dan Williams To: , , , , , , , , , CC: , , , , , , , , , , , , , , , , , , , , , , , , , , , , , Subject: RE: [PATCH v4 00/11] cxl: Add support for CXL feature commands, CXL device patrol scrub control and DDR5 ECS control features Message-ID: <6570cdbaa96e0_45e01294e0@dwillia2-xfh.jf.intel.com.notmuch> References: <20231130192314.1220-1-shiju.jose@huawei.com> Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline In-Reply-To: <20231130192314.1220-1-shiju.jose@huawei.com> X-ClientProxiedBy: MW4P221CA0013.NAMP221.PROD.OUTLOOK.COM (2603:10b6:303:8b::18) To PH8PR11MB8107.namprd11.prod.outlook.com (2603:10b6:510:256::6) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: PH8PR11MB8107:EE_|MN2PR11MB4661:EE_ X-MS-Office365-Filtering-Correlation-Id: 5a318b6e-0019-4554-5a49-08dbf692f2bc 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; X-Microsoft-Antispam-Message-Info: IsVRYbxOHKFm3WEz2jgb3+Ze626v/VrEVwmzLAL/1EZL1mtbrkLy/gux6Va0rCpgFFmyGtKKN6c3VmRdm4Dx7/7K/bvbPMUgUtDMS7bRBHAmVrBxvY6KQPi/hhsc4CNfNqzAxTgLl2tdkKDtZgSVXGIgBZHhASHYamQ7xo0JY5EwIe7PahtRZMsiF8xH889CTnHeEIanHykzfvI9JfxCl+1L120IMEBsrpBF4/6G1q2aJ9NloG8+X368MuYIoTrRWrEC/454St3XE7d9iTnp9gMMvuOTnXksHU2w4OIGcygXKprxh3SquDiUe3r5gJJnFemzrgZ+R8N3es8vBEzrA633p2qh+9atooPsAiCrEo3X31l4UD0qeRzI1hrwKsFBh98fa2ogD4FScrTUSRpG8UJ5daEzS0+lEx2X8QFkrt+vKlHpaaQnz5OdZQ2QFNhWLUW2v9eeKJEwi9b+s3rrsIwE0XJA2V5JozEnYkl2PdusdTagCUECTu+uIh/7g7q/vgFj0fH7kEpHqZeONGZLG2yLcYgp0oNFMRxhCUr3va0hbG/uD0zshhrSlyR+R1TtXYct1Mztf+za9JX1x7aIkpoCG2O0wZt8j+Hztg+MQF8= 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:(13230031)(39860400002)(346002)(376002)(366004)(396003)(136003)(230922051799003)(64100799003)(1800799012)(186009)(451199024)(38100700002)(2906002)(921008)(82960400001)(26005)(83380400001)(86362001)(5660300002)(6486002)(478600001)(6512007)(9686003)(6506007)(6666004)(8936002)(8676002)(4326008)(7416002)(7406005)(316002)(66476007)(41300700001)(66556008)(66946007);DIR:OUT;SFP:1102; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?PUPP/FnQJ3TsIT0uEPPCYzTZ4gByfCX4bUPW8UPv1Z4z5uV/S2wNIJr3Ddtv?= =?us-ascii?Q?Xtv2MAdX+jBLw/b+TlpT9wQhPms6j1/Vu+5hTTOatahv+kv3heccRdRZHlQI?= =?us-ascii?Q?3suwa/Du+fsX4pGcqBu5fp1DanQw5SB31dZnP8gNCn3fu43dDf27qBvdZFiD?= =?us-ascii?Q?DUW0Z504fSBz3TATcy75jVvak1Ndzsz+BkStmrq3y34girYuEeklp4zYggzt?= =?us-ascii?Q?o7qplK0b42BECee9P0JvgNCM3cty1BtwF8cKW3sNMnhpVsxP8ceXPYk5pYP4?= =?us-ascii?Q?LxAZCCZA9mSQWFT+HRS+381Vj56nh/SiAdRa6xoftPtWsB0szz3Ttv2UG1N9?= =?us-ascii?Q?rpB+I1pDg+QOiEQd1HOzXwJgnTvjgRFy8DjOVkPaVZxO27VTZVQFA4nl9A61?= =?us-ascii?Q?jPv0H6v1ASTJYarZstb9/h0sNEB86IrdKAQo3cMh9BzcG2zzC0csZkyw4Mp+?= =?us-ascii?Q?lygqnlSk4zr2zsYlIY9bXN2D+xRShyreR7Z62xYovLZ3qP+iMbFne4sPzy6x?= =?us-ascii?Q?2kIenaJ8EplMpARR/sG36Ga/QUjlIVyEpMI5YWckhL9+ZDrBlr4KMf71H8rg?= =?us-ascii?Q?4EEtZrWiwlowQV4qHP7ek/jq2nxSCC16mJAZyxDQOvIG7Nm7tqnqmggijssL?= =?us-ascii?Q?TvdPsDlQjUCR+swCixmgq/xABkBskYc8smdcBhVmk/2mC7gqwI6vfNKhRFzj?= =?us-ascii?Q?mW57j1dxCllac0elGTbSd/Ht9cXn/ADWv3qWTblpHleS0TaAWcO2I0tFen7x?= =?us-ascii?Q?oo9Nd8qA3mTpcd2N0pS4Stnu2S8n9br0XJtKnb8YE8x46ufj6v7yjVc18Gug?= =?us-ascii?Q?eRKk9IltwHhHGiE+eF+8ox+ryN5Y9hBBLjXO6qBKHtDuYs5YP8I72ISWPC1r?= =?us-ascii?Q?K0+kcy1CvC1559sYHbqhUc7HFZaOLR3p+42yahmBA8Y8CCqgqmmaHNzIRjqv?= =?us-ascii?Q?kiuPUW+S35xqt1+h0S+Sx0yc4Esrq8IsYUNIgwzS3KJjQ8rwdKnmazvwm8BA?= =?us-ascii?Q?MkHfhVCatEbTOg+Ll3zQhwYZyzDukDsEvjofaS0Crbwi/zCRJSCzfc5JhNL+?= =?us-ascii?Q?WNg3hetgwYjZeDvEkzp/tHyD9iF/t/V+Iqwms19pvSKaaFxaqKMjlNH5nUKv?= =?us-ascii?Q?+XsEpophnn9amixV9/iVLJRd2MX7aQKfxGQEiUYG4L5JtO+Xhmszq28GOOIS?= =?us-ascii?Q?W9G/KLe3xnxejICkg4uIyYNNikvL7/doHNBzfqHAdkx9X3E3YkQs/2q05Z5z?= =?us-ascii?Q?JfraSOlwNaDgBlpevw/mgNImyLPTCJFGW3sOEhQ8sozHXsyeFXTvELODeAfy?= =?us-ascii?Q?t3h+RDQWcuzn1Ac1R64cDGS1ItVsQNKjYVCnasWiP4cQe4aOLh8zkqGaYB4X?= =?us-ascii?Q?dTGah9TeJ0mWIbAqu4lrtuuzXpdb/kiOb4Xln7jnakyUdhJsc6xA8cdiZZCm?= =?us-ascii?Q?7BuZMPzZp0ysJE956HOwh22xEkQ2EN30W21t+7cDn6GbZ/3oYliJjzmRd6bL?= =?us-ascii?Q?7gsh6CORKcVeygTg7rFYLzSxl28I1hnJSPsvN4OMmBY+4BgJGM7Mo4+oAePM?= =?us-ascii?Q?Gn+cHADrp477s2WX9C9swvxu2cVCX9ylyI/rCJhe5P+xLaxjFl/4L+s8dEGe?= =?us-ascii?Q?QQ=3D=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 5a318b6e-0019-4554-5a49-08dbf692f2bc X-MS-Exchange-CrossTenant-AuthSource: PH8PR11MB8107.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Dec 2023 19:38:40.2850 (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: wcd8rZiD/SlV6/oZSkGs5W0wSbv/UkwEJ64NmpZQrDdihnXjnJQrTP3AALlCqZqiWihtTV89aaVZcqlUPJ65lt4LJlZ063hV2ySTCCW+roY= X-MS-Exchange-Transport-CrossTenantHeadersStamped: MN2PR11MB4661 X-OriginatorOrg: intel.com X-Rspamd-Queue-Id: F2407120020 X-Rspam-User: X-Rspamd-Server: rspam05 X-Stat-Signature: noux7qy8icr65s9hxe7zmrp1645q1o89 X-HE-Tag: 1701891527-292229 X-HE-Meta: U2FsdGVkX1/Tz+NVSXDjLo5qOexJlOMshcPqMITdQJ3XYVEvl9HEy0iLImO2FaQIvY8zroyHo1Ev45cRN8833YdTxqGjLRX8w22kaEPb/vPKEv9llCoA+02QSe/6AhYZHVH4ufjAsLxW3XG7LIubznaNmjB33FzWDRRaw6EkPNweSRN2b4xNY7bPhciX45ghO/7vDYL0/J7+LXxtAA8lzflL6nqoB8dE0N+l140SYLLG2gvj2ZrfaYxtylZ901nalBCyo+1MUaNKtHrB26GuNzpTF3XciV6CmtDu5yWja/SGxTsIX5jIQwIMrTB5OzCiI4HMXin0MGyXjdhuphqatcqtQS9sHwCK2OR7pnOAb1sZy3sAun5h+5yfjAxelzLwVLSBX2bbUkg4nvvP00LRHW1RHqW6BebB5ZtaGGwhLnpxQ9ILAcRiopmwg0dtSprZhOxpu6lPlbvR1Gx4qSkwVr5xJcoemj0KeWeLIhkzSu2nv43AUSiEIblNEaarwBoCI1UTxR8Xd6jW5lv6mpU0n7CETDaFwaJQZ7LIJZMBTq4pUesUNBBVR8ZLhZJwpyldw5AOcCeysAB2EOLPeIFouhgZdgDyyJf0g6nolreuepyYfcGjzntuW0stq0WJ2Wp8s0D2Jmr2icC4Nq0f+KgAnZB8q2CpVXurNc2GRsM0ITnVIZuc24EGzpktG2EZa+g3KGVPaevV8lY0Vcob6HOHcCvITpyj9BebUC++fA8xTH+QfQW62UQChsReo1jZKHpxeEKzFYyOiz17zxI6SZhjYFblRJpq7dzOQg6did1X6PY87s5rL0RL6s5EHp/PG3oRG8iwK6YRAWJ8GGLp77sUy14LVYiSPaiK/yJ98TS601LIPi3t8uizEoakUypcLos8J+UWDH4CS2iWtOwdkS7nzwl9aCE7OoDJM5py34FLwwFW6GRQDOYrC0GAAbFTSL0KGH9CeVt/AEfy8o5X/XI 9q325KVm vljHhUIwE4NqMfEf7X/ENuXxt27uBVPzGzwEupdEqtkon+Uwcuy4kxJ01WgSyu8qgpuRNbM62O2Fatja1CeS1ajmTa2yyL52sp8PlyRYQLV60yC+iUSKr9B8Q7otJDnXkjE+YtPfUz2Hn+yiHfAzyRNMOlM55ciksUlqi+pocL7XTBZEfFvfOjaeYvlbkIKQNXRDlqPUwJhLqemWa2znH31TpXH25XQ8+S9fGLTrvYHrAARlYGRnGwTmZyXYHyyEFee541oxVUiGRBYw= X-Bogosity: Ham, tests=bogofilter, spamicity=0.000211, version=1.2.4 Sender: owner-linux-mm@kvack.org Precedence: bulk X-Loop: owner-majordomo@kvack.org List-ID: List-Subscribe: List-Unsubscribe: Hi Shiju, I have some general feedback at this point before digging too deep into the details: shiju.jose@ wrote: > From: Shiju Jose > > 1. Add support for CXL feature mailbox commands. > 2. Add CXL device scrub driver supporting patrol scrub control and DDR5 ECS > control features. > 3. Add scrub driver supports configuring memory scrubs in the system. > 4. Add scrub attributes for DDR5 ECS control to the memory scrub driver. For a new a subsystem that is meant to generically abstract a "memory scrub" facility the "DDR5 ECS" naming has too much precision. How much of this interface is DDR5 ECS specific and how much of it is applicable to a theoretical DDR6 scrub implementation? My primary reaction is to boil down this interface so that only generic scrub details are visible in the ABI, and DDR5 specifics are invisible in the sysfs ABI. For example the Linux NVDIMM subsystem has an address-range-scrub facility that is independent of the specific memory technology scrub mechanism. That one is based on ACPI NFIT, but I realize you also looked at enabling the ACPI RASF scrub interface. It would be useful if this patchset could plausibly enable one non-CXL scrubber along with the CXL one. > 5. Register CXL device patrol scrub and ECS with scrub control driver. > 6. Add documentation for common attributes in the scrub configure driver. Going forward, please include the Documentation in the patch that adds the new ABI, it improves the readability / story-telling of the patches. It also makes it easier to analyze which code is needed for which ABI, and whether a given ABI is justified. The regionY nomenclature in the sysfs ABI looks like a potential opportunity to align with the "memregion" id scheme. See all the callers of memregion_alloc() where those are tagging device-backed physical address ranges with a common id namespace. It would be great if the memory-scrub ABI reported failures in terms of region-ids that correlate with CXL, DAX, or NVDIMM regions. > 7. Add documentation for CXL memory device scrub control attributes. Do the CXL specifics need to be in the ABI? One thing I missed was how the series of log entries are conveyed. For CXL in contrast to what NVDIMM did for address range scrub is that CXL makes use of trace-events to emit log records. That allows the sysfs ABI to remain relatively simple, but the various trace-events can get into more protocol specific details. For example, see cxl_trigger_poison_list() and trace_cxl_poison() as a way to genericly trigger the listing of a flow of device-specific details. In other words put the DDR5 ECS specifics in the trace-event, not the sysfs ABI if possible. Lastly, dynamically defined sysfs groups are less palatable than statically defined. See cxl_region_target_visible() for a scheme for statically defining a runtime variable number of attributes. Specifically I would like to see a way to define this new subsystem without scrub_create_attrs() and all the runtime attribute definition. Overall, I like the general approach to define a common subsystem for this, and get off the treadmill of reinventing custom scrub interfaces per bus, but that also requires that it be generic enough to subsume a number of those per-bus-scrub-types.