From owner-freebsd-arch@FreeBSD.ORG Sun Mar 17 11:18:00 2013 Return-Path: Delivered-To: freebsd-arch@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by hub.freebsd.org (Postfix) with ESMTP id 517C5E91 for ; Sun, 17 Mar 2013 11:18:00 +0000 (UTC) (envelope-from pawel@dawidek.net) Received: from mail.dawidek.net (garage.dawidek.net [91.121.88.72]) by mx1.freebsd.org (Postfix) with ESMTP id 11BE4C1C for ; Sun, 17 Mar 2013 11:17:59 +0000 (UTC) Received: from localhost (89-73-195-149.dynamic.chello.pl [89.73.195.149]) by mail.dawidek.net (Postfix) with ESMTPSA id 0B3BB1F1; Sun, 17 Mar 2013 12:14:42 +0100 (CET) Date: Sun, 17 Mar 2013 12:19:31 +0100 From: Pawel Jakub Dawidek To: Konstantin Belousov Subject: Re: chflagsat(2). Message-ID: <20130317111931.GD1364@garage.freebsd.pl> References: <20130317004908.GB1364@garage.freebsd.pl> <20130317064338.GN3794@kib.kiev.ua> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="fOHHtNG4YXGJ0yqR" Content-Disposition: inline In-Reply-To: <20130317064338.GN3794@kib.kiev.ua> X-OS: FreeBSD 10.0-CURRENT amd64 User-Agent: Mutt/1.5.21 (2010-09-15) Cc: freebsd-arch@FreeBSD.org X-BeenThere: freebsd-arch@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: Discussion related to FreeBSD architecture List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 17 Mar 2013 11:18:00 -0000 --fOHHtNG4YXGJ0yqR Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sun, Mar 17, 2013 at 08:43:38AM +0200, Konstantin Belousov wrote: > On Sun, Mar 17, 2013 at 01:49:08AM +0100, Pawel Jakub Dawidek wrote: > > Hi. > >=20 > > The following patch adds chflagsat(2) syscall to complete other *at > > syscalls: > >=20 > > http://people.freebsd.org/~pjd/patches/chflagsat.patch > >=20 > > Note that the name chflagsat was carefully choosen instead of > > fchflagsat, to not repeat POSIX (more likely Linux) mistakes of using > > fchmodat, fchownat, futimesat, etc. names when they really shouldn't > > start with an 'f'. >=20 > This is the only point I do not agree with the patch. I prefer to have > the syscall name consistent with the other syscalls. There are quite a few *at() syscalls and only few of them have bogus 'f' prefix (fchmodat, fchownat, fstatat, futimesat). Most of them don't (bindat, connectat, linkat, mkdirat, mkfifoat, mknodat, openat, renameat, symlinkat, unlinkat). 'f' is of course bogus, because the syscalls don't operate on descriptors, but on paths. Also note that POSIX is moving aways from those mistakes and eventhough they standarized futimesat(), they now use utimensat() for nanosecond precision timestamps to stop those mistakes. This is actually the argument jilles' convinced me to use chflagsat instead of fchflagsat, which is originally used. --=20 Pawel Jakub Dawidek http://www.wheelsystems.com FreeBSD committer http://www.FreeBSD.org Am I Evil? Yes, I Am! http://tupytaj.pl --fOHHtNG4YXGJ0yqR Content-Type: application/pgp-signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v2.0.19 (FreeBSD) iEYEARECAAYFAlFFpsMACgkQForvXbEpPzTNuQCg05X9vRhXLwqb4PaSiWyuHmpz EKsAnRKukxsilLbJQUIVN9nZ2BrguQ1B =/X6D -----END PGP SIGNATURE----- --fOHHtNG4YXGJ0yqR--