From: "Michael S. Zick" <lkml@morethan.org>
To: tomaz.mertelj@guest.arnes.si
Cc: Harald Welte <HaraldWelte@viatech.com>,
Jean Delvare <khali@linux-fr.org>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org
Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU
Date: Fri, 12 Jun 2009 14:57:24 +0000 [thread overview]
Message-ID: <200906120957.27223.lkml@morethan.org> (raw)
In-Reply-To: <20090612_144621_045117.tomaz.mertelj@guest.arnes.si>
[-- Attachment #1: Type: text/plain, Size: 1810 bytes --]
On Fri June 12 2009, tomaz.mertelj@guest.arnes.si wrote:
> > >
> > > Ah, 25 degrees C is room temperature - real hard for the junction
> temperature
> > > to be 25 degrees C with power applied; lacking an infinitely perfect
> heatsink.
> > >
> > > Look for an "off by one" error in shifting or masking the value.
> >
> > there is no shifting and the masking is 0xffffffff :)
> >
> > it might be that the BIOS is doing something wrong when programming the
> > calibration MSR's at early botoup. I would need the contents of MSR
> > 0x1160 ... 0x116C as well as 0x1152 and 0x1153 to be able to determine
> that.
> >
>
> This is what I get on VB7002.. vith C7-D cpu:
>
>
> DEBNAS:/home# sensors
>
> via-cputemp-isa-0000
> Adapter: ISA adapter
> Core 0: +27.0 C
>
> DEBNAS:/home# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166
> 0x1167
> 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsr $r ; done
> MSR register 0x1160 => 08:04:98:10:b8:0b:6f:f4
> MSR register 0x1161 => 08:04:98:10:b8:0a:8f:f4
> MSR register 0x1162 => 08:04:98:10:b7:f7:7f:f4
> MSR register 0x1163 => 08:04:98:10:b7:f1:1f:f4
> MSR register 0x1164 => 08:04:98:10:b7:ee:cf:f4
> MSR register 0x1165 => 08:04:98:10:b8:0b:4f:f4
> MSR register 0x1166 => 08:04:98:10:b7:f7:7f:f4
> MSR register 0x1167 => 08:04:98:10:b7:ef:8f:f4
> MSR register 0x1168 => 08:04:98:10:b7:fc:3f:f4
> MSR register 0x1169 => 08:04:98:10:b8:0d:6f:f4
> MSR register 0x116a => 08:04:98:10:b7:f3:8f:f4
> MSR register 0x116b => 08:04:98:10:b8:0b:bf:f4
> MSR register 0x116c => 08:04:98:10:b8:00:6f:f4
> MSR register 0x1152 => 08:04:98:10:b7:f2:2f:f4
> MSR register 0x1153 => 08:04:98:10:b8:0b:3f:f4
> DEBNAS:/home#
>
>
The author had a problem in that routine -
try this one (32bit machines only) -
Mike
> Tomaz Mertelj
>
>
>
[-- Attachment #2: rdmsrll.c --]
[-- Type: text/x-csrc, Size: 782 bytes --]
/* By Ron Minnich @ wiki.laptop.org */
/* No rights or license mentioned. */
#define _LARGEFILE64_SOURCE
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>
int main (int argc, char *argv[])
{
unsigned char buf[8];
int fd_msr, i;
unsigned long long addr = 0;
if (argc < 2) {
printf("usage:rdmsr reg\n");
exit(1);
}
/* assignment zeros high 32bits */
addr = strtoul(argv[1], NULL, 0);
fd_msr = open("/dev/cpu/0/msr", O_RDONLY);
lseek64(fd_msr, (off64_t)addr, SEEK_SET);
read(fd_msr, buf, 8);
printf("MSR register 0x%llx => ", addr);
for (i = 7; i > 0; i--)
printf("%2.2x:", buf[i]);
printf("%2.2x\n", buf[i]);
return(0);
}
[-- Attachment #3: Type: text/plain, Size: 153 bytes --]
_______________________________________________
lm-sensors mailing list
lm-sensors@lm-sensors.org
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
WARNING: multiple messages have this Message-ID (diff)
From: "Michael S. Zick" <lkml@morethan.org>
To: tomaz.mertelj@guest.arnes.si
Cc: Harald Welte <HaraldWelte@viatech.com>,
Jean Delvare <khali@linux-fr.org>,
Andrew Morton <akpm@linux-foundation.org>,
linux-kernel@vger.kernel.org, lm-sensors@lm-sensors.org
Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature
Date: Fri, 12 Jun 2009 09:57:24 -0500 [thread overview]
Message-ID: <200906120957.27223.lkml@morethan.org> (raw)
In-Reply-To: <20090612_144621_045117.tomaz.mertelj@guest.arnes.si>
[-- Attachment #1: Type: text/plain, Size: 1810 bytes --]
On Fri June 12 2009, tomaz.mertelj@guest.arnes.si wrote:
> > >
> > > Ah, 25 degrees C is room temperature - real hard for the junction
> temperature
> > > to be 25 degrees C with power applied; lacking an infinitely perfect
> heatsink.
> > >
> > > Look for an "off by one" error in shifting or masking the value.
> >
> > there is no shifting and the masking is 0xffffffff :)
> >
> > it might be that the BIOS is doing something wrong when programming the
> > calibration MSR's at early botoup. I would need the contents of MSR
> > 0x1160 ... 0x116C as well as 0x1152 and 0x1153 to be able to determine
> that.
> >
>
> This is what I get on VB7002.. vith C7-D cpu:
>
>
> DEBNAS:/home# sensors
>
> via-cputemp-isa-0000
> Adapter: ISA adapter
> Core 0: +27.0 C
>
> DEBNAS:/home# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166
> 0x1167
> 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsr $r ; done
> MSR register 0x1160 => 08:04:98:10:b8:0b:6f:f4
> MSR register 0x1161 => 08:04:98:10:b8:0a:8f:f4
> MSR register 0x1162 => 08:04:98:10:b7:f7:7f:f4
> MSR register 0x1163 => 08:04:98:10:b7:f1:1f:f4
> MSR register 0x1164 => 08:04:98:10:b7:ee:cf:f4
> MSR register 0x1165 => 08:04:98:10:b8:0b:4f:f4
> MSR register 0x1166 => 08:04:98:10:b7:f7:7f:f4
> MSR register 0x1167 => 08:04:98:10:b7:ef:8f:f4
> MSR register 0x1168 => 08:04:98:10:b7:fc:3f:f4
> MSR register 0x1169 => 08:04:98:10:b8:0d:6f:f4
> MSR register 0x116a => 08:04:98:10:b7:f3:8f:f4
> MSR register 0x116b => 08:04:98:10:b8:0b:bf:f4
> MSR register 0x116c => 08:04:98:10:b8:00:6f:f4
> MSR register 0x1152 => 08:04:98:10:b7:f2:2f:f4
> MSR register 0x1153 => 08:04:98:10:b8:0b:3f:f4
> DEBNAS:/home#
>
>
The author had a problem in that routine -
try this one (32bit machines only) -
Mike
> Tomaz Mertelj
>
>
>
[-- Attachment #2: rdmsrll.c --]
[-- Type: text/x-csrc, Size: 782 bytes --]
/* By Ron Minnich @ wiki.laptop.org */
/* No rights or license mentioned. */
#define _LARGEFILE64_SOURCE
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <unistd.h>
#include <stdio.h>
#include <stdlib.h>
int main (int argc, char *argv[])
{
unsigned char buf[8];
int fd_msr, i;
unsigned long long addr = 0;
if (argc < 2) {
printf("usage:rdmsr reg\n");
exit(1);
}
/* assignment zeros high 32bits */
addr = strtoul(argv[1], NULL, 0);
fd_msr = open("/dev/cpu/0/msr", O_RDONLY);
lseek64(fd_msr, (off64_t)addr, SEEK_SET);
read(fd_msr, buf, 8);
printf("MSR register 0x%llx => ", addr);
for (i = 7; i > 0; i--)
printf("%2.2x:", buf[i]);
printf("%2.2x\n", buf[i]);
return(0);
}
next prev parent reply other threads:[~2009-06-12 14:57 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-06-12 14:46 [lm-sensors] =?UTF-8?Q?Re: [PATCH] hwmon: Add driver for VIA CPU tomaz.mertelj
2009-06-12 14:46 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature= tomaz.mertelj
2009-06-12 14:57 ` Michael S. Zick [this message]
2009-06-12 14:57 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
2009-06-12 15:12 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU Michael S. Zick
2009-06-12 15:12 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
2009-06-12 15:25 ` [lm-sensors] =?UTF-8?Q?Re: [PATCH] hwmon: Add driver for VIA CPU tomaz.mertelj
2009-06-12 15:25 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature= tomaz.mertelj
2009-06-12 15:31 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU Michael S. Zick
2009-06-12 15:31 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
2009-06-12 16:04 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU Michael S. Zick
2009-06-12 16:04 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
2009-06-12 17:38 ` [lm-sensors] Michael S. Zick
2009-06-12 17:38 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
2009-06-12 16:48 ` [lm-sensors] =?UTF-8?Q?Re: [PATCH] hwmon: Add driver for VIA CPU tomaz.mertelj
2009-06-12 16:48 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature= tomaz.mertelj
2009-06-12 18:16 ` [lm-sensors] =?UTF-8?Q?Re: [PATCH] hwmon: Add driver for VIA CPU tomaz.mertelj
2009-06-12 18:16 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature= tomaz.mertelj
2009-06-12 18:20 ` [lm-sensors] Michael S. Zick
2009-06-12 18:20 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature Michael S. Zick
-- strict thread matches above, loose matches on Subject: below --
2009-06-09 8:34 Harald Welte
2009-06-10 17:40 ` Tomaz Mertelj
2009-06-11 22:32 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core Andrew Morton
2009-06-12 8:12 ` Jean Delvare
2009-06-12 9:27 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU Harald Welte
2009-06-12 11:46 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core Michael S. Zick
2009-06-12 12:54 ` [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU Harald Welte
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=200906120957.27223.lkml@morethan.org \
--to=lkml@morethan.org \
--cc=HaraldWelte@viatech.com \
--cc=akpm@linux-foundation.org \
--cc=khali@linux-fr.org \
--cc=linux-kernel@vger.kernel.org \
--cc=lm-sensors@lm-sensors.org \
--cc=tomaz.mertelj@guest.arnes.si \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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.