Whamcloud - gitweb
Revert "LU-2600 osd-zfs: batched object accounting"
[fs/lustre-release.git] / lustre / osd-zfs / osd_quota.c
index 69f1d5d..b51be3b 100644 (file)
 #include <obd.h>
 #include "osd_internal.h"
 
-#include <sys/dnode.h>
-#include <sys/spa.h>
-#include <sys/zap.h>
-#include <sys/dmu_tx.h>
-#include <sys/dsl_prop.h>
-#include <sys/txg.h>
-
-/*
- * the structure tracks per-ID change/state
- */
-struct zfs_id_change {
-       struct hlist_node       zic_hash;
-       __u64                   zic_id;
-       atomic_t                zic_num;
-};
-
-/*
- * callback data for cfs_hash_for_each_safe()
- * used in txg commit and OSD cleanup path
- */
-struct hash_cbdata {
-       struct osd_device       *hcb_osd;
-       uint64_t                 hcb_zapid;
-       dmu_tx_t                *hcb_tx;
-};
-
 /**
  * Helper function to retrieve DMU object id from fid for accounting object
  */
-static inline uint64_t osd_quota_fid2dmu(const struct lu_fid *fid)
+uint64_t osd_quota_fid2dmu(const struct lu_fid *fid)
 {
        LASSERT(fid_is_acct(fid));
        if (fid_oid(fid) == ACCT_GROUP_OID)
@@ -68,475 +42,6 @@ static inline uint64_t osd_quota_fid2dmu(const struct lu_fid *fid)
        return DMU_USERUSED_OBJECT;
 }
 
