From: "Michael S. Zick" <lkml@morethan.org>
Reply-To: lkml@morethan.org
To: tomaz.mertelj@guest.arnes.si
Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core	temperature
Date: Fri, 12 Jun 2009 09:57:24 -0500
User-Agent: KMail/1.9.9
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
References: <20090612_144621_045117.tomaz.mertelj@guest.arnes.si>
In-Reply-To: <20090612_144621_045117.tomaz.mertelj@guest.arnes.si>
MIME-Version: 1.0
Content-Type: Multipart/Mixed;
  boundary="Boundary-00=_XzmMKrOF44zXK8P"
Message-Id: <200906120957.27223.lkml@morethan.org>
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 3201
Lines: 117

--Boundary-00=_XzmMKrOF44zXK8P
Content-Type: text/plain;
  charset="us-ascii"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

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
> 
> 
> 


--Boundary-00=_XzmMKrOF44zXK8P
Content-Type: text/x-csrc;
  charset="iso 8859-15";
  name="rdmsrll.c"
Content-Transfer-Encoding: 7bit
Content-Disposition: attachment;
	filename="rdmsrll.c"

/* 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);
}


--Boundary-00=_XzmMKrOF44zXK8P--
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/