Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 28 Oct 2010 23:31:09 +0200
From:      Tijl Coosemans <tijl@coosemans.org>
To:        freebsd-arch@freebsd.org
Cc:        Attilio Rao <attilio@freebsd.org>, Warner Losh <imp@freebsd.org>
Subject:   Re: [PATCH] Headers for the x86 subtree
Message-ID:  <201010282331.19825.tijl@coosemans.org>
In-Reply-To: <201010281013.03261.jhb@freebsd.org>
References:  <AANLkTiktoYyxmE8nyGeoc4_ov35fR7iN83444MfhYg-e@mail.gmail.com> <AANLkTimWx5QtADFv65BQX%2BgMnnDvWYw6PgUZCSS5khdE@mail.gmail.com> <201010281013.03261.jhb@freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
--nextPart1436314.ToryqBqT7b
Content-Type: Text/Plain;
  charset="us-ascii"
Content-Transfer-Encoding: quoted-printable

On Thursday 28 October 2010 16:13:02 John Baldwin wrote:
> On Thursday, October 28, 2010 3:44:21 am Attilio Rao wrote:
>> 2010/10/27 John Baldwin <jhb@freebsd.org>:
>>> On Wednesday, October 27, 2010 10:56:06 am Attilio Rao wrote:
>>>> This patch should convert a (simple and 100% shared between amd64 and
>>>> i386 header) under the x86 sub-tree. Please note that in this patch I
>>>> "svn cp" the file from sys/amd64/include/mptable.h into
>>>> sys/x86/include/mptable.h:
>>>> http://www.freebsd.org/~attilio/headers-x86.diff
>>>>
>>>> This is someway a POC, that I really want to get in. The idea is
>>>> simple and someway follows the pc98 case (even if not entirely): the
>>>> files under machine/include/* became just mere stubs for x86/include/*
>>>> contents and redirect there.
>>>> This won't particulary help reducing the number of available files,
>>>> but generally removing verbatim and would also be the way to go for
>>>> handling MFCs.
>>>> If you find this is the right way I'll commit the fix and start moving
>>>> other files as time permits.
>>>
>>> No, we want to do this differently because we also want this to work in
>>> userland.  (e.g. I'd like to outright move mca.h to x86/include and then
>>> use '#include <x86/mca.h>' in both kernel and userland for it).  We'd n=
eed
>>> some special glue to setup an 'x86' symlink during a kernel build that
>>> points to @/x86/include as we do now to setup an 'i386' link for pc98
>>> kernels.
>>>
>>> We'd also need to install the x86 headers into /usr/include during an
>>> installworld.  Warner has some more pointers on this I think.
>>=20
>> I spoke with Warner briefly about it.
>> One question I'm having now, though, is how getting co-living of pc98
>> and x86 now, as we are basically overriding the same infrastructure
>> (MACHINE_CPUARCH) in the i386/amd64 case?
>> Do you have ideas about that?
>=20
> I'm still doing testing, but this seems to be working so far.  I am
> moving mca.h as my current test.
>=20
> Index: include/Makefile
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
> --- include/Makefile	(revision 214386)
> +++ include/Makefile	(working copy)
> @@ -116,8 +116,11 @@
>  .endfor
> =20
>  .if ${MACHINE} !=3D ${MACHINE_CPUARCH}
> -_MARCH=3D${MACHINE_CPUARCH}
> +_MARCHS=3D	${MACHINE_CPUARCH}
>  .endif
> +.if ${MACHINE_CPUARCH} =3D=3D "i386" || ${MACHINE_CPUARCH} =3D=3D "amd64"
> +_MARCHS+=3D	x86
> +.endif

Can't MACHINE_CPUARCH be set to "x86" in the i386, amd64 and pc98
cases? This patch wouldn't be needed then.

--nextPart1436314.ToryqBqT7b
Content-Type: application/pgp-signature; name=signature.asc 
Content-Description: This is a digitally signed message part.

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.16 (FreeBSD)

iF4EABEIAAYFAkzJ66cACgkQfoCS2CCgtisIhgEAhPtUZJMFamWhivkzFNCNGjQf
PUdu+BKMYL8YUz9vTpAA+wQXWIXk3V7gOt50SFvY6Mnl7XaJRiqw1hdCkiUQ3Z+C
=TIUO
-----END PGP SIGNATURE-----

--nextPart1436314.ToryqBqT7b--



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