* [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection for
@ 2011-06-01 15:50 Guenter Roeck
2011-06-01 16:57 ` [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection Jean Delvare
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Guenter Roeck @ 2011-06-01 15:50 UTC (permalink / raw)
To: lm-sensors
Commit a321cedb12904114e2ba5041a3673ca24deb09c9 excludes CPU models 0xe, 0xf,
0x16, and 0x1a from TjMax temperature adjustment, even though several of those
CPUs are known to have TiMax other than 100 degrees C, and even though the code
in adjust_tjmax() explicitly handles those CPUs and points to a Web document
listing several of the affected CPU IDs.
Reinstate original TjMax adjustment if TjMax can not be determined using the
IA32_TEMPERATURE_TARGET register.
Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
Cc: Huaxu Wan <huaxu.wan@linux.intel.com>
Cc: Carsten Emde <C.Emde@osadl.org>
Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu>
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Yong Wang <yong.y.wang@linux.intel.com>
Cc: Rudolf Marek <r.marek@assembler.cz>
Cc: Fenghua Yu <fenghua.yu@intel.com>
---
v2: Call adjust_tjmax() unconditionally if reading IA32_TEMPERATURE_TARGET
failed or did not return a reasonable value for TjMax.
Note: This patch is a candidate for -stable [ .35 to .39]
drivers/hwmon/coretemp.c | 19 ++-----------------
1 files changed, 2 insertions(+), 17 deletions(-)
diff --git a/drivers/hwmon/coretemp.c b/drivers/hwmon/coretemp.c
index 0eeff46..1680977 100644
--- a/drivers/hwmon/coretemp.c
+++ b/drivers/hwmon/coretemp.c
@@ -304,24 +304,9 @@ static int get_tjmax(struct cpuinfo_x86 *c, u32 id, struct device *dev)
/*
* An assumption is made for early CPUs and unreadable MSR.
- * NOTE: the given value may not be correct.
+ * NOTE: the calculated value may not be correct.
*/
-
- switch (c->x86_model) {
- case 0xe:
- case 0xf:
- case 0x16:
- case 0x1a:
- dev_warn(dev, "TjMax is assumed as 100 C!\n");
- return 100000;
- case 0x17:
- case 0x1c: /* Atom CPUs */
- return adjust_tjmax(c, id, dev);
- default:
- dev_warn(dev, "CPU (model=0x%x) is not supported yet,"
- " using default TjMax of 100C.\n", c->x86_model);
- return 100000;
- }
+ return adjust_tjmax(c, id, dev);
}
static void __devinit get_ucode_rev_on_cpu(void *edx)
--
1.7.3.1
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection
2011-06-01 15:50 [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection for Guenter Roeck
@ 2011-06-01 16:57 ` Jean Delvare
2011-06-01 17:21 ` Yu, Fenghua
2011-06-01 18:03 ` Guenter Roeck
2 siblings, 0 replies; 4+ messages in thread
From: Jean Delvare @ 2011-06-01 16:57 UTC (permalink / raw)
To: lm-sensors
On Wed, 1 Jun 2011 08:50:46 -0700, Guenter Roeck wrote:
> Commit a321cedb12904114e2ba5041a3673ca24deb09c9 excludes CPU models 0xe, 0xf,
> 0x16, and 0x1a from TjMax temperature adjustment, even though several of those
> CPUs are known to have TiMax other than 100 degrees C, and even though the code
> in adjust_tjmax() explicitly handles those CPUs and points to a Web document
> listing several of the affected CPU IDs.
>
> Reinstate original TjMax adjustment if TjMax can not be determined using the
> IA32_TEMPERATURE_TARGET register.
Tested on my 3 machines, looks alright.
Acked-by: Jean Delvare <khali@linux-fr.org>
> Note: This patch is a candidate for -stable [ .35 to .39]
Yes please.
--
Jean Delvare
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection
2011-06-01 15:50 [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection for Guenter Roeck
2011-06-01 16:57 ` [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection Jean Delvare
@ 2011-06-01 17:21 ` Yu, Fenghua
2011-06-01 18:03 ` Guenter Roeck
2 siblings, 0 replies; 4+ messages in thread
From: Yu, Fenghua @ 2011-06-01 17:21 UTC (permalink / raw)
To: lm-sensors
> -----Original Message-----
> From: Guenter Roeck [mailto:guenter.roeck@ericsson.com]
> Sent: Wednesday, June 01, 2011 8:51 AM
> To: lm-sensors@lm-sensors.org; Jean Delvare
> Cc: Guenter Roeck; Huaxu Wan; Carsten Emde; Valdis Kletnieks; Henrique
> de Moraes Holschuh; Yong Wang; Rudolf Marek; Yu, Fenghua
> Subject: [PATCH v2] hwmon: (coretemp) Fix TjMax detection for older
> CPUs
>
> Commit a321cedb12904114e2ba5041a3673ca24deb09c9 excludes CPU models
> 0xe, 0xf,
> 0x16, and 0x1a from TjMax temperature adjustment, even though several
> of those
> CPUs are known to have TiMax other than 100 degrees C, and even though
> the code
> in adjust_tjmax() explicitly handles those CPUs and points to a Web
> document
> listing several of the affected CPU IDs.
>
> Reinstate original TjMax adjustment if TjMax can not be determined
> using the
> IA32_TEMPERATURE_TARGET register.
>
> Signed-off-by: Guenter Roeck <guenter.roeck@ericsson.com>
> Cc: Huaxu Wan <huaxu.wan@linux.intel.com>
> Cc: Carsten Emde <C.Emde@osadl.org>
> Cc: Valdis Kletnieks <valdis.kletnieks@vt.edu>
> Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
> Cc: Yong Wang <yong.y.wang@linux.intel.com>
> Cc: Rudolf Marek <r.marek@assembler.cz>
> Cc: Fenghua Yu <fenghua.yu@intel.com>
> ---
> v2: Call adjust_tjmax() unconditionally if reading
> IA32_TEMPERATURE_TARGET
> failed or did not return a reasonable value for TjMax.
>
> Note: This patch is a candidate for -stable [ .35 to .39]
>
> drivers/hwmon/coretemp.c | 19 ++-----------------
> 1 files changed, 2 insertions(+), 17 deletions(-)
>
> diff --git a/drivers/hwmon/coretemp.c b/drivers/hwmon/coretemp.c
> index 0eeff46..1680977 100644
> --- a/drivers/hwmon/coretemp.c
> +++ b/drivers/hwmon/coretemp.c
> @@ -304,24 +304,9 @@ static int get_tjmax(struct cpuinfo_x86 *c, u32
> id, struct device *dev)
>
> /*
> * An assumption is made for early CPUs and unreadable MSR.
> - * NOTE: the given value may not be correct.
> + * NOTE: the calculated value may not be correct.
> */
> -
> - switch (c->x86_model) {
> - case 0xe:
> - case 0xf:
> - case 0x16:
> - case 0x1a:
> - dev_warn(dev, "TjMax is assumed as 100 C!\n");
> - return 100000;
> - case 0x17:
> - case 0x1c: /* Atom CPUs */
> - return adjust_tjmax(c, id, dev);
> - default:
> - dev_warn(dev, "CPU (model=0x%x) is not supported yet,"
> - " using default TjMax of 100C.\n", c->x86_model);
> - return 100000;
> - }
> + return adjust_tjmax(c, id, dev);
> }
>
> static void __devinit get_ucode_rev_on_cpu(void *edx)
> --
> 1.7.3.1
Acked-by: Fenghua Yu <fenghua.yu@intel.com>
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection
2011-06-01 15:50 [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection for Guenter Roeck
2011-06-01 16:57 ` [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection Jean Delvare
2011-06-01 17:21 ` Yu, Fenghua
@ 2011-06-01 18:03 ` Guenter Roeck
2 siblings, 0 replies; 4+ messages in thread
From: Guenter Roeck @ 2011-06-01 18:03 UTC (permalink / raw)
To: lm-sensors
On Wed, Jun 01, 2011 at 12:57:15PM -0400, Jean Delvare wrote:
> On Wed, 1 Jun 2011 08:50:46 -0700, Guenter Roeck wrote:
> > Commit a321cedb12904114e2ba5041a3673ca24deb09c9 excludes CPU models 0xe, 0xf,
> > 0x16, and 0x1a from TjMax temperature adjustment, even though several of those
> > CPUs are known to have TiMax other than 100 degrees C, and even though the code
> > in adjust_tjmax() explicitly handles those CPUs and points to a Web document
> > listing several of the affected CPU IDs.
> >
> > Reinstate original TjMax adjustment if TjMax can not be determined using the
> > IA32_TEMPERATURE_TARGET register.
>
> Tested on my 3 machines, looks alright.
>
> Acked-by: Jean Delvare <khali@linux-fr.org>
>
> > Note: This patch is a candidate for -stable [ .35 to .39]
>
> Yes please.
>
Applied, thanks. I'll create a separate patch for -stable once the patch is accepted
into 3.0-rc, to avoid conflicts and unnecessary work for the -stable maintainers.
Thanks,
Guenter
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2011-06-01 18:03 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2011-06-01 15:50 [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection for Guenter Roeck
2011-06-01 16:57 ` [lm-sensors] [PATCH v2] hwmon: (coretemp) Fix TjMax detection Jean Delvare
2011-06-01 17:21 ` Yu, Fenghua
2011-06-01 18:03 ` Guenter Roeck
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.