[Ocfs2-devel] [PATCH 4/4] ocfs2/dlm: Handle EAGAIN for compatibility
    Sunil Mushran 
    sunil.mushran at oracle.com
       
    Mon Jan 25 16:57:41 PST 2010
    
    
  
Mainline commit aad1b15310b9bcd59fa81ab8f2b1513b59553ea8 made
dlm_begin_reco_handler() return -EAGAIN instead of EAGAIN.
This patch makes dlm_send_begin_reco_message() handle both
error numbers for compatibility reasons.
Signed-off-by: Sunil Mushran <sunil.mushran at oracle.com>
---
 fs/ocfs2/dlm/dlmrecovery.c |    8 +++++++-
 1 files changed, 7 insertions(+), 1 deletions(-)
diff --git a/fs/ocfs2/dlm/dlmrecovery.c b/fs/ocfs2/dlm/dlmrecovery.c
index cfb2ae9..ad71221 100644
--- a/fs/ocfs2/dlm/dlmrecovery.c
+++ b/fs/ocfs2/dlm/dlmrecovery.c
@@ -2639,7 +2639,13 @@ retry:
 			     "begin reco msg (%d)\n", dlm->name, nodenum, ret);
 			ret = 0;
 		}
-		if (ret == -EAGAIN) {
+
+		/*
+		 * Prior to commit aad1b15310b9bcd59fa81ab8f2b1513b59553ea8,
+		 * dlm_begin_reco_handler() returned EAGAIN and not -EAGAIN.
+		 * We are handling both for compatibility reasons.
+		 */
+		if (ret == -EAGAIN || ret == EAGAIN) {
 			mlog(0, "%s: trying to start recovery of node "
 			     "%u, but node %u is waiting for last recovery "
 			     "to complete, backoff for a bit\n", dlm->name,
-- 
1.5.6.5
    
    
More information about the Ocfs2-devel
mailing list