On Mon, 11 Oct 2010 12:04:09 -0500
Steve French <[email protected]> wrote:
> find readable file is a common operation and the number of open files
> can be huge (thousands)
>
I don't think so. It gets called from:
get_cifs_acl
set_cifs_acl
...and those are only used when the cifsacl mount option is used, which
no one in their right mind does.
Opening the same inode thousands of times is a pretty atypical
workload. Even if it isn't though, we still return as soon as we find
the first usable open file.
But lets assume worst case -- we have some application that opens the
same inode thousands of times and none of them are usable. I'd still
argue that walking the entire list is a trivial amount of cpu time.
Keeping a list ordered like this is fragile and easily broken. I think
we shouldn't rely on it here.
--
Jeff Layton <[email protected]>
--
To unsubscribe from this list: send the line "unsubscribe linux-cifs" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
