From owner-freebsd-acpi@FreeBSD.ORG Tue Apr 19 20:24:28 2011 Return-Path: Delivered-To: freebsd-acpi@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 0E60F106564A for ; Tue, 19 Apr 2011 20:24:28 +0000 (UTC) (envelope-from jamesbrandongooch@gmail.com) Received: from mail-ww0-f50.google.com (mail-ww0-f50.google.com [74.125.82.50]) by mx1.freebsd.org (Postfix) with ESMTP id 92E038FC1C for ; Tue, 19 Apr 2011 20:24:27 +0000 (UTC) Received: by wwc33 with SMTP id 33so84091wwc.31 for ; Tue, 19 Apr 2011 13:24:26 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=Wz2eLWm+X6biQjSgc5nTlnYv/NkctF3R4NUF/1aDGSE=; b=pM/wi9DcFfq7xAU4pAWRGFASAHRk2T381vJP/7vdUTV/RR4aR2JHBBGLBWYW+Gvb0o Qlgl4qEWduBOwRMRGQ//92n8rgCJHkACuCFKRoqr6BPsvBwyXI3yspV2dCofDULAqlId AFyks3GJqxLryDCY7N8SU11rY2/dIYUSFgLvQ= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=dB7wWKb+cQhOZOSm+gTPc5HwzSWegQgDy38nMR6EmV2oURik7mQqx/2gsQvEQnNRVP MInk8g/SXf+NjaX22mNs7+x+EiDAOv2fDkJprhq7Ih7GNd+nk+TgcPMhMtjQk0UgEIQR 2GxBWYaJsakRQMYDfNSYt0JLJjFK2Boh0RIA0= MIME-Version: 1.0 Received: by 10.216.67.135 with SMTP id j7mr141470wed.54.1303244666236; Tue, 19 Apr 2011 13:24:26 -0700 (PDT) Received: by 10.216.19.13 with HTTP; Tue, 19 Apr 2011 13:24:25 -0700 (PDT) In-Reply-To: References: <20110420003133.4dc391f6.taku@tackymt.homeip.net> <20110420004614.3099947d.taku@tackymt.homeip.net> Date: Tue, 19 Apr 2011 15:24:25 -0500 Message-ID: From: Brandon Gooch To: Warner Losh Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable Cc: freebsd-acpi@freebsd.org Subject: Re: [PATCH] Move scsuspend()/scresume() forward to EVENTHANDLERs 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, 19 Apr 2011 20:24:28 -0000 On Tue, Apr 19, 2011 at 3:02 PM, Warner Losh wrote: > Having chased boogiemen in this area before, I think this patch makes goo= d sense even though it breaks the device model a little. =A0However, the ha= rdware in question really is special on x86... > > Warner Sorry Warner, would you mind elaborating on the above? Do you mean that the patch moves to being too x86-specific for syscons handling? Sorry if I'm being too dense. -Brandon > On Apr 19, 2011, at 12:48 PM, Brandon Gooch wrote: > >> On Tue, Apr 19, 2011 at 10:46 AM, Taku YAMAMOTO wrote: >>> On Wed, 20 Apr 2011 00:31:33 +0900 >>> Taku YAMAMOTO wrote: >>> >>>> A patch is attached. >>> Mailman ate it ;) >>> Here it is. >>> >>> -- >>> -|-__ =A0 YAMAMOTO, Taku >>> =A0| __ < =A0 =A0 >>> >>> =A0 =A0 =A0- A chicken is an egg's way of producing more eggs. - >>> >> >> Throughout testing, I've seen zero regressions. In fact, an issue that >> existed prior to this patch involving minor screen corruption (dirty >> VGA buffers) is gone (Intel Mobile 965 Express). >> >> Thank you so much guys! >> >> -Brandon