Whamcloud - gitweb
LU-8147 osd-zfs: fix osd_mount error path
[fs/lustre-release.git] / lustre / osd-zfs / osd_handler.c
index c87cc00..334310d 100644 (file)
@@ -27,7 +27,7 @@
  * Copyright (c) 2009, 2010, Oracle and/or its affiliates. All rights reserved.
  * Use is subject to license terms.
  *
- * Copyright (c) 2012, 2014, Intel Corporation.
+ * Copyright (c) 2012, 2015, Intel Corporation.
  */
 /*
  * This file is part of Lustre, http://www.lustre.org/
@@ -173,10 +173,15 @@ static void osd_trans_commit_cb(void *cb_data, int error)
 
 static int osd_trans_cb_add(struct thandle *th, struct dt_txn_commit_cb *dcb)
 {
-       struct osd_thandle *oh;
+       struct osd_thandle *oh = container_of0(th, struct osd_thandle,
+                                              ot_super);
 
-       oh = container_of0(th, struct osd_thandle, ot_super);
-       list_add(&dcb->dcb_linkage, &oh->ot_dcb_list);
+       LASSERT(dcb->dcb_magic == TRANS_COMMIT_CB_MAGIC);
+       LASSERT(&dcb->dcb_func != NULL);
+       if (dcb->dcb_flags & DCB_TRANS_STOP)
+               list_add(&dcb->dcb_linkage, &oh->ot_stop_dcb_list);
+       else
+               list_add(&dcb->dcb_linkage, &oh->ot_dcb_list);
 
        return 0;
 }
@@ -209,9 +214,8 @@ static int osd_trans_start(const struct lu_env *env, struct dt_device *d,
                struct osd_device *osd = osd_dt_dev(d);
                /* dmu will call commit callback with error code during abort */
                if (!lu_device_is_md(&d->dd_lu_dev) && rc == -ENOSPC)
-                       CERROR("%s: failed to start transaction due to ENOSPC. "
-                              "Metadata overhead is underestimated or "
-                              "grant_ratio is too low.\n", osd->od_svname);
+                       CERROR("%s: failed to start transaction due to ENOSPC"
+                              "\n", osd->od_svname);
                else
                        CERROR("%s: can't assign tx: rc = %d\n",
                               osd->od_svname, rc);
@@ -247,6 +251,22 @@ static void osd_unlinked_list_emptify(struct osd_device *osd,
        }
 }
 
+static void osd_trans_stop_cb(struct osd_thandle *oth, int result)
+{
+       struct dt_txn_commit_cb *dcb;
+       struct dt_txn_commit_cb *tmp;
+
+       /* call per-transaction stop callbacks if any */
+       list_for_each_entry_safe(dcb, tmp, &oth->ot_stop_dcb_list,
+                                dcb_linkage) {
+               LASSERTF(dcb->dcb_magic == TRANS_COMMIT_CB_MAGIC,
+                        "commit callback entry: magic=%x name='%s'\n",
+                        dcb->dcb_magic, dcb->dcb_name);
+               list_del_init(&dcb->dcb_linkage);
+               dcb->dcb_func(NULL, &oth->ot_super, dcb, result);
+       }
+}
+
 /*
  * Concurrency: shouldn't matter.
  */
