From mboxrd@z Thu Jan 1 00:00:00 1970 From: Matthew Wilcox Subject: Re: bash: Correct usage of F_SETFD Date: Tue, 23 Nov 2010 07:42:39 -0700 Message-ID: <20101123144239.GL6178@parisc-linux.org> References: <20101122201620.GA16687@us.ibm.com> <101122221640.AA32947.SM@caleb.INS.CWRU.Edu> <4CEAEE3A.4090004@redhat.com> <20101123000446.GA24667@us.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Eric Blake , chet.ramey@case.edu, linux-fsdevel@vger.kernel.org, bug-bash@gnu.org, chet@po.cwru.edu To: Sukadev Bhattiprolu Return-path: Received: from palinux.external.hp.com ([192.25.206.14]:53128 "EHLO mail.parisc-linux.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753121Ab0KWOmm (ORCPT ); Tue, 23 Nov 2010 09:42:42 -0500 Content-Disposition: inline In-Reply-To: <20101123000446.GA24667@us.ibm.com> Sender: linux-fsdevel-owner@vger.kernel.org List-ID: On Mon, Nov 22, 2010 at 04:04:46PM -0800, Sukadev Bhattiprolu wrote: > Eric Blake [eblake@redhat.com] wrote: > | On 11/22/2010 03:16 PM, Chet Ramey wrote: > | >> include/filecntl.h in bash-4.1 has following: > | >> > | >> #define SET_CLOSE_ON_EXEC(fd) (fcntl ((fd), F_SETFD, FD_CLOEXEC)) > | >> > | >> Is that really the correct/intended usage of F_SETFD ? > | > > | > F_SETFD Set the close-on-exec flag associated with fildes to > | > the low order bit of arg (0 or 1 as above). > > Is that the POSIX definition ? Following man page does not limit F_SETFD to > FD_CLOEXEC: > > http://www.kernel.org/doc/man-pages/online/pages/man2/fcntl.2.html > > F_SETFD (long) > Set the file descriptor flags to the value specified by arg. The POSIX definition can be found here: http://www.opengroup.org/onlinepubs/009695399/functions/fcntl.html > | In practice, there aren't any such systems; but POSIX warns that current > | practice is no indicator of future systems, and that read-modify-write > | is the only way to use F_SETFD. > > Yes, that seems to make more sense. I think future flags will be created such that they default to off, and bash would have to affirmitively set them in order to use them. So if bash is the one creating its file descriptors, there's no need to use R/M/W since it knows what the state of them are. -- Matthew Wilcox Intel Open Source Technology Centre "Bill, look, we understand that you're interested in selling us this operating system, but compare it to ours. We can't possibly take such a retrograde step."