Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:4 as permitted sender) client-ip=2620:137:e000::3:4;
Date:   Mon, 9 Oct 2023 08:44:40 +0200
From:   Maciej =?utf-8?Q?Wiecz=C3=B3r-Retman?= 
        <maciej.wieczor-retman@intel.com>
To:     Reinette Chatre <reinette.chatre@intel.com>
CC:     <fenghua.yu@intel.com>, <tglx@linutronix.de>, <mingo@redhat.com>,
        <bp@alien8.de>, <dave.hansen@linux.intel.com>, <corbet@lwn.net>,
        <x86@kernel.org>, <hpa@zytor.com>, <linux-kernel@vger.kernel.org>,
        <linux-doc@vger.kernel.org>, <ilpo.jarvinen@linux.intel.com>,
        <bmoger@amd.com>
Subject: Re: [PATCH v4 0/4] x86/resctrl: Non-contiguous bitmasks in Intel CAT
Message-ID: <pve5jzaocwr6okpekrmtqut7gza2yeij4lbkweelk4ei7yxwvx@qrsqfhaebuza>
References: <cover.1696493034.git.maciej.wieczor-retman@intel.com>
 <533ccc1b-7566-444e-8549-184fd272f666@intel.com>
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <533ccc1b-7566-444e-8549-184fd272f666@intel.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?MXM3SGM4WENlUGkyWVdrd3NyM0ZYeVNnd3E5cWxGb2RHdS9yQW81VnlpTFJE?=
 =?utf-8?B?ZUFVSGd6MnRFaHFPbDZBeXVVUGNhNjhacU9CMDkyZkhKU1hCcEs3ckNlNlZL?=
 =?utf-8?B?bis0WWJRVVFKMHc3LzNFNWpEUHZHdGFSTDRqaCswaVFwdm1mSTZ6ZExxZzJV?=
 =?utf-8?B?UXBicUgvRlFjRlQwWlNHbm9xL01TZE9vQUZKeEFTdnZBT3h6c2FIZzRIMTcr?=
 =?utf-8?B?ZThreEd5UWx4alRFSDNkcXZaWDRQOVJzbVE5RDJqUDIzd1NkazdHK3JCOHVS?=
 =?utf-8?B?bHZPY1VkZFBTRWcwS0tKVHQ0UXJJcHQ4YUV3UkFELzJOb0dnYUdmZElTbzNL?=
 =?utf-8?B?K2NvYUxFZ1pzQW5ZTUt4L0hESWs1bjRQaXFJbmE5RlVpZFpNdFdqbDlXaE5F?=
 =?utf-8?B?aXYxR0ZJRGFiQ0d3amdWVUdXSG96cXZBL2Q0c2V2cExHZEtrM2h1T1BaYzJt?=
 =?utf-8?B?d2pTK09TQVg5aVJYK2hIL2pYK0JnMTVNendhS2VnY1NwbHUrQTVpRGFrekd5?=
 =?utf-8?B?dVFDcVBpV0s4OHZVRWJiYjg3SUxFY1hCOUpuNDRVNzdXYzZ6T25qSkQ2RTVa?=
 =?utf-8?B?RzNxK1JRTEZJckVyUmFDSjNmK0VjSm1EMFdRc1lYMVU1S05IL21ZWE9tTTNn?=
 =?utf-8?B?QWNteFV5bFBKdThSdkJ5UHQ5V3Y0Qjd5cGx4ZlB2SFRpeFAzZW1JcVkvTXMz?=
 =?utf-8?B?c3drSjljclRqQVRTRVZabk5NbW1CYkk2UDZqYVhIRHc0T0JRU3JIamVjaDA5?=
 =?utf-8?B?dDNha2hsMmRrWklBdGZrMDFORUV2TFdTNkFxa3UwaDgrWkZhamxlNVV5NGFH?=
 =?utf-8?B?cWZDVms2dTJiUTVSaXM2TFFEQUVxSGp4ZXkzbFFwL3hnTEh4MnhnYkNyY2U0?=
 =?utf-8?B?akljdVVoWWZRUnFrTnQyZEhRWnRGUzY2WldDUW1CMjVDVDJjY1JaNFN0bUtL?=
 =?utf-8?B?eEpHbVRHQi9JbDBXdkltVjJNNVFqSFdGVVMxZHJYejVYZWpZY0hGUGVRcEVw?=
 =?utf-8?B?T0dxdnNvbndUaThOTFRabjMvaWZ2cyt6WExzbkpEK3N6U092MkdwdWZDK3Ni?=
 =?utf-8?B?NG1IeGVlRkljeUpSdzBadUYvVHJQODVkR2ZnK2hONVFLZzFoa3JQa1dNTUVL?=
 =?utf-8?B?dVFVWENJclk4V3g1YVg5NlpoVklJUk9pN3hLa29pRVdOM0o1dkMwM0RwMjRF?=
 =?utf-8?B?MmRQY2libWJEZEhTMEc1SjRSQ1doM1BicGhEbkpJMGlWOVdRL0RVR2NOVFRB?=
 =?utf-8?B?Q2pFa0J3enl0Q0M1anY2Y1lZOHFKMkJXeHhHejE2UDZNYWw1aFpUekNsN0xV?=
 =?utf-8?B?K2lSRmxaZEs3MXZtNHQ0Zjl4bHNvZS9RR1BqV3hjWVZ2KzAza0orRUM2NUZZ?=
 =?utf-8?B?WHdOaGRvbmdBVUNOWmQwWVY0bFZmcDhnVUNXN1NYSTM2SUp6eHNmaXl2TkRx?=
 =?utf-8?B?bWJvRVg2QjRremNyNnQwdWpsN2xTYml1ZWpxNFBLb3FmZjNhbWZ2TGJwbzIr?=
 =?utf-8?B?MzlYTHBjYkhHV3U2Qm8ySFg5cnZObVhNM3llSStoc1lWWjVDanYzOG1XSjFh?=
 =?utf-8?B?WFRWRGRtMUdNSS93d3BJTy9rMmJMbWs0ZVdqcElRaEV2ZkN4MXlFOTRZcXNj?=
 =?utf-8?B?QVh5ck5WaWQrMGdHZVJpQVQ2ZW1DcmhrLzBMcnlzaGJTL0dEWkxuTElyVm51?=
 =?utf-8?B?aFpBVkJSMlA5VE1HSmtwMm4vcFc2b2NVMmhDQjh0RElra2NrdzMvSTNnQ0U4?=
 =?utf-8?B?MS9ULzhJbW1uUjJoaUQyd0xCWCtMZ2E4WDNKdTNycUpxMmxHZWZTV3lnbk5L?=
 =?utf-8?B?cFcza20weHdjZWxHczQxOEh6N3ZLa1RxRXdmZlkvbklXSGVBNVU2b1krdC9I?=
 =?utf-8?B?MGU0R0RPUkRyTXBsVTBINHpWSWIrc3BjOENYMU9xeWJQeFByd1ptU1ptWU0y?=
 =?utf-8?B?eGM3NmFFRFV3VTJuUnJNRW9PRFdpN1Joa29CQ3cyQ3FmaDNYaVlnNnVia05x?=
 =?utf-8?B?SjFWdjRrenRQQ3JMdUxtdEhLb1hSSCsySUZVMDJrcUlEc2puMERCeldZeWIy?=
 =?utf-8?B?RTl1Vmg5YnBmckNpcEh6Wm1UUHVjY0ZBQ1pWNGdhbVRzWm9sVEVCNmFuazFw?=
 =?utf-8?B?MmtsdXo1REkyUDhzc0F5VHFxSE14OEd3LzB0OUFMeUdrMmJ2Q1NCMEtyK1d6?=
 =?utf-8?Q?bxt2Rp0nMKaz+1ISYHQjTlQ=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: f499bed6-de15-4c48-a68a-08dbc8933b24
