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: <81a32a39-217f-0b4b-beec-46d909b41303@intel.com>
Date:   Fri, 5 May 2023 16:20:56 -0700
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:102.0) Gecko/20100101
 Firefox/102.0 Thunderbird/102.10.1
Subject: Re: [RFC PATCH 6/7] x86/resctrl: Allow a device to override an
 existing schemata entry
Content-Language: en-US
To:     Tony Luck <tony.luck@intel.com>, Fenghua Yu <fenghua.yu@intel.com>,
        "Thomas Gleixner" <tglx@linutronix.de>,
        Ingo Molnar <mingo@redhat.com>,
        "Borislav Petkov" <bp@alien8.de>,
        Dave Hansen <dave.hansen@linux.intel.com>,
        "H. Peter Anvin" <hpa@zytor.com>,
        Shaopeng Tan <tan.shaopeng@fujitsu.com>,
        Jamie Iles <quic_jiles@quicinc.com>,
        James Morse <james.morse@arm.com>,
        Babu Moger <babu.moger@amd.com>
CC:     <x86@kernel.org>, <linux-kernel@vger.kernel.org>
References: <20230420220636.53527-1-tony.luck@intel.com>
 <20230420220636.53527-7-tony.luck@intel.com>
From:   Reinette Chatre <reinette.chatre@intel.com>
In-Reply-To: <20230420220636.53527-7-tony.luck@intel.com>
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?N0NxRHlvcHg1Nnhva0hxNSs0d2xoTEhtNzVaU01iQ3BvNy9jdUJzaFhTTEdp?=
 =?utf-8?B?TFR3d2g4SUVLd0ZBQWFPRW1iT1Vmb3pHVWVMZmpCWDFTVWFIYW9uN2lKTzRJ?=
 =?utf-8?B?UWVIbDRobjlMMjE2QldiclBCaUNZY2tKemxIM2xSZ1dkM2tjZG0yUHQxZCtU?=
 =?utf-8?B?Q1p3Nm1sQUxjV0h4dHhrMkdqeXhKeVlTejZ3SlVJWmhOUmlzeHR5RWUrczBK?=
 =?utf-8?B?RkF6MnViSEZEWUlRbzBQbSt1ZGdubzVRK0I0NXRuclpCL0NhZTd5a1JweVF3?=
 =?utf-8?B?N05QSlhFdlBnazEzSmxXa05tSXk5VkFhRjlqcWpmM3NPdk5OWFRNR05YaGdx?=
 =?utf-8?B?U3JpS2xVMGxHdEtRL2hldEQzM2dhUW1uRmV4RGt4RUs3TzNMVVliZFJDQVJu?=
 =?utf-8?B?RXJDdVhlMEREaFF6RUxqTCt0Vmd1cnFrZEVGZHNkSmpMSWduaTM5dXBNSm1h?=
 =?utf-8?B?ZUFxb3c5UFhMSFUzRnRnb0t2clpUSTdqbmZLL3dPV0lYMVNTOU90UnVuWm14?=
 =?utf-8?B?ZXlYOHQyRFVTQkxYaUtxdWhmZ0ZhQ2NCdGJ1b3AwMEJNUld6T0dXRUtJRGpH?=
 =?utf-8?B?MERpUi83RTAvNU9kdklXdFFtK2FtWHo2TC9jUnNBT1BPMW0raUsyNjZNblFz?=
 =?utf-8?B?b1VoTVBhV21CRWU3RnRvS1Jpb1RPL3dYdHdTVFJtbFZJcThQd0gwQmJsdGtH?=
 =?utf-8?B?OUFKTFVWM293R3cxa1BscU9kWGx6WFR6Wmx2T3Y2YUNxUHhrMzNaWDJFMFVq?=
 =?utf-8?B?Qmh2b0pTckFTK0tFRXJmN09ZMGxHTnRqZXY1MXVMQmRxT2ZOUDlQdlVjaUly?=
 =?utf-8?B?TmFOSllNSzdWYTJjOU5UY1puYnJmS09YUEZTWEl4WjNET2NEelg1WmswSjBo?=
 =?utf-8?B?VWtJTVQ1OWE4bEFrbjE3ZThMelhKcUxvVURucUtwVDFodTRHdmFYcW1VK1dj?=
 =?utf-8?B?V2RRRlF2WkFOZ2ppS2JUZmwzUmFibDZLSkpneUdoU2xSSFZXVmZnOFZWY2xj?=
 =?utf-8?B?RkY0blhwRkZsalF2bHNOamJ1TDJ5WFRxVjNmWVE3ZFpTTENRV05ZNkJCWnVH?=
 =?utf-8?B?KzdmZDhpeHpHZ05yU3ZjYkdtd1grZThSMm5nNVRZa0FnVTRnTWVmVzF1NzBz?=
 =?utf-8?B?eUxDNktTWGFvWVhBemhBTzBYOExSa1pKbVBGVFBScFFXRGhnZDgvSUUza2o4?=
 =?utf-8?B?QjczNWFJaUFHR085Q25nNFBwT2d5YXZnR3ZMcmdwaDZQZmtUbW5RZUJmNmw3?=
 =?utf-8?B?V0d2R3RzSUNWc2loUTk4VmI0UlI4aVBkWEQ0WDRRaDV1MnhYZjRYb21QYW5p?=
 =?utf-8?B?TU14eVNEOU5RYUlqcGRLK2lZaXZZb0JPSFNQLzlkOW1teVZjMXZhM25EMHlt?=
 =?utf-8?B?a25OM2piMXBFVmR5UDZadzNlSG1aWmEvUCsxeFR0NXBzS3dLczNOdG93M3Jv?=
 =?utf-8?B?UWNsVEVzdUhveWNMb21qRVdvMHZRVUhnNWJDc1B0ZmwxdE1zaVplMzd0VlJE?=
 =?utf-8?B?L1RYZVVtZWRUV1dhdVkxaWFNZmRXZmd6Vm05UDZuRVpVcjJRb3B1V1NCL3Rj?=
 =?utf-8?B?ZEEzUThzRHRyekN4Q2c1bUFaMmlJWHh6V01NVlB1TGdKUjdJNnk3WWgwYzBO?=
 =?utf-8?B?c2RHNVdoTTFvT3NaTzVKYURwMWEzMHhTSWtYZ2taSHEwMVNyUWlJOEc3Z0x3?=
 =?utf-8?B?WDJFcHV1TGM0QWRZOCtqbXJFbytNMmZ5dkhiWmxTSm51V1c4eW1NWTZDQVlO?=
 =?utf-8?B?Y25lSFJDWFUrNEJ3WFRSK3VJUkhySHNIWjhRMzZNL1dhdE94VnovWWUvQ1Y5?=
 =?utf-8?B?RmVvY29KeHpEa0c3UElGYU44U1dGZEVGeDhMZ1dSTkM4d0ZZNG9mb0IyN1Z3?=
 =?utf-8?B?RStKVVppS0tkckxLV1VGRnpoMDRVejJCQWJONHNkN1NKTHIxRTMwckIzUWUv?=
 =?utf-8?B?Y3FWdVdkNjhkNC81WnZ3M0hMTEhpTWpPOERWZVQ0aEs4dFA1RFV3TEEvNVh2?=
 =?utf-8?B?eW11MjgvL1Bicms2TURUbW1VVUhoeGlVdHBwTTJvL25PWG4vaFUxaEZ5K0M5?=
 =?utf-8?B?d3Q2M1pCR21TS2g5ZFEvczdNYnNQbTJPYVlFNHIrZHVBb3FLbk9OSkZ0MUVI?=
 =?utf-8?B?VHQwNzloRUNUaWhEazVmbi9qRWQxVzNIUEV0amIzNEt5WXR1VVl1MlBXQTNW?=
 =?utf-8?B?aFE9PQ==?=
X-MS-Exchange-CrossTenant-Network-Message-Id: 44b01598-4273-4e26-db91-08db4dbf61fd
X-MS-Exchange-CrossTenant-AuthSource: SJ2PR11MB7573.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 May 2023 23:20:58.1984
 (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: dF1m1hlp2uTM6c/i+K5jo9yOAvYat09qJp6H8WjwugCNesIl7jtAemGlqlSCDIsA2igL3C2YwvwqhARcTjh1qWRLtKMKvDSHCd3x+cQG5dQ=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BL1PR11MB5953
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-8.8 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,RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE 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 Tony,

On 4/20/2023 3:06 PM, Tony Luck wrote:
> Remove that entry from the resctrl_schema_all list when driver
> is loaded. Put it back again when driver is unloaded.

This is unexpected. It sounds like the system would advertise
a supported resource with appropriate properties but for some reason this
is not correct, optional in some way, or perhaps resources are 
conflicting? Where would it be decided whether the overriding driver
should be loaded and why can that logic not be in enumeration
within resctrl?

Reinette