linux-kernel - RE: [PATCH v13 11/27] x86/resctrl: Implement resctrl_arch_config

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <SJ1PR11MB60839776B024017D65EF4E65FC64A@SJ1PR11MB6083.namprd11.prod.outlook.com>
Date: Tue, 27 May 2025 21:41:48 +0000
From: "Luck, Tony" <tony.luck@...el.com>
To: Peter Newman <peternewman@...gle.com>
CC: "Chatre, Reinette" <reinette.chatre@...el.com>, Babu Moger
	<babu.moger@....com>, "corbet@....net" <corbet@....net>, "tglx@...utronix.de"
	<tglx@...utronix.de>, "mingo@...hat.com" <mingo@...hat.com>, "bp@...en8.de"
	<bp@...en8.de>, "dave.hansen@...ux.intel.com" <dave.hansen@...ux.intel.com>,
	"james.morse@....com" <james.morse@....com>, "dave.martin@....com"
	<dave.martin@....com>, "fenghuay@...dia.com" <fenghuay@...dia.com>,
	"x86@...nel.org" <x86@...nel.org>, "hpa@...or.com" <hpa@...or.com>,
	"paulmck@...nel.org" <paulmck@...nel.org>, "akpm@...ux-foundation.org"
	<akpm@...ux-foundation.org>, "thuth@...hat.com" <thuth@...hat.com>,
	"rostedt@...dmis.org" <rostedt@...dmis.org>, "ardb@...nel.org"
	<ardb@...nel.org>, "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>,
	"daniel.sneddon@...ux.intel.com" <daniel.sneddon@...ux.intel.com>,
	"jpoimboe@...nel.org" <jpoimboe@...nel.org>, "alexandre.chartre@...cle.com"
	<alexandre.chartre@...cle.com>, "pawan.kumar.gupta@...ux.intel.com"
	<pawan.kumar.gupta@...ux.intel.com>, "thomas.lendacky@....com"
	<thomas.lendacky@....com>, "perry.yuan@....com" <perry.yuan@....com>,
	"seanjc@...gle.com" <seanjc@...gle.com>, "Huang, Kai" <kai.huang@...el.com>,
	"Li, Xiaoyao" <xiaoyao.li@...el.com>, "kan.liang@...ux.intel.com"
	<kan.liang@...ux.intel.com>, "Li, Xin3" <xin3.li@...el.com>,
	"ebiggers@...gle.com" <ebiggers@...gle.com>, "xin@...or.com" <xin@...or.com>,
	"Mehta, Sohil" <sohil.mehta@...el.com>, "andrew.cooper3@...rix.com"
	<andrew.cooper3@...rix.com>, "mario.limonciello@....com"
	<mario.limonciello@....com>, "linux-doc@...r.kernel.org"
	<linux-doc@...r.kernel.org>, "linux-kernel@...r.kernel.org"
	<linux-kernel@...r.kernel.org>, "Wieczor-Retman, Maciej"
	<maciej.wieczor-retman@...el.com>, "Eranian, Stephane" <eranian@...gle.com>,
	"Xiaojian.Du@....com" <Xiaojian.Du@....com>, "gautham.shenoy@....com"
	<gautham.shenoy@....com>
Subject: RE: [PATCH v13 11/27] x86/resctrl: Implement
 resctrl_arch_config_cntr() to assign a counter with ABMC


> Thanks for applying my suggestion[1] about the array entry sizes, but
> you needed one more dereference:

> -       size_t tsize = sizeof(hw_dom->arch_mbm_states[0]);
> +       size_t tsize = sizeof(*hw_dom->arch_mbm_states[0]);

> -       size_t tsize = sizeof(d->mbm_states[0]);
> +       size_t tsize = sizeof(*d->mbm_states[0]);

Indeed yes. Thanks.

-Tony