mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Tvrtko Ursulin <tvrtko.ursulin@sophos.com>
To: Andreas Gruenbacher <agruen@suse.de>
Cc: Eric Paris <eparis@redhat.com>,
	Christoph Hellwig <hch@infradead.org>,
	Matt Helsley <matthltc@us.ibm.com>,
	"torvalds@linux-foundation.org" <torvalds@linux-foundation.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"viro@zeniv.linux.org.uk" <viro@zeniv.linux.org.uk>,
	"akpm@linux-foundation.org" <akpm@linux-foundation.org>,
	Michael Kerrisk <michael.kerrisk@gmail.com>
Subject: Re: [GIT PULL] notification tree - try 37!
Date: Tue, 17 Aug 2010 11:12:41 +0100	[thread overview]
Message-ID: <201008171112.42097.tvrtko.ursulin@sophos.com> (raw)
In-Reply-To: <201008171201.10142.agruen@suse.de>

On Tuesday 17 Aug 2010 11:01:09 Andreas Gruenbacher wrote:
> On Tuesday 17 August 2010 11:45:25 Tvrtko Ursulin wrote:
> > Why no timeouts? It sounds like a feasible way to work around listeners
> > which have stopped working. (Timeout and -ETIME for example to be clear,
> > not allowing access).
>
> From the kernel's point of view, there is no way to guess how long those
> timeouts should be.  Watching for progress can be implemented in user space
> though.

That is why I said the timeout can be configurable. But I agree (and raise you
double :) that not only the kernel can not guess how long the timeout should
be, but the application can not know as well. One possible way to let fanotify
know that listeners is making progress is via some sort of a heartbeat
message.  Then application can set the timeout as a fail safe and perm events
can take as long as needed and we still prevent clogging the system.

How would you create a robust solution purely from userspace? With the current
interface?

> Setting errno to ETIME as a result of trying to access a file is likely to
> break some applications which are not prepared to receive this error
> condition; we cannot do that.

Why you think EPERM will be handled better?

> I'm quite sure that both of these issues have been discussed already.

Ok, I obviosuly missed it. Do you have a pointer perhaps?

Tvrtko

Sophos Plc, The Pentagon, Abingdon Science Park, Abingdon, OX14 3YP, United Kingdom.
Company Reg No 2096520. VAT Reg No GB 348 3873 20.

  reply	other threads:[~2010-08-17 10:12 UTC|newest]

Thread overview: 45+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-06 15:58 Eric Paris
2010-08-06 23:34 ` Matt Helsley
2010-08-07  0:06   ` Christoph Hellwig
2010-08-07 19:15     ` Eric Paris
2010-08-07 20:55       ` Matt Helsley
2010-08-16 20:32       ` Andreas Gruenbacher
2010-08-17  3:39         ` Eric Paris
2010-08-17  4:03           ` Matt Helsley
2010-08-17  8:09           ` Andreas Gruenbacher
2010-08-17 15:08             ` Eric Paris
2010-08-19 20:24               ` Andreas Gruenbacher
2010-08-19 20:32                 ` Andreas Gruenbacher
2010-08-19 20:42                 ` Eric Paris
2010-08-19 21:07                   ` Andreas Gruenbacher
2010-08-19 21:22                     ` Andreas Gruenbacher
2010-08-20  3:50                     ` Eric Paris
2010-08-20 12:38                       ` Andreas Gruenbacher
2010-08-23 16:46                         ` Eric Paris
2010-08-23 22:38                           ` Andreas Gruenbacher
2010-08-20  0:00             ` Andreas Gruenbacher
2010-08-17  8:38           ` Andreas Gruenbacher
2010-08-17 15:24             ` Eric Paris
2010-08-17 15:48               ` Andreas Gruenbacher
2010-08-18 14:18               ` Andreas Gruenbacher
2010-08-17  9:45           ` Tvrtko Ursulin
2010-08-17 10:01             ` Andreas Gruenbacher
2010-08-17 10:12               ` Tvrtko Ursulin [this message]
2010-08-17 10:55                 ` Tvrtko Ursulin
2010-08-17 15:27                   ` Eric Paris
2010-08-18 15:47         ` [GIT PULL] notification tree: directory events Andreas Gruenbacher
2010-08-18 15:59           ` Eric Paris
2010-08-18 16:42             ` Christoph Hellwig
2010-08-18 17:07               ` Eric Paris
2010-08-19 12:44             ` Andreas Gruenbacher
2010-08-19 15:00               ` Eric Paris
2010-08-19 23:41                 ` Andreas Gruenbacher
2010-08-20  3:38                   ` Eric Paris
2010-08-20  5:19                     ` Andreas Dilger
2010-08-20  9:21                       ` Christoph Hellwig
2010-08-20 15:29                         ` Andreas Gruenbacher
2010-08-20 20:39                           ` Andreas Dilger
2010-08-20  9:09                     ` Tvrtko Ursulin
2010-08-20 11:07                     ` Andreas Gruenbacher
2010-08-20 11:25                     ` Andreas Gruenbacher
2010-08-20 12:16                     ` Andreas Gruenbacher

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=201008171112.42097.tvrtko.ursulin@sophos.com \
    --to=tvrtko.ursulin@sophos.com \
    --cc=agruen@suse.de \
    --cc=akpm@linux-foundation.org \
    --cc=eparis@redhat.com \
    --cc=hch@infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=matthltc@us.ibm.com \
    --cc=michael.kerrisk@gmail.com \
    --cc=torvalds@linux-foundation.org \
    --cc=viro@zeniv.linux.org.uk \
    /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®