Cc: hch@lst.de, geoff@gclare.org.uk, subrata@linux.vnet.ibm.com,
       linux-kernel@vger.kernel.org
Content-Type: text/plain; charset="us-ascii"
Date: Thu, 27 Sep 2007 19:19:50 +0200
From: "Michael Kerrisk" <mtk-manpages@gmx.net>
In-Reply-To: <Pine.LNX.4.64.0709271000420.763@alien.or.mcafeemobile.com>
Message-ID: <20070927171950.252220@gmx.net>
MIME-Version: 1.0
References: <46FB9FB9.8080201@gmx.net>
 <Pine.LNX.4.64.0709271000420.763@alien.or.mcafeemobile.com>
Subject: Re: Revised signalfd man-page
To: Davide Libenzi <davidel@xmailserver.org>
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1704
Lines: 51

> Von: Davide Libenzi <davidel@xmailserver.org>
> On Thu, 27 Sep 2007, Michael Kerrisk wrote:
[...]
> > .\" FIXME Davide, what does the following mean?  How (in userspace
> > .\" terms) does a sighand structure become orphaned?
> > The
> > .BR read (2)
> > call can also return 0,
> > in case the sighand structure to which the signalfd was attached,
> > has been orphaned.
> 
> You can remove the five lines above, in virtue of the fact that Linus 
> merged my simplification patch.

Okay.

[...]
> > .\" FIXME Davide, a question: why rename these fields to be
> > .\" different from their siginfo_t counterparts?  At the very least,
> > .\" it would have been nicer to use the same names with prefixes
> > .\" such as (say) "fdsi_" (thus, e.g., fdsi_signo); that would have
> > .\" made grepping both kernel and userland source code easier
> > .\" (e.g, to find instances of "si_signo" in a siginfo_t or a
> > .\" signalfd_siginfo structure), and also would have avoided
> > .\" weirdnesses like trimming "errno" down to err.
> > .\"
> 
> That is fine for me. I'd like ssi_* (for Signalfd SigInfo). Ok?

Yes, equally good.  Would you write a patch for this, and 
CC me please?

Thanks,

Michael

-- 
Michael Kerrisk
maintainer of Linux man pages Sections 2, 3, 4, 5, and 7 

Want to help with man page maintenance?  
Grab the latest tarball at
http://www.kernel.org/pub/linux/docs/manpages , 
read the HOWTOHELP file and grep the source 
files for 'FIXME'.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/