Re: [PATCH] jbd2: Create proc entry with bdevname+i_ino.

Joel Becker
2008-09-15T01:10:42Z

On Sun, Sep 14, 2008 at 12:06:35PM -0400, Christoph Hellwig wrote:
> On Sun, Sep 14, 2008 at 05:03:06AM -0400, Theodore Tso wrote:
> > On Sun, Sep 14, 2008 at 01:40:03AM -0700, Mark Fasheh wrote:
> > > On Mon, Sep 08, 2008 at 04:31:14PM -0700, Joel Becker wrote:
> > > > jbd2 currently creates statistics files for each loaded journal in
> > > > /proc/fs/jbd2/<bdevname>. ocfs2 loads multiple journals when recovering
> > > > other nodes, and the multiple journals on a given bdev collide in the
> > > > proc namespace.
> > >
> > > Did this patch ever get reviewied? We'd really like it in so that Ocfs2 can
> > > use JBD2...
> >
> > Thanks for pinging me; it fell through the cracks. I'll make sure it
> > gets put into the queue for merging at the next merge window....
>
> Could you please do it the proper way as describe in my reply to the
> thread?

Christoph,
Not sure what you mean here. Do you mean switching over
wholesale without the JBD compat config option? We're currently
tracking a spinlock deadlock in JBD2, so I don't know if we're ready for
that.
If you don't get this, I'll try to track you down tomorrow.

Joel

--

"You look in her eyes, the music begins to play.
Hopeless romantics, here we go again."

Joel Becker
Principal Software Developer
Oracle
E-mail: joel.becker@oracle.com
Phone: (650) 506-8127