Re: powerd adaptive mode latching

Andrew Atrens wrote:
 > # cvs diff -u acpi_perf.c
 > Index: acpi_perf.c
 > ===================================================================
 > RCS file: /usr/repo/freebsd/src/sys/dev/acpica/acpi_perf.c,v
 > retrieving revision 1.26
 > diff -u -r1.26 acpi_perf.c
 > - --- acpi_perf.c 22 Mar 2007 18:16:40 -0000      1.26
 > +++ acpi_perf.c 16 Nov 2007 20:11:02 -0000
 > _at__at_ -300,9 +300,9 _at__at_
 >                         continue;
 > 
 >                 /* Check for duplicate entries */
 > - -               if (count > 0 &&
 > - -                   sc->px_states[count - 1].core_freq ==
 > - -                       sc->px_states[count].core_freq)
 > +               if (count > 0 &&
 > +                   CPUFREQ_CMP(sc->px_states[count - 1].core_freq,
 > +                               sc->px_states[count].core_freq))
 >                         continue;
 > 
 >                 count++;
 > 
 > 
 > It's so simple that it's kind of silly that it (or something like it)
 > hasn't been committed yet...
 > 
 > Anyone out there with a commit bit interested in submitting this ?

It seems that nobody is responsible for those things right
now (or nobody feels responsible).  My PR bin/115513 is
also still open.  I'm using that patch for a long time
already and wouldn't want to live without it; it improves
interactive behaviour a lot on my notebook with powerd.

Best regards
Oliver

-- 
Oliver Fromme, secnetix GmbH & Co. KG, Marktplatz 29, 85567 Grafing b. M.
Handelsregister: Registergericht Muenchen, HRA 74606,  Geschäftsfuehrung:
secnetix Verwaltungsgesellsch. mbH, Handelsregister: Registergericht Mün-
chen, HRB 125758,  Geschäftsführer: Maik Bachmann, Olaf Erb, Ralf Gebhart

FreeBSD-Dienstleistungen, -Produkte und mehr:  http://www.secnetix.de/bsd

"Being really good at C++ is like being really good
at using rocks to sharpen sticks."
        -- Thant Tessman