Date: Fri, 10 Jun 2022 01:02:01 +0400 From: Sergey Kandaurov <pluknet@gmail.com> To: Jilles Tjoelker <jilles@freebsd.org> Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: svn commit: r319663 - head/lib/libc/sys Message-ID: <CAE-mSOK%2B6yZncp8HarxyXJwK-gSyaSws7r59_zSY1Se=xWZ06Q@mail.gmail.com> In-Reply-To: <201706072121.v57LLEH2001634@repo.freebsd.org> References: <201706072121.v57LLEH2001634@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
--000000000000c48a5505e10a2227 Content-Type: text/plain; charset="UTF-8" On Thu, 8 Jun 2017 at 01:21, Jilles Tjoelker <jilles@freebsd.org> wrote: > Author: jilles > Date: Wed Jun 7 21:21:14 2017 > New Revision: 319663 > URL: https://svnweb.freebsd.org/changeset/base/319663 > > Log: > libc: Remove futimens() and utimensat() compat stubs. > > The futimens() and utimensat() compat stubs allowed using these > functions on > kernels that did not have the system calls yet (10.2, old 11-current). > > Also remove the documentation of the [ENOTSUP] error that could occur > with > an old kernel. > > A -DNO_CLEAN build may fail because the depend files refer to the deleted > files. > > Deleted: > head/lib/libc/sys/futimens.c > head/lib/libc/sys/utimensat.c > Modified: > head/lib/libc/sys/Makefile.inc > head/lib/libc/sys/utimensat.2 > > Modified: head/lib/libc/sys/Makefile.inc > > ============================================================================== > --- head/lib/libc/sys/Makefile.inc Wed Jun 7 21:18:28 2017 > (r319662) > +++ head/lib/libc/sys/Makefile.inc Wed Jun 7 21:21:14 2017 > (r319663) > @@ -37,10 +37,6 @@ SRCS+= \ > > SRCS+= getdents.c lstat.c mknod.c stat.c > > -SRCS+= futimens.c utimensat.c > -NOASM+= futimens.o utimensat.o > -PSEUDO+= _futimens.o _utimensat.o > - > SRCS+= pipe.c > > INTERPOSED = \ > > Modified: head/lib/libc/sys/utimensat.2 > > ============================================================================== > --- head/lib/libc/sys/utimensat.2 Wed Jun 7 21:18:28 2017 > (r319662) > +++ head/lib/libc/sys/utimensat.2 Wed Jun 7 21:21:14 2017 > (r319663) > @@ -31,7 +31,7 @@ > .\" @(#)utimes.2 8.1 (Berkeley) 6/4/93 > .\" $FreeBSD$ > .\" > -.Dd January 17, 2016 > +.Dd June 7, 2017 > .Dt UTIMENSAT 2 > .Os > .Sh NAME > @@ -267,10 +267,6 @@ argument is not an absolute path and > is neither > .Dv AT_FDCWD > nor a file descriptor associated with a directory. > -.It Bq Er ENOTSUP > -The running kernel does not support this system call and > -.Dv AT_SYMLINK_NOFOLLOW > -is used with a path relative to a file descriptor. > .El > .Sh SEE ALSO > .Xr chflags 2 , > I believe the compatibility part can be removed too: diff --git a/lib/libc/sys/utimensat.2 b/lib/libc/sys/utimensat.2 index 2af452898c9..53602d68ba8 100644 --- a/lib/libc/sys/utimensat.2 +++ b/lib/libc/sys/utimensat.2 @@ -169,18 +169,6 @@ operate on the current working directory. .El .Sh RETURN VALUES .Rv -std -.Sh COMPATIBILITY -If the running kernel does not support this system call, -a wrapper emulates it using -.Xr fstatat 2 , -.Xr futimesat 2 -and -.Xr lutimes 2 . -As a result, timestamps will be rounded down to the nearest microsecond, -.Dv UTIME_OMIT -is not atomic and -.Dv AT_SYMLINK_NOFOLLOW -is not available with a path relative to a file descriptor. .Sh ERRORS These system calls will fail if: .Bl -tag -width Er --000000000000c48a5505e10a2227 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable <div dir=3D"ltr"><div dir=3D"ltr"><div class=3D"gmail_default" style=3D"fon= t-family:monospace,monospace"></div></div><br><div class=3D"gmail_quote"><d= iv dir=3D"ltr" class=3D"gmail_attr">On Thu, 8 Jun 2017 at 01:21, Jilles Tjo= elker <<a href=3D"mailto:jilles@freebsd.org" target=3D"_blank">jilles@fr= eebsd.org</a>> wrote:<br></div><blockquote class=3D"gmail_quote" style= =3D"margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding= -left:1ex">Author: jilles<br> Date: Wed Jun=C2=A0 7 21:21:14 2017<br> New Revision: 319663<br> URL: <a href=3D"https://svnweb.freebsd.org/changeset/base/319663" rel=3D"no= referrer" target=3D"_blank">https://svnweb.freebsd.org/changeset/base/31966= 3</a><br> <br> Log:<br> =C2=A0 libc: Remove futimens() and utimensat() compat stubs.<br> <br> =C2=A0 The futimens() and utimensat() compat stubs allowed using these func= tions on<br> =C2=A0 kernels that did not have the system calls yet (10.2, old 11-current= ).<br> <br> =C2=A0 Also remove the documentation of the [ENOTSUP] error that could occu= r with<br> =C2=A0 an old kernel.<br> <br> =C2=A0 A -DNO_CLEAN build may fail because the depend files refer to the de= leted<br> =C2=A0 files.<br> <br> Deleted:<br> =C2=A0 head/lib/libc/sys/futimens.c<br> =C2=A0 head/lib/libc/sys/utimensat.c<br> Modified:<br> =C2=A0 head/lib/libc/sys/Makefile.inc<br> =C2=A0 head/lib/libc/sys/utimensat.2<br> <br> Modified: head/lib/libc/sys/Makefile.inc<br> =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=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D<br> --- head/lib/libc/sys/Makefile.inc=C2=A0 =C2=A0 =C2=A0 Wed Jun=C2=A0 7 21:1= 8:28 2017=C2=A0 =C2=A0 =C2=A0 =C2=A0 (r319662)<br> +++ head/lib/libc/sys/Makefile.inc=C2=A0 =C2=A0 =C2=A0 Wed Jun=C2=A0 7 21:2= 1:14 2017=C2=A0 =C2=A0 =C2=A0 =C2=A0 (r319663)<br> @@ -37,10 +37,6 @@ SRCS+=3D=C2=A0 =C2=A0 =C2=A0 =C2=A0\<br> <br> =C2=A0SRCS+=3D getdents.c lstat.c mknod.c stat.c<br> <br> -SRCS+=3D futimens.c utimensat.c<br> -NOASM+=3D futimens.o utimensat.o<br> -PSEUDO+=3D _futimens.o _utimensat.o<br> -<br> =C2=A0SRCS+=3D pipe.c<br> <br> =C2=A0INTERPOSED =3D \<br> <br> Modified: head/lib/libc/sys/utimensat.2<br> =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=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D<br> --- head/lib/libc/sys/utimensat.2=C2=A0 =C2=A0 =C2=A0 =C2=A0Wed Jun=C2=A0 7= 21:18:28 2017=C2=A0 =C2=A0 =C2=A0 =C2=A0 (r319662)<br> +++ head/lib/libc/sys/utimensat.2=C2=A0 =C2=A0 =C2=A0 =C2=A0Wed Jun=C2=A0 7= 21:21:14 2017=C2=A0 =C2=A0 =C2=A0 =C2=A0 (r319663)<br> @@ -31,7 +31,7 @@<br> =C2=A0.\"=C2=A0 =C2=A0 =C2=A0@(#)utimes.2=C2=A0 =C2=A08.1 (Berkeley) 6= /4/93<br> =C2=A0.\" $FreeBSD$<br> =C2=A0.\"<br> -.Dd January 17, 2016<br> +.Dd June 7, 2017<br> =C2=A0.Dt UTIMENSAT 2<br> =C2=A0.Os<br> =C2=A0.Sh NAME<br> @@ -267,10 +267,6 @@ argument is not an absolute path and<br> =C2=A0is neither<br> =C2=A0.Dv AT_FDCWD<br> =C2=A0nor a file descriptor associated with a directory.<br> -.It Bq Er ENOTSUP<br> -The running kernel does not support this system call and<br> -.Dv AT_SYMLINK_NOFOLLOW<br> -is used with a path relative to a file descriptor.<br> =C2=A0.El<br> =C2=A0.Sh SEE ALSO<br> =C2=A0.Xr chflags 2 ,<br></blockquote><div><br></div><div><div style=3D"fon= t-family:monospace,monospace" class=3D"gmail_default">I believe the compati= bility part can be removed too:</div><div style=3D"font-family:monospace,mo= nospace" class=3D"gmail_default"><br></div><div style=3D"font-family:monosp= ace,monospace" class=3D"gmail_default">diff --git a/lib/libc/sys/utimensat.= 2 b/lib/libc/sys/utimensat.2<br>index 2af452898c9..53602d68ba8 100644<br>--= - a/lib/libc/sys/utimensat.2<br>+++ b/lib/libc/sys/utimensat.2<br>@@ -169,1= 8 +169,6 @@ operate on the current working directory.<br>=C2=A0.El<br>=C2= =A0.Sh RETURN VALUES<br>=C2=A0.Rv -std<br>-.Sh COMPATIBILITY<br>-If the run= ning kernel does not support this system call,<br>-a wrapper emulates it us= ing<br>-.Xr fstatat 2 ,<br>-.Xr futimesat 2<br>-and<br>-.Xr lutimes 2 .<br>= -As a result, timestamps will be rounded down to the nearest microsecond,<b= r>-.Dv UTIME_OMIT<br>-is not atomic and<br>-.Dv AT_SYMLINK_NOFOLLOW<br>-is = not available with a path relative to a file descriptor.<br>=C2=A0.Sh ERROR= S<br>=C2=A0These system calls will fail if:<br>=C2=A0.Bl -tag -width Er<br>= </div></div></div></div> --000000000000c48a5505e10a2227--
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAE-mSOK%2B6yZncp8HarxyXJwK-gSyaSws7r59_zSY1Se=xWZ06Q>