Date: Thu, 08 Nov 2007 17:54:49 -0500 From: "Andrew Atrens" <atrens@nortel.com> To: Simon Barner <barner@FreeBSD.org> Cc: Benjamin Lutz <mail@maxlor.com>, freebsd-current@FreeBSD.org, Matus Harvan <mharvan@inf.ethz.ch> Subject: Re: powerd adaptive mode latching Message-ID: <473393B9.8070605@nortel.com> In-Reply-To: <20071108224313.GA1927@dose.local.invalid> References: <472E9D0B.5080409@csub.edu> <20071106002940.GB1817@styx.ethz.ch> <472FD321.3010509@nortel.com> <20071108132222.GC1674@dose.local.invalid> <47334C71.1010102@nortel.com> <20071108224313.GA1927@dose.local.invalid>
next in thread | previous in thread | raw e-mail | index | archive | help
-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 Simon Barner wrote: >>> Please see kern/114722. The patch from the PR works fine with my >>> T61 (T7300). >>> >>> Funny enough, I contacted re@ to get this into 7.0 only two minutes ago. >>> >>> For the archives, the similar bug described in bin/117375 already seems >>> to be adressed in RELENG_7. >> both pr's are open .. and >> >> releng_7 and head are both at v 1.26 of acpi_perf.c >> >> so, no it's not fixed, *anywhere*. :) > > It's true that both PRs are still open, but: > > 1) kern/114722 should fix your problem (CPUFREQ_CMP takes care of almost > identical frequencies). Have you already had a chance to verify that? > > 2) bin/117375 talks about exactly identical frequencies, which should be > handled by acpi_perf.c (1.26, line 303-306). However, the reporter > (Cc'ed) of that PR runs FreeBSD 6.2-p8 which already contains the > removal of duplicate entries (MFC from acpi_perf.c 1.24). Well both head and releng7 are using 1.26 and the fix is not there. it's as simple as that. Just re-read the patch specified in 114722 and then take look at http://fxr.watson.org/fxr/source/dev/acpica/acpi_perf.c the issue is that the frequencies aren't identical - it's that they are *nearly* identical. here's the CPUFREQ_CMP macro - /usr/include/sys/cpu.h:#define CPUFREQ_CMP(x, y) (abs((x) - (y)) < 25) as you can see the macro is dead simple - it just considers any two frequencies within 25Hz of each other to be identical. in fact when you look at it, and look at my powerd.c patch both are doing the same thing, except that one is doing it in the kernel, the other in user space. I actually like 114722. :) Someone should commit it!!! :) :) :) - --Andrew -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.4 (FreeBSD) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org iD8DBQFHM5O58It2CaCdeMwRArG8AJ9OeS09bkODR2RejneYB/beOjJ+eACdG14j wXH0OYZzPR01KvJQ9TD/M60= =YCg9 -----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?473393B9.8070605>