@@ -292,6 +312,8 @@ static int osd_trans_stop(const struct lu_env *env, struct dt_device *dt,
                CDEBUG(D_OTHER, "%s: transaction hook failed: rc = %d\n",
                       osd->od_svname, rc);
 
+       osd_trans_stop_cb(oh, rc);
+
        LASSERT(oh->ot_tx);
        txg = oh->ot_tx->tx_txg;
 
@@ -330,6 +352,7 @@ static struct thandle *osd_trans_create(const struct lu_env *env,
 
        oh->ot_tx = tx;
        INIT_LIST_HEAD(&oh->ot_dcb_list);
+       INIT_LIST_HEAD(&oh->ot_stop_dcb_list);
        INIT_LIST_HEAD(&oh->ot_unlinked_list);
        INIT_LIST_HEAD(&oh->ot_sa_list);
        sema_init(&oh->ot_sa_lock, 1);
@@ -417,6 +440,11 @@ static int osd_objset_statfs(struct osd_device *osd, struct obd_statfs *osfs)
 
        dmu_objset_space(os, &refdbytes, &availbytes, &usedobjs, &availobjs);
 
+       memset(osfs, 0, sizeof(*osfs));
+
+       /* We're a zfs filesystem. */
+       osfs->os_type = UBERBLOCK_MAGIC;
+
        /*
         * ZFS allows multiple block sizes.  For statfs, Linux makes no
         * proper distinction between bsize and frsize.  For calculations
@@ -441,18 +469,17 @@ static int osd_objset_statfs(struct osd_device *osd, struct obd_statfs *osfs)
         * Rather than report this via os_bavail (which makes users unhappy if
         * they can't fill the filesystem 100%), reduce os_blocks as well.
         *
-        * Reserve 0.78% of total space, at least 4MB for small filesystems,
+        * Reserve 0.78% of total space, at least 16MB for small filesystems,
         * for internal files to be created/unlinked when space is tight.
         */
        CLASSERT(OSD_STATFS_RESERVED_SIZE > 0);
-       if (likely(osfs->os_blocks >= OSD_STATFS_RESERVED_SIZE))
+       reserved = OSD_STATFS_RESERVED_SIZE >> bshift;
+       if (likely(osfs->os_blocks >= reserved << OSD_STATFS_RESERVED_SHIFT))
                reserved = osfs->os_blocks >> OSD_STATFS_RESERVED_SHIFT;
-       else
-               reserved = OSD_STATFS_RESERVED_SIZE >> bshift;
 
        osfs->os_blocks -= reserved;
-       osfs->os_bfree  -= MIN(reserved, osfs->os_bfree);
-       osfs->os_bavail -= MIN(reserved, osfs->os_bavail);
+       osfs->os_bfree  -= min(reserved, osfs->os_bfree);
+       osfs->os_bavail -= min(reserved, osfs->os_bavail);
 
        /*
         * The availobjs value returned from dmu_objset_space() is largely
@@ -470,18 +497,13 @@ static int osd_objset_statfs(struct osd_device *osd, struct obd_statfs *osfs)
        /* ZFS XXX: fill in backing dataset FSID/UUID
           memcpy(osfs->os_fsid, .... );*/
 
-       /* We're a zfs filesystem. */
-       osfs->os_type = UBERBLOCK_MAGIC;
-
-       /* ZFS XXX: fill in appropriate OS_STATE_{DEGRADED,READONLY} flags
-          osfs->os_state = vf_to_stf(vfsp->vfs_flag);
-          if (sb->s_flags & MS_RDONLY)
-          osfs->os_state = OS_STATE_READONLY;
-        */
-
        osfs->os_namelen = MAXNAMELEN;
        osfs->os_maxbytes = OBD_OBJECT_EOF;
 
+       if (!spa_writeable(dmu_objset_spa(os)) ||
+           osd->od_dev_set_rdonly || osd->od_prop_rdonly)
+               osfs->os_state |= OS_STATE_READONLY;
+
        return 0;
 }
 
@@ -539,7 +561,7 @@ static void osd_conf_get(const struct lu_env *env,
         */
        param->ddp_max_name_len = MAXNAMELEN;
        param->ddp_max_nlink    = 1 << 31; /* it's 8byte on a disk */
-       param->ddp_block_shift  = 12; /* XXX */
+       param->ddp_symlink_max  = PATH_MAX;
        param->ddp_mount_type   = LDD_MT_ZFS;
 
        param->ddp_mntopts      = MNTOPT_USERXATTR;
@@ -550,20 +572,22 @@ static void osd_conf_get(const struct lu_env *env,
        /* for maxbytes, report same value as ZPL */
        param->ddp_maxbytes     = MAX_LFS_FILESIZE;
 
-       /* Default reserved fraction of the available space that should be kept
-        * for error margin. Unfortunately, there are many factors that can
-        * impact the overhead with zfs, so let's be very cautious for now and
-        * reserve 20% of the available space which is not given out as grant.
-        * This tunable can be changed on a live system via procfs if needed. */
-       param->ddp_grant_reserved = 20;
-
        /* inodes are dynamically allocated, so we report the per-inode space
         * consumption to upper layers. This static value is not really accurate
         * and we should use the same logic as in udmu_objset_statfs() to
         * estimate the real size consumed by an object */
        param->ddp_inodespace = OSD_DNODE_EST_COUNT;
-       /* per-fragment overhead to be used by the client code */
-       param->ddp_grant_frag = osd_blk_insert_cost(osd);
+       /* Although ZFS isn't an extent-based filesystem, the metadata overhead
+        * (i.e. 7 levels of indirect blocks, see osd_blk_insert_cost()) should
+        * not be accounted for every single new block insertion.
+        * Instead, the maximum extent size is set to the number of blocks that
+        * can fit into a single contiguous indirect block. There would be some
+        * cases where this crosses indirect blocks, but it also won't have 7
+        * new levels of indirect blocks in that case either, so it will still
+        * have enough reserved space for the extra indirect block */
+       param->ddp_max_extent_blks =
+               (1 << (DN_MAX_INDBLKSHIFT - SPA_BLKPTRSHIFT));
+       param->ddp_extent_tax = osd_blk_insert_cost(osd);
 }
 
 /*
@@ -605,7 +629,7 @@ static int osd_ro(const struct lu_env *env, struct dt_device *d)
 
        CERROR("%s: *** setting device %s read-only ***\n",
               osd->od_svname, LUSTRE_OSD_ZFS_NAME);
-       osd->od_rdonly = 1;
+       osd->od_dev_set_rdonly = 1;
        spa_freeze(dmu_objset_spa(osd->od_os));
 
        RETURN(0);
@@ -717,6 +741,13 @@ static void osd_recordsize_changed_cb(void *arg, uint64_t newval)
        osd->od_max_blksz = newval;
 }
 
+static void osd_readonly_changed_cb(void *arg, uint64_t newval)
+{
+       struct osd_device *osd = arg;
+
+       osd->od_prop_rdonly = !!newval;
+}
+
 /*
  * This function unregisters all registered callbacks.  It's harmless to
  * unregister callbacks that were never registered so it is used to safely
@@ -730,6 +761,8 @@ static void osd_objset_unregister_callbacks(struct osd_device *o)
                                   osd_xattr_changed_cb, o);
        (void) dsl_prop_unregister(ds, zfs_prop_to_name(ZFS_PROP_RECORDSIZE),
                                   osd_recordsize_changed_cb, o);
+       (void) dsl_prop_unregister(ds, zfs_prop_to_name(ZFS_PROP_READONLY),
+                                  osd_readonly_changed_cb, o);
 
        if (o->arc_prune_cb != NULL) {
                arc_remove_prune_callback(o->arc_prune_cb);
@@ -761,6 +794,11 @@ static int osd_objset_register_callbacks(struct osd_device *o)
        if (rc)
                GOTO(err, rc);
 
+       rc = -dsl_prop_register(ds, zfs_prop_to_name(ZFS_PROP_READONLY),
+                               osd_readonly_changed_cb, o);
+       if (rc)
+               GOTO(err, rc);
+
        o->arc_prune_cb = arc_add_prune_callback(arc_prune_func, o);
 err:
        dsl_pool_config_exit(dp, FTAG);
@@ -965,10 +1003,6 @@ static int osd_mount(const struct lu_env *env,
        if (rc)
                GOTO(err, rc);
 
-       rc = osd_convert_root_to_new_seq(env, o);
-       if (rc)
-               GOTO(err, rc);
-
        /* Use our own ZAP for inode accounting by default, this can be changed
         * via procfs to estimate the inode usage from the block usage */
        o->od_quota_iused_est = 0;
@@ -994,7 +1028,8 @@ static int osd_mount(const struct lu_env *env,
        osd_unlinked_drain(env, o);
 err:
        if (rc) {
-               dmu_objset_disown(o->od_os, o);
+               if (o->od_os)
+                       dmu_objset_disown(o->od_os, o);
                o->od_os = NULL;
        }
 
@@ -1054,13 +1089,19 @@ static struct lu_device *osd_device_alloc(const struct lu_env *env,
                                          struct lu_device_type *type,
                                          struct lustre_cfg *cfg)
 {
-       struct osd_device *dev;
-       int                rc;
+       struct osd_device       *dev;
+       struct osd_seq_list     *osl;
+       int                     rc;
 
        OBD_ALLOC_PTR(dev);
        if (dev == NULL)
                return ERR_PTR(-ENOMEM);
 
+       osl = &dev->od_seq_list;
+       INIT_LIST_HEAD(&osl->osl_seq_list);
+       rwlock_init(&osl->osl_seq_list_lock);
+       sema_init(&osl->osl_seq_init_sem, 1);
+
        rc = dt_device_init(&dev->od_dt_dev, type);
        if (rc == 0) {
                rc = osd_device_init0(env, dev, cfg);
@@ -1334,7 +1375,7 @@ static struct obd_ops osd_obd_device_ops = {
        .o_fid_alloc    = osd_fid_alloc
 };
 
-int __init osd_init(void)
+static int __init osd_init(void)
 {
        int rc;
 
@@ -1353,18 +1394,17 @@ int __init osd_init(void)
        return rc;
 }
 
-void __exit osd_exit(void)
+static void __exit osd_exit(void)
 {
        class_unregister_type(LUSTRE_OSD_ZFS_NAME);
        lu_kmem_fini(osd_caches);
 }
 
 extern unsigned int osd_oi_count;
-CFS_MODULE_PARM(osd_oi_count, "i", int, 0444,
-               "Number of Object Index containers to be created, "
-               "it's only valid for new filesystem.");
+module_param(osd_oi_count, int, 0444);
+MODULE_PARM_DESC(osd_oi_count, "Number of Object Index containers to be created, it's only valid for new filesystem.");
 
-MODULE_AUTHOR("Sun Microsystems, Inc. <http://www.lustre.org/>");
+MODULE_AUTHOR("OpenSFS, Inc. <http://www.lustre.org/>");
 MODULE_DESCRIPTION("Lustre Object Storage Device ("LUSTRE_OSD_ZFS_NAME")");
 MODULE_VERSION(LUSTRE_VERSION_STRING);
 MODULE_LICENSE("GPL");