> -----Original Message----- > From: R, Durgadoss > Sent: Wednesday, June 08, 2011 3:06 AM > To: Jean Delvare; Guenter Roeck > Cc: lm-sensors@xxxxxxxxxxxxxx; Carsten Emde; Yu, Fenghua > Subject: RE: [PATCH] hwmon: (coretemp) Further relax temperature range > checks > > Hi Jean, > > > 4* Regardless of the technically correct answer to point 3 above, I > > have never seen any multi-core CPU supported by the coretemp > driver > > report different tjmax values for different cores. Did anyone ever > > see this? If not, I would feel comfortable reading tjmax from > > MSR_TEMPERATURE_TARGET only once per package for all models. This > > would make driver initialization faster. > > > > When I tested in 3 of my machines, it did not Show different TjMax. > So, I agree that we should have the TjMax per package. According to SDM, MSR_TEMPERATURE_TARGET scope is thread. Reading it per package is not correct. Coding this MSR based on observation is not right and may cause problem in other cases or in the future. Thanks. -Fenghua _______________________________________________ lm-sensors mailing list lm-sensors@xxxxxxxxxxxxxx http://lists.lm-sensors.org/mailman/listinfo/lm-sensors
