From owner-freebsd-acpi@FreeBSD.ORG Tue Mar 20 07:06:18 2007 Return-Path: X-Original-To: acpi@freebsd.org 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 593E516A401; Tue, 20 Mar 2007 07:06:18 +0000 (UTC) (envelope-from rea-fbsd@codelabs.ru) Received: from pobox.codelabs.ru (pobox.codelabs.ru [144.206.177.45]) by mx1.freebsd.org (Postfix) with ESMTP id 0BF6C13C459; Tue, 20 Mar 2007 07:06:17 +0000 (UTC) (envelope-from rea-fbsd@codelabs.ru) Received: from codelabs.ru (pobox.codelabs.ru [144.206.177.45]) by pobox.codelabs.ru with esmtpsa (TLSv1:AES256-SHA:256) id 1HTYQB-0001Tp-MK; Tue, 20 Mar 2007 10:06:16 +0300 Date: Tue, 20 Mar 2007 10:06:11 +0300 From: Eygene Ryabinkin To: Jung-uk Kim , Nate Lawson Message-ID: <20070320070610.GE96806@codelabs.ru> References: <200703161256.23996.jkim@FreeBSD.org> <53B52415C756A84E8A169F0E3673A32925FF83@IMCSRV6.MITRE.ORG> <20070319163113.GB96806@codelabs.ru> <45FEF6A6.30209@root.org> <45FF3239.7080808@root.org> <200703161256.23996.jkim@FreeBSD.org> <20070319163113.GB96806@codelabs.ru> <200703191535.27842.jkim@FreeBSD.org> <200703191538.32383.jkim@FreeBSD.org> MIME-Version: 1.0 Content-Type: text/plain; charset=koi8-r Content-Disposition: inline In-Reply-To: <45FF3239.7080808@root.org> <200703191538.32383.jkim@FreeBSD.org> Sender: rea-fbsd@codelabs.ru X-Spam-Status: No, score=-3.4 required=4.0 tests=ALL_TRUSTED,AWL,BAYES_00 Cc: acpi@freebsd.org, current@freebsd.org Subject: Re: [HEADSUP] ACPI-CA 20070126 import 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: Tue, 20 Mar 2007 07:06:18 -0000 Jung-uk, Nate, good day. Mon, Mar 19, 2007 at 06:00:41PM -0700, Nate Lawson wrote: > > > I think it's not the acpi-ca patch, it's the recent acpi_ec.c change. > > Try it with rev 1.68 of that file. > > > > Or better, try with rev 1.71 that I just committed. Fixed a bug where > we weren't re-enabling the GPE, which could cause exactly this issue. Nate, you're completely right: I've applied the 1.70 -> 1.71 patch on top of the Jung-uk's one and it enabled the backlight controls to work again. But the "queued" behaviour were still there. Mon, Mar 19, 2007 at 03:38:30PM -0400, Jung-uk Kim wrote: > > As a quick test, can you try this? > > --- src/sys/dev/acpica/Osd/OsdSchedule.c > +++ src/sys/dev/acpica/Osd/OsdSchedule.c > pri = 5; > break; > case OSL_NOTIFY_HANDLER: > - pri = 3; > + pri = 15; > break; > case OSL_DEBUGGER_THREAD: > pri = 0; Jung-uk, and this change enabled the backlignt to go up or down after the second press of any F-button, so the "event queue" length got down to one. Again, your patch was applied to the yesterday's clean FreeBSD HEAD and the Nate's one for the acpi_ec.c 1.70 -> 1.71 was applied on top of it. > BTW, it is not really a patch, i.e., I just typed it in. :-) All I > want you to test is to change 3 -> 15. > > I hope I didn't confuse you. No, you're not confused me at all -- I am doing such "patches" almost regularily ;)) Thank you both for your response and work! -- Eygene