X-MS-Exchange-CrossTenant-AuthSource: MN0PR11MB6231.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Oct 2023 06:44:48.3480
 (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: 8XmYS+gttF2kNCnARtodlg8CyI56woj/AgOMZtfaIyZrIz8KWPwK/x+TrbXgFukAsS2zO4qhn7MVTNjbSnE6yAV4eLbpygTYl+zEQMw0Tn0=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR11MB7241
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=2.7 required=5.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RCVD_IN_SBL_CSS,SPF_HELO_NONE,SPF_PASS autolearn=no
	autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on howler.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (howler.vger.email [0.0.0.0]); Sun, 08 Oct 2023 23:45:08 -0700 (PDT)
X-Spam-Level: **

On 2023-10-06 at 10:53:52 -0700, Reinette Chatre wrote:
>Hi Maciej,
>
>On 10/5/2023 1:14 AM, Maciej Wieczor-Retman wrote:
>> Until recently Intel CPUs didn't support using non-contiguous 1s
>> in Cache Allocation Technology (CAT). Writing a bitmask with
>> non-contiguous 1s to the resctrl schemata file would fail.
>> 
>> Intel CPUs that support non-contiguous 1s can be identified through a
>> CPUID leaf mentioned in the "Intel® 64 and IA-32 Architectures
>> Software Developer’s Manual" document available at:
>> https://www.intel.com/content/www/us/en/developer/articles/technical/intel-sdm.html
>> 
>> Add kernel support for detecting if non-contiguous 1s in Cache
>> Allocation Technology (CAT) are supported by the hardware. Also add a
>> new resctrl FS file to output this information to the userspace.
>> Keep the hardcoded value for Haswell CPUs only since they do not have
>> CPUID enumeration support for Cache allocation.
>> 
>> Since the selftests/resctrl files are going through many rewrites and
>> cleanups the appropriate selftest is still a work in progress. For
>> basic selftesting capabilities use the bash script attached below this
>> paragraph. It checks whether various bitmasks written into resctrl FS
>> generate output consistent with reported feature support.
>
>This work conflicts with Babu's series [1] that is also ready for inclusion.
>We could wait for outcome of next level review to determine who will need
>to rebase. It may help to provide a snippet of the conflict resolution
>in anticipation of Babu's series being merged first (I will propose exactly
>the same to Babu for the scenario of this work merged first).
>
>Reinette
>
>[1] https://lore.kernel.org/lkml/20231003235430.1231238-1-babu.moger@amd.com/

Thanks for fiding this issue. I can see how to resolve the conflict but
where can I put the solution?

I'm guessing in the cover letter?

I'm going to resend the series very soon to apply Babu's comment and
tags. I'll then attach the snippet you mentioned wherever you think it
would fit best.

-- 
Kind regards
Maciej Wieczór-Retman