Whamcloud - gitweb
LU-11765 ofd: return EAGAIN during 1st CLEANUP_ORPHAN
[fs/lustre-release.git] / lustre / ofd / ofd_dev.c
index 685bdf0..af33796 100644 (file)
@@ -320,13 +320,7 @@ static bool match_symlink_param(char *param)
                sval = strchr(param, '=');
                if (sval != NULL) {
                        paramlen = sval - param;
-                       if (strncmp(param, "writethrough_cache_enable",
-                                   paramlen) == 0 ||
-                           strncmp(param, "readcache_max_filesize",
-                                   paramlen) == 0 ||
-                           strncmp(param, "read_cache_enable",
-                                   paramlen) == 0 ||
-                           strncmp(param, "brw_stats", paramlen) == 0)
+                       if (strncmp(param, "brw_stats", paramlen) == 0)
                                return true;
                }
        }
@@ -359,11 +353,11 @@ static int ofd_process_config(const struct lu_env *env, struct lu_device *d,
 
        switch (cfg->lcfg_command) {
        case LCFG_PARAM: {
-               struct obd_device       *obd = ofd_obd(m);
                /* For interoperability */
-               struct cfg_interop_param   *ptr = NULL;
-               struct lustre_cfg          *old_cfg = NULL;
-               char                       *param = NULL;
+               struct cfg_interop_param *ptr = NULL;
+               struct lustre_cfg *old_cfg = NULL;
+               char *param = NULL;
+               ssize_t count;
 
                param = lustre_cfg_string(cfg, 1);
                if (param == NULL) {
@@ -397,14 +391,16 @@ static int ofd_process_config(const struct lu_env *env, struct lu_device *d,
                        break;
                }
 
-               rc = class_process_proc_param(PARAM_OST, obd->obd_vars, cfg,
-                                             d->ld_obd);
-               if (rc > 0 || rc == -ENOSYS) {
-                       CDEBUG(D_CONFIG, "pass param %s down the stack.\n",
-                              param);
-                       /* we don't understand; pass it on */
-                       rc = next->ld_ops->ldo_process_config(env, next, cfg);
+               count = class_modify_config(cfg, PARAM_OST,
+                                           &d->ld_obd->obd_kset.kobj);
+               if (count > 0) {
+                       rc = 0;
+                       break;
                }
+               CDEBUG(D_CONFIG, "pass param %s down the stack.\n",
+                      param);
+               /* we don't understand; pass it on */
+               rc = next->ld_ops->ldo_process_config(env, next, cfg);
                break;
        }
        case LCFG_SPTLRPC_CONF: {
@@ -699,73 +695,7 @@ static struct lu_device_operations ofd_lu_ops = {
        .ldo_prepare            = ofd_prepare,
 };
 
-LPROC_SEQ_FOPS(lprocfs_nid_stats_clear);
-
-/**
- * Initialize all needed procfs entries for OFD device.
- *
- * \param[in] ofd      OFD device
- *
- * \retval             0 if successful
- * \retval             negative value on error
- */
-static int ofd_procfs_init(struct ofd_device *ofd)
-{
-       struct obd_device               *obd = ofd_obd(ofd);
-       struct proc_dir_entry           *entry;
-       int                              rc = 0;
-
-       ENTRY;
-
-       /* lprocfs must be setup before the ofd so state can be safely added
-        * to /proc incrementally as the ofd is setup */
-       obd->obd_vars = lprocfs_ofd_obd_vars;
-       rc = lprocfs_obd_setup(obd, false);
-       if (rc) {
-               CERROR("%s: lprocfs_obd_setup failed: %d.\n",
-                      obd->obd_name, rc);
-               RETURN(rc);
-       }
-
-       rc = lprocfs_alloc_obd_stats(obd, LPROC_OFD_STATS_LAST);
-       if (rc) {
-               CERROR("%s: lprocfs_alloc_obd_stats failed: %d.\n",
-                      obd->obd_name, rc);
-               GOTO(obd_cleanup, rc);
-       }
-
-       entry = lprocfs_register("exports", obd->obd_proc_entry, NULL, NULL);
-       if (IS_ERR(entry)) {
-               rc = PTR_ERR(entry);
-               CERROR("%s: error %d setting up lprocfs for %s\n",
-                      obd->obd_name, rc, "exports");
-               GOTO(obd_cleanup, rc);
-       }
-       obd->obd_proc_exports_entry = entry;
-
-       entry = lprocfs_add_simple(obd->obd_proc_exports_entry, "clear",
-                                  obd, &lprocfs_nid_stats_clear_fops);
-       if (IS_ERR(entry)) {
-               rc = PTR_ERR(entry);
-               CERROR("%s: add proc entry 'clear' failed: %d.\n",
-                      obd->obd_name, rc);
-               GOTO(obd_cleanup, rc);
-       }
-
-       ofd_stats_counter_init(obd->obd_stats);
-
-       rc = lprocfs_job_stats_init(obd, LPROC_OFD_STATS_LAST,
-                                   ofd_stats_counter_init);
-       if (rc)
-               GOTO(obd_cleanup, rc);
-       RETURN(0);
-obd_cleanup:
-       lprocfs_obd_cleanup(obd);
-       lprocfs_free_obd_stats(obd);
-
-       return rc;
-}
-
+#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 14, 53, 0)
 /**
  * Expose OSD statistics to OFD layer.
  *
@@ -779,8 +709,31 @@ obd_cleanup:
  */
 static void ofd_procfs_add_brw_stats_symlink(struct ofd_device *ofd)
 {
-       struct obd_device       *obd = ofd_obd(ofd);
-       struct obd_device       *osd_obd = ofd->ofd_osd_exp->exp_obd;
+       struct obd_device *obd = ofd_obd(ofd);
+       struct obd_device *osd_obd = ofd->ofd_osd_exp->exp_obd;
+       struct kobj_type *osd_type;
+       int i;
+
+       osd_type = get_ktype(&ofd->ofd_osd->dd_kobj);
+       for (i = 0; osd_type->default_attrs[i]; i++) {
+               if (strcmp(osd_type->default_attrs[i]->name,
+                          "read_cache_enable") == 0) {
+                       ofd->ofd_read_cache_enable =
+                               osd_type->default_attrs[i];
+               }
+
+               if (strcmp(osd_type->default_attrs[i]->name,
+                          "readcache_max_filesize") == 0) {
+                       ofd->ofd_read_cache_max_filesize =
+                               osd_type->default_attrs[i];
+               }
+
+               if (strcmp(osd_type->default_attrs[i]->name,
+                          "writethrough_cache_enable") == 0) {
+                       ofd->ofd_write_cache_enable =
+                               osd_type->default_attrs[i];
+               }
+       }
 
        if (obd->obd_proc_entry == NULL)
                return;
@@ -788,21 +741,8 @@ static void ofd_procfs_add_brw_stats_symlink(struct ofd_device *ofd)
        lprocfs_add_symlink("brw_stats", obd->obd_proc_entry,
                            "../../%s/%s/brw_stats",
                            osd_obd->obd_type->typ_name, obd->obd_name);
-
-       lprocfs_add_symlink("read_cache_enable", obd->obd_proc_entry,
-                           "../../%s/%s/read_cache_enable",
-                           osd_obd->obd_type->typ_name, obd->obd_name);
-
-       lprocfs_add_symlink("readcache_max_filesize",
-                           obd->obd_proc_entry,
-                           "../../%s/%s/readcache_max_filesize",
-                           osd_obd->obd_type->typ_name, obd->obd_name);
-
-       lprocfs_add_symlink("writethrough_cache_enable",
-                           obd->obd_proc_entry,
-                           "../../%s/%s/writethrough_cache_enable",
-                           osd_obd->obd_type->typ_name, obd->obd_name);
 }
+#endif
 
 /**
  * Cleanup all procfs entries in OFD.
@@ -852,18 +792,19 @@ int ofd_fid_fini(const struct lu_env *env, struct ofd_device *ofd)
  */
 int ofd_fid_init(const struct lu_env *env, struct ofd_device *ofd)
 {
-       struct seq_server_site  *ss = &ofd->ofd_seq_site;
-       struct lu_device        *lu = &ofd->ofd_dt_dev.dd_lu_dev;
-       char                    *obd_name = ofd_name(ofd);
-       char                    *name = NULL;
-       int                     rc = 0;
+       struct seq_server_site *ss = &ofd->ofd_seq_site;
+       struct lu_device *lu = &ofd->ofd_dt_dev.dd_lu_dev;
+       char *obd_name = ofd_name(ofd);
+       char *name = NULL;
+       int len = strlen(obd_name) + 7;
+       int rc = 0;
 
        ss = &ofd->ofd_seq_site;
        lu->ld_site->ld_seq_site = ss;
        ss->ss_lu = lu->ld_site;
        ss->ss_node_id = ofd->ofd_lut.lut_lsd.lsd_osd_index;
 
-       OBD_ALLOC(name, sizeof(obd_name) * 2 + 10);
+       OBD_ALLOC(name, len);
        if (name == NULL)
                return -ENOMEM;
 
@@ -874,7 +815,7 @@ int ofd_fid_init(const struct lu_env *env, struct ofd_device *ofd)
        rc = seq_server_init(env, ss->ss_server_seq, ofd->ofd_osd, obd_name,
                             LUSTRE_SEQ_SERVER, ss);
        if (rc) {
-               CERROR("%s : seq server init error %d\n", obd_name, rc);
+               CERROR("%s: seq server init error: rc = %d\n", obd_name, rc);
                GOTO(out_server, rc);
        }
        ss->ss_server_seq->lss_space.lsr_index = ss->ss_node_id;
@@ -883,15 +824,11 @@ int ofd_fid_init(const struct lu_env *env, struct ofd_device *ofd)
        if (ss->ss_client_seq == NULL)
                GOTO(out_server, rc = -ENOMEM);
 
-       /*
-        * It always printed as "%p", so that the name is unique in the kernel,
-        * even if the filesystem is mounted twice. So sizeof(.) * 2 is enough.
-        */
-       snprintf(name, sizeof(obd_name) * 2 + 7, "%p-super", obd_name);
+       snprintf(name, len, "%s-super", obd_name);
        rc = seq_client_init(ss->ss_client_seq, NULL, LUSTRE_SEQ_DATA,
                             name, NULL);
        if (rc) {
-               CERROR("%s : seq client init error %d\n", obd_name, rc);
+               CERROR("%s: seq client init error: rc = %d\n", obd_name, rc);
                GOTO(out_client, rc);
        }
 
@@ -908,7 +845,7 @@ out_server:
                ss->ss_server_seq = NULL;
        }
 out_name:
-       OBD_FREE(name, sizeof(obd_name) * 2 + 10);
+       OBD_FREE(name, len);
 
        return rc;
 }
@@ -1021,7 +958,8 @@ int ofd_fiemap_get(const struct lu_env *env, struct ofd_device *ofd,
 }
 
 
-static int ofd_lock_unlock_region(struct ldlm_namespace *ns,
+static int ofd_lock_unlock_region(const struct lu_env *env,
+                                 struct ldlm_namespace *ns,
                                  struct ldlm_res_id *res_id,
                                  unsigned long long begin,
                                  unsigned long long end)
@@ -1032,7 +970,7 @@ static int ofd_lock_unlock_region(struct ldlm_namespace *ns,
 
        LASSERT(begin <= end);
 
-       rc = tgt_extent_lock(ns, res_id, begin, end, &lh, LCK_PR, &flags);
+       rc = tgt_extent_lock(env, ns, res_id, begin, end, &lh, LCK_PR, &flags);
        if (rc != 0)
                return rc;
 
@@ -1060,7 +998,8 @@ static int ofd_lock_unlock_region(struct ldlm_namespace *ns,
  * \retval             0 if successful
  * \retval             negative value on error
  */
-static int lock_zero_regions(struct ldlm_namespace *ns,
+static int lock_zero_regions(const struct lu_env *env,
+                            struct ldlm_namespace *ns,
                             struct ldlm_res_id *res_id,
                             struct fiemap *fiemap)
 {
@@ -1076,7 +1015,7 @@ static int lock_zero_regions(struct ldlm_namespace *ns,
                if (fiemap_start[i].fe_logical > begin) {
                        CDEBUG(D_OTHER, "ost lock [%llu,%llu]\n",
                               begin, fiemap_start[i].fe_logical);
-                       rc = ofd_lock_unlock_region(ns, res_id, begin,
+                       rc = ofd_lock_unlock_region(env, ns, res_id, begin,
                                                    fiemap_start[i].fe_logical);
                        if (rc)
                                RETURN(rc);
@@ -1088,7 +1027,7 @@ static int lock_zero_regions(struct ldlm_namespace *ns,
        if (begin < (fiemap->fm_start + fiemap->fm_length)) {
                CDEBUG(D_OTHER, "ost lock [%llu,%llu]\n",
                       begin, fiemap->fm_start + fiemap->fm_length);
-               rc = ofd_lock_unlock_region(ns, res_id, begin,
+               rc = ofd_lock_unlock_region(env, ns, res_id, begin,
                                fiemap->fm_start + fiemap->fm_length);
        }
 
@@ -1189,7 +1128,7 @@ static int ofd_get_info_hdl(struct tgt_session_info *tsi)
                if (fm_key->lfik_oa.o_valid & OBD_MD_FLFLAGS &&
                    fm_key->lfik_oa.o_flags & OBD_FL_SRVLOCK) {
                        ost_fid_build_resid(fid, &fti->fti_resid);
-                       rc = lock_zero_regions(ofd->ofd_namespace,
+                       rc = lock_zero_regions(tsi->tsi_env, ofd->ofd_namespace,
                                               &fti->fti_resid, fiemap);
                        if (rc == 0)
                                rc = ofd_fiemap_get(tsi->tsi_env, ofd, fid,
@@ -1281,7 +1220,8 @@ static int ofd_getattr_hdl(struct tgt_session_info *tsi)
                if (unlikely(tsi->tsi_ost_body->oa.o_flags & OBD_FL_FLUSH))
                        lock_mode = LCK_PW;
 
-               rc = tgt_extent_lock(tsi->tsi_tgt->lut_obd->obd_namespace,
+               rc = tgt_extent_lock(tsi->tsi_env,
+                                    tsi->tsi_tgt->lut_obd->obd_namespace,
                                     &tsi->tsi_resid, 0, OBD_OBJECT_EOF, &lh,
                                     lock_mode, &flags);
                if (rc != 0)
@@ -1413,7 +1353,7 @@ out:
                res = ldlm_resource_get(ofd->ofd_namespace, NULL,
                                        &tsi->tsi_resid, LDLM_EXTENT, 0);
                if (!IS_ERR(res)) {
-                       ldlm_res_lvbo_update(res, NULL, 0);
+                       ldlm_res_lvbo_update(tsi->tsi_env, res, NULL, 0);
                        ldlm_resource_putref(res);
                }
        }
@@ -1681,6 +1621,7 @@ static int ofd_create_hdl(struct tgt_session_info *tsi)
                u64 next_id;
                int created = 0;
                int count;
+               int rc2;
 
                if (!(oa->o_valid & OBD_MD_FLFLAGS) ||
                    !(oa->o_flags & OBD_FL_DELORPHAN)) {
@@ -1768,11 +1709,14 @@ static int ofd_create_hdl(struct tgt_session_info *tsi)
                        granted = 0;
                }
 
-               rc = ostid_set_id(&rep_oa->o_oi, ofd_seq_last_oid(oseq));
+               rc2 = ostid_set_id(&rep_oa->o_oi, ofd_seq_last_oid(oseq));
+               rc = rc ? : rc2;
        }
        EXIT;
        ofd_counter_incr(exp, LPROC_OFD_STATS_CREATE,
                         tsi->tsi_jobid, 1);
+       if (unlikely(!oseq->os_last_id_synced))
+               oseq->os_last_id_synced = 1;
 out:
        mutex_unlock(&oseq->os_create_lock);
 out_nolock:
@@ -2038,8 +1982,8 @@ static int ofd_punch_hdl(struct tgt_session_info *tsi)
                  oa->o_flags & OBD_FL_SRVLOCK;
 
        if (srvlock) {
-               rc = tgt_extent_lock(ns, &tsi->tsi_resid, start, end, &lh,
-                                    LCK_PW, &flags);
+               rc = tgt_extent_lock(tsi->tsi_env, ns, &tsi->tsi_resid, start,
+                                    end, &lh, LCK_PW, &flags);
                if (rc != 0)
                        RETURN(rc);
        }
@@ -2082,7 +2026,7 @@ out:
                if (!IS_ERR(res)) {
                        struct ost_lvb *res_lvb;
 
-                       ldlm_res_lvbo_update(res, NULL, 0);
+                       ldlm_res_lvbo_update(tsi->tsi_env, res, NULL, 0);
                        res_lvb = res->lr_lvb_data;
                        repbody->oa.o_valid |= OBD_MD_FLBLOCKS;
                        repbody->oa.o_blocks = res_lvb->lvb_blocks;
@@ -2249,7 +2193,7 @@ static int ofd_ladvise_hdl(struct tgt_session_info *tsi)
 
                        ioo.ioo_oid = body->oa.o_oi;
                        ioo.ioo_bufcnt = 1;
-                       rc = tgt_extent_lock(exp->exp_obd->obd_namespace,
+                       rc = tgt_extent_lock(env, exp->exp_obd->obd_namespace,
                                             &tsi->tsi_resid, start, end - 1,
                                             &lockh, LCK_PR, &flags);
                        if (rc != 0)
@@ -2912,7 +2856,7 @@ static int ofd_init0(const struct lu_env *env, struct ofd_device *m,
        spin_lock_init(&m->ofd_flags_lock);
        m->ofd_raid_degraded = 0;
        m->ofd_checksum_t10pi_enforce = 0;
-       m->ofd_syncjournal = 0;
+       m->ofd_sync_journal = 0;
        ofd_slc_set(m);
        m->ofd_soft_sync_limit = OFD_SOFT_SYNC_LIMIT_DEFAULT;
 
@@ -2929,7 +2873,7 @@ static int ofd_init0(const struct lu_env *env, struct ofd_device *m,
        /* set this lu_device to obd, because error handling need it */
        obd->obd_lu_dev = &m->ofd_dt_dev.dd_lu_dev;
 
-       rc = ofd_procfs_init(m);
+       rc = ofd_tunables_init(m);
        if (rc) {
                CERROR("Can't init ofd lprocfs, rc %d\n", rc);
                RETURN(rc);
@@ -2959,7 +2903,9 @@ static int ofd_init0(const struct lu_env *env, struct ofd_device *m,
                GOTO(err_fini_proc, rc);
        }
 
+#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 14, 53, 0)
        ofd_procfs_add_brw_stats_symlink(m);
+#endif
 
        snprintf(info->fti_u.name, sizeof(info->fti_u.name), "%s-%s",
                 "filter"/*LUSTRE_OST_NAME*/, obd->obd_uuid.uuid);