From: Christoph Hellwig <hch@infradead.org>
To: Oleg Drokin <green@linuxhacker.ru>
Cc: linux-fsdevel@vger.kernel.org
Subject: Re: Attempt at "stat light" implementation
Date: Tue, 7 Apr 2009 13:49:21 -0400 [thread overview]
Message-ID: <20090407174921.GA24424@infradead.org> (raw)
In-Reply-To: <20090407062356.GA1336463@fiona.linuxhacker.ru>
On Tue, Apr 07, 2009 at 10:23:56AM +0400, Oleg Drokin wrote:
> Hello!
>
> I quickly realized that perhaps we can use just one extra syscall for fstat, and stat/lstat could be
> implemented as statat with just current cwd, though I do not know how desirable that is.
>
> Also I though that it would be kind of useful to allow the bitmask to be compatible with existing statat
> flags usage so that we do not need a new statat.
I think the best way to do it is to just define additional flags for the
*statat family, that if present only request partial stat information.
> Also perhaps the ->getattr just needs a prototype change to accept the flags argument and if the FS cares
> about it - to use it, but I did not want to do this huge patch touching every FS right now if only to save
> my time should this approach to be determined as undesirable for one reason or another, so
> for now I add a ->getattr_light
Yes, no additional method please. Adding the mask argument to getattr
is a pretty trivial search and replace.
next prev parent reply other threads:[~2009-04-07 17:49 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-04-07 6:23 Attempt at "stat light" implementation Oleg Drokin
2009-04-07 10:23 ` Andreas Dilger
2009-04-07 14:54 ` Oleg Drokin
2009-04-07 17:52 ` Christoph Hellwig
2009-04-07 15:22 ` jim owens
2009-04-07 15:38 ` Oleg Drokin
2009-04-07 16:20 ` jim owens
2009-04-07 17:32 ` Jamie Lokier
2009-04-07 17:38 ` Jeff Garzik
2009-04-07 17:56 ` Jamie Lokier
2009-04-07 18:21 ` Andreas Dilger
2009-04-07 19:30 ` Ulrich Drepper
2009-04-12 20:55 ` Jamie Lokier
2009-04-07 17:41 ` Oleg Drokin
2009-04-07 17:54 ` Jamie Lokier
2009-04-07 18:00 ` Oleg Drokin
2009-04-07 18:18 ` Andreas Dilger
2009-04-07 18:31 ` Nicholas Miell
2009-04-07 17:49 ` Christoph Hellwig [this message]
2009-04-07 17:56 ` Oleg Drokin
2009-04-07 18:28 ` Andreas Dilger
2009-04-07 18:50 ` Jamie Lokier
2009-04-07 19:48 ` Jeff Garzik
2009-04-07 19:47 ` Jeff Garzik
2009-04-07 20:18 ` Evgeniy Polyakov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090407174921.GA24424@infradead.org \
--to=hch@infradead.org \
--cc=green@linuxhacker.ru \
--cc=linux-fsdevel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.