From owner-cvs-all@FreeBSD.ORG Thu May 25 15:46:40 2006 Return-Path: X-Original-To: cvs-all@freebsd.org Delivered-To: cvs-all@freebsd.org Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125]) by hub.freebsd.org (Postfix) with ESMTP id 5C37516A53C; Thu, 25 May 2006 15:46:40 +0000 (UTC) (envelope-from imp@bsdimp.com) Received: from harmony.bsdimp.com (vc4-2-0-87.dsl.netrack.net [199.45.160.85]) by mx1.FreeBSD.org (Postfix) with ESMTP id C80C743D58; Thu, 25 May 2006 15:46:39 +0000 (GMT) (envelope-from imp@bsdimp.com) Received: from localhost (localhost.village.org [IPv6:::1] (may be forged)) by harmony.bsdimp.com (8.13.4/8.13.4) with ESMTP id k4PFj580033149; Thu, 25 May 2006 09:45:06 -0600 (MDT) (envelope-from imp@bsdimp.com) Date: Thu, 25 May 2006 09:45:05 -0600 (MDT) Message-Id: <20060525.094505.-924281119.imp@bsdimp.com> To: scottl@samsco.org From: "M. Warner Losh" In-Reply-To: <4475CB00.30808@samsco.org> References: <200605241726.k4OHQG59037586@repoman.freebsd.org> <20060525111035.GI27819@FreeBSD.org> <4475CB00.30808@samsco.org> X-Mailer: Mew version 4.2 on Emacs 21.3 / Mule 5.0 (SAKAKI) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: cvs-src@freebsd.org, glebius@freebsd.org, src-committers@freebsd.org, cvs-all@freebsd.org Subject: Re: cvs commit: src/sys/dev/pccbb pccbb.c X-BeenThere: cvs-all@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: CVS commit messages for the entire tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 May 2006 15:46:47 -0000 In message: <4475CB00.30808@samsco.org> Scott Long writes: : Gleb Smirnoff wrote: : > On Wed, May 24, 2006 at 05:26:16PM +0000, Warner Losh wrote: : > W> imp 2006-05-24 17:26:16 UTC : > W> : > W> FreeBSD src repository : > W> : > W> Modified files: : > W> sys/dev/pccbb pccbb.c : > W> Log: : > W> Suspend the children before we turn off card events in hardware. This : > W> was done, I believe, to work around some cards having issues in the : > W> suspend case. I think that this helped my Sony VAIO TS505 work better : > W> when it had certain wireless cards in it and I did a apm -z. I've not : > ^^^^^^^ : > W> tested suspend/resume on other laptops in a long time, so I hope this : > ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ : > W> doesn't cause greif. Please let me know if it does. : > ^^^^^^^^^^^^^^^^^^^ : > : > Is this a joke, that I can't understand? :( : > : > This file has broken APM suspend since revision 1.122, which was made : > almost a year ago. I've told that several times, and I still hope that : > this will be fixed before RELENG_7. The problem is not in suspending : > children, since suspend is broken w/o any cards in slot. The problem is : > in DELAY -> tsleep() conversion. The latter is not working, when : > system is suspending. : > : : Maybe it would help if Warner published patches and called for testers : before committing stuff like this? No. that wouldn't help. Before the 1.122 commit, I circulated patches for weeks. No one reported the problem. By the time the problem came to light, there were already other commits to the file. More circulation of patches won't help. Since RELENG_7 is a year away at this point, there's plenty of time to get it right. It will get fixed before then, or I'll commit a workaround in the cbb code to for that bug in the ACPI/APM code. I need actual hardware that is known to exhibt the problem. Toward that vain, I'm going through my collection of really old laptops and trying each one out to see if I can get it to fail. This process takes about 2 hours per laptop, or longer, because I have to upgrade them, find the missing power supply, missing video dongles, etc. So if you want me to work on this at all, hire me as a consultant, or lighten up. It will get done. Warner