[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20120705174454.GD16586@moon>
Date: Thu, 5 Jul 2012 21:44:54 +0400
From: Cyrill Gorcunov <gorcunov@...nvz.org>
To: Pavel Emelyanov <xemul@...allels.com>
Cc: "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-fsdevel@...r.kernel.org" <linux-fsdevel@...r.kernel.org>,
Al Viro <viro@...IV.linux.org.uk>,
Alexey Dobriyan <adobriyan@...il.com>,
Andrew Morton <akpm@...ux-foundation.org>,
James Bottomley <jbottomley@...allels.com>
Subject: Re: [rfc 3/7] procfs: Add ability to plugin auxiliary fdinfo
providers
On Wed, Jul 04, 2012 at 11:39:30AM +0400, Pavel Emelyanov wrote:
> On 06/27/2012 03:01 PM, Cyrill Gorcunov wrote:
> > This patch brings ability to plugin auxiliary fdinfo providers.
> > For example in further patches eventfd, evenpoll and fsnotify
> > will print out information associated with files.
> >
> > This feature is CONFIG_CHECKPOINT_RESTORE guarded to eliminate
> > overhead for those who don't need it at all (this
> > unfortunately makes patch bigger than I wanted).
>
> How about introducing one more file_operations member and call it in the
> seq_show() (if exists)? Only inotify, eventfd and eventpoll will implement
> one. This will allow to avoid all this complexity with drivers.
Hi Pavel, yes, this might be an option, if people agree with this approach
I'll redo my patches (this will require to remake all the rest patches so
I would like to know people opinion before start doint that). Anyone?
Cyrill
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists