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: <703e6dfd-68d6-6def-183d-fb99b39692b3@intel.com>
Date:   Tue, 18 Oct 2022 09:26:41 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.3.3
Subject: Re: [PATCH v5 0/2] x86/resctrl: Fix min_cbm_bits for AMD and code
 cleanup
To:     <tglx@linutronix.de>, <mingo@redhat.com>, <bp@alien8.de>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "x86@kernel.org" <x86@kernel.org>
CC:     <fenghua.yu@intel.com>, <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>, Jonathan Corbet <corbet@lwn.net>,
        Babu Moger <babu.moger@amd.com>
References: <166430959655.372014.14294247239089851375.stgit@bmoger-ubuntu>
Content-Language: en-US
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <166430959655.372014.14294247239089851375.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?emlIKyt2MTVkQ1RuMU9NS08wREVTZFYrYko5ajVFR1VBd053Tzl4WDRySE5m?=
 =?utf-8?B?b282Znp5THZGR2dHU1A5c2FTRDZ1VXgrQlZQcmdvblVoYUN0V3JOaE1zL0Fp?=
 =?utf-8?B?N1o3UDZPQVAvanpvU3lOdUFvdWRKdkd6ZFVkTHRvdUlDRk04bzEvZ2xyVUZP?=
 =?utf-8?B?aGtJK2tFa29zTnNTQzVXM3JVc1dYME1SbXo3VFVKWDY4MW1scDE1c25zM1Nz?=
 =?utf-8?B?KzFpelVlWW80TGFveHZ4clhRcnJkQm9IVUdIcnRJRHVlSGFmYnBMWHltS3ZI?=
 =?utf-8?B?eFJwNVNLbFVGU0tPVytNWC9WVDRyZktZdFNZcmp4eWZaK0RrcCsyMWp3cXkr?=
 =?utf-8?B?a3JLNXhPTEVnOGdxVWM2V21pSlR1QzNkZjFheUJaZzkrY1ZNdkRHajF5ZDZS?=
 =?utf-8?B?L2JlN3IxUGloVCticks4b3UycjROZmlsUldjVEtSangyK2ZpQWhNQ3JvVmZ3?=
 =?utf-8?B?SysxbFJaZW9IYnlXOWsyUWhqdndiMjhQdGhiQnJqcWZuU0Q0YVM0cW54aE4x?=
 =?utf-8?B?L1ozdkl4ZjU4ODRkWmZqQ21zSmlURkhwZEk1S01ubllxR1ZuNVErM2dyVTIy?=
 =?utf-8?B?QnhJRy8vcnIwbmFBSUlmUWFoQnRqaXkxdzAySFpxaWZMK0NSOHhmSUlaU2cv?=
 =?utf-8?B?Lyt3ZysydjV1bnltV251TnRtVmc3dEZUR0tQVGJ6djNGN3JwRk9QbXNxbElG?=
 =?utf-8?B?eWYxc3V5YVBoM1JmT2czUHRoVm1DS1p5V2pUYzVHcHg5OGZxdTRXYkNWRE5Y?=
 =?utf-8?B?VURvQ3E4SHozRTlzNEhyc2dwc3U0K1M4UmVGMUpWOTZEa1UyNUJQT3dIOTJ3?=
 =?utf-8?B?VEd6dTNiUFBKWWFQTEdOcmcxdEl5UmhDRlRnMWorNGVwR0x4TFhwK2FqNk5C?=
 =?utf-8?B?dGZJbFREenhlbWZuWnA2TlU3ZWdkTkhGQmo4eGZuSXhOQ2hnLzBJOGZNY3VV?=
 =?utf-8?B?VkQ1N25RUXBLbVN6U1pkOUhJdkNqWWxpUGNMRzg3VjJ1VkN2NFJOdTBoZEJQ?=
 =?utf-8?B?OE9EVEJlYXR3UkRLeVZ6MjdYWEtXazcyaS9iS3QvVnBRejdjQkx1NmdiZkdr?=
 =?utf-8?B?NGhFUHkxVnZ6TllkNHVFdVFJdDEwbk5HSUI2dXBadWVubnN5M0NJMkpVaUJM?=
 =?utf-8?B?L2xmd1lDY05XRG9yaUYzMU4veUpjbmVjTHJVK1VsakFSbmhuL0FvbU1mekFj?=
 =?utf-8?B?KzR0TXRHUWlPWG96T1BidXBxU0dFQmRCU0p6S1NIaVlSUkJCM2c1QVdEQjda?=
 =?utf-8?B?VTNJcjBjdXh4YU1LeENlcDVhWGJLSVhDckdONkNBK1Z3Y21zRE9ZNnhPRHlr?=
 =?utf-8?B?eEMrWVRuZEEzUGlDVnQzMWFZZ1RPYWlJMGNWR0hKMkRFdXN5WGZIR2JRZXJi?=
 =?utf-8?B?S0E1VHEwUGtlUzRjTmVrQXE3U3QrNVZXN2NrS1F6NUt0Y2dQbW1iWklQWVpz?=
 =?utf-8?B?bGJoZHBGQlhEd3hyTkZBNHFNM0FGRmRlUkhlcGNhUHl6VWQ0bkU0UDRLanZW?=
 =?utf-8?B?UUh4bGFNMnBtU1NVb1d0cGY4RnZISkpCNGFkSmhpalQyQm9vR1hzdU45Z3lm?=
 =?utf-8?B?WXlFMTFLNWVHUUxJamhnem9zQ241SktiOFEvS2gzeVNIQkZjendwcXNHU0Zq?=
 =?utf-8?B?OFJtYmdXVE9iSEtKNC95bHdSeFlQNXZrYjZ5eFZ5QzhQd2JuU2hndWlCQy9i?=
 =?utf-8?B?VVFxZkphVEhyMGNHWmhYb3pvdUpTTWpoYWYrTUs3blE2RUFFdlJ5ZWZHNkNJ?=
 =?utf-8?B?OG9JdkpIb1pUL3FkelllNmZNWURSeFd1TDZyYmM5Q0xXaHp4akZodituakxG?=
 =?utf-8?B?T0taNDdjRW9BSDJnRHhMRUx5QnhtTFdnYkZoc21DTDZock1aRDhid2xxNnBT?=
 =?utf-8?B?bWFXRXNZTHdkejI3YTB0QWJ2bEgrcHA1d283Z2swUWRzVkl6K0s4cDd5V1JP?=
 =?utf-8?B?bjFtdnZlU1RFa0VzbFpOSzZFaEE4QytZQ3Iycm5Bcy9jSmgrVy9FSm1ybE9T?=
 =?utf-8?B?RWtpQ2pST0gyRGFUS3pTb0kwd0M2R2pnWm9uY055dHlVMGg5cS96OHFTZzls?=
 =?utf-8?B?Tk8wbHk3MUJ1amV5Qmp0UXg0Z0lMN2RvZGZvQUdMRzJieDMxQXI2dGgwSnVS?=
 =?utf-8?B?MDNmcVh4L0hGdW9jTlU5c0JPRVp5c2xrYmg5cEsyZ25hOTFTaHBTNlVHY2Vv?=
 =?utf-8?B?SXc9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: cadfdf73-efdf-4745-a4fd-08dab1258ce1
