[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <20131119111719.GA25036@quack.suse.cz>
Date: Tue, 19 Nov 2013 12:17:19 +0100
From: Jan Kara <jack@...e.cz>
To: Christoph Hellwig <hch@...radead.org>
Cc: Dave Chinner <david@...morbit.com>,
Dmitry Monakhov <dmonakhov@...nvz.org>,
linux-ext4@...r.kernel.org, linux-fsdevel@...r.kernel.org,
Jan Kara <jack@...e.cz>
Subject: Re: [PATCH] ext4: add generic uevent infrastructure
On Tue 19-11-13 02:25:01, Christoph Hellwig wrote:
> On Tue, Nov 19, 2013 at 12:36:21PM +1100, Dave Chinner wrote:
> > I certain agree with that. I'd also like to see ENOSPC notifications
> > as that would obliviate the need for distros like RHEL to ship
> > systemtap scripts to generate such notifications for admins....
>
> The ENOSPC case would be a natural tag on to Jan's quota notification,
> and I have a vague memory that someone started implementing it or
> at least talked about it.
Yes, quota netlink interface is technically very easy to extend to also
provide ENOSPC notifications. It's just that the name of generic netlink
family is 'VFS_DQUOT' so ENOSPC notifications do not fit very well with
that name but it isn't too bad either.
> > > Also Jan Kara has done quota netlink notifications a while ago, which
> > > fit into the same sort of niche.
> >
> > The question I'm asking is whether we really want a new interface
> > for these events? Shouldn't we really try to use an existing
> > filesystem event interface for generating these events
>
> Good quetion, and the quota netlink notifications would be the natural
> place to tag on at least some of this.
>
> > (e.g.
> > fanotify) rather than adding yet another disjoint filesystem event
> > interface to the kernel?
>
> It needs to be a per-fs interface, and as Dmitry pointed out fanotify
> is a per-file one.
I agree fanotify is really a bad fit.
Honza
--
Jan Kara <jack@...e.cz>
SUSE Labs, CR
--
To unsubscribe from this list: send the line "unsubscribe linux-ext4" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists