From owner-freebsd-acpi@FreeBSD.ORG Sun Dec 12 06:22:56 2004 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 9091A16A4CE for ; Sun, 12 Dec 2004 06:22:56 +0000 (GMT) Received: from otter3.centtech.com (moat3.centtech.com [207.200.51.50]) by mx1.FreeBSD.org (Postfix) with ESMTP id 3A10A43D41 for ; Sun, 12 Dec 2004 06:22:56 +0000 (GMT) (envelope-from anderson@centtech.com) Received: from [192.168.42.22] (andersonbox2.centtech.com [192.168.42.22]) by otter3.centtech.com (8.12.3/8.12.3) with ESMTP id iBC6MtOJ014891; Sun, 12 Dec 2004 00:22:55 -0600 (CST) (envelope-from anderson@centtech.com) Message-ID: <41BBE3BE.8090201@centtech.com> Date: Sun, 12 Dec 2004 00:22:54 -0600 From: Eric Anderson User-Agent: Mozilla/5.0 (X11; U; FreeBSD i386; en-US; rv:1.7.3) Gecko/20041110 X-Accept-Language: en-us, en MIME-Version: 1.0 To: Nate Lawson References: <41B4E577.9060502@root.org> <41B50754.10604@centtech.com> <41B50C12.6070103@root.org> <20041208095845.GA896@galgenberg.net> <41B76AF2.3040204@root.org> <20041210180446.GA768@galgenberg.net> <41B9EE4E.8030703@root.org> <20041210205417.GB768@galgenberg.net> <41BB5146.1080102@root.org> In-Reply-To: <41BB5146.1080102@root.org> Content-Type: text/plain; charset=us-ascii; format=flowed Content-Transfer-Encoding: 7bit cc: acpi@freebsd.org Subject: Re: suspend/resume improved? X-BeenThere: freebsd-acpi@freebsd.org X-Mailman-Version: 2.1.1 Precedence: list List-Id: ACPI and power management development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 12 Dec 2004 06:22:56 -0000 Nate Lawson wrote: > Ulrich Spoerlein wrote: > >> On Fri, 10.12.2004 at 10:43:26 -0800, Nate Lawson wrote: >> >>>> Line 444 is the culprit: >>>> ACPI_FLUSH_CPU_CACHE (); >>>> >>>> Status = AcpiHwRegisterWrite (ACPI_MTX_DO_NOT_LOCK, >>>> ACPI_REGISTER_PM1A_CONTROL, PM1AControl); >>>> if (ACPI_FAILURE (Status)) >>>> { >>>> return_ACPI_STATUS (Status); >>>> } >>>> >>>> Putting the loop before AcpiHwRegisterWrite will enter infinite loop, >>>> putting it after it -> Reset >>> >>> >>> Interesting. If you add a "DELAY(10000);" before the register write, >>> does this help? A more likely issue is that we need to write pm1a/b >>> at the same time. Let me think about this and get you more info later. >> >> >> >> It "looks" like the delay does nothing, however I noticed that all three >> LEDs (num/scroll/caps lock) were lighting up for 1-2s. >> >> I then swapped pm1a and pm1b, but nothing changed. > > > That's not quite what I meant. The acpi-ca code splits the write of > SLP_TYP and SLP_EN into two separate writes. I suspect some BIOSes > don't like this. Try the attached patch that combines them. I just tried this patch (my laptop is having similar responses) - no change. I'm off to adding some manual debug code to the acpi source files you mentioned in an earlier email. I can't seem to find Warner's beep code patch though - do you have a pointer? Eric -- ------------------------------------------------------------------------ Eric Anderson Sr. Systems Administrator Centaur Technology When in doubt, mumble; when in trouble, delegate; when in charge, ponder ------------------------------------------------------------------------