From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-pl1-f194.google.com ([209.85.214.194]:38506 "EHLO mail-pl1-f194.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1728372AbeKHLNU (ORCPT ); Thu, 8 Nov 2018 06:13:20 -0500 Subject: Re: [PATCH 2/4] x86/amd_nb: add support for newer PCI topologies To: Bjorn Helgaas , Srinivas Pandruvada Cc: Borislav Petkov , "Woods, Brian" , Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , "x86@kernel.org" , Clemens Ladisch , Jean Delvare , Pu Wen , Jia Zhang , Takashi Iwai , Andy Whitcroft , Colin Ian King , Myron Stowe , Sumeet Pawnikar , "linux-kernel@vger.kernel.org" , "linux-hwmon@vger.kernel.org" , "linux-pci@vger.kernel.org" References: <20181102195925.GB160487@google.com> <20181102232948.GC26770@zn.tnic> <20181105214537.GA19420@google.com> <20181105215650.GG26868@zn.tnic> <20181106214256.GA65443@google.com> <20181106220059.GA4139@zn.tnic> <20181106232040.GA85755@google.com> <75748b089ee696d5cbaa5c0ce68bad228699894c.camel@linux.intel.com> <20181107213103.GA41183@google.com> <20181107231411.GB41183@google.com> From: Guenter Roeck Message-ID: <2c4b9e7e-6558-e5ce-50e6-58aaec22fd1c@roeck-us.net> Date: Wed, 7 Nov 2018 17:40:14 -0800 MIME-Version: 1.0 In-Reply-To: <20181107231411.GB41183@google.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit Sender: linux-hwmon-owner@vger.kernel.org List-Id: linux-hwmon@vger.kernel.org On 11/7/18 3:14 PM, Bjorn Helgaas wrote: > >> >> There is no INT3401 on any newer atom or core platforms, so you can't >> enumerate on this device. We don't control what ACPI device is present >> on a system. It depends on what the other non-Linux OS is using. > > Sure, you can't *force* OEMs to supply a given ACPI device, but you > can certainly say "if you want this functionality, supply INT3401 > devices." That's what you do with PNP0A03 (PCI host bridges), for > example. If an OEM doesn't supply PNP0A03 devices, the system can > boot just fine as long as you don't need PCI. > > This model of using the PCI IDs forces OS vendors to release updates > for every new platform. I guess you must have considered that and > decided whatever benefit you're getting was worth the cost. > I really dislike where this is going. Board vendors - and that included Intel when Intel was still selling boards - have a long history of only making mandatory methods available in ACPI. Pretty much all of them don't make hardware monitoring information available via ACPI. This is a pain especially for laptops where the information is provided by an embedded controller. On systems with Super-IO chips with dedicated hardware monitoring functionality, they often go as far as signing mutual NDAs with chip vendors, which lets both the board and the chip vendor claim that they can not provide chip specifications to third parties, aka users. You are pretty much extending that to CPU temperature monitoring. The fallout, if adopted, will be that it will effectively no longer be possible to monitor the temperature on chips supporting this "feature". I do not think that would be a good idea. Guenter