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: <57cf9b61-82f4-f6d4-7f43-c3f94de7aaf3@amd.com>
Date:   Fri, 19 Aug 2022 11:00:21 -0500
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101
 Thunderbird/91.12.0
Subject: Re: [RFC 0/2] Stop the abuse of Linux-* _OSI strings
Content-Language: en-US
To:     Karol Herbst <kherbst@redhat.com>, Lyude Paul <lyude@redhat.com>
Cc:     linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
        rafael@kernel.org, Len Brown <lenb@kernel.org>,
        nouveau@lists.freedesktop.org, hdegoede@redhat.com,
        ddadap@nvidia.com, kai.heng.feng@canonical.com,
        Dell.Client.Kernel@dell.com
References: <20220819142519.5684-1-mario.limonciello@amd.com>
 <CACO55tuw_2QWFMr0t6-JfUiQ4M7V3ZMKC7jHYTyERypaU3TekQ@mail.gmail.com>
From:   "Limonciello, Mario" <mario.limonciello@amd.com>
In-Reply-To: <CACO55tuw_2QWFMr0t6-JfUiQ4M7V3ZMKC7jHYTyERypaU3TekQ@mail.gmail.com>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?T1dUUTR3dzNtb1NNWEUwU0I3NUk1QVpPZVAxd2EvZjFqQ1BWYkFIT1RpeWsr?=
 =?utf-8?B?QWZFOHZSK1YwMzBzZzVmbFBHaUVjMG1Gb0Z3dUptL2ZYd1IvVHlLN2REdmJK?=
 =?utf-8?B?NDNEV1F5VHBWSVVwZStPZmx0dVBwNHQ4WnRUeGZETlZVY0hKb2hqNDdzSlhq?=
 =?utf-8?B?WnBWOStXalFFRnpNb2MwZ01UYW4yc0F3UHdQZEsyL0t1Tzc5Y0lkcHRXTjB4?=
 =?utf-8?B?TFA4c1ZYcmMxaklqQmh3THNFaXFhMXdickRpaWxjMGh6RmtUZExzUTVhYUdX?=
 =?utf-8?B?ZmVuSjFGVFg4Y05MMmJ3YjhWZXUxM0YvQ0dxblRES1J2SEZyTklZbTBsOWRQ?=
 =?utf-8?B?d1FwZnNZcVd3N2psb1R1aGV0YkpXYzI3dTBhNEJnaTBUbmVLWTFJenluL1oz?=
 =?utf-8?B?TWhOYmpzS2s4ZjFidndoZUI1WDJqU3RGcjA2QVA2NkxydVEyOGRmN0cwbFBr?=
 =?utf-8?B?cUdka0daYUUrOVI3WUF4Y0ViLzJyTzREZDFtb2Vidk9qOVJJckk3QWp2STA5?=
 =?utf-8?B?bkRmZ2hNOUw4Y1pqZ0xwZXJxeU9jTlc5U2gyQ2tzcnhxN3ZzTDVkK2thNVJ2?=
 =?utf-8?B?ZGMvUHlMNDhXS0pvMkp5WXBqbjFab3gwejF1VFRNb3IreE9aSnNpeTlGb0Ro?=
 =?utf-8?B?WmFWWW94TkJmYkljcG8xNW85VmNxa0grWnNMOXpmUlpZM2NTT3pUUEFZV292?=
 =?utf-8?B?SnpBbnE3aEJMQ1NJSkZLU0NURHhmOGFlNEZEbDdLV3ZUYU1ENElvQllodDNz?=
 =?utf-8?B?cHFOa01hQkcrQXEvbUpxMm50ZlVrUVRxYU56UUlkR21McTdJeGJla3pZOXRJ?=
 =?utf-8?B?ZG5QeHZ6REdKMWt6cG95alRWOGVQVm5nNk9ZWktrbHp5M1pqSTVQTUN2ZFNR?=
 =?utf-8?B?SmVFaHdSNzM5b2VrVUZ4RHpHcXdDNmNGdW5acFlDeGU4VlhGQkxZUHZJSmls?=
 =?utf-8?B?K0hSNVBKdjdhaDF2WlVDRHdRWW5IT3FaYTBPOXV3NEtSTDdQRGVVMW8xbzUx?=
 =?utf-8?B?WTJHYTlQZkZWVUloRmFhVnU4NE1SZllzZENJQzVFeTZzdndKekxoM0ZlUmY0?=
 =?utf-8?B?cHFINFk4c0tEUUhycGg4ZzFUTWpKbVBQSDRqWWVYdlI1QXdhLzVEK0hrYkNk?=
 =?utf-8?B?aDJxQktYTlVPbWdhSWxveTBxMXErV1o5SWsrUmlMK3RoWDVxeStVaXhjQzlO?=
 =?utf-8?B?YWdLZnk4SVRueXVZRVFMWnN6WjhHTDVKQ2JKMFc1UFJObXd4VUdvMzdIZ3Zt?=
 =?utf-8?B?eEIwS1B5VmhGdEppTUJtZlppaWhSdUZtRk45Y0lRYklidUpJSVEranpqYnY1?=
 =?utf-8?B?cDZ1WEhTd2FWUnVxVEVWOHpocEZyNFZyNHNTOHZjV1dlRUxzVzRaOXFpZzJm?=
 =?utf-8?B?a2ZGSjVtNU03SzdtNlRreFg0Z1NXaEpGcjZFbVJyb25VWm9yaExOMDFydEcz?=
 =?utf-8?B?cEpId3dnbC81Vk53MTdGK3pUcnlQTnRoZGZyMmNTamUxekFxVU5vVVpUd1p0?=
 =?utf-8?B?aEgxdTU4Vk9hK1BtNlZOZGdkVjM3TGs4c3N2TEt6Q2NNYmwwS3dHSStvYUJq?=
 =?utf-8?B?WmY2ekhmaGQwRG5wbTluOEswTzFoQXZUNFlLZEVIVDV1RmkwMm51aHdLeVNa?=
 =?utf-8?B?TDRFUzlLNXhqeERxN0NCNVhxRmZEd0FrSTE1Vmh3NGI2MDcwTkZ0bGdzY2s1?=
 =?utf-8?B?aDZZcGNYMmhkN2E1dXp4Smk1Q2o0eERha3RicGVFNzRHOTJjYTdyL0cxa1A3?=
 =?utf-8?B?YWVvQnBSa0FpV3VxVmM0UUdTZzVBZmR5V2ZkMnZYK0NmOVkwR2VoMlN2VWZz?=
 =?utf-8?B?blVKbDRKZmJ0N2xmT0tvSkwrUVpScnNrWkJ1eWRuK3QwTngwbzB6am5nZ3JE?=
 =?utf-8?B?Yit2ODFhUm0vYnNnKy9IL3VmZm1ZUktRVFlLMDRhUXUraTRRZGFEcGtENlc1?=
 =?utf-8?B?dFJvd0lpWGJJTi9FUVR1UnBsaDBJQUVJVmFGTnVicEprY1A2eVhFVlhaclh4?=
 =?utf-8?B?R3NnajM1ZTRLUndUYjFoS2lEc2NXT20wSXhobzBFTWZhYTJpN2RKQUdwZFIy?=
 =?utf-8?B?OUplbGkvVC9GVTNuNjVRQjhjL1hJaDJHSFlweXcrQ2dKYm5lUkhVdnZBSWUv?=
 =?utf-8?Q?JbuVzIcfXDLwQsp9DxDezbNMM?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ff00e1e6-72ca-445e-e110-08da81fbed12
