From owner-freebsd-standards@FreeBSD.ORG Mon Aug 20 10:44:19 2012 Return-Path: Delivered-To: freebsd-standards@freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [69.147.83.52]) by hub.freebsd.org (Postfix) with ESMTP id 8FE92106566C for ; Mon, 20 Aug 2012 10:44:19 +0000 (UTC) (envelope-from kostikbel@gmail.com) Received: from mail.zoral.com.ua (mx0.zoral.com.ua [91.193.166.200]) by mx1.freebsd.org (Postfix) with ESMTP id E2A988FC0A for ; Mon, 20 Aug 2012 10:44:18 +0000 (UTC) Received: from skuns.kiev.zoral.com.ua (localhost [127.0.0.1]) by mail.zoral.com.ua (8.14.2/8.14.2) with ESMTP id q7KAiQmW015682; Mon, 20 Aug 2012 13:44:26 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: from deviant.kiev.zoral.com.ua (kostik@localhost [127.0.0.1]) by deviant.kiev.zoral.com.ua (8.14.5/8.14.5) with ESMTP id q7KAiEI8085892; Mon, 20 Aug 2012 13:44:14 +0300 (EEST) (envelope-from kostikbel@gmail.com) Received: (from kostik@localhost) by deviant.kiev.zoral.com.ua (8.14.5/8.14.5/Submit) id q7KAiENY085891; Mon, 20 Aug 2012 13:44:14 +0300 (EEST) (envelope-from kostikbel@gmail.com) X-Authentication-Warning: deviant.kiev.zoral.com.ua: kostik set sender to kostikbel@gmail.com using -f Date: Mon, 20 Aug 2012 13:44:14 +0300 From: Konstantin Belousov To: "Jukka A. Ukkonen" Message-ID: <20120820104414.GK33100@deviant.kiev.zoral.com.ua> References: <201208181010.q7IAABqP017818@freefall.freebsd.org> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="bgLLobvf7eP6VP5c" Content-Disposition: inline In-Reply-To: <201208181010.q7IAABqP017818@freefall.freebsd.org> User-Agent: Mutt/1.4.2.3i X-Virus-Scanned: clamav-milter 0.95.2 at skuns.kiev.zoral.com.ua X-Virus-Status: Clean X-Spam-Status: No, score=-4.0 required=5.0 tests=ALL_TRUSTED,AWL,BAYES_00 autolearn=ham version=3.2.5 X-Spam-Checker-Version: SpamAssassin 3.2.5 (2008-06-10) on skuns.kiev.zoral.com.ua Cc: freebsd-standards@freebsd.org Subject: Re: standards/170346: Changes to support waitid() and related stuff X-BeenThere: freebsd-standards@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: Standards compliance List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 20 Aug 2012 10:44:19 -0000 --bgLLobvf7eP6VP5c Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Sat, Aug 18, 2012 at 10:10:11AM +0000, Jukka A. Ukkonen wrote: Some my notes: - the style requires tab after #define, not space, in particular, in sys/wait.h. Other style notes: there should be no space between () and variable for type cast, and space, not tab, between 'case' and case constant. You are adding uneccessary blank lines, esp. in the kern_wait6. The libc/gen/waitid.c shall be reformatted fully. - you can reduce the namespace pollution by using struct __siginfo instead of siginfo_t in the new prototypes added into sys/wait.h. - do not use the low syscall numbers for new syscalls, just add at the end of the table. - you need to provide compat32 translation for new syscalls, see compat/freebsd32/syscalls.master as the starting point. - the comment 'New siginfo stuff...' seems to not provide any useful information. - I do not think we need to enumerate other OSes in comments which provide similar functionality. Just noting that resource usage is provided for both zombies and snapshotted for live process will be good. - please wrap long line in kern_wait6. - why do you zero siginfo in case of returning EINVAL from kern_wait6 ? Same question for assigning -1 to td_reval[0]. td_retval is ignored at all if error is indicated. There seems to be similar fragment for WNOHANG, why do you zero siginfo for the case when the process is not found ? Shouldn't the caller not to copy out the siginfo at all in this case ? - in waitid.c, why do you even consider the case of ret != 0 or -1 ? How can it be ? --bgLLobvf7eP6VP5c Content-Type: application/pgp-signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.12 (FreeBSD) iEYEARECAAYFAlAyFP4ACgkQC3+MBN1Mb4gYWACgyfnSJcg7g5ivfS5vDRzVdKIK +g8AnA99Y0qkn+9wTFTBvYLoNIvahOKG =+zTx -----END PGP SIGNATURE----- --bgLLobvf7eP6VP5c--