Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 19 Mar 2006 13:30:39 +0200
From:      Ruslan Ermilov <ru@FreeBSD.org>
To:        Alexander Leidinger <netchild@FreeBSD.org>
Cc:        cvs-src@FreeBSD.org, src-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/amd64/linux32 linux32_sysvec.c syscalls.master src/sys/compat/linux linux_file.c linux_getcwd.c linux_ioctl.c linux_ipc.c linux_mib.c linux_misc.c linux_signal.c linux_socket.c linux_stats.c linux_sysctl.c linux_uid16.c
Message-ID:  <20060319113039.GD89287@ip.net.ua>
In-Reply-To: <200603191110.k2JBAYxC085982@repoman.freebsd.org>
References:  <200603191110.k2JBAYxC085982@repoman.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help

--Qrgsu6vtpU/OV/zm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Hi Alexander,

Linuxulator is still broken after my preliminary fix referenced
below, on both amd64 and alpha.  On amd64, it now stops here:

: cc -c -O2 -frename-registers -pipe -fno-strict-aliasing  -Wall -Wredundan=
t-decls -Wnested-externs -Wstrict-prototypes  -Wmissing-prototypes -Wpointe=
r-arith -Winline -Wcast-qual  -Wundef -fformat-extensions -std=3Dc99 -g -no=
stdinc -I-  -I. -I/usr/src/sys -I/usr/src/sys/contrib/altq -D_KERNEL -DHAVE=
_KERNEL_OPTION_HEADERS -include opt_global.h -fno-common -finline-limit=3D8=
000 --param inline-unit-growth=3D100 --param large-function-growth=3D1000  =
-fno-omit-frame-pointer -mcmodel=3Dkernel -mno-red-zone  -mfpmath=3D387 -mn=
o-sse -mno-sse2 -mno-mmx -mno-3dnow  -msoft-float -fno-asynchronous-unwind-=
tables -ffreestanding -Werror  /usr/src/sys/compat/linux/linux_stats.c
: /usr/src/sys/compat/linux/linux_stats.c:233: warning: "struct linux_lstat=
_args" declared inside parameter list
: /usr/src/sys/compat/linux/linux_stats.c:233: warning: its scope is only t=
his definition or declaration, which is probably not what you want
: /usr/src/sys/compat/linux/linux_stats.c:234: warning: no previous prototy=
pe for 'linux_lstat'
: /usr/src/sys/compat/linux/linux_stats.c: In function `linux_lstat':
: /usr/src/sys/compat/linux/linux_stats.c:242: error: dereferencing pointer=
 to incomplete type
: /usr/src/sys/compat/linux/linux_stats.c:245: error: dereferencing pointer=
 to incomplete type
: *** Error code 1
:=20
: Stop in /usr/obj/usr/src/sys/HAMMER.
: *** Error code 1

On alpha/LINT, it will break similarly, because you didn't provide
support for them.

You obviously didn't build test your change on anything except i386,
or you would notice that inclusion of "opt_compat.h" is needed so
the sources can see the COMPAT_LINUX32/COMPAT_IA32 options.

On Sun, Mar 19, 2006 at 11:10:33AM +0000, Ruslan Ermilov wrote:
> ru          2006-03-19 11:10:33 UTC
>=20
>   FreeBSD src repository
>=20
>   Modified files:
>     sys/amd64/linux32    linux32_sysvec.c syscalls.master=20
>     sys/compat/linux     linux_file.c linux_getcwd.c linux_ioctl.c=20
>                          linux_ipc.c linux_mib.c linux_misc.c=20
>                          linux_signal.c linux_socket.c=20
>                          linux_stats.c linux_sysctl.c=20
>                          linux_uid16.c=20
>   Log:
>   Unbreak COMPAT_LINUX32 option support on amd64.
>  =20
>   Broken by:      netchild
>  =20
>   Revision  Changes    Path
>   1.17      +1 -0      src/sys/amd64/linux32/linux32_sysvec.c
>   1.7       +1 -0      src/sys/amd64/linux32/syscalls.master
>   1.94      +1 -0      src/sys/compat/linux/linux_file.c
>   1.24      +1 -0      src/sys/compat/linux/linux_getcwd.c
>   1.135     +2 -0      src/sys/compat/linux/linux_ioctl.c
>   1.46      +2 -0      src/sys/compat/linux/linux_ipc.c
>   1.26      +2 -0      src/sys/compat/linux/linux_mib.c
>   1.175     +1 -0      src/sys/compat/linux/linux_misc.c
>   1.54      +2 -0      src/sys/compat/linux/linux_signal.c
>   1.65      +1 -0      src/sys/compat/linux/linux_socket.c
>   1.78      +1 -0      src/sys/compat/linux/linux_stats.c
>   1.17      +2 -0      src/sys/compat/linux/linux_sysctl.c
>   1.19      +2 -0      src/sys/compat/linux/linux_uid16.c


Cheers,
--=20
Ruslan Ermilov
ru@FreeBSD.org
FreeBSD committer

--Qrgsu6vtpU/OV/zm
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (FreeBSD)

iD8DBQFEHUDfqRfpzJluFF4RAtQMAJ9qV2kVXxYnNWO/tnzzIlffnZCL1wCgnMd8
cXuVyqH4qPQrjyZ7QbhCagY=
=ioJo
-----END PGP SIGNATURE-----

--Qrgsu6vtpU/OV/zm--



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