Whamcloud - gitweb
LU-14316 llite: quiet spurious ioctl warning 27/41427/2
authorAndreas Dilger <adilger@whamcloud.com>
Fri, 5 Feb 2021 20:13:10 +0000 (13:13 -0700)
committerOleg Drokin <green@whamcloud.com>
Mon, 8 Feb 2021 21:55:38 +0000 (21:55 +0000)
Calling "lfs setstripe" prints a suprious warning about using the old
ioctl(LL_IOC_LOV_GETSTRIPE) when that is not actually the case.

Remove the ioctl warning for now and deal with related issues later.

Fixes: 364ec95f3688 ("LU-9367 llite: restore ll_file_getstripe in ll_lov_setstripe")
Signed-off-by: Andreas Dilger <adilger@whamcloud.com>
Change-Id: I20f5a7adb60a30fce27e49827bd46229e2ce7057
Reviewed-on: https://review.whamcloud.com/41427
Reviewed-by: John L. Hammond <jhammond@whamcloud.com>
Tested-by: jenkins <devops@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
lustre/lov/lov_pack.c

index 528f067..c0d5745 100644 (file)
@@ -363,7 +363,6 @@ int lov_getstripe(const struct lu_env *env, struct lov_object *obj,
        struct lov_user_md_v1 lum;
        size_t lmmk_size, lum_size = 0;
        ssize_t lmm_size;
-       static bool printed;
        int rc = 0;
 
        ENTRY;
@@ -376,14 +375,6 @@ int lov_getstripe(const struct lu_env *env, struct lov_object *obj,
                GOTO(out, rc = -EIO);
        }
 
-       if (!printed) {
-               LCONSOLE_WARN("%s: using old ioctl(LL_IOC_LOV_GETSTRIPE) on "
-                             DFID", use llapi_layout_get_by_path()\n",
-                             current->comm,
-                             PFID(&obj->lo_cl.co_lu.lo_header->loh_fid));
-               printed = true;
-       }
-
        lmmk_size = lov_comp_md_size(lsm);
 
        OBD_ALLOC_LARGE(lmmk, lmmk_size);