From owner-svn-src-head@freebsd.org Sun Sep 29 09:25:46 2019 Return-Path: Delivered-To: svn-src-head@mailman.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.nyi.freebsd.org (Postfix) with ESMTP id B297DFA9FE; Sun, 29 Sep 2019 09:25:46 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from kib.kiev.ua (kib.kiev.ua [IPv6:2001:470:d5e7:1::1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) server-signature RSA-PSS (4096 bits)) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 46h0WZ35Wzz4L7T; Sun, 29 Sep 2019 09:25:46 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from tom.home (kib@localhost [127.0.0.1]) by kib.kiev.ua (8.15.2/8.15.2) with ESMTPS id x8T9PWdt041380 (version=TLSv1.3 cipher=TLS_AES_256_GCM_SHA384 bits=256 verify=NO); Sun, 29 Sep 2019 12:25:35 +0300 (EEST) (envelope-from kostikbel@gmail.com) DKIM-Filter: OpenDKIM Filter v2.10.3 kib.kiev.ua x8T9PWdt041380 Received: (from kostik@localhost) by tom.home (8.15.2/8.15.2/Submit) id x8T9PWti041378; Sun, 29 Sep 2019 12:25:32 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: tom.home: kostik set sender to kostikbel@gmail.com using -f Date: Sun, 29 Sep 2019 12:25:32 +0300 From: Konstantin Belousov To: Warner Losh Cc: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r352846 - head/lib/libc/sys Message-ID: <20190929092532.GO44691@kib.kiev.ua> References: <201909281715.x8SHFmZR034892@repo.freebsd.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <201909281715.x8SHFmZR034892@repo.freebsd.org> User-Agent: Mutt/1.12.2 (2019-09-21) X-Spam-Status: No, score=-1.0 required=5.0 tests=ALL_TRUSTED,BAYES_00, DKIM_ADSP_CUSTOM_MED,FORGED_GMAIL_RCVD,FREEMAIL_FROM, NML_ADSP_CUSTOM_MED autolearn=no autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on tom.home X-Rspamd-Queue-Id: 46h0WZ35Wzz4L7T X-Spamd-Bar: ----- Authentication-Results: mx1.freebsd.org; none X-Spamd-Result: default: False [-5.99 / 15.00]; NEURAL_HAM_MEDIUM(-0.99)[-0.991,0]; NEURAL_HAM_LONG(-1.00)[-1.000,0]; REPLY(-4.00)[] X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: SVN commit messages for the src tree for head/-current List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 29 Sep 2019 09:25:46 -0000 On Sat, Sep 28, 2019 at 05:15:48PM +0000, Warner Losh wrote: > Author: imp > Date: Sat Sep 28 17:15:48 2019 > New Revision: 352846 > URL: https://svnweb.freebsd.org/changeset/base/352846 > > Log: > Revert the mode_t -> int changes and add a warning in the BUGS section instead. > > While FreeBSD's implementation of these expect an int inside of libc, that's an > implementation detail that we can hide from the user as it's the natural > promotion of the current mode_t type and before it is used in the kernel, it's > converted back to the narrower type that's the current definition of mode_t. As > such, documenting int is at best confusing and at worst misleading. Instead add > a note that these args are variadic and as such calling conventions may differ > from non-variadic arguments. > > Modified: > head/lib/libc/sys/mq_open.2 > head/lib/libc/sys/open.2 > > Modified: head/lib/libc/sys/mq_open.2 > ============================================================================== > --- head/lib/libc/sys/mq_open.2 Sat Sep 28 14:20:28 2019 (r352845) > +++ head/lib/libc/sys/mq_open.2 Sat Sep 28 17:15:48 2019 (r352846) > @@ -37,7 +37,7 @@ > .\" > .\" $FreeBSD$ > .\" > -.Dd September 15, 2014 > +.Dd September 28, 2019 > .Dt MQ_OPEN 2 > .Os > .Sh NAME > @@ -133,7 +133,7 @@ Create a message queue. > It requires two additional arguments: > .Fa mode , > which is of type > -.Vt int , > +.Vt mode_t , > and > .Fa attr , > which is a pointer to an > @@ -317,6 +317,13 @@ This implementation places strict requirements on the > it must begin with a slash > .Pq Ql / > and contain no other slash characters. > +.Pp > +The > +.Fa mode > +and > +.Fa attr > +arguments are variadic and may result in different calling conventions > +than might otherwise be expected. > .Sh COPYRIGHT > Portions of this text are reprinted and reproduced in electronic form > from IEEE Std 1003.1, 2004 Edition, Standard for Information Technology -- > > Modified: head/lib/libc/sys/open.2 > ============================================================================== > --- head/lib/libc/sys/open.2 Sat Sep 28 14:20:28 2019 (r352845) > +++ head/lib/libc/sys/open.2 Sat Sep 28 17:15:48 2019 (r352846) > @@ -28,7 +28,7 @@ > .\" @(#)open.2 8.2 (Berkeley) 11/16/93 > .\" $FreeBSD$ > .\" > -.Dd September 17, 2019 > +.Dd September 28, 2019 > .Dt OPEN 2 > .Os > .Sh NAME > @@ -61,7 +61,7 @@ In this case > and > .Fn openat > require an additional argument > -.Fa "int mode" , > +.Fa "mode_t mode" , > and the file is created with mode > .Fa mode > as described in > @@ -615,3 +615,8 @@ permits searches. > The present implementation of the > .Fa openat > checks the current permissions of directory instead. > +.Pp > +The > +.Fa mode > +argument is variadic and may result in different calling conventions > +than might otherwise be expected. This note is also very confusing. Assume that somebody knows calling conventions and then see this note. Now she would be equally misdirected because it is completely not clear what different conventions are mentioned there, esp. because there are no. Also, as I noted before, this note in whatever form does not belongs to BUSS section.