Re: Merge window closed - 2.6.39-rc1 out

[Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>]

On Wed, Mar 30, 2011 at 06:29:09PM -0400, Christoph Hellwig wrote:
>  (1) it still has the totally stupid interface of a global dispatch table.
>      Either it does make sense to have different providers, in which
>      case you want the dispatch table on a per-superblock or whatever
>      granularity, or you really want just one and can have static calls
>      into it, and get rid of this whole layer
>  (2) it still requires totally pointless calls from local filesystem to
>      initialize a pool ID.  The filesystem really should not need to
>      care about any of this.

Thank you for taking a look at the patches. I think what you are saying
is that the opt-in interface (where only those fs that want it) are using,
is not proper. It should be in for everybody? Hmm.. let me ponder this.

>  (3) it's still lacking a good user submitted with it.  And with that
>      I don't mean junk code shoved into staging where it's bitrotting.

I think when one takes user cases and try to use one type of matrix (say,
how many customers are using it) but use another (how many enterprise
clients are clamoring for it) you get drastically different numbers.

If you just use one ruler to evaluate acceptance, then maybe
unicore32 (just one company), or ILO (small community compared to SCST)
should have not been merged b/c of that? But if you think of other
acceptance rules then it makes sense.

> 
> It also has an entirely new bug, in that it assumes every inode has
> a dentry on the alias list.  Did you only ever test this with filesystem
> that do not have export operations?  Otherwise the no-dentry case should
> be fairly trivial to trigger due the placement of the hooks.

Aha! Thank you for spotting that. Dan did post a patch for this:
https://lkml.org/lkml/2011/3/3/291

And he should have pulled it in his tree <sigh> Will ask him to create
a branch with this bug-fix.
> 
> And of course there's still no convincing use case for all of this.