-/*
- * a note about locking:
- *  entries in per-OSD cache never go before umount,
- *  so there is no need in locking for lookups.
- *
- *  entries in per-txg deltas never go before txg is closed,
- *  there is no concurrency between removal/insertions.
- *
- * also, given all above, there is no need in reference counting.
- */
-static struct zfs_id_change *osd_zfs_lookup_by_id(cfs_hash_t *hash, __u64 id)
-{
-       struct zfs_id_change    *za = NULL;
-       struct hlist_node       *hnode;
-       cfs_hash_bd_t            bd;
-
-       cfs_hash_bd_get(hash, &id, &bd);
-       hnode = cfs_hash_bd_peek_locked(hash, &bd, &id);
-       if (hnode != NULL)
-               za = container_of0(hnode, struct zfs_id_change, zic_hash);
-
-       return za;
-}
-
-static struct zfs_id_change *lookup_or_create_by_id(struct osd_device *osd,
-                                               cfs_hash_t *hash, __u64 id)
-{
-       struct zfs_id_change    *za, *tmp;
-       struct hlist_node       *hnode;
-       cfs_hash_bd_t            bd;
-
-       za = osd_zfs_lookup_by_id(hash, id);
-       if (likely(za != NULL))
-               return za;
-
-       OBD_ALLOC_PTR(za);
-       if (unlikely(za == NULL))
-               return NULL;
-
-       za->zic_id = id;
-
-       cfs_hash_bd_get(hash, &id, &bd);
-       spin_lock(&osd->od_known_txg_lock);
-       hnode = cfs_hash_bd_findadd_locked(hash, &bd, &id, &za->zic_hash, 1);
-       LASSERT(hnode != NULL);
-       tmp = container_of0(hnode, struct zfs_id_change, zic_hash);
-       spin_unlock(&osd->od_known_txg_lock);
-
-       if (tmp == za) {
-               /*
-                * our structure got into the hash
-                */
-       } else {
-               /* somebody won the race, we wasted the cycles */
-               OBD_FREE_PTR(za);
-       }
-
-       return tmp;
-}
-
-/*
- * used to maintain per-txg deltas
- */
-static int osd_zfs_acct_id(const struct lu_env *env, cfs_hash_t *hash,
-                          __u64 id, int delta, struct osd_thandle *oh)
-{
-       struct osd_device       *osd = osd_dt_dev(oh->ot_super.th_dev);
-       struct zfs_id_change    *za;
-
-       LASSERT(hash);
-       LASSERT(oh->ot_tx);
-       LASSERT(oh->ot_tx->tx_txg == osd->od_known_txg);
-       LASSERT(osd->od_acct_delta != NULL);
-
-       za = lookup_or_create_by_id(osd, hash, id);
-       if (unlikely(za == NULL))
-               return -ENOMEM;
-
-       atomic_add(delta, &za->zic_num);
-
-       return 0;
-}
-
-/*
- * this function is used to maintain current state for given ID:
- * at the beginning it initializes the cache from correspoding ZAP
- */
-static void osd_zfs_acct_cache_init(const struct lu_env *env,
-                                   struct osd_device *osd,
-                                   cfs_hash_t *hash, __u64 oid,
-                                   __u64 id, int delta,
-                                   struct osd_thandle *oh)
-{
-       char                    *buf  = osd_oti_get(env)->oti_buf;
-       struct hlist_node       *hnode;
-       cfs_hash_bd_t            bd;
-       struct zfs_id_change    *za, *tmp;
-       __u64                    v;
-       int                      rc;
-
-       za = osd_zfs_lookup_by_id(hash, id);
-       if (likely(za != NULL))
-               goto apply;
-
-       /*
-        * any concurrent thread is running in the same txg, so no on-disk
-        * accounting ZAP can be modified until this txg is closed
-        * thus all the concurrent threads must be getting the same value
-        * from that ZAP and we don't need to serialize lookups
-        */
-       snprintf(buf, sizeof(osd_oti_get(env)->oti_buf), "%llx", id);
-       /* XXX: we should be using zap_lookup_int_key(), but it consumes
-        *      20 bytes on the stack for buf .. */
-       rc = -zap_lookup(osd->od_objset.os, oid, buf, sizeof(uint64_t), 1, &v);
-       if (rc == -ENOENT) {
-               v = 0;
-       } else if (unlikely(rc != 0)) {
-               CERROR("%s: can't access accounting zap %llu\n",
-                      osd->od_svname, oid);
-               return;
-       }
-
-       OBD_ALLOC_PTR(za);
-       if (unlikely(za == NULL)) {
-               CERROR("%s: can't allocate za\n", osd->od_svname);
-               return;
-       }
-
-       za->zic_id = id;
-       atomic_set(&za->zic_num, v);
-
-       cfs_hash_bd_get(hash, &id, &bd);
-       spin_lock(&osd->od_known_txg_lock);
-       hnode = cfs_hash_bd_findadd_locked(hash, &bd, &id, &za->zic_hash, 1);
-       LASSERT(hnode != NULL);
-       tmp = container_of0(hnode, struct zfs_id_change, zic_hash);
-       spin_unlock(&osd->od_known_txg_lock);
-
-       if (tmp == za) {
-               /* our structure got into the hash */
-               if (rc == -ENOENT) {
-                       /* there was no entry in ZAP yet, we have
-                        * to initialize with 0, so that accounting
-                        * reports can find that and then find our
-                        * cached value. */
-                       v = 0;
-                       rc = -zap_update(osd->od_objset.os, oid, buf,
-                                        sizeof(uint64_t), 1, &v, oh->ot_tx);
-                       if (unlikely(rc != 0))
-                               CERROR("%s: can't initialize: rc = %d\n",
-                                      osd->od_svname, rc);
-               }
-       } else {
-               /* somebody won the race, we wasted the cycles */
-               OBD_FREE_PTR(za);
-               za = tmp;
-       }
-
-apply:
-       LASSERT(za != NULL);
-       atomic_add(delta, &za->zic_num);
-}
-
-static __u32 acct_hashfn(cfs_hash_t *hash_body, const void *key, unsigned mask)
-{
-       const __u64     *id = key;
-       __u32            result;
-
-       result = (__u32) *id;
-       return result % mask;
-}
-
-static void *acct_key(struct hlist_node *hnode)
-{
-       struct zfs_id_change    *ac;
-
-       ac = hlist_entry(hnode, struct zfs_id_change, zic_hash);
-       return &ac->zic_id;
-}
-
-static int acct_hashkey_keycmp(const void *key,
-                              struct hlist_node *compared_hnode)
-{
-       struct zfs_id_change    *ac;
-       const __u64             *id = key;
-
-       ac = hlist_entry(compared_hnode, struct zfs_id_change, zic_hash);
-       return *id == ac->zic_id;
-}
-
-static void *acct_hashobject(struct hlist_node *hnode)
-{
-       return hlist_entry(hnode, struct zfs_id_change, zic_hash);
-}
-
-static cfs_hash_ops_t acct_hash_operations = {
-       .hs_hash        = acct_hashfn,
-       .hs_key         = acct_key,
-       .hs_keycmp      = acct_hashkey_keycmp,
-       .hs_object      = acct_hashobject,
-};
-
-#define ACCT_HASH_OPS (CFS_HASH_NO_LOCK|CFS_HASH_NO_ITEMREF|CFS_HASH_ADD_TAIL)
-
-int osd_zfs_acct_init(const struct lu_env *env, struct osd_device *o)
-{
-       int rc = 0;
-       ENTRY;
-
-       spin_lock_init(&o->od_known_txg_lock);
-
-       /* global structure representing current state for given ID */
-       o->od_acct_usr = cfs_hash_create("usr", 4, 4, 4, 0, 0, 0,
-                                        &acct_hash_operations,
-                                        ACCT_HASH_OPS);
-       if (o->od_acct_usr == NULL)
-               GOTO(out, rc = -ENOMEM);
-
-       o->od_acct_grp = cfs_hash_create("grp", 4, 4, 4, 0, 0, 0,
-                                        &acct_hash_operations,
-                                        ACCT_HASH_OPS);
-       if (o->od_acct_grp == NULL)
-               GOTO(out, rc = -ENOMEM);
-
-out:
-       RETURN(rc);
-}
-
-static int osd_zfs_delete_item(cfs_hash_t *hs, cfs_hash_bd_t *bd,
-                              struct hlist_node *node, void *data)
-{
-       struct hash_cbdata      *d = data;
-       struct zfs_id_change    *za;
-       __u64                    v;
-       char                     buf[12];
-       int                      rc;
-
-       za = hlist_entry(node, struct zfs_id_change, zic_hash);
-
-       /*
-        * XXX: should we try to fix accounting we failed to update before?
-        */
-#if LUSTRE_VERSION_CODE < OBD_OCD_VERSION(2, 5, 70, 0)
-       /*
-        * extra checks to ensure our cache matches on-disk state
-        */
-       snprintf(buf, sizeof(buf), "%llx", za->zic_id);
-       rc = -zap_lookup(d->hcb_osd->od_objset.os, d->hcb_zapid,
-                        buf, sizeof(uint64_t), 1, &v);
-       /* pairs with zero value are removed by ZAP automatically */
-       if (rc == -ENOENT)
-               v = 0;
-       if (atomic_read(&za->zic_num) != v) {
-               CERROR("%s: INVALID ACCOUNTING FOR %llu %d != %lld: rc = %d\n",
-                      d->hcb_osd->od_svname, za->zic_id,
-                      atomic_read(&za->zic_num), v, rc);
-               /* XXX: to catch with automated testing */
-               LBUG();
-       }
-#else
-#warning "remove this additional check before release"
-#endif
-
-       cfs_hash_bd_del_locked(hs, bd, node);
-       OBD_FREE_PTR(za);
-
-       return 0;
-}
-
-void osd_zfs_acct_fini(const struct lu_env *env, struct osd_device *o)
-{
-       struct hash_cbdata      cbdata;
-
-       cbdata.hcb_osd = o;
-
-       /* release object accounting cache (owners) */
-       cbdata.hcb_zapid = o->od_iusr_oid;
-
-       if (o->od_acct_usr) {
-               cfs_hash_for_each_safe(o->od_acct_usr, osd_zfs_delete_item,
-                                      &cbdata);
-               cfs_hash_putref(o->od_acct_usr);
-               o->od_acct_usr = NULL;
-       }
-
-       /* release object accounting cache (groups) */
-       cbdata.hcb_zapid = o->od_igrp_oid;
-
-       if (o->od_acct_grp) {
-               cfs_hash_for_each_safe(o->od_acct_grp, osd_zfs_delete_item,
-                                      &cbdata);
-               cfs_hash_putref(o->od_acct_grp);
-               o->od_acct_grp = NULL;
-       }
-}
-
-static int osd_zfs_commit_item(cfs_hash_t *hs, cfs_hash_bd_t *bd,
-                              struct hlist_node *node, void *data)
-{
-       struct hash_cbdata      *d = data;
-       struct osd_device       *osd = d->hcb_osd;
-       struct zfs_id_change    *za;
-       int                      rc;
-
-       za = hlist_entry(node, struct zfs_id_change, zic_hash);
-
-       rc = -zap_increment_int(osd->od_objset.os, d->hcb_zapid, za->zic_id,
-                               atomic_read(&za->zic_num), d->hcb_tx);
-       if (unlikely(rc != 0))
-               CERROR("%s: quota update for UID "LPU64" failed: rc = %d\n",
-                      osd->od_svname, za->zic_id, rc);
-
-       cfs_hash_bd_del_locked(hs, bd, node);
-       OBD_FREE_PTR(za);
-
-       return 0;
-}
-
-/*
- * this function is called as part of txg commit procedure,
- * no more normal changes are allowed to this txg.
- * we go over all the changes cached in per-txg structure
- * and apply them to actual ZAPs
- */
-#ifdef HAVE_DSL_SYNC_TASK_DO_NOWAIT
-static void osd_zfs_acct_update(void *arg, void *arg2, dmu_tx_t *tx)
-#else
-static void osd_zfs_acct_update(void *arg, dmu_tx_t *tx)
-#endif
-{
-       struct osd_zfs_acct_txg *zat = arg;
-       struct osd_device       *osd = zat->zat_osd;
-       struct hash_cbdata       cbdata;
-
-       cbdata.hcb_osd = osd;
-       cbdata.hcb_tx = tx;
-
-       CDEBUG(D_OTHER, "COMMIT %llu on %s\n", tx->tx_txg, osd->od_svname);
-
-       /* apply changes related to the owners */
-       cbdata.hcb_zapid = osd->od_iusr_oid;
-       cfs_hash_for_each_safe(zat->zat_usr, osd_zfs_commit_item, &cbdata);
-
-       /* apply changes related to the groups */
-       cbdata.hcb_zapid = osd->od_igrp_oid;
-       cfs_hash_for_each_safe(zat->zat_grp, osd_zfs_commit_item, &cbdata);
-
-       cfs_hash_putref(zat->zat_usr);
-       cfs_hash_putref(zat->zat_grp);
-
-       OBD_FREE_PTR(zat);
-}
-
-#ifdef HAVE_DSL_SYNC_TASK_DO_NOWAIT
-#define dsl_sync_task_nowait(pool, func, arg, blocks, tx) \
-       dsl_sync_task_do_nowait(pool, NULL, func, arg, NULL, blocks, tx)
-#endif
-
-/*
- * if any change to the object accounting is going to happen,
- * we create one structure per txg to track all the changes
- * and register special routine to be called as part of txg
- * commit procedure.
- */
-int osd_zfs_acct_trans_start(const struct lu_env *env, struct osd_thandle *oh)
-{
-       struct osd_device       *osd = osd_dt_dev(oh->ot_super.th_dev);
-       struct osd_zfs_acct_txg *ac = NULL;
-       int                      rc = 0, add_work = 0;
-
-       if (likely(oh->ot_tx->tx_txg == osd->od_known_txg)) {
-               /* already created */
-               return 0;
-       }
-
-       OBD_ALLOC_PTR(ac);
-       if (unlikely(ac == NULL))
-               return -ENOMEM;
-
-       ac->zat_usr = cfs_hash_create("usr", 4, 4, 4, 0, 0, 0,
-                                     &acct_hash_operations,
-                                     ACCT_HASH_OPS);
-       if (unlikely(ac->zat_usr == NULL)) {
-               CERROR("%s: can't allocate hash for accounting\n",
-                       osd->od_svname);
-               GOTO(out, rc = -ENOMEM);
-       }
-
-       ac->zat_grp = cfs_hash_create("grp", 4, 4, 4, 0, 0, 0,
-                                     &acct_hash_operations,
-                                     ACCT_HASH_OPS);
-       if (unlikely(ac->zat_grp == NULL)) {
-               CERROR("%s: can't allocate hash for accounting\n",
-                       osd->od_svname);
-               GOTO(out, rc = -ENOMEM);
-       }
-
-       spin_lock(&osd->od_known_txg_lock);
-       if (oh->ot_tx->tx_txg != osd->od_known_txg) {
-               osd->od_acct_delta = ac;
-               osd->od_known_txg = oh->ot_tx->tx_txg;
-               add_work = 1;
-       }
-       spin_unlock(&osd->od_known_txg_lock);
-
-       /* schedule a callback to be run in the context of txg
-        * once the latter is closed and syncing */
-       if (add_work) {
-               spa_t *spa = dmu_objset_spa(osd->od_objset.os);
-               LASSERT(ac->zat_osd == NULL);
-               ac->zat_osd = osd;
-               dsl_sync_task_nowait(spa_get_dsl(spa),
-                                       osd_zfs_acct_update,
-                                       ac, 128, oh->ot_tx);
-
-               /* no to be freed now */
-               ac = NULL;
-       }
-
-out:
-       if (ac != NULL) {
-               /* another thread has installed new structure already */
-               if (ac->zat_usr)
-                       cfs_hash_putref(ac->zat_usr);
-               if (ac->zat_grp)
-                       cfs_hash_putref(ac->zat_grp);
-               OBD_FREE_PTR(ac);
-       }
-
-       return rc;
-}
-
-void osd_zfs_acct_uid(const struct lu_env *env, struct osd_device *osd,
-                     __u64 uid, int delta, struct osd_thandle *oh)
-{
-       int rc;
-
-       /* add per-txg job to update accounting */
-       rc = osd_zfs_acct_trans_start(env, oh);
-       if (unlikely(rc != 0))
-               return;
-
-       /* maintain per-OSD cached value */
-       osd_zfs_acct_cache_init(env, osd, osd->od_acct_usr,
-                               osd->od_iusr_oid, uid, delta, oh);
-
-       /* maintain per-TXG delta */
-       osd_zfs_acct_id(env, osd->od_acct_delta->zat_usr, uid, delta, oh);
-
-}
-
-void osd_zfs_acct_gid(const struct lu_env *env, struct osd_device *osd,
-                     __u64 gid, int delta, struct osd_thandle *oh)
-{
-       int rc;
-
-       /* add per-txg job to update accounting */
-       rc = osd_zfs_acct_trans_start(env, oh);
-       if (unlikely(rc != 0))
-               return;
-
-       /* maintain per-OSD cached value */
-       osd_zfs_acct_cache_init(env, osd, osd->od_acct_grp,
-                               osd->od_igrp_oid, gid, delta, oh);
-
-       /* maintain per-TXG delta */
-       osd_zfs_acct_id(env, osd->od_acct_delta->zat_grp, gid, delta, oh);
-}
-
 /**
  * Space Accounting Management
  */
