2009-06-12 15:25:22

by Tomaz Mertelj

Subject: =?UTF-8?Q?Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature= ?=

>
> The author had a problem in that routine -
> try this one (32bit machines only) -
>
> Mike

DEBNAS:/home# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166
0x1167
0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsrll $r ;
done
MSR register 0x1160 => 08:04:83:94:bf:c2:39:c8
MSR register 0x1161 => 08:04:83:94:bf:ce:12:88
MSR register 0x1162 => 08:04:83:94:bf:d0:7a:a8
MSR register 0x1163 => 08:04:83:94:bf:9e:1f:88
MSR register 0x1164 => 08:04:83:94:bf:9c:ff:78
MSR register 0x1165 => 08:04:83:94:bf:87:6e:18
MSR register 0x1166 => 08:04:83:94:bf:a1:87:b8
MSR register 0x1167 => 08:04:83:94:bf:f1:cc:b8
MSR register 0x1168 => 08:04:83:94:bf:c6:22:08
MSR register 0x1169 => 08:04:83:94:bf:cf:b8:68
MSR register 0x116a => 08:04:83:94:bf:a3:57:d8
MSR register 0x116b => 08:04:83:94:bf:b3:a8:d8
MSR register 0x116c => 08:04:83:94:bf:ea:04:48
MSR register 0x1152 => 08:04:83:94:bf:84:85:e8
MSR register 0x1153 => 08:04:83:94:bf:d6:fb:18

I
I hope this is better.

Tomaz

2009-06-12 15:31:19

by Michael S. Zick

Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature

On Fri June 12 2009, [email protected] wrote:
> >
> > The author had a problem in that routine -
> > try this one (32bit machines only) -
> >
> > Mike
>
> DEBNAS:/home# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166
> 0x1167
> 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsrll $r ;
> done
> MSR register 0x1160 => 08:04:83:94:bf:c2:39:c8
> MSR register 0x1161 => 08:04:83:94:bf:ce:12:88
> MSR register 0x1162 => 08:04:83:94:bf:d0:7a:a8
> MSR register 0x1163 => 08:04:83:94:bf:9e:1f:88
> MSR register 0x1164 => 08:04:83:94:bf:9c:ff:78
> MSR register 0x1165 => 08:04:83:94:bf:87:6e:18
> MSR register 0x1166 => 08:04:83:94:bf:a1:87:b8
> MSR register 0x1167 => 08:04:83:94:bf:f1:cc:b8
> MSR register 0x1168 => 08:04:83:94:bf:c6:22:08
> MSR register 0x1169 => 08:04:83:94:bf:cf:b8:68
> MSR register 0x116a => 08:04:83:94:bf:a3:57:d8
> MSR register 0x116b => 08:04:83:94:bf:b3:a8:d8
> MSR register 0x116c => 08:04:83:94:bf:ea:04:48
> MSR register 0x1152 => 08:04:83:94:bf:84:85:e8
> MSR register 0x1153 => 08:04:83:94:bf:d6:fb:18
>
> I
> I hope this is better.
>

The tool is broke - it is not reporting read failures.
Evidently those register ranges are masked out somewhere;
will check on why/where.

Mike
> Tomaz
>
>
>
>

2009-06-12 16:04:35

by Michael S. Zick

Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature

On Fri June 12 2009, Michael S. Zick wrote:
> On Fri June 12 2009, [email protected] wrote:
> > >
> > > The author had a problem in that routine -
> > > try this one (32bit machines only) -
> > >
> > > Mike
> >
> > DEBNAS:/home# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166
> > 0x1167
> > 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsrll $r ;
> > done
> > MSR register 0x1160 => 08:04:83:94:bf:c2:39:c8
> > MSR register 0x1161 => 08:04:83:94:bf:ce:12:88
> > MSR register 0x1162 => 08:04:83:94:bf:d0:7a:a8
> > MSR register 0x1163 => 08:04:83:94:bf:9e:1f:88
> > MSR register 0x1164 => 08:04:83:94:bf:9c:ff:78
> > MSR register 0x1165 => 08:04:83:94:bf:87:6e:18
> > MSR register 0x1166 => 08:04:83:94:bf:a1:87:b8
> > MSR register 0x1167 => 08:04:83:94:bf:f1:cc:b8
> > MSR register 0x1168 => 08:04:83:94:bf:c6:22:08
> > MSR register 0x1169 => 08:04:83:94:bf:cf:b8:68
> > MSR register 0x116a => 08:04:83:94:bf:a3:57:d8
> > MSR register 0x116b => 08:04:83:94:bf:b3:a8:d8
> > MSR register 0x116c => 08:04:83:94:bf:ea:04:48
> > MSR register 0x1152 => 08:04:83:94:bf:84:85:e8
> > MSR register 0x1153 => 08:04:83:94:bf:d6:fb:18
> >
> > I
> > I hope this is better.
> >
>
> The tool is broke - it is not reporting read failures.
> Evidently those register ranges are masked out somewhere;
> will check on why/where.
>

