Date: Fri, 16 Jul 2010 19:12:45 -0400 From: Jung-uk Kim <jkim@FreeBSD.org> To: "M. Warner Losh" <imp@bsdimp.com> Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org, brde@optusnet.com.au Subject: Re: svn commit: r210005 - head/sys/modules/acpi/acpi Message-ID: <201007161912.53042.jkim@FreeBSD.org> In-Reply-To: <20100716.154216.582095686063971712.imp@bsdimp.com> References: <201007131657.o6DGvq9L097594@svn.freebsd.org> <201007161514.04955.jkim@FreeBSD.org> <20100716.154216.582095686063971712.imp@bsdimp.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On Friday 16 July 2010 05:42 pm, M. Warner Losh wrote: > In message: <201007161514.04955.jkim@FreeBSD.org> > > Jung-uk Kim <jkim@FreeBSD.org> writes: > : On Thursday 15 July 2010 08:04 am, Bruce Evans wrote: > : > On Tue, 13 Jul 2010, Jung-uk Kim wrote: > : > > Log: > : > > Define SMP unconditionally for amd64 and remove opt_global.h > : > > from SRCS. Note it is done just for correctness sake because > : > > we do not build, ship, or support acpi.ko on amd64. > : > > > : > > Prodded by: bde > : > > : > Thanks. > : > > : > It was much more interesting than I noticed. Modules are > : > supposed to be independent of all configuration options, > : > especially global ones, especially especially SMP. Perhaps > : > this is too much to ask for acpi. Does it work now, after you > : > removed an SMP ifdef from it? > : > : I removed "#ifdef SMP" from acpi.c because 'smp_started' is > : available for both UP and SMP kernels. On the other hand, I > : cannot remove these easily from sys/amd64/acpica/acpi_wakeup.c, > : i.e., it requires some symbols from sys/amd64/amd64/mp_machdep.c, > : which are only available in SMP case. > : > : FYI, we never had to worry about acpi.ko on i386 because we > : didn't implement suspend/resume for SMP case in the first place. > : X-) > > Does this mean we do implement it on amd64 now? I may have to > start trying to use it again... Yes, I did that for a while ago. 8-) Jung-uk Kim
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201007161912.53042.jkim>