i386/145718: [est] [patch] fix freq calculation from MSR for CPUs with fractional multipliers

Andre Albsmeier Andre.Albsmeier at siemens.com
Fri Apr 26 16:50:02 UTC 2013


The following reply was made to PR i386/145718; it has been noted by GNATS.

From: Andre Albsmeier <Andre.Albsmeier at siemens.com>
To: bug-followup at FreeBSD.org, Andre.Albsmeier at siemens.com
Cc:  
Subject: Re: i386/145718: [est] [patch] fix freq calculation from MSR for
 CPUs with fractional multipliers
Date: Fri, 26 Apr 2013 18:34:30 +0200

 And another additional patch to make this work with an E6600 CPU
 (266 MHz clock). While we are here, fix the unit of the voltage
 as well (Mv -> mV).
 
 --- est.c.ORI	2013-04-26 18:30:02.000000000 +0200
 +++ est.c	2013-04-26 18:30:02.000000000 +0200
 @@ -1192,6 +1192,7 @@
  	switch (bus) {
  	case 100:
  	case 133:
 +	case 267:
  	case 333:
  		return (1);
  	default:
 @@ -1252,7 +1253,7 @@
  	fp[0].volts = volts;
  	fp[0].id16 = id;
  	fp[0].power = CPUFREQ_VAL_UNKNOWN;
 -	device_printf(dev, "Guessed high setting of %d MHz @ %d Mv\n", freq,
 +	device_printf(dev, "Guessed high setting of %d MHz @ %d mV\n", freq,
  	    volts);
  
  	/* Second, the low frequency. */
 @@ -1269,7 +1270,7 @@
  	fp[1].volts = volts;
  	fp[1].id16 = id;
  	fp[1].power = CPUFREQ_VAL_UNKNOWN;
 -	device_printf(dev, "Guessed low setting of %d MHz @ %d Mv\n", freq,
 +	device_printf(dev, "Guessed low setting of %d MHz @ %d mV\n", freq,
  	    volts);
  
  	/* Table is already terminated due to M_ZERO. */


More information about the freebsd-i386 mailing list