public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Michael Kerrisk" <mtk-manpages@gmx.net>
To: Davide Libenzi <davidel@xmailserver.org>
Cc: hch@lst.de, geoff@gclare.org.uk, subrata@linux.vnet.ibm.com,
	linux-kernel@vger.kernel.org
Subject: Re: Revised signalfd man-page
Date: Thu, 27 Sep 2007 19:19:50 +0200	[thread overview]
Message-ID: <20070927171950.252220@gmx.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0709271000420.763@alien.or.mcafeemobile.com>

> 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'.


  reply	other threads:[~2007-09-27 17:20 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-09-27 12:19 Revised signalfd man-page Michael Kerrisk
2007-09-27 12:40 ` Heikki Orsila
2007-09-27 12:44   ` Michael Kerrisk
2007-09-27 17:05 ` Davide Libenzi
2007-09-27 17:19   ` Michael Kerrisk [this message]
2007-10-15  6:54   ` Michael Kerrisk
2007-10-15 18:14     ` Davide Libenzi
2007-10-17  6:16       ` Michael Kerrisk
2007-10-17 22:52         ` Davide Libenzi

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20070927171950.252220@gmx.net \
    --to=mtk-manpages@gmx.net \
    --cc=davidel@xmailserver.org \
    --cc=geoff@gclare.org.uk \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=subrata@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox