On February 25, 2002 07:28 pm, Alexander Viro wrote:
> On Sat, 23 Feb 2002, Daniel Phillips wrote:
> > Please tell me who wrote this:
> >
> > struct super_operations {
> > struct inode *(*alloc_inode)(struct super_block *sb);
> > void (*destroy_inode)(struct inode *)
>
> I had. With inodes it _does_ provide things that can't be done
> without these methods. Namely, common allocation of generic and
> fs-private part *on* *the* *fast* *path* *for* *class* *with*
> *many* *instances*.
>
> The latter parts are missing in case of superblocks. We don't
> allocate hundreds of thousands of superblocks. Moreover, ones
> allocated live much longer than normal struct inode.
>
> IOW, common allocation is worthless in this case and that's the
> only rationale for ->alloc_inode()/->destroy_inode().
You are being random. I'll leave the patch as it stands, I'm satisfied with
it. If you want to change it, go ahead, you have it in your mailbox. Tear
the whole thing up and rewrite it if you like. Just don't delay this
important work because of stupid personality issues.
-- Daniel - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
This archive was generated by hypermail 2b29 : Thu Feb 28 2002 - 21:00:22 EST