To make the code clearer, use rb_entry() instead of container_of() to
deal with rbtree.

Signed-off-by: Geliang Tang <geliangt...@gmail.com>
---
 fs/proc/generic.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/fs/proc/generic.c b/fs/proc/generic.c
index 7eb3cef..f842650 100644
--- a/fs/proc/generic.c
+++ b/fs/proc/generic.c
@@ -57,9 +57,9 @@ static struct proc_dir_entry *pde_subdir_find(struct 
proc_dir_entry *dir,
        struct rb_node *node = dir->subdir.rb_node;
 
        while (node) {
-               struct proc_dir_entry *de = container_of(node,
-                                                        struct proc_dir_entry,
-                                                        subdir_node);
+               struct proc_dir_entry *de = rb_entry(node,
+                                                    struct proc_dir_entry,
+                                                    subdir_node);
                int result = proc_match(len, name, de);
 
                if (result < 0)
@@ -80,8 +80,9 @@ static bool pde_subdir_insert(struct proc_dir_entry *dir,
 
        /* Figure out where to put new node */
        while (*new) {
-               struct proc_dir_entry *this =
-                       container_of(*new, struct proc_dir_entry, subdir_node);
+               struct proc_dir_entry *this = rb_entry(*new,
+                                                      struct proc_dir_entry,
+                                                      subdir_node);
                int result = proc_match(de->namelen, de->name, this);
 
                parent = *new;
-- 
2.9.3

Reply via email to