Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 8 Jun 2010 17:25:24 -0400
From:      John Baldwin <jhb@freebsd.org>
To:        "M. Warner Losh" <imp@bsdimp.com>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r208921 - in head/sys: amd64/amd64 conf i386/i386 x86/x86
Message-ID:  <201006081725.24684.jhb@freebsd.org>
In-Reply-To: <20100608.132210.10150326092157406.imp@bsdimp.com>
References:  <201006081804.o58I473g073875@svn.freebsd.org> <201006081446.09423.jhb@freebsd.org> <20100608.132210.10150326092157406.imp@bsdimp.com>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tuesday 08 June 2010 3:22:10 pm M. Warner Losh wrote:
> In message: <201006081446.09423.jhb@freebsd.org>
>             John Baldwin <jhb@freebsd.org> writes:
> : On Tuesday 08 June 2010 2:04:07 pm John Baldwin wrote:
> : > Author: jhb
> : > Date: Tue Jun  8 18:04:07 2010
> : > New Revision: 208921
> : > URL: http://svn.freebsd.org/changeset/base/208921
> : > 
> : > Log:
> : >   Move the machine check support code to the x86 tree since it is 
identical
> : >   on i386 and amd64.
> : >   
> : >   Requested by:	alc
> : 
> : It would be nice to consolidate <machine/mca.h> as that is identical on 
both 
> : platforms, but that moving to x86/include is trickier as the header needs 
to 
> : be available in userland, probably as <machine/mca.h> still.  <x86/mca.h> 
> : would work ok (i.e. in /usr/include/x86/mca.h), but that makes things 
trickier 
> : in the kernel as the file should really live in sys/x86/include, not 
sys/x86 
> : directly.
> : 
> : I'm open to suggestions on if this is feasible and if so how to do it.
> 
> I believe we had a long talk about this before.
> 
> copy the current {i386,amd64}/mca.h to x86/mca.h
> new {i386,amd64}/mca.h == #include <x86/mca.h> (no copyright notice, etc)
> 
> just like we do for the pc98 stuff.  The mca.h file would live in
> sys/x86/include.

How does this work for the kernel?  <x86/mca.h> doesn't map to 
sys/x86/include/mca.h.  For machine we make a symlink that points to 
sys/<machine>/include.  Are you proposing an x86 symlink in the kernel build 
directory that for i386, pc98, and amd64 that points to sys/x86/include?

Ah, looks like you are.  This is handled by _ILINKS in kern.post.mk currently.
If you will fix all the glue magic so #include <x86/foo.h> works in kernel and 
userland I have several headers I can move (apicreg.h, mca.h, etc.).

-- 
John Baldwin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201006081725.24684.jhb>