mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Jesse Pollard <pollard@tomcat.admin.navo.hpc.mil>
To: phillips@innominate.de,
	Jamie Lokier <lk@tantalophile.demon.co.uk>,
	linux-kernel@vger.kernel.org
Subject: Re: FS callback routines
Date: Thu, 11 Jan 2001 10:39:25 -0600 (CST)	[thread overview]
Message-ID: <200101111639.KAA72525@tomcat.admin.navo.hpc.mil> (raw)

Daniel Phillips <phillips@innominate.de>:
> Jamie Lokier wrote:
> > 
> > Daniel Phillips wrote:
> > >         DN_OPEN       A file in the directory was opened
> > >
> > > You open the top level directory and register for events.  When somebody
> > > opens a subdirectory of the top level directory, you receive
> > > notification and register for events on the subdirectory, and so on,
> > > down to the file that is actually modified.
> > 
> > If it worked, and I'm not sure the timing would be reliable enough, the
> > daemon would only have to have open every directory being accessed by
> > every program in the system.  Hmm.  Seems like overkill when you're only
> > interested in files that are being modified.
> 
> It gets to close some too.  Normally just the directories in the path to
> the file(s) being modified would be open.
> 
> Good point about the timing.  A directory should not disappear before an
> in-flight notification has been serviced.  I doubt the current scheme
> enforces this.  There is no more room for 'works most of the time' in
> this than there is in our memory page handling.
> 
> > It would be much, much more reliable to do a walk over d_parent in
> > dnotify.c.  Your idea is a nice way to flag kernel dentries such that
> > you don't do d_parent walks unnecessarily.
> 
> It's bottom-up vs top-down.  It's worth analyzing the top-down approach
> a little more, it does solve a lot of problems (and creates some as you
> pointed out, or at least makes some existing problems more obvious). 
> For make it's really quite nice.  The make daemon only needs to register
> in the top level directory of the source tree.  I think this solves the
> hard link problem too, because each path that's interested in
> notification will receive it.

It makes security checks impossible though. You would have to reboot
the system every time a directory changes permission to block unauthorized
monitoring of files that are no longer accessable by the user.

-------------------------------------------------------------------------
Jesse I Pollard, II
Email: pollard@navo.hpc.mil

Any opinions expressed are solely my own.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

             reply	other threads:[~2001-01-11 16:39 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-01-11 16:39 Jesse Pollard [this message]
2001-01-11 17:53 ` Daniel Phillips
  -- strict thread matches above, loose matches on Subject: below --
2001-01-10 15:18 Jesse Pollard
2001-01-09 14:05 Jesse Pollard
2001-01-09 15:41 ` Daniel Phillips
2001-01-10 10:48   ` Jamie Lokier
2001-01-09  1:21 Sean R. Bright
2001-01-09 11:22 ` Philipp Matthias Hahn
2001-01-09 11:34 ` Daniel Stodden
2001-01-08 23:12 Michael D. Crawford
2001-01-09  2:37 ` Sean R. Bright
2001-01-09  3:48 ` David Weinehall
2001-01-09 13:07 ` Daniel Phillips
2001-01-10 10:56   ` Jamie Lokier
2001-01-10 18:25     ` Daniel Phillips
2001-01-11 14:30     ` Daniel Phillips
2001-01-11 15:37       ` Jamie Lokier
2001-01-11 16:11         ` Daniel Phillips

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=200101111639.KAA72525@tomcat.admin.navo.hpc.mil \
    --to=pollard@tomcat.admin.navo.hpc.mil \
    --cc=linux-kernel@vger.kernel.org \
    --cc=lk@tantalophile.demon.co.uk \
    --cc=phillips@innominate.de \
    /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

all inboxes | Powered by JetHome®