Date: Mon, 8 Oct 2007 18:31:06 +0300 From: Kostik Belousov <kostikbel@gmail.com> To: Roman Divacky <rdivacky@freebsd.org> Cc: current@freebsd.org, peter@freebsd.org Subject: Re: pad argument for mmap/lseek/etc. Message-ID: <20071008153106.GN2180@deviant.kiev.zoral.com.ua> In-Reply-To: <20071007140237.GA44735@freebsd.org> References: <20071007140237.GA44735@freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--Kynn+LdAwU9N+JqL
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable
On Sun, Oct 07, 2007 at 04:02:37PM +0200, Roman Divacky wrote:
> hi
>=20
> 3 months ago Peter Wemm removed "pad" argument from mmap/lseek/etc. sysca=
lls
> arguments.
>=20
> Unfortunately he forgot to update the struct XYZ_args definitions shipped
> with the actual functions like this:
>=20
> vm/vm_mmap.c:
>=20
> #ifndef _SYS_SYSPROTO_H_
> struct mmap_args {
> void *addr;
> size_t len;
> int prot;
> int flags;
> int fd;
> long pad;
> off_t pos;
> };
> #endif
>=20
> /*
> * MPSAFE
> */
> int
> mmap(td, uap)
> struct thread *td;
> struct mmap_args *uap;
>=20
> it would be nice to get this removed. I'd submit a patch but I dont
> know what syscalls this affected.
You should lookup the corresponding commit that obviously touched
syscalls.master (seems to be rev. 1.232), and get the syscalls list from
there. Then, you should copy the old structure definitions to freebsd6_XXX
functions, and delete the paddings inplace.
The interesting question is whether the _SYS_SYSPROTO_H and absense of the
sys/sysproto.h supported. Or, shall we remove these #ifdef blocks instead
of fixing them in whole kernel.
--Kynn+LdAwU9N+JqL
Content-Type: application/pgp-signature
Content-Disposition: inline
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2.0.4 (FreeBSD)
iD8DBQFHCk06C3+MBN1Mb4gRApzUAKDIZVtxRXcKRMj+TuKihD6Y5PJwzwCdGcKT
kwYDwtOw4ECnizLy9/JzIAk=
=LwWF
-----END PGP SIGNATURE-----
--Kynn+LdAwU9N+JqL--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20071008153106.GN2180>