Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 11 May 2009 21:58:14 +0300
From:      Kostik Belousov <kostikbel@gmail.com>
To:        John Baldwin <jhb@freebsd.org>
Cc:        freebsd-hackers@freebsd.org, Ighighi <ighighi@gmail.com>
Subject:   Re: POSIXfy readlink() call
Message-ID:  <20090511185814.GD1948@deviant.kiev.zoral.com.ua>
In-Reply-To: <200905111446.14439.jhb@freebsd.org>
References:  <46FDBAFC.1010000@gmail.com> <200905111405.07289.jhb@freebsd.org> <20090511183309.GB1948@deviant.kiev.zoral.com.ua> <200905111446.14439.jhb@freebsd.org>

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

--8nuB+PMTZvqSz3PP
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Mon, May 11, 2009 at 02:46:14PM -0400, John Baldwin wrote:
> On Monday 11 May 2009 2:33:09 pm Kostik Belousov wrote:
> > On Mon, May 11, 2009 at 02:05:07PM -0400, John Baldwin wrote:
> > > On Friday 28 September 2007 10:39:56 pm Ighighi wrote:
> >                         ^^^^^
>=20
> Yes, I had this stuck in the back of my head from when it first appeared.
>=20
> > > > The POXIX prototype for readlink(2) is:
> > > > ssize_t readlink(const char *restrict path, char *restrict buf, siz=
e_t=20
> > > > bufsize);
> > >=20
> > > It can't simply be corrected as it would change the ABI and thus requ=
ires=20
> a=20
> > > new system call, etc.  However, do you really expect a symlink to be=
=20
> longer=20
> > > than 2^31 on a 64-bit machine?
> >=20
> > Yes, I agree that this is ABI change.
> >=20
> > Meantime,
> > r176215 | ru | 2008-02-12 22:09:04 +0200 (Tue, 12 Feb 2008) | 5 lines
> >=20
> > Change readlink(2)'s return type and type of the last argument
> > to match POSIX.
> >=20
> > Prodded by:     Alexey Lyashkov
> >=20
> > I tried to convince ru@ that ABI breakage is not good, but has not
> > succeeded.
>=20
> Ugh, is this only in HEAD?  If so, I will back it out for 8.0.  If this m=
ade=20
> it into a release then this is a far bigger mess.  Oh, good, this is only=
 in=20
> 8.  I will fix this ASAP.  I can just add the new syscall I guess.

You need to symver the syscalls. It requires some ugly games with our
syscall stubs, because gnu ld only honor .symver in the same object where
the symbol is defined. I did prototyped this some time ago, by including
a file with appropriate .symver from all stubs.

--8nuB+PMTZvqSz3PP
Content-Type: application/pgp-signature
Content-Disposition: inline

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

iEYEARECAAYFAkoIdUYACgkQC3+MBN1Mb4ifzACgrLpVJ3GBpJiibydOVyKaPbZY
v/YAnAo8BjIyRA0+wApIBZong+B3aHWf
=7S0J
-----END PGP SIGNATURE-----

--8nuB+PMTZvqSz3PP--



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