staging: lustre: mdc: quiet console message for known -EINTR
authorAndreas Dilger <andreas.dilger@intel.com>
Sun, 29 Jan 2017 00:04:32 +0000 (19:04 -0500)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 3 Feb 2017 12:01:37 +0000 (13:01 +0100)
If a user process is waiting for MDS recovery during close, but the
process is interrupted, the file is still closed but it prints a
message on the console. Quiet the console message for -EINTR, since
this is expected behaviour.

Signed-off-by: Andreas Dilger <andreas.dilger@intel.com>
Intel-bug-id: https://jira.hpdd.intel.com/browse/LU-6627
Reviewed-on: http://review.whamcloud.com/14911
Reviewed-by: Frank Zago <fzago@cray.com>
Reviewed-by: Emoly Liu <emoly.liu@intel.com>
Reviewed-by: Oleg Drokin <oleg.drokin@intel.com>
Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/lustre/lustre/llite/file.c

index 0ee02f1..a1e51a5 100644 (file)
@@ -122,26 +122,25 @@ static int ll_close_inode_openhandle(struct obd_export *md_exp,
                                     enum mds_op_bias bias,
                                     void *data)
 {
-       struct obd_export *exp = ll_i2mdexp(inode);
+       const struct ll_inode_info *lli = ll_i2info(inode);
        struct md_op_data *op_data;
        struct ptlrpc_request *req = NULL;
-       struct obd_device *obd = class_exp2obd(exp);
        int rc;
 
-       if (!obd) {
-               /*
-                * XXX: in case of LMV, is this correct to access
-                * ->exp_handle?
-                */
-               CERROR("Invalid MDC connection handle %#llx\n",
-                      ll_i2mdexp(inode)->exp_handle.h_cookie);
+       if (!class_exp2obd(md_exp)) {
+               CERROR("%s: invalid MDC connection handle closing " DFID "\n",
+                      ll_get_fsname(inode->i_sb, NULL, 0),
+                      PFID(&lli->lli_fid));
                rc = 0;
                goto out;
        }
 
        op_data = kzalloc(sizeof(*op_data), GFP_NOFS);
+       /*
+        * We leak openhandle and request here on error, but not much to be
+        * done in OOM case since app won't retry close on error either.
+        */
        if (!op_data) {
-               /* XXX We leak openhandle and request here. */
                rc = -ENOMEM;
                goto out;
        }
@@ -170,10 +169,9 @@ static int ll_close_inode_openhandle(struct obd_export *md_exp,
        }
 
        rc = md_close(md_exp, op_data, och->och_mod, &req);
-       if (rc) {
-               CERROR("%s: inode "DFID" mdc close failed: rc = %d\n",
-                      ll_i2mdexp(inode)->exp_obd->obd_name,
-                      PFID(ll_inode2fid(inode)), rc);
+       if (rc && rc != -EINTR) {
+               CERROR("%s: inode " DFID " mdc close failed: rc = %d\n",
+                      md_exp->exp_obd->obd_name, PFID(&lli->lli_fid), rc);
        }
 
        if (op_data->op_bias & (MDS_HSM_RELEASE | MDS_CLOSE_LAYOUT_SWAP) &&
@@ -192,8 +190,7 @@ out:
        och->och_fh.cookie = DEAD_HANDLE_MAGIC;
        kfree(och);
 
-       if (req) /* This is close request */
-               ptlrpc_req_finished(req);
+       ptlrpc_req_finished(req);
        return rc;
 }