Re: Patch to implement getattr [patch]

Alexander Viro (viro@math.psu.edu)
Wed, 14 Jul 1999 20:31:22 -0400 (EDT)


On 14 Jul 1999, Magnus Ahltorp wrote:

> > 1. Having getattr is nice, but I'ld rather turned the old code into
> > generic_getattr() and let the filesystems refer to it.
>
> Do you mean that implementation of the getattr operation should be
> mandatory, or do you mean that generic_getattr shold be called when
> getattr is not implemented, like in my patch, and that generic_getattr
> should be provided as a convenience function?

Either the former, or (at least) providing a wrapper that either calls
->getattr() or does the generic stuff.
>
> > 2. I think that having a wrapper around the thing (vfs_getattr()) would be
> > the right thing. Just look how arch/*/* stuff does stat() emulation for
> > personalities with different struct stat layout. set_fs();sys_stat();set_fs();
> > and then reshuffle the stuff...<shudder>
>
> I cannot find any stat emulation in arch/*/*. Do you have specific
> examples?

grep for irix_xstat(), solaris_stat(), solaris_stat64(), etc.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/