From owner-freebsd-current@FreeBSD.ORG Wed Sep 28 10:57:59 2011 Return-Path: Delivered-To: freebsd-current@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id DA4A61065674; Wed, 28 Sep 2011 10:57:59 +0000 (UTC) (envelope-from taku@tackymt.homeip.net) Received: from basalt.tackymt.homeip.net (unknown [IPv6:2001:3e0:577:0:20d:61ff:fecc:2253]) by mx1.freebsd.org (Postfix) with ESMTP id 627508FC1A; Wed, 28 Sep 2011 10:57:59 +0000 (UTC) Received: from basalt.tackymt.homeip.net (localhost [127.0.0.1]) by basalt.tackymt.homeip.net (Postfix) with ESMTP id 6B7FD83DE; Wed, 28 Sep 2011 19:57:58 +0900 (JST) X-Virus-Scanned: amavisd-new at tackymt.homeip.net Received: from localhost ([127.0.0.1]) by basalt.tackymt.homeip.net (basalt.tackymt.homeip.net [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 2TB9SDxVdR3K; Wed, 28 Sep 2011 19:57:56 +0900 (JST) Received: from biotite.tackymt.homeip.net (i114-186-112-119.s41.a019.ap.plala.or.jp [114.186.112.119]) by basalt.tackymt.homeip.net (Postfix) with ESMTP; Wed, 28 Sep 2011 19:57:56 +0900 (JST) Date: Wed, 28 Sep 2011 19:57:56 +0900 From: Taku YAMAMOTO To: Gavin Atkinson Message-Id: <20110928195756.5042d262.taku@tackymt.homeip.net> In-Reply-To: <1317125181.95805.13.camel@buffy.york.ac.uk> References: <8bce9b8a86d5c7a83095d8e58f794f64@i-pi.pl> <9e25323fa87abb93af1946c9ed2c399e@i-pi.pl> <1317125181.95805.13.camel@buffy.york.ac.uk> X-Mailer: Sylpheed 3.1.0 (GTK+ 2.22.1; i386-portbld-freebsd9.0) Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="Multipart=_Wed__28_Sep_2011_19_57_56_+0900_5AF0L+23D6u6bcgr" Cc: "crsnet.pl" , Adrian Chadd , freebsd-current@freebsd.org, Hans Petter Selasky Subject: Re: [Solved] FreeBSD 9-Beta3 on X300 2 problems X-BeenThere: freebsd-current@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Discussions about the use of FreeBSD-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 28 Sep 2011 10:58:00 -0000 This is a multi-part message in MIME format. --Multipart=_Wed__28_Sep_2011_19_57_56_+0900_5AF0L+23D6u6bcgr Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit On Tue, 27 Sep 2011 13:06:21 +0100 Gavin Atkinson wrote: > On Tue, 2011-09-27 at 19:53 +0800, Adrian Chadd wrote: > > Hans, > > > > Why haven't those patches been committed? > > This patch is an absolute hack, and shouldn't be committed as it is. > > I would, however, appreciate some help in determining the correct > solution. The solution may well involve not suspending/resuming hpet(4) > or the other timers on the normal DEVICE_SUSPEND()/DEVICE_RESUME() path > but instead doing them as the last thing to be suspended, or it may Like the attached patches do? > instead involve reworking the USB code (and potentially other code) to > not need to sleep during suspend/resume. I don't know the right > solution, but would really like to work with somebody who does. > > Please also see the thread "Choosing between DELAY(useconds) and > pause()" on -current. > > Gavin > > _______________________________________________ > freebsd-current@freebsd.org mailing list > http://lists.freebsd.org/mailman/listinfo/freebsd-current > To unsubscribe, send any mail to "freebsd-current-unsubscribe@freebsd.org" -- -|-__ YAMAMOTO, Taku | __ < - A chicken is an egg's way of producing more eggs. - --Multipart=_Wed__28_Sep_2011_19_57_56_+0900_5AF0L+23D6u6bcgr Content-Type: text/plain; name="subr_bus-revsusp.patch" Content-Disposition: attachment; filename="subr_bus-revsusp.patch" Content-Transfer-Encoding: 7bit --- sys/kern/subr_bus.c.orig 2010-12-02 13:28:01.000000000 +0900 +++ sys/kern/subr_bus.c 2010-12-08 12:37:05.524727855 +0900 @@ -3385,15 +3385,13 @@ int bus_generic_suspend(device_t dev) { int error; - device_t child, child2; + device_t child; - TAILQ_FOREACH(child, &dev->children, link) { + TAILQ_FOREACH_REVERSE(child, &dev->children, device_list, link) { error = DEVICE_SUSPEND(child); if (error) { - for (child2 = TAILQ_FIRST(&dev->children); - child2 && child2 != child; - child2 = TAILQ_NEXT(child2, link)) - DEVICE_RESUME(child2); + while ((child = TAILQ_NEXT(child, link)) != NULL) + DEVICE_RESUME(child); return (error); } } --Multipart=_Wed__28_Sep_2011_19_57_56_+0900_5AF0L+23D6u6bcgr Content-Type: text/plain; name="acpi-hpet-reorder.patch" Content-Disposition: attachment; filename="acpi-hpet-reorder.patch" Content-Transfer-Encoding: 7bit --- sys/dev/acpica/acpi.c.orig 2010-07-16 08:24:06.000000000 +0900 +++ sys/dev/acpica/acpi.c 2010-07-21 20:10:59.056243391 +0900 @@ -1702,7 +1702,7 @@ acpi_probe_order(ACPI_HANDLE handle, int *order = 1; else if (acpi_MatchHid(handle, "PNP0C01") || acpi_MatchHid(handle, "PNP0C02")) *order = 2; - else if (acpi_MatchHid(handle, "PNP0C09")) + else if (acpi_MatchHid(handle, "PNP0C09") || acpi_MatchHid(handle, "PNP0103")) *order = 3; else if (acpi_MatchHid(handle, "PNP0C0F")) *order = 4; --Multipart=_Wed__28_Sep_2011_19_57_56_+0900_5AF0L+23D6u6bcgr--