@@ -558,19 +63,18 @@ void osd_zfs_acct_gid(const struct lu_env *env, struct osd_device *osd,
  * \retval -ve - failure
  */
 static int osd_acct_index_lookup(const struct lu_env *env,
-                                struct dt_object *dtobj,
-                                struct dt_rec *dtrec,
-                                const struct dt_key *dtkey,
-                                struct lustre_capa *capa)
+                               struct dt_object *dtobj,
+                               struct dt_rec *dtrec,
+                               const struct dt_key *dtkey,
+                               struct lustre_capa *capa)
 {
        struct osd_thread_info  *info = osd_oti_get(env);
        char                    *buf  = info->oti_buf;
        struct lquota_acct_rec  *rec  = (struct lquota_acct_rec *)dtrec;
        struct osd_object       *obj = osd_dt_obj(dtobj);
        struct osd_device       *osd = osd_obj2dev(obj);
-       uint64_t                 oid;
-       struct zfs_id_change    *za = NULL;
        int                      rc;
+       uint64_t                 oid;
        ENTRY;
 
        rec->bspace = rec->ispace = 0;
@@ -605,20 +109,8 @@ static int osd_acct_index_lookup(const struct lu_env *env,
 
        /* as for inode accounting, it is not maintained by DMU, so we just
         * use our own ZAP to track inode usage */
-       if (oid == DMU_USERUSED_OBJECT) {
-               za = osd_zfs_lookup_by_id(osd->od_acct_usr,
-                                        *((__u64 *)dtkey));
-       } else if (oid == DMU_GROUPUSED_OBJECT) {
-               za = osd_zfs_lookup_by_id(osd->od_acct_grp,
-                                        *((__u64 *)dtkey));
-       }
-       if (za) {
-               rec->ispace = atomic_read(&za->zic_num);
-       } else {
-               rc = -zap_lookup(osd->od_objset.os, obj->oo_db->db_object,
-                               buf, sizeof(uint64_t), 1, &rec->ispace);
-       }
-
+       rc = -zap_lookup(osd->od_objset.os, obj->oo_db->db_object,
+                        buf, sizeof(uint64_t), 1, &rec->ispace);
        if (rc == -ENOENT)
                /* user/group has not created any file yet */
                CDEBUG(D_QUOTA, "%s: id %s not found in accounting ZAP\n",
@@ -657,13 +149,6 @@ static struct dt_it *osd_it_acct_init(const struct lu_env *env,
        memset(it, 0, sizeof(*it));
        it->oiq_oid = osd_quota_fid2dmu(lu_object_fid(lo));
 
-       if (it->oiq_oid == DMU_GROUPUSED_OBJECT)
-               it->oiq_hash = osd->od_acct_grp;
-       else if (it->oiq_oid == DMU_USERUSED_OBJECT)
-               it->oiq_hash = osd->od_acct_usr;
-       else
-               LBUG();
-
        /* initialize zap cursor */
        rc = -udmu_zap_cursor_init(&it->oiq_zc, &osd->od_objset, it->oiq_oid,0);
        if (rc)
@@ -767,7 +252,6 @@ static int osd_it_acct_rec(const struct lu_env *env,
        struct osd_object       *obj = it->oiq_obj;
        struct osd_device       *osd = osd_obj2dev(obj);
        int                      bytes_read;
-       struct zfs_id_change    *za;
        int                      rc;
        ENTRY;
 
@@ -796,23 +280,16 @@ static int osd_it_acct_rec(const struct lu_env *env,
 
        /* inode accounting is not maintained by DMU, so we use our own ZAP to
         * track inode usage */
-       za = osd_zfs_lookup_by_id(it->oiq_hash, it->oiq_id);
-       if (za != NULL) {
-               /* found in the cache */
-               rec->ispace = atomic_read(&za->zic_num);
-       } else {
-                rc = -zap_lookup(osd->od_objset.os,
-                                 it->oiq_obj->oo_db->db_object,
-                                 buf, sizeof(uint64_t), 1, &rec->ispace);
-                if (rc == -ENOENT) {
-                       /* user/group has not created any file yet */
-                       CDEBUG(D_QUOTA, "%s: id %s not found in ZAP\n",
-                              osd->od_svname, buf);
-                       rc = 0;
-               }
-       }
+       rc = -zap_lookup(osd->od_objset.os, it->oiq_obj->oo_db->db_object,
+                        buf, sizeof(uint64_t), 1, &rec->ispace);
+       if (rc == -ENOENT)
+               /* user/group has not created any file yet */
+               CDEBUG(D_QUOTA, "%s: id %s not found in accounting ZAP\n",
+                      osd->od_svname, buf);
+       else if (rc)
+               RETURN(rc);
 
-       RETURN(rc);
+       RETURN(0);
 }
 
 /**