From owner-cvs-all Fri Nov 2 1: 2:15 2001 Delivered-To: cvs-all@freebsd.org Received: from mail6.speakeasy.net (mail6.speakeasy.net [216.254.0.206]) by hub.freebsd.org (Postfix) with ESMTP id 7957F37B40F for ; Fri, 2 Nov 2001 01:01:34 -0800 (PST) Received: (qmail 78700 invoked from network); 2 Nov 2001 09:01:33 -0000 Received: from unknown (HELO laptop.baldwin.cx) ([64.81.54.73]) (envelope-sender ) by mail6.speakeasy.net (qmail-ldap-1.03) with SMTP for ; 2 Nov 2001 09:01:33 -0000 Message-ID: X-Mailer: XFMail 1.4.0 on FreeBSD X-Priority: 3 (Normal) Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 8bit MIME-Version: 1.0 In-Reply-To: <200111011634.fA1GY8M18688@freefall.freebsd.org> Date: Fri, 02 Nov 2001 01:01:33 -0800 (PST) From: John Baldwin To: Mitsuru IWASAKI Subject: RE: cvs commit: src/sys/conf files src/sys/dev/acpica acpi.c src Cc: cvs-all@FreeBSD.org, cvs-committers@FreeBSD.org Sender: owner-cvs-all@FreeBSD.ORG Precedence: bulk List-ID: List-Archive: (Web Archive) List-Help: (List Instructions) List-Subscribe: List-Unsubscribe: X-Loop: FreeBSD.ORG On 01-Nov-01 Mitsuru IWASAKI wrote: > iwasaki 2001/11/01 08:34:08 PST > > Modified files: > sys/conf files > sys/dev/acpica acpi.c > sys/dev/syscons syscons.c > sys/i386/apm apm.c > sys/i386/isa clock.c > sys/sys kernel.h > Added files: > sys/kern subr_power.c > sys/sys power.h > Log: > Some fix for the recent apm module changes. > - Now that apm loadable module can inform its existence to other kernel > components (e.g. i386/isa/clock.c:startrtclock()'s TCS hack). > - Exchange priority of SI_SUB_CPU and SI_SUB_KLD for above purpose. > - Add simple arbitration mechanism for APM vs. ACPI. This prevents > the kernel enables both of them. > - Remove obsolete `#ifdef DEV_APM' related code. > - Add abstracted interface for Powermanagement operations. Public apm(4) > functions, such as apm_suspend(), should be replaced new interfaces. > Currently only power_pm_suspend (successor of apm_suspend) is > implemented. > > Reviewed by: peter, arch@ and audit@ Hmm, I actually really don't like swapping those SYSINIT's. Probably what you should do is do whwat the alpha does and initalize the timecounters during the SI_SUB_CLOCKS sysinit. If SI_SUB_CLOCKS is before SI_SUB_KLD, then you should create a SI_SUB_TC after SI_SUB_KLD and initialize timecounters there. Logically, we need the CPU up and initialized before we start trying to figure out what modules we have and how to init them. At least IMO. -- John Baldwin -- http://www.FreeBSD.org/~jhb/ PGP Key: http://www.baldwin.cx/~john/pgpkey.asc "Power Users Use the Power to Serve!" - http://www.FreeBSD.org/ To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe cvs-all" in the body of the message