The tool does not report read failures.
**BUT** if you remember to "modprobe msr" or have it built in...
Translation: Cockpit error. ;)

root@cb01:~# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166 0x1167 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsrll $r ; done
MSR register 0x1160 => 00:00:00:00:00:00:05:dc
MSR register 0x1161 => 00:00:00:00:00:00:03:e8
MSR register 0x1162 => 00:00:00:00:00:00:00:00
MSR register 0x1163 => 00:00:00:00:00:00:00:00
MSR register 0x1164 => 00:00:00:00:00:00:00:5f
MSR register 0x1165 => 00:00:00:00:00:00:00:64
MSR register 0x1166 => 00:00:00:00:00:00:00:7d
MSR register 0x1167 => 00:00:00:00:00:00:00:00
MSR register 0x1168 => 00:00:00:00:00:00:01:8d
MSR register 0x1169 => 00:00:00:00:00:00:00:32
MSR register 0x116a => 00:00:00:00:00:00:00:03
MSR register 0x116b => 00:00:00:00:00:00:01:a6
MSR register 0x116c => 00:00:00:00:00:00:03:34
MSR register 0x1152 => e7:2c:b3:9f:fb:ff:ff:ff
MSR register 0x1153 => 04:06:0c:09:fe:ff:ff:ff

Which is the same data as reported by msr-tools.

Mike
> Mike
> > Tomaz
> >
> >
> >
> >
>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to [email protected]
> More majordomo info at http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at http://www.tux.org/lkml/
>
>

2009-06-12 17:38:32

by Michael S. Zick

Subject: Re: [lm-sensors] [PATCH] hwmon: Add driver for VIA CPU core temperature

On Fri June 12 2009, Michael S. Zick wrote:
>
> The tool does not report read failures.
> **BUT** if you remember to "modprobe msr" or have it built in...
> Translation: Cockpit error. ;)
>
> root@cb01:~# for r in 0x1160 0x1161 0x1162 0x1163 0x1164 0x1165 0x1166 0x1167 0x1168 0x1169 0x116a 0x116b 0x116c 0x1152 0x1153 ; do ./rdmsrll $r ; done
> MSR register 0x1160 => 00:00:00:00:00:00:05:dc
> MSR register 0x1161 => 00:00:00:00:00:00:03:e8
> MSR register 0x1162 => 00:00:00:00:00:00:00:00
> MSR register 0x1163 => 00:00:00:00:00:00:00:00
> MSR register 0x1164 => 00:00:00:00:00:00:00:5f
> MSR register 0x1165 => 00:00:00:00:00:00:00:64
> MSR register 0x1166 => 00:00:00:00:00:00:00:7d
> MSR register 0x1167 => 00:00:00:00:00:00:00:00
> MSR register 0x1168 => 00:00:00:00:00:00:01:8d
> MSR register 0x1169 => 00:00:00:00:00:00:00:32
> MSR register 0x116a => 00:00:00:00:00:00:00:03
> MSR register 0x116b => 00:00:00:00:00:00:01:a6
> MSR register 0x116c => 00:00:00:00:00:00:03:34
> MSR register 0x1152 => e7:2c:b3:9f:fb:ff:ff:ff
> MSR register 0x1153 => 04:06:0c:09:fe:ff:ff:ff
>
> Which is the same data as reported by msr-tools.
>

And the output of sensors with H.W. driver built-in:

root@cb01:~# sensors
acpitz-virtual-0
Adapter: Virtual device
temp1: +48.8?C (crit = +94.8?C)

via-cputemp-isa-0000
Adapter: ISA adapter
Core 0: +50.0?C

Which looks reasonable to me, on this machine.

Mike
> Mike