diff options
| author | Jeff Layton <jlayton@redhat.com> | 2010-10-28 11:16:44 -0400 |
|---|---|---|
| committer | Steve French <sfrench@us.ibm.com> | 2010-11-02 19:20:23 +0000 |
| commit | b647c35f77af9c07d336247b23014596e9f0a593 (patch) | |
| tree | 653949250681fd1c23ad529b631da793f95778a6 /fs/cifs/cifsglob.h | |
| parent | 413e661c136c52290de1ee19a1b049a4da9dbf51 (diff) | |
| download | lwn-b647c35f77af9c07d336247b23014596e9f0a593.tar.gz lwn-b647c35f77af9c07d336247b23014596e9f0a593.zip | |
cifs: convert tlink_tree to a rbtree
Radix trees are ideal when you want to track a bunch of pointers and
can't embed a tracking structure within the target of those pointers.
The tradeoff is an increase in memory, particularly if the tree is
sparse.
In CIFS, we use the tlink_tree to track tcon_link structs. A tcon_link
can never be in more than one tlink_tree, so there's no impediment to
using a rb_tree here instead of a radix tree.
Convert the new multiuser mount code to use a rb_tree instead. This
should reduce the memory required to manage the tlink_tree.
Signed-off-by: Jeff Layton <jlayton@redhat.com>
Signed-off-by: Steve French <sfrench@us.ibm.com>
Diffstat (limited to 'fs/cifs/cifsglob.h')
| -rw-r--r-- | fs/cifs/cifsglob.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/fs/cifs/cifsglob.h b/fs/cifs/cifsglob.h index f259e4d7612d..b577bf0a1bb3 100644 --- a/fs/cifs/cifsglob.h +++ b/fs/cifs/cifsglob.h @@ -336,7 +336,8 @@ struct cifsTconInfo { * "get" on the container. */ struct tcon_link { - unsigned long tl_index; + struct rb_node tl_rbnode; + uid_t tl_uid; unsigned long tl_flags; #define TCON_LINK_MASTER 0 #define TCON_LINK_PENDING 1 |