Whamcloud - gitweb
LU-4245 ldlm: do not fill empty lvbo buffer 85/8285/2
authorBobi Jam <bobijam.xu@intel.com>
Fri, 15 Nov 2013 10:21:40 +0000 (18:21 +0800)
committerOleg Drokin <oleg.drokin@intel.com>
Mon, 2 Dec 2013 12:07:44 +0000 (12:07 +0000)
Don't fill empty lvbo buffer if server does not prepare it.

Signed-off-by: Bobi Jam <bobijam.xu@intel.com>
Change-Id: I22a2ddd016dcc43c00b2b3d6be5d18921bf26d67
Reviewed-on: http://review.whamcloud.com/8285
Tested-by: Jenkins
Reviewed-by: Jinshan Xiong <jinshan.xiong@intel.com>
Reviewed-by: Andreas Dilger <andreas.dilger@intel.com>
Tested-by: Maloo <hpdd-maloo@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
lustre/ldlm/ldlm_lockd.c

index 084e3e3..5d55852 100644 (file)
@@ -1400,7 +1400,7 @@ existing_lock:
                                         req, lock);
                                buflen = req_capsule_get_size(&req->rq_pill,
                                                &RMF_DLM_LVB, RCL_SERVER);
-                               if (buflen >= 0) {
+                               if (buflen > 0) {
                                        buflen = ldlm_lvbo_fill(lock, buf,
                                                                buflen);
                                        if (buflen >= 0)
@@ -1410,9 +1410,9 @@ existing_lock:
                                                        buflen, RCL_SERVER);
                                        else
                                                rc = buflen;
-                               }
-                               else
+                               } else {
                                        rc = buflen;
+                               }
                        }
                 } else {
                         lock_res_and_lock(lock);