Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <4d79d134-c6e9-3e98-e670-5870eff44153@intel.com>
Date:   Fri, 16 Sep 2022 08:53:36 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.2.2
Subject: Re: [PATCH v4 02/13] x86/resctrl: Remove arch_has_empty_bitmaps
Content-Language: en-US
To:     Babu Moger <babu.moger@amd.com>, <corbet@lwn.net>,
        <tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>
CC:     <fenghua.yu@intel.com>, <dave.hansen@linux.intel.com>,
        <x86@kernel.org>, <hpa@zytor.com>, <paulmck@kernel.org>,
        <akpm@linux-foundation.org>, <quic_neeraju@quicinc.com>,
        <rdunlap@infradead.org>, <damien.lemoal@opensource.wdc.com>,
        <songmuchun@bytedance.com>, <peterz@infradead.org>,
        <jpoimboe@kernel.org>, <pbonzini@redhat.com>,
        <chang.seok.bae@intel.com>, <pawan.kumar.gupta@linux.intel.com>,
        <jmattson@google.com>, <daniel.sneddon@linux.intel.com>,
        <sandipan.das@amd.com>, <tony.luck@intel.com>,
        <james.morse@arm.com>, <linux-doc@vger.kernel.org>,
        <linux-kernel@vger.kernel.org>, <bagasdotme@gmail.com>,
        <eranian@google.com>
