Re: Patch[1/1] Adding Core Thermal Threshold Support to_Coretemp

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

 



Hi Gong,

> > +	/* Mask the thermal vector in the lapic */
> > +	l = apic_read(APIC_LVTTHMR);
> > +	apic_write(APIC_LVTTHMR, l | APIC_LVT_MASKED);
> > +
> > +	rdmsr_on_cpu(data->id, MSR_IA32_THERM_INTERRUPT,&eax,&edx);
> 
> apic_write and rdmsr_on_cpu maybe don't point to the same CPU.
> Maybe it is a potential issue. The below apic_xxx etc. are same situations.

Yes. Agreed. Figured out that there is a way to execute a function on a
particular CPU, if we know cpu->id. So, shall modify the patch accordingly
and submit it. 
Sorry for the late response. It took some time to find out how to do this.


Thanks,
Durgadoss
_______________________________________________
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