From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) (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 91A0F8F5B for ; Fri, 19 Dec 2025 17:05:22 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=fail smtp.client-ip=192.198.163.18 ARC-Seal:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766163924; cv=fail; b=TroiizTzyp//J2dmihMmO4vKRSLQVBg7mGZb5L5sVXkURjCOYElWiKKVtJ02oVjoMvXsBUsZXXWrb/MERSHzViPhIdg77ryjW/wRpT2Mjg2+22affzhGZ8XR/gzbValr2K3XfNolcLiW9DZRMMla+3bp02DB/q6IU5gFiDyda/w= ARC-Message-Signature:i=2; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1766163924; c=relaxed/simple; bh=U7IDAN5rRAtP7YpcWn0HR2d9R4DE8cqmPyzm/kLUAL8=; h=Message-ID:Date:Subject:To:CC:References:From:In-Reply-To: Content-Type:MIME-Version; b=k5hTqYRXscr/rwK6zyMby9mX2svQ6KVO1jgkV2WcltKghC+R7uA7lkSMK8AjCMhxjxu6pJHy8FXvwB0Pcm4fK3xrj5N8cWqjFEUbM2ZD73AAi/7zs5qGMy/nCWOfxPGbonoJIk0+ROKpti2yZ0Mi5p+6a+pcuWT1p7aDEfs/AUA= ARC-Authentication-Results:i=2; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com; spf=pass smtp.mailfrom=intel.com; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b=O78EivmP; arc=fail smtp.client-ip=192.198.163.18 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=intel.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=intel.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=intel.com header.i=@intel.com header.b="O78EivmP" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1766163922; x=1797699922; h=message-id:date:subject:to:cc:references:from: in-reply-to:content-transfer-encoding:mime-version; bh=U7IDAN5rRAtP7YpcWn0HR2d9R4DE8cqmPyzm/kLUAL8=; b=O78EivmPwyOEFb0mMuf3fF88iApXS9YzGlxuKUPpM9wWUULVjQeLcJnr s+WdeZHyyfx20QC9ZlU/gOtOGS1sQpoRAzpMy9Kog2zbG2E0IyAD3EX9N 2BZwM8hZ5PWWKQA6pCKacdW7RV1/uX1sM17DJhsEiar1YLHO+gxjqf2qq ajzgIuRz0bJYDsgSUkeu5u+3vGstM8RjRozKjrrS1qacQYbz9xD5knaKJ WtqTx3MWNp1xCxvUAu8C/nt825iCdeRnFYviy7x1ee9KN7gob3Bq6r97l 4Yjb80NIQEvfQwX6whdPr3pGOKcH+oVUg96nUUYJDpA+MThtPv2hehnpc g==; X-CSE-ConnectionGUID: Fl0hVwcLTGmSzZy2w3//Xg== X-CSE-MsgGUID: iiM2F3IEQlC4AHD9M6esEA== X-IronPort-AV: E=McAfee;i="6800,10657,11647"; a="67322679" X-IronPort-AV: E=Sophos;i="6.21,161,1763452800"; d="scan'208";a="67322679" Received: from orviesa010.jf.intel.com ([10.64.159.150]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Dec 2025 09:05:21 -0800 X-CSE-ConnectionGUID: yVC9mIZ6RpWG0dFMzxWJzA== X-CSE-MsgGUID: Hmt+Wg/5Sym8ysoZoLrPxQ== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.21,161,1763452800"; d="scan'208";a="198165373" Received: from orsmsx901.amr.corp.intel.com ([10.22.229.23]) by orviesa010.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 19 Dec 2025 09:05:22 -0800 Received: from ORSMSX903.amr.corp.intel.com (10.22.229.25) by ORSMSX901.amr.corp.intel.com (10.22.229.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.29; Fri, 19 Dec 2025 09:05:21 -0800 Received: from ORSEDG903.ED.cps.intel.com (10.7.248.13) by ORSMSX903.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.29 via Frontend Transport; Fri, 19 Dec 2025 09:05:21 -0800 Received: from BL0PR03CU003.outbound.protection.outlook.com (52.101.53.25) by edgegateway.intel.com (134.134.137.113) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.29; Fri, 19 Dec 2025 09:05:17 -0800 ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=lbNbsvuUs/+roK1YUvKR6ktqS1yg+UrebtnZkzmtFAtUlP2cN7Rsm5pVswzgn8OdcQsp1653gU1mWc8q+lR6kCBgKNPzeklfAF94Be7cMoM5blFp+2WNc+0Rttm90ZXkLl6BmUhFJNh5yjZ7V8VNUeGwnfQ/A3xNS3ZcdgZP+xHz8dSoO+LGwdH1rEQSPcDi6+zYKYM2r2LZjeLhQj8qfysQxb9fd/p3bFq0V5UJ9HquhjbuWIPg2r6I92A2U2qOeltw7pk35hEhG2vBUKA7hu6/4fo28pRFneuZnC5YJeYS2bIbEFLetqG9Z0uUgLbr8EY56LrOchQVBvOcvnEBow== 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=S1pO//HV78EcDY4hfiBPg1DtzNn93dDeTcWM7xrU4I4=; b=BA4LyYvWWximBzAB2mj7kN4P9yu2JbuMW624ANHaLyOqyk4TngY8S0jQFfaSpplbA0HdFDWaJKjo8fVi+6PUi0OQ2eefXPRLRuanKkr6M+tODC5o/jbXOlzZ4ZtlhJKUvx6RcrlGYAkz0p7q3DSmZalErAgBcLpmb9YJmYFGR/Wv+ImRyVt372p+fHFEMsdfCaDTFmnBjL8MUkTPPJu12UvQhuKbljD9M8TQUUqukLzjXOtNO5H+KVssFygzbn4NFRXRugAtUR5DQRomphKSUrcB+5kyHNcO6sVOdcDnq7awPrqGtVJuEgrUcfQC7ggfyzvAuOO5tAQLBIQKAiIwCA== 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 SJ2PR11MB7573.namprd11.prod.outlook.com (2603:10b6:a03:4d2::10) by CY8PR11MB7106.namprd11.prod.outlook.com (2603:10b6:930:52::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.9434.6; Fri, 19 Dec 2025 17:05:12 +0000 Received: from SJ2PR11MB7573.namprd11.prod.outlook.com ([fe80::61a:aa57:1d81:a9cf]) by SJ2PR11MB7573.namprd11.prod.outlook.com ([fe80::61a:aa57:1d81:a9cf%3]) with mapi id 15.20.9434.009; Fri, 19 Dec 2025 17:05:12 +0000 Message-ID: Date: Fri, 19 Dec 2025 09:05:10 -0800 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/3] fs/resctrl: Add helpers to check io_alloc support and enabled state To: Aaron Tomlin CC: , , , , , , , , , References: <20251215230257.1798865-1-atomlin@atomlin.com> <20251215230257.1798865-2-atomlin@atomlin.com> From: Reinette Chatre Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-ClientProxiedBy: MW4P222CA0018.NAMP222.PROD.OUTLOOK.COM (2603:10b6:303:114::23) To SJ2PR11MB7573.namprd11.prod.outlook.com (2603:10b6:a03:4d2::10) Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: SJ2PR11MB7573:EE_|CY8PR11MB7106:EE_ X-MS-Office365-Filtering-Correlation-Id: 92266d8f-8701-4c2f-41af-08de3f20c5c3 X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0;ARA:13230040|376014|7416014|366016|1800799024; X-Microsoft-Antispam-Message-Info: =?utf-8?B?bk1tcUlKTkhHdzU1UjdGdG9lS1RpNlFlYllLelVKdzE1U3c4S3pDVmcvOXNh?= =?utf-8?B?eG1GNTcweTBJdlNUMTdMR1VpOWcvRVNDR2o0c3FnMDRlbldqTDBYNHVSNWVt?= =?utf-8?B?SEpMd0pRSnB6dlJkd29TenB0bUhDWnJwY0pCQ21KUGFhekZ3Vm0vRGdOdzFh?= =?utf-8?B?M2hjd2o2VlBRbS9QT3JlS3RCYmxHUnVJenFxS3N2QWZYSDVsem1CVVR0eWRi?= =?utf-8?B?T1JqR1diL2NGejBSZmNpRmR3U2FrQk9LMUdxd0w0akppTUpzblJWNGdkUzFQ?= =?utf-8?B?MEsveS90V044c053UHduRFRWU21JN2g3YzlJNkVDSVl4L3A2ck5WdlkxQjht?= =?utf-8?B?MUdkTlZNOEFuZHJvemxvT1VEV2c2VkQzM29iTHo0RjVBS2poeWEvbDBXMUE1?= =?utf-8?B?T2k1eUZReUtvZEM4WFQ1TlMyMnRzb3dlZTRGVFRjZnZQdUk0VGpZS2JBVmdu?= =?utf-8?B?SUY5MU14ZnZ3WElaZk1udW1ldzlseG9JeGNjOUdINXFFMS9oZ2JVRVJCb0hl?= =?utf-8?B?R0M4RmlIdnpWVEg3WWEvaUZIN3g2OG1IUVlVRlloL2kvTGV1WTNWaG1oY2FI?= =?utf-8?B?SkdKVTUzc0ZlYmlaZDh0RXptVm5TTXlGZmVkRXd4Qit6c21BcTBiU1VPbnhR?= =?utf-8?B?cTZuQ2lLUFBGbldtdktTaHU0TnhycUVXdHdhcERqK2FaVWdnREdRSXg2c3p2?= =?utf-8?B?V2l6QmhUK0d1UlppQ3NLaXdrcUJLWVkvWS90bk1IZ1BVVkF3MGVHMThWWnc3?= =?utf-8?B?QnVDNEJqbEk2YXJjZ25nZGJnc04vWEsyQlBQY2Nlc004ejFaZGwwRUEwS2lk?= =?utf-8?B?TElhRlVidm1ISXgrWGlpTmduYWZsbmx6ZU5tcHZmYjN0YU5oSG1yb0Z3c3Js?= =?utf-8?B?aDl4UE1sWmU0anNjUWtTTnZUMytoVmsyWTRpYWtWRWtTUlpDTEdVUnJXTm1R?= =?utf-8?B?QkV3UGRmUG1pM1ZJVmRkUmNlcEpuREYxbDRmSXI1RWV5dEtJWkJLcmY0TlBM?= =?utf-8?B?K3FLUEdlNWNvQ3lpVFUzTDVWMWpCb1oyVkFORzdHd1g2czZZdlZ5VDlsK0My?= =?utf-8?B?ZkZrUE9yc2JEc3BDMS9tZklqZkdmczlhNjQ2UEpHUUMzRU00M0NsRWY0a1Uy?= =?utf-8?B?NVZ1OVBkNnNPblFlazJXMVo0Z0VqVXBHbm4vY1o5NU5GWGtOREd2ODZ5djZw?= =?utf-8?B?R0lmeWZ1a0N1WGZTWEhudzBOMVhsMEpFMzJJckRiQkhULzJnMGdXaHUzaUFV?= =?utf-8?B?VFlGb0ZxeWU0enFUSkZXTGZtR1JVOVUvclNmSzVsOGJXNitSNWtmYmVNd3Vq?= =?utf-8?B?MVpnVGdWbDZjcVh5VUFmSkxwQnpOclBlODNlRWRFbjlzbnpwdk16YXhuMEVk?= =?utf-8?B?QlY1VVpvS0pDSGx5WmM2b2hHSTdaa1l4TEI4YXBITHhscFN3ZGJFTmtOSGll?= =?utf-8?B?L0c2TytmcnM4a0R3N0o5emwyYXZKc3hjbkd4TTFTbDdhQnpkVUU3SW91dHFv?= =?utf-8?B?eUYwT0lGa0tBQzd3ejB4M1ZBaS8xR2xVWXFINjN5SElSS0Z1WVZlSDdCQjg2?= =?utf-8?B?U2thSUxic1k4bXpIMTVVbzZSWHhGVkM2aklBWmJHL09KdWo4SDcxd0V2QU5r?= =?utf-8?B?MTdVNEkyNkwxUXNNTVV3QmMxZkpLUFBxS21wQUhsaFA0QUhHLzNRdHRQRHY0?= =?utf-8?B?WDdjWHYwS2xQaHh2cm1sanFtSXVCdkJZZVdzYzNHS3ZaVC9sYjlNd3hsbkwr?= =?utf-8?B?eG1UUHZyczcyaGRQK2xqTWxXSzJRbzJldmc3WHZGcmg0RE1jZEJES0xkNE11?= =?utf-8?B?bVZTZW8rWmw1MXZQVVNTMkhXbzlPWC93SkY0Y0dVakwwZFBKMW94QUMxZEpj?= =?utf-8?B?WUhpMU9ReGZ0UDBwT1U0ZlJKQllMdldSaERwZkZsclVlVU1TcW1zaW1yNk5l?= =?utf-8?Q?ofVip6epfgqBBvZa/06Ry2RjB4w5DE4b?= X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:SJ2PR11MB7573.namprd11.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230040)(376014)(7416014)(366016)(1800799024);DIR:OUT;SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?VFdPMW9HOXQyelQ0ano0NEMvemlGR05Fdmw0dGVoS3N4L2xocnFmOVladFJI?= =?utf-8?B?NlVHLzF4STZxdUZXS3VWUmsremIxZHQxekc2SlR1ZVR4WXQrMmpua1VHNFBp?= =?utf-8?B?QVM1UUptNWVDM1BJU3NaQnIwcEVzVU9YSERJc0tmamtLRWhleHZBcTNVUEtk?= =?utf-8?B?d1kzTDhJbTVNTUpIbVloWFhYczBZMGtRenFsL1phQlB5S2dEZUNwaGx6R0p1?= =?utf-8?B?cFlBa2I4TlJKWTFOK3lpVzFJYk12dUduM0JzQkkvVHFEQWdxbitPeEJsK0JT?= =?utf-8?B?SlcwaHcxeHBXM1pLUlZVVEtFdHlvbnB2dWU5ajVvZC9hbTVZRnh3WUdVY3ZV?= =?utf-8?B?VFU3S3BHaEpFV2RTdGZkOHBpbW01OTFJaUNxdGllcXdUclcyazd6TkxxT2pn?= =?utf-8?B?NUt5Sk1KQk9PNG9EWlA2ZWpzWlBIVExqT0FrVFpFcHZ1aWpIcTl1TGtmNVlX?= =?utf-8?B?RUk1SVIxc3pDUWJxa3gvcnR4czdjUjBrRURTN2t3S3RuS0pVV2VxRkl3RjFY?= =?utf-8?B?c3VTVUliUjBYWE9FQnkyTmJaZi9vWkJBelZwekR0MHU1bDRJYnpkSmtIUmZG?= =?utf-8?B?c0tCemx1am5NNWJMbURCZ3FlelpNWHkva2ZtL3FlVkZYSklhSVNYYkhFZmE3?= =?utf-8?B?ZnJzTUlXUDNLYlE5SWVpdzlESUoxQWtvREVwaWhaZkd1bU5GVXd6SEdvVVJj?= =?utf-8?B?VGNWMEVMN3ZZU3NPSXA2bnd3bDNEbUhMUWVQZU9CMThJejVVYjNMV2ltRXBv?= =?utf-8?B?Z2R2R2Z6ckpWa29BMnM0RXRwcEdGbEJlVlYycHo4cDZUSk5aZmZqSEdBRDJM?= =?utf-8?B?TjJNL2xONENReWtZM0dBUkJKWTNRZmt6bjJ0RStNY0U4UzdUbzVibFZseU9W?= =?utf-8?B?dW9wUGQ1T3FhL2hGYmZOdlNRNXlIa0pES0NkV1oxR3FzWmhETytFWmdiQmhl?= =?utf-8?B?amthVkpodEVheDNDNkpHR0FoMjNVTzhTTDdURU91bGxmVDlqSTZTVjNCcjli?= =?utf-8?B?ZzdMTlkvRFc0SkxOVWJaZEpCQ3l3TE9xSlUyZEZ2NC9ZR1R4eUU4Zy80QjZH?= =?utf-8?B?YzMrSVVXbG1nWkI1K1psdEJ0YUtXbVp5YitPYjZtSTJKZzVhT2ROR2JLNGFL?= =?utf-8?B?cHJXUis3Q2NMbHdwSW5wejhPd3k0US84YVFuUHYxLytsNGRSQTBKRUpjWDdE?= =?utf-8?B?UTdpV3RmOGFBQmVXQmhScnFrTnNkNVQ3RnpGRzlLblhvZHBTRkFiUW1FNG9C?= =?utf-8?B?RUhiejgxeGUwYWdjcFdUUmlRUXlBWTdRRUZoOXpLQ21JVnBieUFiV2NnUUhJ?= =?utf-8?B?cEpnekhSaWpMTE1WamdldjhzMFRSL3BDaVNlQzJQa3RBOWpadHFhc1lWRk5Z?= =?utf-8?B?MTAvMzkxUmkvMXJYZ0NBbEU0V2pDS2pHTlh5VHFaZGthTWNUaUc2K1dNNEpM?= =?utf-8?B?TVRWbTIrcWZUSzRDU3RwUFc0dDU5TUs4Sno0SzF1UTREbThBN3E1UDhUdVhQ?= =?utf-8?B?QTc5SFZOaTZMc1VqSTYxcmpxT0doZ3pIQWpQazhyV09mRVlyZ2hYVXNUajlD?= =?utf-8?B?NnlmOTJ6UHJCK0daWGtrTThqLzRTbXdnMWlkUGlFL0hYc2YvSEJPVXBoU0lr?= =?utf-8?B?dnhsR3ZXVXpIMnFUbDNyTWd2bUxlWEd3Zk1pL0JCVzNaemRsa2hQc1dDQmdn?= =?utf-8?B?bUVWMElUZTg3RFBZWnpSRHhiNWRYZWl5NmpmT09FQUZQbExhN0RVeCtLWTA2?= =?utf-8?B?NXltNUc2dUlYVkM4ZzdLS1A3djgzTXpOUExFdG5NSFVnNEc5VVl0YTR3U2c3?= =?utf-8?B?TE9oUCthaXpyMWVTYjMzdDdRcm5odXNrbWpDdHJFRXpaeENacTFSQitVRXhl?= =?utf-8?B?RldmeC8zTVRob1ZFK0xxQllXWjFBV0ZXVUttZzhuTTVJRTFlTVc4TE4wK3Ju?= =?utf-8?B?cVY1VEJGOTF4SHhrdFN2bUFrZE1kRXJuL3BNOW1wWERIcHRHOXFwNTdFTnRZ?= =?utf-8?B?TUlUS29Fc01zRnJST0dyaUNhMnhieGZTVnMzTXFDdXRGUFN6aGc3ajNGUGZH?= =?utf-8?B?cXBKS21uZDdxUk41ZktzbFQxdjB1TUJxZ3BtR2xOOEt3aGlNeGszajNZbitR?= =?utf-8?B?V0tTTk1YODM4RVNnYkw5VSs4Zk1WTS9BdzRmU1dIbmFra3lDV0MrNFkxenZC?= =?utf-8?B?K0dZUE42Q0k2SkJWTVhWSTJnVUpxMzBHMkRWaVNSQzhiNlFldnFVaWxzbmhr?= =?utf-8?B?cU02ckNQSEgzMmVMZEVYUHE0S1BaQm9yTXI0dzZRLzJkV1UxTFdMbWl3OE1S?= =?utf-8?B?OUhIOGFEc2NZdTV6cGdGekM4cnQwZnE4bDczdm1OcVA5andhbGt6cW1Wc2NP?= =?utf-8?Q?tbrz+y6Q6k9S4gPQ=3D?= X-MS-Exchange-CrossTenant-Network-Message-Id: 92266d8f-8701-4c2f-41af-08de3f20c5c3 X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Dec 2025 17:05:12.4211 (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: cIeRe0MvXK4ikP27gyBEka6621zzVQDAKMbwt7gp6ris6Dj1VAhy9NkvyaHhd7t/pHK72BZ6tXxa7kOGxM6ofTkShyun1On1qzaJOGiZODU= X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7106 X-OriginatorOrg: intel.com Hi Aaron, On 12/18/25 3:22 PM, Aaron Tomlin wrote: > On Tue, Dec 16, 2025 at 09:01:32PM -0800, Reinette Chatre wrote: >> How does this patch benefit the goal of this submission, which is to set >> identical CBM on all domains? >> >> If this change benefited this submission I should find these new helpers >> used in later patches but they are not. This seems to be an unrelated and >> unnecessary change mixed with this submission. It does not fix a bug nor >> does it support this submission and thus just adds noise when considering >> the new feature for inclusion. This patch can be dropped. > > Hi Reinette, > > Thank you for your feedback regarding the relevance of this patch to the > overall objective. > > To be clear, this change was included as part of the broader series. Whilst > I grant that this patch is not a strong requirement, I deem it a useful > prerequisite clean-up. In my view, streamlining the existing infrastructure > before layering on new features is a sound practice that prevents the > accumulation of technical debt. You are correct that new features should be layered on a clean foundation. One problem is that these two patches are independent. This change is misrepresented as a pre-requisite of the new feature. Presenting it in this way impacts how the feature itself is considered. > > Given that this refactoring provides a cleaner foundation, I would propose > that it remain in the series rather than being excluded. I believe the > long-term benefit to the maintainability of the resctrl code outweighs the > concern of it being "noise" in the context of this specific feature. I have a different view on how this patch impacts maintainability. There are several techniques to help developers. For example, while the new functions introduced in this patch intend to be helpful to developer by including the comment "This function must be called under the cpu hotplug lock and and rdtgroup mutex" the right way to communicate this is to use lockdep_assert_cpus_held() and lockdep_assert_held(&rdtgroup_mutex). Existence of these tools demonstrate that even while knowing what the right thing to do is, mistakes can still appear. Something else required by these new functions is that rdt_last_cmd_clear() needs to be called beforehand. This is not possible to automate like the examples above and thus relies on developer to "get right". As one that have seen many patches flow into resctrl I can say with confidence that this is one of the things where there are often mistakes. With that in mind, note how every hunk includes rdt_last_cmd_clear() followed by the rdt_last_cmd_printf() being moved. How the buffer is used cannot be more clear, right? This patch adds a layer of indirection that makes this relationship more difficult to see. It thus does not simplify how to reason about this code. Surely, rdt_last_cmd_clear() is not required to be a few lines away from a call that writes to the buffer but having these calls in the same function/scope makes it obvious where the buffer is cleared and where data is written to it. Also, as resctrl documentation states about the "last_cmd_status" file: "If the command failed, it will provide more information that can be conveyed in the error returns from file operations.". Now, while keeping this in mind, consider, for example how resctrl_io_alloc_write() appears to developer before and after this change. The current implementation is consistent: every time there is a failure it is accompanied by a write to last_cmd_status buffer to make sure the error details are conveyed to user space. After this change the function is inconsistent: some errors result in a print to last_cmd_status and some do not. It is not that the print to last_cmd_status is removed but it is behind another layer of indirection that makes resctrl_io_alloc_write() more difficult to read. A developer can no longer just look at resctrl_io_alloc_write() and learn how it interacts with user space. Same for the error codes. It is important to know and be consistent which error codes are returned to user space. Adding these behind another layer of indirection where that is all that function does seems unnecessary to me. In summary, no, I do not see how this change benefits maintainability. Reinette ps. I will be offline over the holidays and may only be able to continue this discussion in the next year.