References: <166257348081.1043018.11227924488792315932.stgit@bmoger-ubuntu>
 <166257360370.1043018.10126722515228365580.stgit@bmoger-ubuntu>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <166257360370.1043018.10126722515228365580.stgit@bmoger-ubuntu>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?ODM3a3U0aGtXRlB4U21VbzNXMXlCRTdpTWp3R05KYkpQencrZnFMSmZaQitF?=
 =?utf-8?B?dEw2eXI5UWxjWUl4NGNpMEd4ZTNRaG9UWS83V1M4eEhqRXlaUlNzOEQxemxV?=
 =?utf-8?B?ZktHL25JOWQ5azVYREdML0JpelpLTlM3V0VXeXhESjd2M3pSVEVCYnNGQXlT?=
 =?utf-8?B?R3lyTVpQVnBtaDZWcGQwRHQ3U1F5TXZMYzN0T0duSVZSekh4cUNDNEdyZG9Y?=
 =?utf-8?B?bSs5NitDZ1ZQNDNOOVFDaEtFY1kybVJMQ1I2QXU2a1NiVVUyMkw5SDd1YVlY?=
 =?utf-8?B?dHBPOUlKVzFpT1RUd0hiRXl3U3hmaUk0TkRQZFYyTDVNcjVkQlo3ZHFRUzBG?=
 =?utf-8?B?d1hxaGYxTEJXcmRnVlNyRmtseTRtS0RNSW42U1VtK1pHWjJod3djNUF1WVZy?=
 =?utf-8?B?N0wzalZZOXZ6ZHhCS2xlRUUzTUtDazFqd0E3Y0V3OC9QYmJuazVRZldPZ0tO?=
 =?utf-8?B?YytIWlI2aE51NUFWU3JWZ2dsbUNIUmEwak16aVg4NThJWWpNTEFsQm9zOFdG?=
 =?utf-8?B?L0xzZFhZYm93UDlac0dSYTBXekR6aEhtRXg0a0phdFVwV1Foenh0R28yaXpH?=
 =?utf-8?B?eDY1aUM4TTBoMlBaajNmb3JLeHVZRGhFM25pQ096cHd1c3NicjZWT0dVT3Bp?=
 =?utf-8?B?a0V3b0lqK2trUEx5VmVWeHNZMXM0RTI1QzVvOXJiQlBTZ1ZLNmJKaStYaHho?=
 =?utf-8?B?MEFySEVEbmE3OHhGRmQ1bTJLME1jRnRyOVV2aFpyTFJPZnNaREx6NUVvcVht?=
 =?utf-8?B?TzRaU2ptcXFIWGFmY1o0UEF0L3ZENWllNkVMVnVIUUJ4QStDcHVzU20xcU9x?=
 =?utf-8?B?RWsvcC9NNm1hcHZobVVDQzBMSmlITHFxMXlIb2wvUDBvcTdjRUpVNEc2UmUw?=
 =?utf-8?B?TEl5VTF1czArWjQwd3RDejd1SDFqWWQrNVRYZ3Z3bWR3Y0RkVm9MSFoyODlw?=
 =?utf-8?B?YmlETmwzUzVnVWUrMnlDWS9zVGdJRjNkZWVIeFgvN3h3R2tTd2prV3BxMGNi?=
 =?utf-8?B?bXRlcFN5TVc1UFMweUdFYldSNUhOUHdjTittc2tMUjZRWGpIL0RkNVlqMXR0?=
 =?utf-8?B?S2hkdGI0N1F0RzQzdE1rYmd3YTE1Z0Vka0V5NGRNM2FFcUdFNGNvNThVTmk4?=
 =?utf-8?B?NHRyQ3lnQ1hWTFNXU3pFRm0vSnRlMWt1T0NaNFNPZW9yZ2JjSHdPWFhEVUVV?=
 =?utf-8?B?ditXOEJRRDREUElQWUFqbkVMTGN3V1RRZ2V5dTFsbDBxU3R0SDVJOVJUckxz?=
 =?utf-8?B?TXNicUNjcjUyYncvaUpNaERwUGZYWGZiaU5QQnYxWnFnY09NK3pDS3lxd3NF?=
 =?utf-8?B?aTQ1QU5zRnZHY0pjSk90RWNUa3RpdkVpc1VyRTNoSFJvYnk4NUpxcHpFYzN5?=
 =?utf-8?B?MTVWRTV3MHRNM0ZhdHE2Zm5OZ2p6MlFkclBDQk1aZW9uK1lHeEdvbnArdTNI?=
 =?utf-8?B?dHNMMFFTc0NvaEVyRi9nQkVTbitPMkoyc3dNRmpVVldOQXE1Rnp5TDBvRE1L?=
 =?utf-8?B?aFNnTSt6bEsvRmhlZ3RCdUlFR3FpaVFqYjZGaEpteTJ2MzZmcXRXNHJsK0tB?=
 =?utf-8?B?SnJVMVhEY3JoNStBL0tpRFE3am5wNjJwODlQMlQ3YU0wMkt3d1RMTG5SYUgy?=
 =?utf-8?B?bFZMeHlKY2JiY0pCL3F1UzlEMUdhK2dvWVM4N1JBS3FSOWg5dVlndHlLZzVm?=
 =?utf-8?B?VjY5dzFCSVkvbGVYeDNsSFdaLzhWZHVtL201cXl5TDRIamxzZFMxSXR1WThz?=
 =?utf-8?B?eG96WnNRM09TQWE1RTIwVUoyUnFGY0NVSHV3ZVBmY3AyL1pXVTJYUVlOTUR4?=
 =?utf-8?B?VDdTNUhqaWYweGNhOUtTaUw5NzdOL0RQenQwNHByV2U3a3ZxQkV1aTNxL09w?=
 =?utf-8?B?d2hMd2EvaGx4Zm5JUTM2aXVKeFFFVTJsc0pGYVRoaEZ3UUF6bkh6Ukc5RXNR?=
 =?utf-8?B?dXd1Q2R6cDhoNWxNTkN1bm91ZnRVSzVpbll2WGVCdU8yNFEvYWRTQ1h1dGQw?=
 =?utf-8?B?SUR5TXJNQk5PVFFwbnJVbWdzd0hjZ0tOOFk4bzQrdzJVU1ZnMnI3cE01TFI1?=
 =?utf-8?B?ZG9FNExBekkydzEzZnBQZkZackFzZVRUQmFhNFZaekdLT0l2MWVvNkV2QTFP?=
 =?utf-8?B?TXVrTzNoZkJHSStvZVNkZ0lTUkNWTHg3dmpZMjZWbCszTSt1QnV4TkcrSmZl?=
 =?utf-8?B?ZEE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 23b88706-6b7e-405b-9c37-08da97fb9fd3
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Sep 2022 15:53:40.2720
 (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: LXq+SPwo/xLWRlX38L5JwTUHGfY9ohZSx84W5XyEsC7xHNq7/phzaL8uHx4Gyv8yIgE29eHG1XxCc1k6aDZNWVgHcrTwVTADqvvg/MNUoRM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SA0PR11MB4656
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-6.2 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,
        RCVD_IN_DNSWL_MED,SPF_HELO_PASS,SPF_NONE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Babu,

On 9/7/2022 11:00 AM, Babu Moger wrote:
> The field arch_has_empty_bitmaps is not required anymore. The field
> min_cbm_bits is enough to validate the CBM (capacity bit mask) if the
> architecture can support the zero CBM or not.
> 
> Suggested-by: Reinette Chatre <reinette.chatre@intel.com>
> Signed-off-by: Babu Moger <babu.moger@amd.com>
> ---

Thank you.

Reviewed-by: Reinette Chatre <reinette.chatre@intel.com>

Reinette