Drop the spinlock before returning if the export already has a connection.

I'm still not sure I agree that this is an error, but be that as it may,
I leave it for Mike & Peter to work out.

diff --git a/lustre/mds/handler.c b/lustre/mds/handler.c
index fa285e6..d8ea1e7 100644 (file)
--- a/lustre/mds/handler.c
+++ b/lustre/mds/handler.c
@@ -281,8 +281,9 @@ static int mds_connect(struct lustre_handle *conn, struct obd_device *obd,
                 if (!memcmp(cluuid, mcd->mcd_uuid, sizeof(mcd->mcd_uuid))) {
                         LASSERT(exp->exp_obd == obd);
 
-                        if (!list_empty(&exp->exp_conn_chain)) { 
+                        if (!list_empty(&exp->exp_conn_chain)) {
                                 CERROR("existing uuid/export, list not empty!\n");
+                                spin_unlock(&obd->obd_dev_lock);
                                 RETURN(-EALREADY);
                         }
                         conn->addr = (__u64) (unsigned long)exp;