In worst case code do 4 comparison,
just add some new checks to switch check branch faster
now in worst case code do 3 comparison

Signed-off-by: Timofey Titovets <nefelim...@gmail.com>
---
 fs/btrfs/file.c | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/fs/btrfs/file.c b/fs/btrfs/file.c
index da1096eb1..39efda26c 100644
--- a/fs/btrfs/file.c
+++ b/fs/btrfs/file.c
@@ -71,16 +71,17 @@ struct inode_defrag {
 static int __compare_inode_defrag(struct inode_defrag *defrag1,
                                  struct inode_defrag *defrag2)
 {
-       if (defrag1->root > defrag2->root)
-               return 1;
-       else if (defrag1->root < defrag2->root)
+       if (defrag1->root != defrag2->root) {
+               if (defrag1->root > defrag2->root)
+                       return 1;
                return -1;
-       else if (defrag1->ino > defrag2->ino)
-               return 1;
-       else if (defrag1->ino < defrag2->ino)
+       }
+       if (defrag1->ino != defrag2->ino) {
+               if (defrag1->ino > defrag2->ino)
+                       return 1;
                return -1;
-       else
-               return 0;
+       }
+       return 0;
 }
 
 /* pop a record for an inode into the defrag tree.  The lock
-- 
2.13.0

--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to