Re: [PATCH] hwmon: (coretemp) Fix oops on driver load

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



----- Original Message -----
From: "Jean Delvare" <khali@xxxxxxxxxxxx>
To: undersys@xxxxxxxxxxxx
Cc: lm-sensors@xxxxxxxxxxxxxx, "Durgadoss R" <Durgadoss.r@xxxxxxxxx>, "guenter roeck" <guenter.roeck@xxxxxxxxxxxx>
Sent: Tuesday, 6 December, 2011 5:53:41 PM
Subject: Re: [PATCH] hwmon: (coretemp) Fix oops on driver load

On Tue, 06 Dec 2011 07:26:59 +0800 (HKT), undersys wrote:
> The microcode utility I am using is :-
> http://urbanmyth.org/microcode
> 
> The version of micro code I am using is "microcode-20110915.tgz" :-
> http://downloadcenter.intel.com/Detail_Desc.aspx?agr=Y&DwnldID=20429&lang=eng

Did you have to do that with previous kernels too?

undersys ->  I Did not ever need to use the microcode utility.
             Maybe i should in future...

> CPU is :-
> Intel Xeon sossaman, model=14, stepping=4. 
> I don't have the SPEC number on hand let me know if you need more information.

Is this a single or multi-cpu system?

undersys -> multi-cpu (dual socket)

I would like to see the contents of /proc/cpuinfo.

undersys -> attached

> Micro code was working on 2.6.29 and 2.6.39 , then I jumped to 3.0.6.

I admit I don't quite get it for now. While the device probing logic
was modified a lot in the coretemp driver of kernel 3.0, the microcode
check did not change, so I just don't get why the new driver would
behave differently from the previous ones in this respect.

For testing purposes, I made the 2.6.39 version of the coretemp driver
available as a standalone driver:
http://khali.linux-fr.org/devel/misc/coretemp_old/
(Instructions at http://khali.linux-fr.org/devel/misc/INSTALL )

I would like you to test it on your 3.0 kernel and report whether it is
working or not. You'll have to rmmod the original coretemp driver
before loading coretemp_old, you can't have both loaded at once.

If coretemp_old works, then please rmmod it and modprobe coretemp
again, and see if it then works.

undersys ->

I built your module(coretemp_old), works fine no kernel opps.
The only lines printed to "dmesg" are :-
[  133.603204] coretemp coretemp.0: Errata AE18 not fixed, update BIOS or microcode of the CPU!
[  133.603277] coretemp coretemp.1: Errata AE18 not fixed, update BIOS or microcode of the CPU!
[  133.603329] coretemp coretemp.2: Errata AE18 not fixed, update BIOS or microcode of the CPU!
[  133.603379] coretemp coretemp.3: Errata AE18 not fixed, update BIOS or microcode of the CPU!

If i remove it and then attempt to re-probe the new one, the new one fails.


-- 
Jean Delvare
http://khali.linux-fr.org/wishlist.html


Hi Jean,

Answers are inline.

Thanks for your time!
# cat /proc/cpuinfo 
processor	: 0
vendor_id	: GenuineIntel
cpu family	: 6
model		: 14
model name	: Intel(R) Dual Pentium(R) M CPU   000  @ 1.83GHz
stepping	: 4
cpu MHz		: 996.000
cache size	: 2048 KB
physical id	: 0
siblings	: 2
core id		: 0
cpu cores	: 2
apicid		: 0
initial apicid	: 0
fdiv_bug	: no
hlt_bug		: no
f00f_bug	: no
coma_bug	: no
fpu		: yes
fpu_exception	: yes
cpuid level	: 6
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx constant_tsc bts pni monitor vmx est tm2 xtpr dts
bogomips	: 3667.09
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 32 bits virtual
power management:

processor	: 1
vendor_id	: GenuineIntel
cpu family	: 6
model		: 14
model name	: Intel(R) Dual Pentium(R) M CPU   000  @ 1.83GHz
stepping	: 4
cpu MHz		: 996.000
cache size	: 2048 KB
physical id	: 3
siblings	: 2
core id		: 0
cpu cores	: 2
apicid		: 6
initial apicid	: 6
fdiv_bug	: no
hlt_bug		: no
f00f_bug	: no
coma_bug	: no
fpu		: yes
fpu_exception	: yes
cpuid level	: 6
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx constant_tsc bts pni monitor vmx est tm2 xtpr dts
bogomips	: 3666.16
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 32 bits virtual
power management:

processor	: 2
vendor_id	: GenuineIntel
cpu family	: 6
model		: 14
model name	: Intel(R) Dual Pentium(R) M CPU   000  @ 1.83GHz
stepping	: 4
cpu MHz		: 996.000
cache size	: 2048 KB
physical id	: 0
siblings	: 2
core id		: 1
cpu cores	: 2
apicid		: 1
initial apicid	: 1
fdiv_bug	: no
hlt_bug		: no
f00f_bug	: no
coma_bug	: no
fpu		: yes
fpu_exception	: yes
cpuid level	: 6
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx constant_tsc bts pni monitor vmx est tm2 xtpr dts
bogomips	: 3666.16
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 32 bits virtual
power management:

processor	: 3
vendor_id	: GenuineIntel
cpu family	: 6
model		: 14
model name	: Intel(R) Dual Pentium(R) M CPU   000  @ 1.83GHz
stepping	: 4
cpu MHz		: 996.000
cache size	: 2048 KB
physical id	: 3
siblings	: 2
core id		: 1
cpu cores	: 2
apicid		: 7
initial apicid	: 7
fdiv_bug	: no
hlt_bug		: no
f00f_bug	: no
coma_bug	: no
fpu		: yes
fpu_exception	: yes
cpuid level	: 6
wp		: yes
flags		: fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe nx constant_tsc bts pni monitor vmx est tm2 xtpr dts
bogomips	: 3666.16
clflush size	: 64
cache_alignment	: 64
address sizes	: 36 bits physical, 32 bits virtual
power management:

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux