Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v2 03/23] x86/resctrl: Kill off alloc_enabled
To:     James Morse <james.morse@arm.com>, <x86@kernel.org>,
        <linux-kernel@vger.kernel.org>
CC:     Fenghua Yu <fenghua.yu@intel.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>, Borislav Petkov <bp@alien8.de>,
        H Peter Anvin <hpa@zytor.com>,
        Babu Moger <Babu.Moger@amd.com>,
        <shameerali.kolothum.thodi@huawei.com>,
        Jamie Iles <jamie@nuviainc.com>,
        "D Scott Phillips OS" <scott@os.amperecomputing.com>,
        <lcherian@marvell.com>, <bobo.shaobowang@huawei.com>,
        <tan.shaopeng@fujitsu.com>
References: <20211001160302.31189-1-james.morse@arm.com>
 <20211001160302.31189-4-james.morse@arm.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
Message-ID: <e7b0c232-6643-cd3a-1fae-80d368090e39@intel.com>
Date:   Fri, 15 Oct 2021 15:19:26 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:78.0) Gecko/20100101
 Firefox/78.0 Thunderbird/78.14.0
In-Reply-To: <20211001160302.31189-4-james.morse@arm.com>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?b2lmRkY5bzAwMDlVTVBFQzNzb0V4cE96SlpzT2dCbkJIYmZBcE5Veks3R1Fh?=
 =?utf-8?B?OEF6bmhNaFkxNmtYT2JUYUVjTzFKMVp2cW9xUzFGaEsrS0dhM1NRN21hT1Vj?=
 =?utf-8?B?WWVWNlRvYWwrOWw0bHc5eWxjcFEvOFBLWmVFeVVBWWNINFY4MTRzMEdYUTA5?=
 =?utf-8?B?Rmd2VDBjc3ZhUkRzWXhOeUk2ZkE0dG4yR1VjTmRmZ29kTTd2M3NkeTV3S3NB?=
 =?utf-8?B?cTg0UmZxL09ndzFaQ2s0VDFlVjlsQktkenB5eko2M2xDOWd5U2dqZUlyNG1x?=
 =?utf-8?B?OE12bkR4emdMTy91SGVVRVhBMTN3YVZUVHR5aGp3alViN2p2M084MEc4WUxW?=
 =?utf-8?B?Y3pmaTF3bTFKR2xPTWU3aDNEU0hSZmpnbmtiYlU4L2NvTEZ4RXcwTVFkWVdF?=
 =?utf-8?B?YVFXVXNRQWs2TmZKaWZGeHc4dE83ZnhVVTR3ejU4ZUlEZ2w5OUl5cmNXVCtX?=
 =?utf-8?B?WE5rVVk3VEkvS1Q1UGJld2lYUTRKbGpBVzFZMllPcWhzYXBvR3pZMG5Nakd1?=
 =?utf-8?B?KzEzU3NkQWVuSFhTM29XZmFYYUQxUWtNZmZHY1lWVmhTZTlCQTdlTUFJRzIw?=
 =?utf-8?B?Zy8wNWtXbWREdC80WnBvZ3lXZTdJeWhydGU2R2UvTHVNNlczR2gvUWowOSs4?=
 =?utf-8?B?OEk0WkpyRWpoUUVkbWdVNmdzRDI2cGhHMDh5eDVGZkFzcTNjcnZxWmMvYUgw?=
 =?utf-8?B?QWJnWWJwNkJWYXoydUU3L0YzMXFndnhxV2VGVm44ejJ3Q001OGJaeVpsRXdT?=
 =?utf-8?B?OXhicFM3UEhHc0RvL1lnYkMrL3BaTGxiMHNHb2s2SkhUelk1cGZNRlFRSzBW?=
 =?utf-8?B?VkpRY2NOUjFaUVFLWlN3MnlXdG9UYm9qczNIbjl0dE1jYW54T3dsYTVsUW1E?=
 =?utf-8?B?YXlWR2pIK3RhUWpFbjdsM2lIR3VMOXhBbHRyVVhCdWIxdHNzZmU3VU5tNElj?=
 =?utf-8?B?QkxkZjl3K3ZLZUJ4TkVDYVZERVhUY3ZRQUJFaFB3dE5WTURMaWd6K2Q4d3Bv?=
 =?utf-8?B?TEliRFliN0sxVC9od1czSlJzM0FEdytOdjIvU0hHTDVaTk9aYk9aN0F5dFVI?=
 =?utf-8?B?b25MOEpOTndSbXZSYmYvSlN1YnN4Sk8vSU8xRnZ0REdySTZqRnp5VmkxazNN?=
 =?utf-8?B?SWR6VVJIcnFtamU0TjQycmJqVGxtWVdKbkhsYmJmRjhRamkxM2VBRnBoRzFP?=
 =?utf-8?B?cnNrRlJLS0RzTUl4amJkQkFxMEJLRjdRK1c5NmF3RCs1ZytXVzNsQlVXSFlh?=
 =?utf-8?B?RmI1S0FvRGZMdWFYRFNhZlRaaFhxc1lIa3k2Y1NCQ01JTEc2Vmp2NUJvbjlw?=
 =?utf-8?B?Vmh4RXNvM2llWkJWZmJ3K1ozaHA0OWxWTnlSNy9WdGhhS1pRR0FGcjIraU5y?=
 =?utf-8?B?WHB5aG1ucEVkcmgvc3NhMkxyY1NtSDRRTGZqbGkyUzE1MUMyWXdNL2lmalps?=
 =?utf-8?B?dnN4OE5ZaElUU0xBd1BvSmVObDlNYkV3K09QTFNJY2xtejBrYWovc1VIVVJj?=
 =?utf-8?B?VUtPc3dsQ3MrUWl5bjZ5UzdLTURNZUMwdXFQMHp4RlZTUXJmb2t6WmluSnJl?=
 =?utf-8?B?WW9JczZBbUFFMGhHU3NkdlNFZ1BWUllBeFBwcldHeU5za0dJUVh5b0svcWFX?=
 =?utf-8?B?aXRtTGQ0OGQ1bkhkczNFLy9GOG85U3ZYdDkvTURIRDdBazVqNnFpaWw5cnBI?=
 =?utf-8?B?eDNsUGJUWjVHRjArN1dRTVdDQitxcmtYT3dKTW0zNVorN2ZnYk9JN1RmeWJn?=
 =?utf-8?Q?nEJbYVNb3bJgs/nyH+VHElxcMPJzbueDliECG5G?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 2af5d451-6826-4c4a-cfa3-08d99029dc66
X-MS-Exchange-CrossTenant-AuthSource: BN0PR11MB5744.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 15 Oct 2021 22:19:31.7988
 (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: RBczra6iOoqTuDtEfp1D/hrIEmUKzLEVS3w6u1cLbT4/JSQ9EL+MYgmnLxMNahfsoZivSACo09jpBMjgekz0lMuMkpycp0SX0q3p1MVjFg8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BN6PR1101MB2178
X-OriginatorOrg: intel.com
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi James,

On 10/1/2021 9:02 AM, James Morse wrote:
> rdt_resources_all[] used to have extra entries for L2CODE/L2DATA.
> These were hidden from resctrl by the alloc_enabled value.
> 
> Now that the L2/L2CODE/L2DATA resources have been merged together,
> alloc_enabled doesn't mean anything, it always has the same value as
> alloc_capable which indicates CAT is supported by this cache.

This impacts all resources, not just cache, so perhaps it should read 
"... which indicates allocation is supported by this resource." or 
equivalent.

Reinette