Whamcloud - gitweb
LU-6825 ofd: statfs needs NOT check layout LFSCK status
[fs/lustre-release.git] / lustre / ofd / ofd_obd.c
index 5656ae9..cb7a2e9 100644 (file)
@@ -70,9 +70,7 @@ static int ofd_export_stats_init(struct ofd_device *ofd,
 {
        struct obd_device       *obd = ofd_obd(ofd);
        struct nid_stat         *stats;
-       int                      num_stats;
-       int                      rc, newnid = 0;
-
+       int                      rc;
        ENTRY;
 
        LASSERT(obd->obd_uses_nid_stats);
@@ -81,44 +79,34 @@ static int ofd_export_stats_init(struct ofd_device *ofd,
                /* Self-export gets no proc entry */
                RETURN(0);
 
-       rc = lprocfs_exp_setup(exp, client_nid, &newnid);
-       if (rc) {
-               /* Mask error for already created
-                * /proc entries */
-               if (rc == -EALREADY)
-                       rc = 0;
-               RETURN(rc);
-       }
-
-       if (newnid == 0)
-               RETURN(0);
+       rc = lprocfs_exp_setup(exp, client_nid);
+       if (rc != 0)
+               /* Mask error for already created /proc entries */
+               RETURN(rc == -EALREADY ? 0 : rc);
 
        stats = exp->exp_nid_stats;
-       LASSERT(stats != NULL);
-
-       num_stats = NUM_OBD_STATS + LPROC_OFD_STATS_LAST;
-
-       stats->nid_stats = lprocfs_alloc_stats(num_stats,
-                                              LPROCFS_STATS_FLAG_NOPERCPU);
+       stats->nid_stats = lprocfs_alloc_stats(NUM_OBD_STATS +
+                                               LPROC_OFD_STATS_LAST,
+                                               LPROCFS_STATS_FLAG_NOPERCPU);
        if (stats->nid_stats == NULL)
-               return -ENOMEM;
+               RETURN(-ENOMEM);
 
        lprocfs_init_ops_stats(LPROC_OFD_STATS_LAST, stats->nid_stats);
+
        ofd_stats_counter_init(stats->nid_stats);
-       rc = lprocfs_register_stats(stats->nid_proc, "stats",
-                                   stats->nid_stats);
-       if (rc)
-               GOTO(clean, rc);
 
-       rc = lprocfs_nid_ldlm_stats_init(stats);
-       if (rc) {
+       rc = lprocfs_register_stats(stats->nid_proc, "stats", stats->nid_stats);
+       if (rc != 0) {
                lprocfs_free_stats(&stats->nid_stats);
-               GOTO(clean, rc);
+               GOTO(out, rc);
        }
 
-       RETURN(0);
-clean:
-       return rc;
+       rc = lprocfs_nid_ldlm_stats_init(stats);
+       if (rc != 0)
+               GOTO(out, rc);
+
+out:
+       RETURN(rc);
 }
 
 /**
@@ -566,7 +554,7 @@ int ofd_postrecov(const struct lu_env *env, struct ofd_device *ofd)
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_obd_postrecov(struct obd_device *obd)
+static int ofd_obd_postrecov(struct obd_device *obd)
 {
        struct lu_env            env;
        struct lu_device        *ldev = obd->obd_lu_dev;
@@ -640,19 +628,17 @@ static int ofd_set_info_async(const struct lu_env *env, struct obd_export *exp,
  * \param[in]  key     key name
  * \param[out] vallen  length of key value
  * \param[out] val     the key value to return
- * \param[in]  lsm     not used in OFD
  *
  * \retval             0 if successful
  * \retval             negative value on error
  */
 static int ofd_get_info(const struct lu_env *env, struct obd_export *exp,
-                       __u32 keylen, void *key, __u32 *vallen, void *val,
-                       struct lov_stripe_md *lsm)
+                       __u32 keylen, void *key, __u32 *vallen, void *val)
 {
        struct ofd_thread_info          *info;
        struct ofd_device               *ofd;
        struct ll_fiemap_info_key       *fm_key = key;
-       struct ll_user_fiemap           *fiemap = val;
+       struct fiemap                   *fiemap = val;
        int                              rc = 0;
 
        ENTRY;
@@ -667,7 +653,7 @@ static int ofd_get_info(const struct lu_env *env, struct obd_export *exp,
        if (KEY_IS(KEY_FIEMAP)) {
                info = ofd_info_init(env, exp);
 
-               rc = ostid_to_fid(&info->fti_fid, &fm_key->oa.o_oi,
+               rc = ostid_to_fid(&info->fti_fid, &fm_key->lfik_oa.o_oi,
                                  ofd->ofd_lut.lut_lsd.lsd_osd_index);
                if (rc != 0)
                        RETURN(rc);
@@ -702,16 +688,9 @@ int ofd_statfs_internal(const struct lu_env *env, struct ofd_device *ofd,
        int rc = 0;
        ENTRY;
 
-       down_read(&ofd->ofd_lastid_rwsem);
-       /* Currently, for safe, we do not distinguish which LAST_ID is broken,
-        * we may do that in the future.
-        * Return -ENOSPC until the LAST_ID rebuilt. */
-       if (unlikely(ofd->ofd_lastid_rebuilding))
-               GOTO(out, rc = -ENOSPC);
-
        spin_lock(&ofd->ofd_osfs_lock);
        if (cfs_time_before_64(ofd->ofd_osfs_age, max_age) || max_age == 0) {
-               obd_size unstable;
+               u64 unstable;
 
                /* statfs data are too old, get up-to-date one.
                 * we must be cautious here since multiple threads might be
@@ -747,7 +726,7 @@ int ofd_statfs_internal(const struct lu_env *env, struct ofd_device *ofd,
                         * w/o the ofd_osfs_lock. Those ones got added to
                         * the cached statfs data that we are about to crunch.
                         * Take them into account in the new statfs data */
-                       osfs->os_bavail -= min_t(obd_size, osfs->os_bavail,
+                       osfs->os_bavail -= min_t(u64, osfs->os_bavail,
                                               unstable >> ofd->ofd_blockbits);
                        /* However, we don't really know if those writes got
                         * accounted in the statfs call, so tell
@@ -781,12 +760,9 @@ int ofd_statfs_internal(const struct lu_env *env, struct ofd_device *ofd,
                if (from_cache)
                        *from_cache = 1;
        }
-
        GOTO(out, rc);
 
 out:
-       up_read(&ofd->ofd_lastid_rwsem);
-
        return rc;
 }
 
@@ -834,7 +810,7 @@ int ofd_statfs(const struct lu_env *env,  struct obd_export *exp,
               osfs->os_bfree << ofd->ofd_blockbits,
               osfs->os_bavail << ofd->ofd_blockbits);
 
-       osfs->os_bavail -= min_t(obd_size, osfs->os_bavail,
+       osfs->os_bavail -= min_t(u64, osfs->os_bavail,
                                 ((ofd->ofd_tot_dirty + ofd->ofd_tot_pending +
                                   osfs->os_bsize - 1) >> ofd->ofd_blockbits));
 
@@ -844,7 +820,7 @@ int ofd_statfs(const struct lu_env *env,  struct obd_export *exp,
                struct filter_export_data *fed;
 
                fed = &obd->obd_self_export->exp_filter_data;
-               osfs->os_bavail -= min_t(obd_size, osfs->os_bavail,
+               osfs->os_bavail -= min_t(u64, osfs->os_bavail,
                                         fed->fed_grant >> ofd->ofd_blockbits);
        }
 
@@ -891,21 +867,19 @@ out:
 
  * \param[in] env      execution environment
  * \param[in] exp      OBD export of OFD device
- * \param[in] oinfo    obd_info with setattr parameters
- * \param[in] oti      not used in OFD
+ * \param[in] oa       setattr parameters
  *
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_echo_setattr(const struct lu_env *env, struct obd_export *exp,
-                    struct obd_info *oinfo, struct obd_trans_info *oti)
+static int ofd_echo_setattr(const struct lu_env *env, struct obd_export *exp,
+                           struct obdo *oa)
 {
        struct ofd_thread_info  *info;
        struct ofd_device       *ofd = ofd_exp(exp);
        struct ldlm_namespace   *ns = ofd->ofd_namespace;
        struct ldlm_resource    *res;
        struct ofd_object       *fo;
-       struct obdo             *oa = oinfo->oi_oa;
        struct lu_fid           *fid = &oa->o_oi.oi_fid;
        struct filter_fid       *ff = NULL;
        int                      rc = 0;
@@ -918,8 +892,7 @@ int ofd_echo_setattr(const struct lu_env *env, struct obd_export *exp,
 
        /* This would be very bad - accidentally truncating a file when
         * changing the time or similar - bug 12203. */
-       if (oa->o_valid & OBD_MD_FLSIZE &&
-           oinfo->oi_policy.l_extent.end != OBD_OBJECT_EOF) {
+       if (oa->o_valid & OBD_MD_FLSIZE) {
                static char mdsinum[48];
 
                if (oa->o_valid & OBD_MD_FLFID)
@@ -1039,7 +1012,6 @@ int ofd_destroy_by_fid(const struct lu_env *env, struct ofd_device *ofd,
  * \param[in] env      execution environment
  * \param[in] exp      OBD export of OFD device
  * \param[in] oa       obdo structure with FID
- * \param[in] oti      not used in OFD
  *
  * Note: this is OBD API method which is common API for server OBDs and
  * client OBDs. Thus some parameters used in client OBDs may not be used
@@ -1048,8 +1020,8 @@ int ofd_destroy_by_fid(const struct lu_env *env, struct ofd_device *ofd,
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_echo_destroy(const struct lu_env *env, struct obd_export *exp,
-                    struct obdo *oa, struct obd_trans_info *oti)
+static int ofd_echo_destroy(const struct lu_env *env, struct obd_export *exp,
+                           struct obdo *oa)
 {
        struct ofd_device       *ofd = ofd_exp(exp);
        struct lu_fid           *fid = &oa->o_oi.oi_fid;
@@ -1086,8 +1058,6 @@ out:
  * \param[in]  env     execution environment
  * \param[in]  exp     OBD export of OFD device
  * \param[in]  oa      obdo structure with FID sequence to use
- * \param[out] ea      contains object ID/SEQ to return
- * \param[in]  oti     not used in OFD
  *
  * Note: this is OBD API method which is common API for server OBDs and
  * client OBDs. Thus some parameters used in client OBDs may not be used
@@ -1096,15 +1066,15 @@ out:
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_echo_create(const struct lu_env *env, struct obd_export *exp,
-                   struct obdo *oa, struct obd_trans_info *oti)
+static int ofd_echo_create(const struct lu_env *env, struct obd_export *exp,
+                          struct obdo *oa)
 {
        struct ofd_device       *ofd = ofd_exp(exp);
        struct ofd_thread_info  *info;
-       obd_seq                  seq = ostid_seq(&oa->o_oi);
+       u64                      seq = ostid_seq(&oa->o_oi);
        struct ofd_seq          *oseq;
        int                      rc = 0, diff = 1;
-       obd_id                   next_id;
+       u64                      next_id;
        int                      count;
 
        ENTRY;
@@ -1171,18 +1141,18 @@ out_sem:
  *
  * \param[in]    env   execution environment
  * \param[in]    exp   OBD export of OFD device
- * \param[in,out] oinfo        contains FID of object to get attributes from and
+ * \param[in,out] oa   contains FID of object to get attributes from and
  *                     is used to return attributes back
  *
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_echo_getattr(const struct lu_env *env, struct obd_export *exp,
-                    struct obd_info *oinfo)
+static int ofd_echo_getattr(const struct lu_env *env, struct obd_export *exp,
+                           struct obdo *oa)
 {
        struct ofd_device       *ofd = ofd_exp(exp);
        struct ofd_thread_info  *info;
-       struct lu_fid           *fid = &oinfo->oi_oa->o_oi.oi_fid;
+       struct lu_fid           *fid = &oa->o_oi.oi_fid;
        struct ofd_object       *fo;
        int                      rc = 0;
 
@@ -1196,18 +1166,18 @@ int ofd_echo_getattr(const struct lu_env *env, struct obd_export *exp,
 
        LASSERT(fo != NULL);
        rc = ofd_attr_get(env, fo, &info->fti_attr);
-       oinfo->oi_oa->o_valid = OBD_MD_FLID;
+       oa->o_valid = OBD_MD_FLID;
        if (rc == 0) {
                __u64 curr_version;
 
-               obdo_from_la(oinfo->oi_oa, &info->fti_attr,
+               obdo_from_la(oa, &info->fti_attr,
                             OFD_VALID_FLAGS | LA_UID | LA_GID);
 
                /* Store object version in reply */
                curr_version = dt_version_get(env, ofd_object_child(fo));
                if ((__s64)curr_version != -EOPNOTSUPP) {
-                       oinfo->oi_oa->o_valid |= OBD_MD_FLDATAVERSION;
-                       oinfo->oi_oa->o_data_version = curr_version;
+                       oa->o_valid |= OBD_MD_FLDATAVERSION;
+                       oa->o_data_version = curr_version;
                }
        }
 
@@ -1302,8 +1272,8 @@ out:
  * \retval             0 if successful
  * \retval             negative value on error
  */
-int ofd_iocontrol(unsigned int cmd, struct obd_export *exp, int len,
-                 void *karg, void *uarg)
+static int ofd_iocontrol(unsigned int cmd, struct obd_export *exp, int len,
+                        void *karg, void *uarg)
 {
        struct lu_env            env;
        struct ofd_device       *ofd = ofd_exp(exp);
@@ -1320,6 +1290,7 @@ int ofd_iocontrol(unsigned int cmd, struct obd_export *exp, int len,
        switch (cmd) {
        case OBD_IOC_ABORT_RECOVERY:
                CERROR("%s: aborting recovery\n", obd->obd_name);
+               obd->obd_force_abort_recovery = 1;
                target_stop_recovery_thread(obd);
                break;
        case OBD_IOC_SYNC: