connection: unlock order doesn't matter

Make kdbus_conn_unlock2() just unlock both mutices, unlock order never
matters.

Signed-off-by: David Herrmann <dh.herrmann@gmail.com>

diff --git a/connection.h b/connection.h
index ad438ebef3fdb6b43eea9144ac9024631df02221..62b1b0633be072dca16d5812144d0c2d9690b44f 100644 (file)
--- a/connection.h
+++ b/connection.h
@@ -221,8 +221,6 @@ static inline void kdbus_conn_lock2(struct kdbus_conn *a, struct kdbus_conn *b)
        if (a < b) {
                mutex_lock(&a->lock);
                mutex_lock_nested(&b->lock, 1);
-       } else if (a == b) {
-               mutex_lock(&a->lock);
        } else {
                mutex_lock(&b->lock);
                mutex_lock_nested(&a->lock, 1);
@@ -239,15 +237,8 @@ static inline void kdbus_conn_lock2(struct kdbus_conn *a, struct kdbus_conn *b)
 static inline void kdbus_conn_unlock2(struct kdbus_conn *a,
                                      struct kdbus_conn *b)
 {
-       if (a < b) {
-               mutex_unlock(&b->lock);
-               mutex_unlock(&a->lock);
-       } else if (a == b) {
-               mutex_unlock(&a->lock);
-       } else {
-               mutex_unlock(&a->lock);
-               mutex_unlock(&b->lock);
-       }
+       mutex_unlock(&a->lock);
+       mutex_unlock(&b->lock);
 }
 
 #endif