X-MS-Exchange-CrossTenant-AuthSource: MN0PR12MB6101.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Aug 2022 16:00:24.1964
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 2akRcDabx+IQfzQvMCAUVJ7bjlS2uv5AMiti4dPUhWsMhYrrz8111Ij5Q6eqc6sftx7ZnXWe1tvn/OD/uhUCaw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR12MB2438
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS,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

On 8/19/2022 10:44, Karol Herbst wrote:
> On Fri, Aug 19, 2022 at 4:25 PM Mario Limonciello
> <mario.limonciello@amd.com> wrote:
>>
>> 3 _OSI strings were introduced in recent years that were intended
>> to workaround very specific problems found on specific systems.
>>
>> The idea was supposed to be that these quirks were only used on
>> those systems, but this proved to be a bad assumption.  I've found
>> at least one system in the wild where the vendor using the _OSI
>> string doesn't match the _OSI string and the neither does the use.
>>
>> So this brings a good time to review keeping those strings in the kernel.
>> There are 3 strings that were introduced:
>>
>> Linux-Dell-Video
>> -> Intended for systems with NVIDIA cards that didn't support RTD3
>> Linux-Lenovo-NV-HDMI-Audio
>> -> Intended for powering on NVIDIA HDMI device
>> Linux-HPI-Hybrid-Graphics
>> -> Intended for changing dGPU output
>>
>> AFAIK the first string is no longer relevant as nouveau now supports
>> RTD3.  If that's wrong, this can be changed for the series.
>>
> 
> Nouveau always supported RTD3, because that's mainly a kernel feature.
> When those were introduced we simply had a bug only hit on a few
> systems. And instead of helping us to debug this, this workaround was
> added :( We were not even asked about this.

My apologies, I was certainly part of the impetus for this W/A in the 
first place while I was at my previous employer.  Your comment 
re-affirms to me that at least the first patch is correct.

> 
> I am a bit curious about the other two though as I am not even sure
> they are needed at all as we put other work arounds in place. @Lyude
> Paul might know more about these.
> 

If the other two really aren't needed anymore, then yeah we should just 
tear all 3 out.  If that's the direction we go, I would appreciate some 
commit IDs to reference in the commit message for tearing them out so 
that if they end up backporting to stable we know how far they should go.