From owner-freebsd-acpi@FreeBSD.ORG Wed Sep 5 17:41:27 2012 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 45F92106566B; Wed, 5 Sep 2012 17:41:27 +0000 (UTC) (envelope-from kob6558@gmail.com) Received: from mail-wi0-f170.google.com (mail-wi0-f170.google.com [209.85.212.170]) by mx1.freebsd.org (Postfix) with ESMTP id A29168FC19; Wed, 5 Sep 2012 17:41:26 +0000 (UTC) Received: by wibhq12 with SMTP id hq12so4603494wib.1 for ; Wed, 05 Sep 2012 10:41:20 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type; bh=oy7uI/TOF6Yg0E2ahzM+1U19hyeXGtHexYUR7Vy3FrM=; b=sy3/QixYFv+hrtv8oq9dRJCyNU5CI9vUFfOmGo2sMCAfvmiAUL/i4i+8JDSpIi6+Dr oQeL5anPLQzK/bCBKrDhh6Bhvxt9K2/8Xr4e58Mqz1O4n01BA+07wxfAQoiObSM9unSI IpJtFrXRu2oGywP5uL4CYYeNnwxUbGopZKQXqjornlCS5LpcjOcRoolSxmhFY7ZAOlvi 85moKn5244NkERiM+sUvaOHniyiF4VyhmFep3AAsAuzzgUIXER4DlgDvfc2FcfaIfa16 0bclbyL+LE89uap27yLfzvW6Bj0DpVR7REId2dI6K11CZxvUQ+bj4RPNuIgu7CUekz65 PV7A== MIME-Version: 1.0 Received: by 10.180.75.209 with SMTP id e17mr39926303wiw.0.1346866880447; Wed, 05 Sep 2012 10:41:20 -0700 (PDT) Received: by 10.223.63.76 with HTTP; Wed, 5 Sep 2012 10:41:20 -0700 (PDT) In-Reply-To: <50477E9F.7030604@FreeBSD.org> References: <4FF95F79.30309@FreeBSD.org> <1341935370.4493.4.camel@powernoodle> <4FFFD891.2060109@FreeBSD.org> <5046F105.80706@FreeBSD.org> <504779D8.4030909@FreeBSD.org> <50477E9F.7030604@FreeBSD.org> Date: Wed, 5 Sep 2012 10:41:20 -0700 Message-ID: From: Kevin Oberman To: Andriy Gapon Content-Type: text/plain; charset=UTF-8 Cc: freebsd-acpi@freebsd.org Subject: Re: improve cx_lowest logic X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 05 Sep 2012 17:41:27 -0000 On Wed, Sep 5, 2012 at 9:32 AM, Andriy Gapon wrote: > on 05/09/2012 19:23 Kevin Oberman said the following: >> On Wed, Sep 5, 2012 at 9:12 AM, Andriy Gapon wrote: >>> on 05/09/2012 18:17 Kevin Oberman said the following: >>>> Thanks so much! This should finally make Cx states work on my >>>> ThinkPad! I really appreciate it. Guess it's time to do my weekly >>>> upgrade of this system. >>> >>> I haven't sneaked in that other commit :-( >> >> Oops! :-( >> >> Oh, well. At least it should make it to /base/stable/9 soon. Right??? >> (I only run release/ or releng/ or for an occasional test.) >> > > It's already in stable/9 :) Ahh! I now see C3/109, but I see some strange behavior. When on AC power, only C1/1 and C2/104 are available, but cx_lowest is C3, even though C3 is not available. If I switch to battery, C1/1, C2/80 and C3/109 are available (???), but cx_lowest is set to C2. I find the Cx value sets a bit odd, but the setting of cx_lowest appears to be a bug, at least to me. I can manually set cx_lowest to C3 and I actually use C3. My suspicion is that there is either a race or a logic issue where x_lowest is reset to the lowest value before the available Cx values are set, so cx_lowest is always set the the lowest Cx state from the previous power configuration. (This is a guess, but it fits what I am seeing very well.) -- R. Kevin Oberman, Network Engineer E-mail: kob6558@gmail.com