Your message dated Sat, 03 Apr 2010 14:15:16 +0000
with message-id <[email protected]>
and subject line Bug#538950: fixed in cpufreqd 2.4.1-1
has caused the Debian Bug report #538950,
regarding Sensors plugin does not obey "ignore" configuration line
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact [email protected]
immediately.)


-- 
538950: http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=538950
Debian Bug Tracking System
Contact [email protected] with problems
--- Begin Message ---
Package: cpufreqd
Version: 2.3.3-4
Tags: patch

I was requested on the Ubuntu bug tracker (
https://bugs.launchpad.net/bugs/403868 )  to send this bug in here, because
Debian and Ubuntu share the same package version (which is also the same as
upstream release).

Current behavior: The sensors plugin does not honor the "ignore" line in
sensors.conf. This makes it impossible to choose between two features with
the same name on different chips.
My sensors.conf:
chip "it8716-*"
chip "k8temp-*"
ignore temp1

Both chips have a feature named temp1, but the one for k8temp is garbage.
Cpufreqd.conf line: sensor=temp1:55-100
On my system, cpufreqd picks the first feature named "temp1" it finds, which
is the one attached to k8temp. There is no way to make it pick the it8716
chip. So you get in the log something like "sensor_evaluate          :
called 45.000-50.000 [temp1:-47.000]"

Expected behavior: if a certain feature on a chip is set to "ignore" in
sensors.conf, cpufreqd's sensor plugin will not use it.

A patch is attached that fixes the problem for me. I have reported the bug
upstream but I'm not sure how often it is updated
--- src/cpufreqd_sensors.c	21 Jul 2008 07:56:28 -0000	1.12
+++ src/cpufreqd_sensors.c	24 Jul 2009 04:24:23 -0000
@@ -171,6 +171,12 @@
 			} else if (strncmp(feat->name, name, MAX_STRING_LEN) != 0) {
 				continue;
 
+			/* not ignored? */
+			} else if(sensors_get_ignored(*chip, feat->number) == 0) {
+				clog(LOG_INFO, "feature %s on chip set to ignore, skipping\n",
+						feat->name);
+ 				continue;
+
 			/* cache it */
 			} else if ((ret = calloc(1, sizeof(struct sensors_monitor))) != NULL) {
 				clog(LOG_DEBUG, "Creating new sensors_monitor for %s\n",

--- End Message ---
--- Begin Message ---
Source: cpufreqd
Source-Version: 2.4.1-1

We believe that the bug you reported is fixed in the latest version of
cpufreqd, which is due to be installed in the Debian FTP archive:

cpufreqd_2.4.1-1.debian.tar.gz
  to main/c/cpufreqd/cpufreqd_2.4.1-1.debian.tar.gz
cpufreqd_2.4.1-1.dsc
  to main/c/cpufreqd/cpufreqd_2.4.1-1.dsc
cpufreqd_2.4.1-1_amd64.deb
  to main/c/cpufreqd/cpufreqd_2.4.1-1_amd64.deb
cpufreqd_2.4.1.orig.tar.bz2
  to main/c/cpufreqd/cpufreqd_2.4.1.orig.tar.bz2



A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed.  If you
have further comments please address them to [email protected],
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Mattia Dongili <[email protected]> (supplier of updated cpufreqd package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing [email protected])


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.8
Date: Sun, 28 Mar 2010 20:29:31 +0900
Source: cpufreqd
Binary: cpufreqd
Architecture: source amd64
Version: 2.4.1-1
Distribution: unstable
Urgency: low
Maintainer: Mattia Dongili <[email protected]>
Changed-By: Mattia Dongili <[email protected]>
Description: 
 cpufreqd   - fully configurable daemon for dynamic frequency and voltage scali
Closes: 454617 538546 538950 547483
Changes: 
 cpufreqd (2.4.1-1) unstable; urgency=low
 .
   * New upstream release (Closes: #547483), now supports libsensors4
     (Closes: #454617) and ignores ignored features (Closes: #538950).
   * Switch to dpkg-source 3.0 (quilt) format (Closes: #538546).
   * Remove patch merged upstream
Checksums-Sha1: 
 8c24a0d945c75b59456538d00a7c6e7e25e123ae 1117 cpufreqd_2.4.1-1.dsc
 0fa0349d139a5aa2f5b14f5cc8397c9a5b5f1ef2 66334 cpufreqd_2.4.1.orig.tar.bz2
 a12e98e5f538f42e4fd28f4e53b8a9829bb1ca17 10084 cpufreqd_2.4.1-1.debian.tar.gz
 b12eb095a5c29ad90cbc8a7d975f3177b618f56b 88570 cpufreqd_2.4.1-1_amd64.deb
Checksums-Sha256: 
 578eece9c8ce1c4c587b81fecd2d4b7df817fd84f99dd82c9f1a18ba01412b4d 1117 
cpufreqd_2.4.1-1.dsc
 43336e43ec84c8330267bd79353c012b359072947fb7a7bceb97eb31b4c66f24 66334 
cpufreqd_2.4.1.orig.tar.bz2
 8d07aab14d6c38211ee534667e48c6443a9259f466a74a2ef7157fe2b2b31a32 10084 
cpufreqd_2.4.1-1.debian.tar.gz
 1e6613e3ef9ce30cbb4d0f8dcc4abc573919c45fada7f9f50ff02c8ec5315474 88570 
cpufreqd_2.4.1-1_amd64.deb
Files: 
 8c135ceea6d882cf668e883a3e77f519 1117 admin optional cpufreqd_2.4.1-1.dsc
 56eaa5b5e36f6f0b754e659620f7a3ed 66334 admin optional 
cpufreqd_2.4.1.orig.tar.bz2
 77a71e790d6296582fdf7fe2b534448a 10084 admin optional 
cpufreqd_2.4.1-1.debian.tar.gz
 dbe8080866ccd55908a09b308ee274e0 88570 admin optional 
cpufreqd_2.4.1-1_amd64.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)

iEYEARECAAYFAkuvQQMACgkQgpRPaOotLEGOQgCffAb3fDOBaahG8LCGDpBliXQc
0UkAoLvt8bljJZG7iq0jrbafvZRsOK9u
=Rc4i
-----END PGP SIGNATURE-----



--- End Message ---

Reply via email to