linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Andrew Morton <akpm@osdl.org>
To: John McCutchan <ttb@tentacle.dhs.org>, Christoph Hellwig <hch@lst.de>
Cc: arnd@arndb.de, rml@novell.com, zab@zabbo.net,
	linux-kernel@vger.kernel.org,
	viro@parcelfarce.linux.theplanet.co.uk
Subject: Re: [patch] inotify.
Date: Fri, 17 Jun 2005 14:33:34 -0700	[thread overview]
Message-ID: <20050617143334.41a31707.akpm@osdl.org> (raw)
In-Reply-To: <20050617175605.GB1981@tentacle.dhs.org>

John McCutchan <ttb@tentacle.dhs.org> wrote:
>
> This was settled a long time ago. Robert, Andrew, and I had an off-list
> discussion months ago, and we all agreed that this was the right
> interface for inotify.

I don't think I ever really affirmatively agreed to anything.  I do recall
various things being discussed at various times and various things being
changed, but from where I sit it's all spread out and foggy.

I certainly remember that good-sounding recommendations which addressed the
things which Christoph doesn't like were convincingly shot down by yourself
and by Robert, but I don't recall why.

Look, this stuff is hard.  This is why I've asked you and Robert again and
again and again to generate some sort of design doc or FAQ which addresses
each of these frequently-asked-questions.  So the poor rest of us can look
through it and say "oh yeah".  Because inotify _is_ a tricky thing, and
standard kernel interface designs _don't_ fit it well.

So.  It's not too late.  Please spend an hour and write up the Inofity
Implementation FAQ?  You probably remember and fully understand what all of
our objections are and I know that you have explanations and rebuttals at
hand.

Please?  Something like:

q: Why does it use an ioctl multiplexer

a: Because ...

etc...

I haven't done a detailed review of the patch in months and I intend to do
another soon.  That FAQ will help!  When I ask more silly questions we can
update it, so those questions will never again be asked.

I know it's unusual process-wise, but inotify is an unusual feature.

  reply	other threads:[~2005-06-17 21:36 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-06-15 17:18 [patch] inotify Robert Love
2005-06-16 17:52 ` Zach Brown
2005-06-16 18:25   ` Robert Love
2005-06-17  1:30     ` Nick Piggin
2005-06-17  1:35       ` Robert Love
2005-06-17 15:15         ` [patch] inotify, improved Robert Love
2005-06-17 15:37           ` Chris Friesen
2005-06-17 15:44             ` Robert Love
2005-06-17 16:11               ` Valdis.Kletnieks
2005-06-17 16:29                 ` Robert Love
2005-06-17 16:36                 ` Chris Friesen
2005-06-17 16:43                   ` Chris Wright
2005-06-17 16:46                   ` Muli Ben-Yehuda
2005-06-17 16:40               ` Chris Friesen
2005-06-17 17:57                 ` John McCutchan
2005-06-17 17:20           ` Zach Brown
2005-06-17 17:54             ` John McCutchan
2005-06-17 17:56               ` Zach Brown
2005-06-17 18:15                 ` John McCutchan
2005-06-17 18:17                   ` Zach Brown
2005-06-17 17:07     ` [patch] inotify Arnd Bergmann
2005-06-17 17:54       ` Christoph Hellwig
2005-06-17 18:12         ` John McCutchan
2005-06-17 18:16         ` Robert Love
2005-06-17 18:28           ` Christoph Hellwig
2005-06-17 18:38             ` Robert Love
2005-06-17 18:45               ` Christoph Hellwig
2005-06-17 18:54                 ` Robert Love
2005-06-17 17:56       ` John McCutchan
2005-06-17 21:33         ` Andrew Morton [this message]
2005-06-17 21:40           ` Robert Love
2005-06-17 23:52             ` Robert Love
2005-06-21  0:51               ` Neil Brown
2005-06-21  2:15                 ` John McCutchan
2005-06-21  2:29                   ` Neil Brown
2005-06-21  2:43                     ` John McCutchan
2005-06-21 15:55                     ` Robert Love
2005-07-14  0:25                       ` Neil Brown
2005-07-14  4:11                         ` John McCutchan
2005-06-18  0:05             ` Arnd Bergmann
2005-06-18  0:57               ` Robert Love
2005-06-18  1:51       ` Chris Wedgwood
  -- strict thread matches above, loose matches on Subject: below --
2005-05-09 16:05 Robert Love
2005-05-09 17:43 ` Coywolf Qi Hunt
2005-01-06 20:00 Robert Love

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=20050617143334.41a31707.akpm@osdl.org \
    --to=akpm@osdl.org \
    --cc=arnd@arndb.de \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=rml@novell.com \
    --cc=ttb@tentacle.dhs.org \
    --cc=viro@parcelfarce.linux.theplanet.co.uk \
    --cc=zab@zabbo.net \
    /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;
as well as URLs for NNTP newsgroup(s).