From owner-svn-src-head@freebsd.org Thu Feb 15 12:23:06 2018 Return-Path: Delivered-To: svn-src-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id CDC2CF19567; Thu, 15 Feb 2018 12:23:06 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from mail105.syd.optusnet.com.au (mail105.syd.optusnet.com.au [211.29.132.249]) by mx1.freebsd.org (Postfix) with ESMTP id B00B8747CD; Thu, 15 Feb 2018 12:23:05 +0000 (UTC) (envelope-from brde@optusnet.com.au) Received: from [192.168.0.102] (c110-21-101-228.carlnfd1.nsw.optusnet.com.au [110.21.101.228]) by mail105.syd.optusnet.com.au (Postfix) with ESMTPS id A80CD1047FC1; Thu, 15 Feb 2018 23:22:57 +1100 (AEDT) Date: Thu, 15 Feb 2018 23:22:56 +1100 (EST) From: Bruce Evans X-X-Sender: bde@besplex.bde.org To: Warner Losh cc: Eitan Adler , src-committers , svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r329296 - head/sbin/devd In-Reply-To: Message-ID: <20180215231649.Q985@besplex.bde.org> References: <201802150322.w1F3Mrie018786@repo.freebsd.org> MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII; format=flowed X-Optus-CM-Score: 0 X-Optus-CM-Analysis: v=2.2 cv=cIaQihWN c=1 sm=1 tr=0 a=PalzARQSbocsUSjMRkwAPg==:117 a=PalzARQSbocsUSjMRkwAPg==:17 a=kj9zAlcOel0A:10 a=6I5d2MoRAAAA:8 a=7Qk2ozbKAAAA:8 a=vSB-gUQo5yuNJ_K1mewA:9 a=CjuIK1q_8ugA:10 a=IjZwj45LgO3ly-622nXo:22 a=1lyxoWkJIXJV6VJUPhuM:22 X-BeenThere: svn-src-head@freebsd.org X-Mailman-Version: 2.1.25 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: Thu, 15 Feb 2018 12:23:07 -0000 On Wed, 14 Feb 2018, Warner Losh wrote: > On Wed, Feb 14, 2018 at 8:51 PM, Eitan Adler wrote: > >> On 14 February 2018 at 19:48, Warner Losh wrote: >>> >>> On Wed, Feb 14, 2018 at 8:39 PM, Warner Losh wrote: >>>> >>>> On Feb 14, 2018 8:23 PM, "Eitan Adler" wrote: >>>> Log: >>>> devd: don't pass &fds in useless parameters to select(2) >>>> >>>> select(2) should be declared as restrict. In addition the only fd in >>>> the fdset is open O_RDONLY, and it's not a socket that can provide OOB >>>> notifications, >>>> >>>> Reviewed by: ian, imp, vangyzen >>>> >>>> Don't put my name on this. I specifically and clearly objected to the >>>> change anf tld yoy not to do it. >>> >>> Stupid phone... >>> >>> I specifically objected to this change. I said not to make it because it >>> wasn't necessary. You did it any way. Don't put "reviewed by" for that. >> Put >>> "objected to but I did it anyway by: imp" It is worse than unnecessary. It breaks at least "any" in the comment before it. "any" means read, write or exeptional descriptors, but now only read descriptors are checked. >> hrm.. rereading the thread I think I missed your original email. Only >> comment I saw was "poll is a better interface". Sorry for mis-stating >> your opinion. > > Yea. It was more of a "don't change it, since it's fine now" and then the > "poll" comment was "better to just change to a better interface." > >> FTR I'd like to fix the declaration of select(2) anyways. > > That's unlikely to end well... I wish you luck... The pattern I used in > devd for select was nearly universal a decade ago... Maybe things have > changed... It can't be nearly universal except possibly for the very special check where you don't care which events occurred, just whether there was one, and also don't wait for any events, but just check for one in a racy way. Bruce