X-MS-Exchange-CrossTenant-AuthSource: CY4PR11MB1862.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Oct 2022 16:26:46.5131
 (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: 6dg7qJNjvt3zCL00TDZuQlW8KYrOlcu8AzjtpYwgWiKQydoal3dPnSkar8F06Jz9XS8gPihi2HCAw7gKx/OTjfvPBcgRWofR1lzSbJbwYYQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PH0PR11MB5191
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-4.7 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_NONE,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 X86 Maintainers,

Could you please consider this fix and cleanup for inclusion?

Thank you very much

Reinette

On 9/27/2022 1:16 PM, Babu Moger wrote:
> This series adds a bug fix for AMD and a minor code cleanup.
> 
> ---
> v5:
>   This is bug. So, submitting these patches as a separate series from the old v4 series.
>   This was a comment from James Morse. 
> 
> v4
> https://lore.kernel.org/lkml/166257348081.1043018.11227924488792315932.stgit@bmoger-ubuntu/
> 
> Babu Moger (2):
>       x86/resctrl: Fix min_cbm_bits for AMD
>       x86/resctrl: Remove arch_has_empty_bitmaps
> 
> 
>  arch/x86/kernel/cpu/resctrl/core.c        | 10 ++--------
>  arch/x86/kernel/cpu/resctrl/ctrlmondata.c |  3 +--
>  include/linux/resctrl.h                   |  6 +++---
>  3 files changed, 6 insertions(+), 13 deletions(-)
> 
> --
> 
>