[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170905164503.GA11478@roeck-us.net>
Date: Tue, 5 Sep 2017 09:45:03 -0700
From: Guenter Roeck <linux@...ck-us.net>
To: Clemens Ladisch <clemens@...isch.de>
Cc: Jean Delvare <jdelvare@...e.com>, linux-hwmon@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH] hwmon: (k10temp) Add support for family 17h
On Tue, Sep 05, 2017 at 04:12:07PM +0200, Clemens Ladisch wrote:
> Guenter Roeck wrote:
> > On 09/04/2017 11:47 PM, Clemens Ladisch wrote:
> >> Guenter Roeck wrote:
> >>> Some of this is guesswork, but afaics it is working. No idea if there
> >>> is a better way to determine the temperature offset.
> >>
> >> The reported value is not an absolute temperature on any CPU.
> >>
> >> As far as I know, the offset is not guaranteed to be fixed for any model,
> >> i.e., it would be pointless to apply the offset observed on one specific
> >> chip.
> >
> > What we should do then, as we did for coretemp, would be to collect the various
> > temperature offsets (and temperature limits, for that matter) and apply per-CPU
> > adjustments. Are the offsets documented somewhere ?
>
> AMD says:
> "Tctl is the processor temperature control value, used by the platform to
> control cooling systems. Tctl is a non-physical temperature on an
> arbitrary scale measured in degrees. It does _not_ represent an actual
> physical temperature like die or case temperature. Instead, it specifies
> the processor temperature relative to the point at which the system must
> supply the maximum cooling for the processor's specified maximum case
> temperature and maximum thermal power dissipation."
>
Pretty much the same as Intel. That doesn't mean we should not (try to) report
the real temperature as good as we can, as at least most of the BIOSes do,
and as all the Windows tools do, and as users expect us to do.
Do we really have to argue about this ? If you insist, I'll drop the
adjustments and refer all resulting inquiries to you.
Thanks,
Guenter
Powered by blists - more mailing lists