X-Git-Url: https://git.whamcloud.com/?a=blobdiff_plain;f=lustre%2Fmdd%2Fmdd_dir.c;h=2a8fd652831f351a4b6674d900a4079f4fb3f9ae;hb=040179b84330689b1b2cd128dcdd711bfc8f0b4e;hp=4625205dd361b2ba72598e2a3371233a074b3213;hpb=6c8ed1d8edb0f6fe1d79172f7fa4a0fcf5053dfe;p=fs%2Flustre-release.git diff --git a/lustre/mdd/mdd_dir.c b/lustre/mdd/mdd_dir.c index 4625205..2a8fd65 100644 --- a/lustre/mdd/mdd_dir.c +++ b/lustre/mdd/mdd_dir.c @@ -1,6 +1,4 @@ -/* -*- mode: c; c-basic-offset: 8; indent-tabs-mode: nil; -*- - * vim:expandtab:shiftwidth=8:tabstop=8: - * +/* * GPL HEADER START * * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. @@ -26,8 +24,10 @@ * GPL HEADER END */ /* - * Copyright 2008 Sun Microsystems, Inc. All rights reserved + * Copyright (c) 2007, 2010, Oracle and/or its affiliates. All rights reserved. * Use is subject to license terms. + * + * Copyright (c) 2011, 2012, Intel Corporation. */ /* * This file is part of Lustre, http://www.lustre.org/ @@ -37,25 +37,14 @@ * * Lustre Metadata Server (mdd) routines * - * Author: Wang Di + * Author: Wang Di */ -#ifndef EXPORT_SYMTAB -# define EXPORT_SYMTAB -#endif #define DEBUG_SUBSYSTEM S_MDS -#include -#include -#include #include -#include #include -#include - -#include #include -#include #include #include "mdd_internal.h" @@ -68,14 +57,14 @@ static struct lu_name lname_dotdot = { sizeof(dotdot) - 1 }; -static int __mdd_lookup(const struct lu_env *env, struct md_object *pobj, - const struct lu_name *lname, struct lu_fid* fid, - int mask); +static int __mdd_lookup(const struct lu_env *, struct md_object *, + const struct lu_name *, struct lu_fid*, int); + static int __mdd_lookup_locked(const struct lu_env *env, struct md_object *pobj, const struct lu_name *lname, struct lu_fid* fid, int mask) { - char *name = lname->ln_name; + const char *name = lname->ln_name; struct mdd_object *mdd_obj = md2mdd_obj(pobj); struct dynlock_handle *dlh; int rc; @@ -89,9 +78,9 @@ __mdd_lookup_locked(const struct lu_env *env, struct md_object *pobj, return rc; } -static int mdd_lookup(const struct lu_env *env, - struct md_object *pobj, const struct lu_name *lname, - struct lu_fid* fid, struct md_op_spec *spec) +int mdd_lookup(const struct lu_env *env, + struct md_object *pobj, const struct lu_name *lname, + struct lu_fid* fid, struct md_op_spec *spec) { int rc; ENTRY; @@ -99,18 +88,17 @@ static int mdd_lookup(const struct lu_env *env, RETURN(rc); } - -static int mdd_parent_fid(const struct lu_env *env, struct mdd_object *obj, - struct lu_fid *fid) +int mdd_parent_fid(const struct lu_env *env, struct mdd_object *obj, + struct lu_fid *fid) { return __mdd_lookup_locked(env, &obj->mod_obj, &lname_dotdot, fid, 0); } /* - * For root fid use special function, whcih does not compare version component - * of fid. Vresion component is different for root fids on all MDTs. + * For root fid use special function, which does not compare version component + * of fid. Version component is different for root fids on all MDTs. */ -static int mdd_is_root(struct mdd_device *mdd, const struct lu_fid *fid) +int mdd_is_root(struct mdd_device *mdd, const struct lu_fid *fid) { return fid_seq(&mdd->mdd_root_fid) == fid_seq(fid) && fid_oid(&mdd->mdd_root_fid) == fid_oid(fid); @@ -155,16 +143,17 @@ static int mdd_is_parent(const struct lu_env *env, GOTO(out, rc = 1); if (parent) mdd_object_put(env, parent); - parent = mdd_object_find(env, mdd, pfid); - - /* cross-ref parent */ - if (parent == NULL) { - if (pf != NULL) - *pf = *pfid; - GOTO(out, rc = -EREMOTE); - } else if (IS_ERR(parent)) - GOTO(out, rc = PTR_ERR(parent)); - p1 = parent; + + parent = mdd_object_find(env, mdd, pfid); + if (IS_ERR(parent)) { + GOTO(out, rc = PTR_ERR(parent)); + } else if (mdd_object_remote(parent)) { + /*FIXME: Because of the restriction of rename in Phase I. + * If the parent is remote, we just assumed lf is not the + * parent of P1 for now */ + GOTO(out, rc = 0); + } + p1 = parent; } EXIT; out: @@ -184,9 +173,8 @@ out: * * returns < 0: if error */ -static int mdd_is_subdir(const struct lu_env *env, - struct md_object *mo, const struct lu_fid *fid, - struct lu_fid *sfid) +int mdd_is_subdir(const struct lu_env *env, struct md_object *mo, + const struct lu_fid *fid, struct lu_fid *sfid) { struct mdd_device *mdd = mdo2mdd(mo); int rc; @@ -232,8 +220,8 @@ static int mdd_dir_is_empty(const struct lu_env *env, RETURN(-ENOTDIR); iops = &obj->do_index_ops->dio_it; - it = iops->init(env, obj, 0, BYPASS_CAPA); - if (it != NULL) { + it = iops->init(env, obj, LUDA_64BITHASH, BYPASS_CAPA); + if (!IS_ERR(it)) { result = iops->get(env, it, (const void *)""); if (result > 0) { int i; @@ -252,7 +240,7 @@ static int mdd_dir_is_empty(const struct lu_env *env, iops->put(env, it); iops->fini(env, it); } else - result = -ENOMEM; + result = PTR_ERR(it); RETURN(result); } @@ -269,7 +257,7 @@ static int __mdd_may_link(const struct lu_env *env, struct mdd_object *obj) /* * Subdir count limitation can be broken through. - */ + */ if (la->la_nlink >= m->mdd_dt_conf.ddp_max_nlink && !S_ISDIR(la->la_mode)) RETURN(-EMLINK); @@ -287,7 +275,7 @@ int mdd_may_create(const struct lu_env *env, struct mdd_object *pobj, int rc = 0; ENTRY; - if (cobj && mdd_object_exists(cobj)) + if (cobj && mdd_object_exists(cobj)) RETURN(-EEXIST); if (mdd_is_dead_obj(pobj)) @@ -297,7 +285,6 @@ int mdd_may_create(const struct lu_env *env, struct mdd_object *pobj, rc = mdd_permission_internal_locked(env, pobj, NULL, MAY_WRITE | MAY_EXEC, MOR_TGT_PARENT); - if (!rc && check_nlink) rc = __mdd_may_link(env, pobj); @@ -308,7 +295,7 @@ int mdd_may_create(const struct lu_env *env, struct mdd_object *pobj, * Check whether can unlink from the pobj in the case of "cobj == NULL". */ int mdd_may_unlink(const struct lu_env *env, struct mdd_object *pobj, - const struct md_attr *ma) + const struct lu_attr *attr) { int rc; ENTRY; @@ -316,8 +303,8 @@ int mdd_may_unlink(const struct lu_env *env, struct mdd_object *pobj, if (mdd_is_dead_obj(pobj)) RETURN(-ENOENT); - if ((ma->ma_attr.la_valid & LA_FLAGS) && - (ma->ma_attr.la_flags & (LUSTRE_APPEND_FL | LUSTRE_IMMUTABLE_FL))) + if ((attr->la_valid & LA_FLAGS) && + (attr->la_flags & (LUSTRE_APPEND_FL | LUSTRE_IMMUTABLE_FL))) RETURN(-EPERM); rc = mdd_permission_internal_locked(env, pobj, NULL, @@ -337,31 +324,58 @@ int mdd_may_unlink(const struct lu_env *env, struct mdd_object *pobj, * VTX feature has been checked already, no need check again. */ static inline int mdd_is_sticky(const struct lu_env *env, - struct mdd_object *pobj, - struct mdd_object *cobj) + struct mdd_object *pobj, + struct mdd_object *cobj) { - struct lu_attr *tmp_la = &mdd_env_info(env)->mti_la; - struct md_ucred *uc = md_ucred(env); - int rc; + struct lu_attr *tmp_la = &mdd_env_info(env)->mti_la; + struct lu_ucred *uc = lu_ucred_assert(env); + int rc; - if (pobj) { - rc = mdd_la_get(env, pobj, tmp_la, BYPASS_CAPA); - if (rc) - return rc; - - if (!(tmp_la->la_mode & S_ISVTX) || - (tmp_la->la_uid == uc->mu_fsuid)) - return 0; - } + if (pobj) { + rc = mdd_la_get(env, pobj, tmp_la, BYPASS_CAPA); + if (rc) + return rc; - rc = mdd_la_get(env, cobj, tmp_la, BYPASS_CAPA); - if (rc) - return rc; - - if (tmp_la->la_uid == uc->mu_fsuid) - return 0; - - return !mdd_capable(uc, CFS_CAP_FOWNER); + if (!(tmp_la->la_mode & S_ISVTX) || + (tmp_la->la_uid == uc->uc_fsuid)) + return 0; + } + + rc = mdd_la_get(env, cobj, tmp_la, BYPASS_CAPA); + if (rc) + return rc; + + if (tmp_la->la_uid == uc->uc_fsuid) + return 0; + + return !md_capable(uc, CFS_CAP_FOWNER); +} + +static int mdd_may_delete_entry(const struct lu_env *env, + struct mdd_object *pobj, int check_perm) +{ + ENTRY; + + LASSERT(pobj != NULL); + if (!mdd_object_exists(pobj)) + RETURN(-ENOENT); + + if (mdd_is_dead_obj(pobj)) + RETURN(-ENOENT); + + if (check_perm) { + int rc; + rc = mdd_permission_internal_locked(env, pobj, NULL, + MAY_WRITE | MAY_EXEC, + MOR_TGT_PARENT); + if (rc) + RETURN(rc); + } + + if (mdd_is_append(pobj)) + RETURN(-EPERM); + + RETURN(0); } /* @@ -369,55 +383,53 @@ static inline int mdd_is_sticky(const struct lu_env *env, * pobj maybe NULL */ int mdd_may_delete(const struct lu_env *env, struct mdd_object *pobj, - struct mdd_object *cobj, struct md_attr *ma, - int check_perm, int check_empty) + struct mdd_object *cobj, struct lu_attr *cattr, + struct lu_attr *src_attr, int check_perm, int check_empty) { int rc = 0; ENTRY; - LASSERT(cobj); + if (pobj) { + rc = mdd_may_delete_entry(env, pobj, check_perm); + if (rc != 0) + RETURN(rc); + } + + if (cobj == NULL) + RETURN(0); + if (!mdd_object_exists(cobj)) RETURN(-ENOENT); - if (pobj) { - if (mdd_is_dead_obj(pobj)) - RETURN(-ENOENT); - - if (check_perm) { - rc = mdd_permission_internal_locked(env, pobj, NULL, - MAY_WRITE | MAY_EXEC, - MOR_TGT_PARENT); - if (rc) - RETURN(rc); - } + if (mdd_is_dead_obj(cobj)) + RETURN(-ESTALE); - if (mdd_is_append(pobj)) - RETURN(-EPERM); - } - if (!(ma->ma_attr_flags & MDS_VTX_BYPASS) && - mdd_is_sticky(env, pobj, cobj)) + if (mdd_is_sticky(env, pobj, cobj)) RETURN(-EPERM); if (mdd_is_immutable(cobj) || mdd_is_append(cobj)) RETURN(-EPERM); - if ((ma->ma_attr.la_valid & LA_FLAGS) && - (ma->ma_attr.la_flags & (LUSTRE_APPEND_FL | LUSTRE_IMMUTABLE_FL))) + if ((cattr->la_valid & LA_FLAGS) && + (cattr->la_flags & (LUSTRE_APPEND_FL | LUSTRE_IMMUTABLE_FL))) RETURN(-EPERM); - if (S_ISDIR(ma->ma_attr.la_mode)) { - struct mdd_device *mdd = mdo2mdd(&cobj->mod_obj); + /* additional check the rename case */ + if (src_attr) { + if (S_ISDIR(src_attr->la_mode)) { + struct mdd_device *mdd = mdo2mdd(&cobj->mod_obj); - if (!S_ISDIR(mdd_object_type(cobj))) - RETURN(-ENOTDIR); + if (!S_ISDIR(cattr->la_mode)) + RETURN(-ENOTDIR); - if (lu_fid_eq(mdo2fid(cobj), &mdd->mdd_root_fid)) - RETURN(-EBUSY); - } else if (S_ISDIR(mdd_object_type(cobj))) - RETURN(-EISDIR); + if (lu_fid_eq(mdo2fid(cobj), &mdd->mdd_root_fid)) + RETURN(-EBUSY); + } else if (S_ISDIR(cattr->la_mode)) + RETURN(-EISDIR); + } - if (S_ISDIR(ma->ma_attr.la_mode) && check_empty) + if (S_ISDIR(cattr->la_mode) && check_empty) rc = mdd_dir_is_empty(env, cobj); RETURN(rc); @@ -436,6 +448,12 @@ int mdd_link_sanity_check(const struct lu_env *env, int rc = 0; ENTRY; + if (!mdd_object_exists(src_obj)) + RETURN(-ENOENT); + + if (mdd_is_dead_obj(src_obj)) + RETURN(-ESTALE); + /* Local ops, no lookup before link, check filename length here. */ if (lname && (lname->ln_namelen > m->mdd_dt_conf.ddp_max_name_len)) RETURN(-ENAMETOOLONG); @@ -458,50 +476,47 @@ int mdd_link_sanity_check(const struct lu_env *env, RETURN(rc); } -const struct dt_rec *__mdd_fid_rec(const struct lu_env *env, - const struct lu_fid *fid) +static int __mdd_index_delete_only(const struct lu_env *env, struct mdd_object *pobj, + const char *name, struct thandle *handle, + struct lustre_capa *capa) { - struct lu_fid_pack *pack = &mdd_env_info(env)->mti_pack; - - fid_pack(pack, fid, &mdd_env_info(env)->mti_fid2); - return (const struct dt_rec *)pack; -} + struct dt_object *next = mdd_object_child(pobj); + int rc; + ENTRY; -/** - * If subdir count is up to ddp_max_nlink, then enable MNLINK_OBJ flag and - * assign i_nlink to 1 which means the i_nlink for subdir count is incredible - * (maybe too large to be represented). It is a trick to break through the - * "i_nlink" limitation for subdir count. - */ -void __mdd_ref_add(const struct lu_env *env, struct mdd_object *obj, - struct thandle *handle) -{ - struct lu_attr *tmp_la = &mdd_env_info(env)->mti_la; - struct mdd_device *m = mdd_obj2mdd_dev(obj); + if (dt_try_as_dir(env, next)) { + rc = next->do_index_ops->dio_delete(env, next, + (struct dt_key *)name, + handle, capa); + } else + rc = -ENOTDIR; - if (!mdd_is_mnlink(obj)) { - if (S_ISDIR(mdd_object_type(obj))) { - if (mdd_la_get(env, obj, tmp_la, BYPASS_CAPA)) - return; - - if (tmp_la->la_nlink >= m->mdd_dt_conf.ddp_max_nlink) { - obj->mod_flags |= MNLINK_OBJ; - tmp_la->la_nlink = 1; - tmp_la->la_valid = LA_NLINK; - mdd_attr_set_internal(env, obj, tmp_la, handle, - 0); - return; - } - } - mdo_ref_add(env, obj, handle); - } + RETURN(rc); } -void __mdd_ref_del(const struct lu_env *env, struct mdd_object *obj, - struct thandle *handle, int is_dot) +static int __mdd_index_insert_only(const struct lu_env *env, + struct mdd_object *pobj, + const struct lu_fid *lf, const char *name, + struct thandle *handle, + struct lustre_capa *capa) { - if (!mdd_is_mnlink(obj) || is_dot) - mdo_ref_del(env, obj, handle); + struct dt_object *next = mdd_object_child(pobj); + int rc; + ENTRY; + + if (dt_try_as_dir(env, next)) { + struct lu_ucred *uc = lu_ucred_check(env); + int ignore_quota; + + ignore_quota = uc ? uc->uc_cap & CFS_CAP_SYS_RESOURCE_MASK : 1; + rc = next->do_index_ops->dio_insert(env, next, + (struct dt_rec*)lf, + (const struct dt_key *)name, + handle, capa, ignore_quota); + } else { + rc = -ENOTDIR; + } + RETURN(rc); } /* insert named index, add reference if isdir */ @@ -509,25 +524,14 @@ static int __mdd_index_insert(const struct lu_env *env, struct mdd_object *pobj, const struct lu_fid *lf, const char *name, int is_dir, struct thandle *handle, struct lustre_capa *capa) { - struct dt_object *next = mdd_object_child(pobj); int rc; ENTRY; - if (dt_try_as_dir(env, next)) { - rc = next->do_index_ops->dio_insert(env, next, - __mdd_fid_rec(env, lf), - (const struct dt_key *)name, - handle, capa); - } else { - rc = -ENOTDIR; - } - - if (rc == 0) { - if (is_dir) { - mdd_write_lock(env, pobj, MOR_TGT_PARENT); - __mdd_ref_add(env, pobj, handle); - mdd_write_unlock(env, pobj); - } + rc = __mdd_index_insert_only(env, pobj, lf, name, handle, capa); + if (rc == 0 && is_dir) { + mdd_write_lock(env, pobj, MOR_TGT_PARENT); + mdo_ref_add(env, pobj, handle); + mdd_write_unlock(env, pobj); } RETURN(rc); } @@ -537,67 +541,722 @@ static int __mdd_index_delete(const struct lu_env *env, struct mdd_object *pobj, const char *name, int is_dir, struct thandle *handle, struct lustre_capa *capa) { - struct dt_object *next = mdd_object_child(pobj); int rc; ENTRY; - if (dt_try_as_dir(env, next)) { - rc = next->do_index_ops->dio_delete(env, next, - (struct dt_key *)name, - handle, capa); - if (rc == 0 && is_dir) { - int is_dot = 0; - - if (name != NULL && name[0] == '.' && name[1] == 0) - is_dot = 1; - mdd_write_lock(env, pobj, MOR_TGT_PARENT); - __mdd_ref_del(env, pobj, handle, is_dot); - mdd_write_unlock(env, pobj); - } - } else - rc = -ENOTDIR; + rc = __mdd_index_delete_only(env, pobj, name, handle, capa); + if (rc == 0 && is_dir) { + mdd_write_lock(env, pobj, MOR_TGT_PARENT); + mdo_ref_del(env, pobj, handle); + mdd_write_unlock(env, pobj); + } RETURN(rc); } -static int -__mdd_index_insert_only(const struct lu_env *env, struct mdd_object *pobj, - const struct lu_fid *lf, const char *name, - struct thandle *handle, struct lustre_capa *capa) +int mdd_declare_llog_record(const struct lu_env *env, struct mdd_device *mdd, + int reclen, struct thandle *handle) { - struct dt_object *next = mdd_object_child(pobj); - int rc; - ENTRY; + int rc; - if (dt_try_as_dir(env, next)) { - rc = next->do_index_ops->dio_insert(env, next, - __mdd_fid_rec(env, lf), - (const struct dt_key *)name, - handle, capa); - } else { - rc = -ENOTDIR; - } - RETURN(rc); + /* XXX: this is a temporary solution to declare llog changes + * will be fixed in 2.3 with new llog implementation */ + + LASSERT(mdd->mdd_capa); + + /* XXX: Since we use the 'mdd_capa' as fake llog object here, we + * have to set the parameter 'size' as INT_MAX or 0 to inform + * OSD that this record write is for a llog write or catalog + * header update, and osd declare function will reserve less + * credits for optimization purpose. + * + * Reserve 6 blocks for a llog write, since the llog file is + * usually small, reserve 2 blocks for catalog header update, + * because we know for sure that catalog header is already + * allocated. + * + * This hack should be removed in 2.3. + */ + + /* record itself */ + rc = dt_declare_record_write(env, mdd->mdd_capa, + DECLARE_LLOG_WRITE, 0, handle); + if (rc) + return rc; + + /* header will be updated as well */ + rc = dt_declare_record_write(env, mdd->mdd_capa, + DECLARE_LLOG_WRITE, 0, handle); + if (rc) + return rc; + + /* also we should be able to create new plain log */ + rc = dt_declare_create(env, mdd->mdd_capa, NULL, NULL, NULL, handle); + if (rc) + return rc; + + /* new record referencing new plain llog */ + rc = dt_declare_record_write(env, mdd->mdd_capa, + DECLARE_LLOG_WRITE, 0, handle); + if (rc) + return rc; + + /* catalog's header will be updated as well */ + rc = dt_declare_record_write(env, mdd->mdd_capa, + DECLARE_LLOG_REWRITE, 0, handle); + + return rc; +} + +int mdd_declare_changelog_store(const struct lu_env *env, + struct mdd_device *mdd, + const struct lu_name *fname, + struct thandle *handle) +{ + struct obd_device *obd = mdd2obd_dev(mdd); + struct llog_ctxt *ctxt; + struct llog_changelog_rec *rec; + struct lu_buf *buf; + int reclen; + int rc; + + /* Not recording */ + if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + return 0; + + reclen = llog_data_len(sizeof(*rec) + + (fname != NULL ? fname->ln_namelen : 0)); + buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_big_buf, reclen); + if (buf->lb_buf == NULL) + return -ENOMEM; + + rec = buf->lb_buf; + rec->cr_hdr.lrh_len = reclen; + rec->cr_hdr.lrh_type = CHANGELOG_REC; + + ctxt = llog_get_context(obd, LLOG_CHANGELOG_ORIG_CTXT); + if (ctxt == NULL) + return -ENXIO; + + rc = llog_declare_add(env, ctxt->loc_handle, &rec->cr_hdr, handle); + llog_ctxt_put(ctxt); + + return rc; +} + +static int mdd_declare_changelog_ext_store(const struct lu_env *env, + struct mdd_device *mdd, + const struct lu_name *tname, + const struct lu_name *sname, + struct thandle *handle) +{ + struct obd_device *obd = mdd2obd_dev(mdd); + struct llog_ctxt *ctxt; + struct llog_changelog_ext_rec *rec; + struct lu_buf *buf; + int reclen; + int rc; + + /* Not recording */ + if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + return 0; + + reclen = llog_data_len(sizeof(*rec) + + (tname != NULL ? tname->ln_namelen : 0) + + (sname != NULL ? 1 + sname->ln_namelen : 0)); + buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_big_buf, reclen); + if (buf->lb_buf == NULL) + return -ENOMEM; + + rec = buf->lb_buf; + rec->cr_hdr.lrh_len = reclen; + rec->cr_hdr.lrh_type = CHANGELOG_REC; + + ctxt = llog_get_context(obd, LLOG_CHANGELOG_ORIG_CTXT); + if (ctxt == NULL) + return -ENXIO; + + rc = llog_declare_add(env, ctxt->loc_handle, &rec->cr_hdr, handle); + llog_ctxt_put(ctxt); + + return rc; +} + +/** Add a changelog entry \a rec to the changelog llog + * \param mdd + * \param rec + * \param handle - currently ignored since llogs start their own transaction; + * this will hopefully be fixed in llog rewrite + * \retval 0 ok + */ +int mdd_changelog_store(const struct lu_env *env, struct mdd_device *mdd, + struct llog_changelog_rec *rec, struct thandle *th) +{ + struct obd_device *obd = mdd2obd_dev(mdd); + struct llog_ctxt *ctxt; + int rc; + + rec->cr_hdr.lrh_len = llog_data_len(sizeof(*rec) + rec->cr.cr_namelen); + rec->cr_hdr.lrh_type = CHANGELOG_REC; + rec->cr.cr_time = cl_time(); + + spin_lock(&mdd->mdd_cl.mc_lock); + /* NB: I suppose it's possible llog_add adds out of order wrt cr_index, + * but as long as the MDD transactions are ordered correctly for e.g. + * rename conflicts, I don't think this should matter. */ + rec->cr.cr_index = ++mdd->mdd_cl.mc_index; + spin_unlock(&mdd->mdd_cl.mc_lock); + + ctxt = llog_get_context(obd, LLOG_CHANGELOG_ORIG_CTXT); + if (ctxt == NULL) + return -ENXIO; + + rc = llog_add(env, ctxt->loc_handle, &rec->cr_hdr, NULL, NULL, th); + llog_ctxt_put(ctxt); + if (rc > 0) + rc = 0; + return rc; +} + +/** Add a changelog_ext entry \a rec to the changelog llog + * \param mdd + * \param rec + * \param handle - currently ignored since llogs start their own transaction; + * this will hopefully be fixed in llog rewrite + * \retval 0 ok + */ +int mdd_changelog_ext_store(const struct lu_env *env, struct mdd_device *mdd, + struct llog_changelog_ext_rec *rec, + struct thandle *th) +{ + struct obd_device *obd = mdd2obd_dev(mdd); + struct llog_ctxt *ctxt; + int rc; + + rec->cr_hdr.lrh_len = llog_data_len(sizeof(*rec) + rec->cr.cr_namelen); + /* llog_lvfs_write_rec sets the llog tail len */ + rec->cr_hdr.lrh_type = CHANGELOG_REC; + rec->cr.cr_time = cl_time(); + + spin_lock(&mdd->mdd_cl.mc_lock); + /* NB: I suppose it's possible llog_add adds out of order wrt cr_index, + * but as long as the MDD transactions are ordered correctly for e.g. + * rename conflicts, I don't think this should matter. */ + rec->cr.cr_index = ++mdd->mdd_cl.mc_index; + spin_unlock(&mdd->mdd_cl.mc_lock); + + ctxt = llog_get_context(obd, LLOG_CHANGELOG_ORIG_CTXT); + if (ctxt == NULL) + return -ENXIO; + + /* nested journal transaction */ + rc = llog_add(env, ctxt->loc_handle, &rec->cr_hdr, NULL, NULL, th); + llog_ctxt_put(ctxt); + if (rc > 0) + rc = 0; + + return rc; +} + +/** Store a namespace change changelog record + * If this fails, we must fail the whole transaction; we don't + * want the change to commit without the log entry. + * \param target - mdd_object of change + * \param parent - parent dir/object + * \param tname - target name string + * \param handle - transacion handle + */ +int mdd_changelog_ns_store(const struct lu_env *env, struct mdd_device *mdd, + enum changelog_rec_type type, unsigned flags, + struct mdd_object *target, struct mdd_object *parent, + const struct lu_name *tname, struct thandle *handle) +{ + struct llog_changelog_rec *rec; + struct lu_buf *buf; + int reclen; + int rc; + ENTRY; + + /* Not recording */ + if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + RETURN(0); + if ((mdd->mdd_cl.mc_mask & (1 << type)) == 0) + RETURN(0); + + LASSERT(target != NULL); + LASSERT(parent != NULL); + LASSERT(tname != NULL); + LASSERT(handle != NULL); + + reclen = llog_data_len(sizeof(*rec) + tname->ln_namelen); + buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_big_buf, reclen); + if (buf->lb_buf == NULL) + RETURN(-ENOMEM); + rec = buf->lb_buf; + + rec->cr.cr_flags = CLF_VERSION | (CLF_FLAGMASK & flags); + rec->cr.cr_type = (__u32)type; + rec->cr.cr_tfid = *mdo2fid(target); + rec->cr.cr_pfid = *mdo2fid(parent); + rec->cr.cr_namelen = tname->ln_namelen; + memcpy(rec->cr.cr_name, tname->ln_name, tname->ln_namelen); + + target->mod_cltime = cfs_time_current_64(); + + rc = mdd_changelog_store(env, mdd, rec, handle); + if (rc < 0) { + CERROR("changelog failed: rc=%d, op%d %s c"DFID" p"DFID"\n", + rc, type, tname->ln_name, PFID(&rec->cr.cr_tfid), + PFID(&rec->cr.cr_pfid)); + RETURN(-EFAULT); + } + + RETURN(0); +} + + +/** Store a namespace change changelog record + * If this fails, we must fail the whole transaction; we don't + * want the change to commit without the log entry. + * \param target - mdd_object of change + * \param tpfid - target parent dir/object fid + * \param sfid - source object fid + * \param spfid - source parent fid + * \param tname - target name string + * \param sname - source name string + * \param handle - transacion handle + */ +static int mdd_changelog_ext_ns_store(const struct lu_env *env, + struct mdd_device *mdd, + enum changelog_rec_type type, + unsigned flags, + struct mdd_object *target, + const struct lu_fid *tpfid, + const struct lu_fid *sfid, + const struct lu_fid *spfid, + const struct lu_name *tname, + const struct lu_name *sname, + struct thandle *handle) +{ + struct llog_changelog_ext_rec *rec; + struct lu_buf *buf; + int reclen; + int rc; + ENTRY; + + /* Not recording */ + if (!(mdd->mdd_cl.mc_flags & CLM_ON)) + RETURN(0); + if ((mdd->mdd_cl.mc_mask & (1 << type)) == 0) + RETURN(0); + + LASSERT(sfid != NULL); + LASSERT(tpfid != NULL); + LASSERT(tname != NULL); + LASSERT(handle != NULL); + + reclen = llog_data_len(sizeof(*rec) + + sname != NULL ? 1 + sname->ln_namelen : 0); + buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_big_buf, reclen); + if (buf->lb_buf == NULL) + RETURN(-ENOMEM); + rec = buf->lb_buf; + + rec->cr.cr_flags = CLF_EXT_VERSION | (CLF_FLAGMASK & flags); + rec->cr.cr_type = (__u32)type; + rec->cr.cr_pfid = *tpfid; + rec->cr.cr_sfid = *sfid; + rec->cr.cr_spfid = *spfid; + rec->cr.cr_namelen = tname->ln_namelen; + memcpy(rec->cr.cr_name, tname->ln_name, tname->ln_namelen); + if (sname) { + rec->cr.cr_name[tname->ln_namelen] = '\0'; + memcpy(rec->cr.cr_name + tname->ln_namelen + 1, sname->ln_name, + sname->ln_namelen); + rec->cr.cr_namelen += 1 + sname->ln_namelen; + } + + if (likely(target != NULL)) { + rec->cr.cr_tfid = *mdo2fid(target); + target->mod_cltime = cfs_time_current_64(); + } else { + fid_zero(&rec->cr.cr_tfid); + } + + rc = mdd_changelog_ext_store(env, mdd, rec, handle); + if (rc < 0) { + CERROR("changelog failed: rc=%d, op%d %s c"DFID" p"DFID"\n", + rc, type, tname->ln_name, PFID(sfid), PFID(tpfid)); + return -EFAULT; + } + + return 0; +} + +static int __mdd_links_add(const struct lu_env *env, + struct mdd_object *mdd_obj, + struct linkea_data *ldata, + const struct lu_name *lname, + const struct lu_fid *pfid, + int first, int check) +{ + int rc; + + if (ldata->ld_leh == NULL) { + rc = first ? -ENODATA : mdd_links_read(env, mdd_obj, ldata); + if (rc) { + if (rc != -ENODATA) + return rc; + rc = linkea_data_new(ldata, + &mdd_env_info(env)->mti_link_buf); + if (rc) + return rc; + } + } + + if (check) { + rc = linkea_links_find(ldata, lname, pfid); + if (rc && rc != -ENOENT) + return rc; + if (rc == 0) + return -EEXIST; + } + + if (OBD_FAIL_CHECK(OBD_FAIL_LFSCK_LINKEA_MORE)) { + struct lu_fid *tfid = &mdd_env_info(env)->mti_fid2; + + *tfid = *pfid; + tfid->f_ver = ~0; + linkea_add_buf(ldata, lname, tfid); + } + + return linkea_add_buf(ldata, lname, pfid); +} + +static int __mdd_links_del(const struct lu_env *env, + struct mdd_object *mdd_obj, + struct linkea_data *ldata, + const struct lu_name *lname, + const struct lu_fid *pfid) +{ + int rc; + + if (ldata->ld_leh == NULL) { + rc = mdd_links_read(env, mdd_obj, ldata); + if (rc) + return rc; + } + + rc = linkea_links_find(ldata, lname, pfid); + if (rc) + return rc; + + linkea_del_buf(ldata, lname); + return 0; +} + +static int mdd_linkea_prepare(const struct lu_env *env, + struct mdd_object *mdd_obj, + const struct lu_fid *oldpfid, + const struct lu_name *oldlname, + const struct lu_fid *newpfid, + const struct lu_name *newlname, + int first, int check, + struct linkea_data *ldata) +{ + int rc = 0; + int rc2 = 0; + ENTRY; + + if (OBD_FAIL_CHECK(OBD_FAIL_FID_IGIF)) + return 0; + + LASSERT(oldpfid != NULL || newpfid != NULL); + + if (mdd_obj->mod_flags & DEAD_OBJ) + /* No more links, don't bother */ + RETURN(0); + + if (oldpfid != NULL) { + rc = __mdd_links_del(env, mdd_obj, ldata, oldlname, oldpfid); + if (rc) { + if ((check == 0) || + (rc != -ENODATA && rc != -ENOENT)) + RETURN(rc); + /* No changes done. */ + rc = 0; + } + } + + /* If renaming, add the new record */ + if (newpfid != NULL) { + /* even if the add fails, we still delete the out-of-date + * old link */ + rc2 = __mdd_links_add(env, mdd_obj, ldata, newlname, newpfid, + first, check); + if (rc2 == -EEXIST) + rc2 = 0; + } + + rc = rc != 0 ? rc : rc2; + + RETURN(rc); +} + +int mdd_links_rename(const struct lu_env *env, + struct mdd_object *mdd_obj, + const struct lu_fid *oldpfid, + const struct lu_name *oldlname, + const struct lu_fid *newpfid, + const struct lu_name *newlname, + struct thandle *handle, + struct linkea_data *ldata, + int first, int check) +{ + int rc2 = 0; + int rc = 0; + ENTRY; + + if (ldata == NULL) { + ldata = &mdd_env_info(env)->mti_link_data; + memset(ldata, 0, sizeof(*ldata)); + rc = mdd_linkea_prepare(env, mdd_obj, oldpfid, oldlname, + newpfid, newlname, first, check, + ldata); + if (rc != 0) + GOTO(out, rc); + } + + if (ldata->ld_lee != NULL) + rc = mdd_links_write(env, mdd_obj, ldata, handle); + EXIT; +out: + if (rc == 0) + rc = rc2; + if (rc) { + int error = 1; + if (rc == -EOVERFLOW || rc == -ENOENT) + error = 0; + if (oldpfid == NULL) + CDEBUG(error ? D_ERROR : D_OTHER, + "link_ea add '%.*s' failed %d "DFID"\n", + newlname->ln_namelen, newlname->ln_name, + rc, PFID(mdd_object_fid(mdd_obj))); + else if (newpfid == NULL) + CDEBUG(error ? D_ERROR : D_OTHER, + "link_ea del '%.*s' failed %d "DFID"\n", + oldlname->ln_namelen, oldlname->ln_name, + rc, PFID(mdd_object_fid(mdd_obj))); + else + CDEBUG(error ? D_ERROR : D_OTHER, + "link_ea rename '%.*s'->'%.*s' failed %d " + DFID"\n", + oldlname->ln_namelen, oldlname->ln_name, + newlname->ln_namelen, newlname->ln_name, + rc, PFID(mdd_object_fid(mdd_obj))); + } + + if (ldata->ld_buf && ldata->ld_buf->lb_len > OBD_ALLOC_BIG) + /* if we vmalloced a large buffer drop it */ + lu_buf_free(ldata->ld_buf); + + return rc; +} + +static inline int mdd_links_add(const struct lu_env *env, + struct mdd_object *mdd_obj, + const struct lu_fid *pfid, + const struct lu_name *lname, + struct thandle *handle, + struct linkea_data *data, int first) +{ + return mdd_links_rename(env, mdd_obj, NULL, NULL, + pfid, lname, handle, data, first, 0); +} + +static inline int mdd_links_del(const struct lu_env *env, + struct mdd_object *mdd_obj, + const struct lu_fid *pfid, + const struct lu_name *lname, + struct thandle *handle) +{ + return mdd_links_rename(env, mdd_obj, pfid, lname, + NULL, NULL, handle, NULL, 0, 0); +} + +/** Read the link EA into a temp buffer. + * Uses the mdd_thread_info::mti_big_buf since it is generally large. + * A pointer to the buffer is stored in \a ldata::ld_buf. + * + * \retval 0 or error + */ +int mdd_links_read(const struct lu_env *env, struct mdd_object *mdd_obj, + struct linkea_data *ldata) +{ + int rc; + + /* First try a small buf */ + LASSERT(env != NULL); + ldata->ld_buf = lu_buf_check_and_alloc(&mdd_env_info(env)->mti_link_buf, + CFS_PAGE_SIZE); + if (ldata->ld_buf->lb_buf == NULL) + return -ENOMEM; + + if (!mdd_object_exists(mdd_obj)) + return -ENODATA; + + rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf, XATTR_NAME_LINK, + BYPASS_CAPA); + if (rc == -ERANGE) { + /* Buf was too small, figure out what we need. */ + lu_buf_free(ldata->ld_buf); + rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf, + XATTR_NAME_LINK, BYPASS_CAPA); + if (rc < 0) + return rc; + ldata->ld_buf = lu_buf_check_and_alloc(ldata->ld_buf, rc); + if (ldata->ld_buf->lb_buf == NULL) + return -ENOMEM; + rc = mdo_xattr_get(env, mdd_obj, ldata->ld_buf, + XATTR_NAME_LINK, BYPASS_CAPA); + } + if (rc < 0) + return rc; + + linkea_init(ldata); + return 0; +} + +/** Read the link EA into a temp buffer. + * Uses the name_buf since it is generally large. + * \retval IS_ERR err + * \retval ptr to \a lu_buf (always \a mti_big_buf) + */ +struct lu_buf *mdd_links_get(const struct lu_env *env, + struct mdd_object *mdd_obj) +{ + struct linkea_data ldata = { 0 }; + int rc; + + rc = mdd_links_read(env, mdd_obj, &ldata); + return rc ? ERR_PTR(rc) : ldata.ld_buf; +} + +int mdd_links_write(const struct lu_env *env, struct mdd_object *mdd_obj, + struct linkea_data *ldata, struct thandle *handle) +{ + const struct lu_buf *buf = mdd_buf_get_const(env, ldata->ld_buf->lb_buf, + ldata->ld_leh->leh_len); + return mdo_xattr_set(env, mdd_obj, buf, XATTR_NAME_LINK, 0, handle, + mdd_object_capa(env, mdd_obj)); +} + +int mdd_declare_links_add(const struct lu_env *env, struct mdd_object *mdd_obj, + struct thandle *handle, struct linkea_data *ldata) +{ + int rc; + int ea_len; + void *linkea; + + if (ldata != NULL && ldata->ld_lee != NULL) { + ea_len = ldata->ld_leh->leh_len; + linkea = ldata->ld_buf->lb_buf; + } else { + ea_len = 4096; + linkea = NULL; + } + + /* XXX: max size? */ + rc = mdo_declare_xattr_set(env, mdd_obj, + mdd_buf_get_const(env, linkea, ea_len), + XATTR_NAME_LINK, 0, handle); + return rc; +} + +static inline int mdd_declare_links_del(const struct lu_env *env, + struct mdd_object *c, + struct thandle *handle) +{ + int rc = 0; + + /* For directory, the linkEA will be removed together + * with the object. */ + if (!S_ISDIR(mdd_object_type(c))) + rc = mdd_declare_links_add(env, c, handle, NULL); + + return rc; +} + +static int mdd_declare_link(const struct lu_env *env, + struct mdd_device *mdd, + struct mdd_object *p, + struct mdd_object *c, + const struct lu_name *name, + struct thandle *handle, + struct lu_attr *la, + struct linkea_data *data) +{ + int rc; + + rc = mdo_declare_index_insert(env, p, mdo2fid(c), name->ln_name,handle); + if (rc) + return rc; + + rc = mdo_declare_ref_add(env, c, handle); + if (rc) + return rc; + + la->la_valid = LA_CTIME | LA_MTIME; + rc = mdo_declare_attr_set(env, p, la, handle); + if (rc != 0) + return rc; + + la->la_valid = LA_CTIME; + rc = mdo_declare_attr_set(env, c, la, handle); + if (rc) + return rc; + + rc = mdd_declare_links_add(env, c, handle, data); + if (rc) + return rc; + + rc = mdd_declare_changelog_store(env, mdd, name, handle); + + return rc; } static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj, struct md_object *src_obj, const struct lu_name *lname, struct md_attr *ma) { - char *name = lname->ln_name; + const char *name = lname->ln_name; struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; struct mdd_object *mdd_tobj = md2mdd_obj(tgt_obj); struct mdd_object *mdd_sobj = md2mdd_obj(src_obj); struct mdd_device *mdd = mdo2mdd(src_obj); struct dynlock_handle *dlh; struct thandle *handle; + struct linkea_data *ldata = &mdd_env_info(env)->mti_link_data; int rc; ENTRY; - mdd_txn_param_build(env, mdd, MDD_TXN_LINK_OP); - handle = mdd_trans_start(env, mdd); + handle = mdd_trans_create(env, mdd); if (IS_ERR(handle)) - RETURN(PTR_ERR(handle)); + GOTO(out_pending, rc = PTR_ERR(handle)); + + memset(ldata, 0, sizeof(*ldata)); + + LASSERT(ma->ma_attr.la_valid & LA_CTIME); + la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; + + rc = mdd_declare_link(env, mdd, mdd_tobj, mdd_sobj, lname, handle, + la, ldata); + if (rc) + GOTO(stop, rc); + + rc = mdd_trans_start(env, mdd, handle); + if (rc) + GOTO(stop, rc); dlh = mdd_pdo_write_lock(env, mdd_tobj, name, MOR_TGT_CHILD); if (dlh == NULL) @@ -608,57 +1267,101 @@ static int mdd_link(const struct lu_env *env, struct md_object *tgt_obj, if (rc) GOTO(out_unlock, rc); - rc = __mdd_index_insert_only(env, mdd_tobj, mdo2fid(mdd_sobj), - name, handle, - mdd_object_capa(env, mdd_tobj)); - if (rc) - GOTO(out_unlock, rc); + rc = mdo_ref_add(env, mdd_sobj, handle); + if (rc) + GOTO(out_unlock, rc); - __mdd_ref_add(env, mdd_sobj, handle); - LASSERT(ma->ma_attr.la_valid & LA_CTIME); - la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; + rc = __mdd_index_insert_only(env, mdd_tobj, mdo2fid(mdd_sobj), + name, handle, + mdd_object_capa(env, mdd_tobj)); + if (rc != 0) { + mdo_ref_del(env, mdd_sobj, handle); + GOTO(out_unlock, rc); + } la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_tobj, la, handle, 0); + rc = mdd_attr_check_set_internal(env, mdd_tobj, la, handle, 0); if (rc) GOTO(out_unlock, rc); la->la_valid = LA_CTIME; rc = mdd_attr_check_set_internal(env, mdd_sobj, la, handle, 0); + if (rc == 0) { + rc = mdd_linkea_prepare(env, mdd_sobj, NULL, NULL, + mdo2fid(mdd_tobj), lname, 0, 0, + ldata); + if (rc == 0) + mdd_links_add(env, mdd_sobj, mdo2fid(mdd_tobj), + lname, handle, ldata, 0); + /* The failure of links_add should not cause the link + * failure, reset rc here */ + rc = 0; + } EXIT; out_unlock: mdd_write_unlock(env, mdd_sobj); mdd_pdo_write_unlock(env, mdd_tobj, dlh); out_trans: - mdd_trans_stop(env, mdd, rc, handle); + if (rc == 0) + rc = mdd_changelog_ns_store(env, mdd, CL_HARDLINK, 0, mdd_sobj, + mdd_tobj, lname, handle); +stop: + mdd_trans_stop(env, mdd, rc, handle); + + if (ldata->ld_buf && ldata->ld_buf->lb_len > OBD_ALLOC_BIG) + /* if we vmalloced a large buffer drop it */ + lu_buf_free(ldata->ld_buf); +out_pending: return rc; } +int mdd_declare_finish_unlink(const struct lu_env *env, + struct mdd_object *obj, + struct md_attr *ma, + struct thandle *handle) +{ + int rc; + + rc = orph_declare_index_insert(env, obj, mdd_object_type(obj), handle); + if (rc) + return rc; + + return mdo_declare_destroy(env, obj, handle); +} + /* caller should take a lock before calling */ int mdd_finish_unlink(const struct lu_env *env, struct mdd_object *obj, struct md_attr *ma, struct thandle *th) { - int rc; + int rc = 0; + int is_dir = S_ISDIR(ma->ma_attr.la_mode); ENTRY; - rc = mdd_iattr_get(env, obj, ma); - if (rc == 0 && ma->ma_attr.la_nlink == 0) { - /* add new orphan and the object - * will be deleted during the object_put() */ - if (__mdd_orphan_add(env, obj, th) == 0) - obj->mod_flags |= ORPHAN_OBJ; + LASSERT(mdd_write_locked(env, obj) != 0); + if (rc == 0 && (ma->ma_attr.la_nlink == 0 || is_dir)) { obj->mod_flags |= DEAD_OBJ; - if (obj->mod_count == 0) - rc = mdd_object_kill(env, obj, ma); - else - /* clear MA_LOV | MA_COOKIE, if we do not - * unlink it in case we get it somewhere */ - ma->ma_valid &= ~(MA_LOV | MA_COOKIE); - } else - ma->ma_valid &= ~(MA_LOV | MA_COOKIE); + /* add new orphan and the object + * will be deleted during mdd_close() */ + if (obj->mod_count) { + rc = __mdd_orphan_add(env, obj, th); + if (rc == 0) + CDEBUG(D_HA, "Object "DFID" is inserted into " + "orphan list, open count = %d\n", + PFID(mdd_object_fid(obj)), + obj->mod_count); + else + CERROR("Object "DFID" fail to be an orphan, " + "open count = %d, maybe cause failed " + "open replay\n", + PFID(mdd_object_fid(obj)), + obj->mod_count); + } else { + rc = mdo_destroy(env, obj, th); + } + } RETURN(rc); } @@ -668,344 +1371,228 @@ int mdd_finish_unlink(const struct lu_env *env, * has mdd_write_lock on cobj already, but not on pobj yet */ int mdd_unlink_sanity_check(const struct lu_env *env, struct mdd_object *pobj, - struct mdd_object *cobj, struct md_attr *ma) + struct mdd_object *cobj, struct lu_attr *cattr) { int rc; ENTRY; - rc = mdd_may_delete(env, pobj, cobj, ma, 1, 1); + rc = mdd_may_delete(env, pobj, cobj, cattr, NULL, 1, 1); RETURN(rc); } +static int mdd_declare_unlink(const struct lu_env *env, struct mdd_device *mdd, + struct mdd_object *p, struct mdd_object *c, + const struct lu_name *name, struct md_attr *ma, + struct thandle *handle, int no_name) +{ + struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; + int rc; + + if (likely(no_name == 0)) { + rc = mdo_declare_index_delete(env, p, name->ln_name, handle); + if (rc) + return rc; + } + + rc = mdo_declare_ref_del(env, p, handle); + if (rc) + return rc; + + LASSERT(ma->ma_attr.la_valid & LA_CTIME); + la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; + la->la_valid = LA_CTIME | LA_MTIME; + rc = mdo_declare_attr_set(env, p, la, handle); + if (rc) + return rc; + + if (c != NULL) { + rc = mdo_declare_ref_del(env, c, handle); + if (rc) + return rc; + + rc = mdo_declare_ref_del(env, c, handle); + if (rc) + return rc; + + la->la_valid = LA_CTIME; + rc = mdo_declare_attr_set(env, c, la, handle); + if (rc) + return rc; + + rc = mdd_declare_finish_unlink(env, c, ma, handle); + if (rc) + return rc; + + rc = mdd_declare_links_del(env, c, handle); + if (rc != 0) + return rc; + + /* FIXME: need changelog for remove entry */ + rc = mdd_declare_changelog_store(env, mdd, name, handle); + } + + return rc; +} + +/** + * Delete name entry and the object. + * Note: no_name == 1 means it only destory the object, i.e. name_entry + * does not exist for this object, and it could only happen during resending + * of remote unlink. see the comments in mdt_reint_unlink. Unfortunately, lname + * is also needed in this case(needed by changelog), so we have to add another + * parameter(no_name)here. XXX: this is only needed in DNE phase I, on Phase II, + * the ENOENT failure should be able to be fixed by redo mechanism. + */ static int mdd_unlink(const struct lu_env *env, struct md_object *pobj, - struct md_object *cobj, const struct lu_name *lname, - struct md_attr *ma) + struct md_object *cobj, const struct lu_name *lname, + struct md_attr *ma, int no_name) { - char *name = lname->ln_name; + const char *name = lname->ln_name; + struct lu_attr *cattr = &mdd_env_info(env)->mti_cattr; struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; struct mdd_object *mdd_pobj = md2mdd_obj(pobj); - struct mdd_object *mdd_cobj = md2mdd_obj(cobj); + struct mdd_object *mdd_cobj = NULL; struct mdd_device *mdd = mdo2mdd(pobj); struct dynlock_handle *dlh; struct thandle *handle; - int rc, is_dir; + int rc, is_dir = 0; ENTRY; - LASSERTF(mdd_object_exists(mdd_cobj) > 0, "FID is "DFID"\n", - PFID(mdd_object_fid(mdd_cobj))); - - rc = mdd_log_txn_param_build(env, cobj, ma, MDD_TXN_UNLINK_OP); - if (rc) - RETURN(rc); - - handle = mdd_trans_start(env, mdd); + /* cobj == NULL means only delete name entry */ + if (likely(cobj != NULL)) { + mdd_cobj = md2mdd_obj(cobj); + if (mdd_object_exists(mdd_cobj) == 0) + RETURN(-ENOENT); + /* currently it is assume, it could only delete + * name entry of remote directory */ + is_dir = 1; + } + + handle = mdd_trans_create(env, mdd); if (IS_ERR(handle)) RETURN(PTR_ERR(handle)); + rc = mdd_declare_unlink(env, mdd, mdd_pobj, mdd_cobj, + lname, ma, handle, no_name); + if (rc) + GOTO(stop, rc); + + rc = mdd_trans_start(env, mdd, handle); + if (rc) + GOTO(stop, rc); + + dlh = mdd_pdo_write_lock(env, mdd_pobj, name, MOR_TGT_PARENT); + if (dlh == NULL) + GOTO(stop, rc = -ENOMEM); + + if (likely(mdd_cobj != NULL)) { + mdd_write_lock(env, mdd_cobj, MOR_TGT_CHILD); + + /* fetch cattr */ + rc = mdd_la_get(env, mdd_cobj, cattr, + mdd_object_capa(env, mdd_cobj)); + if (rc) + GOTO(cleanup, rc); + + is_dir = S_ISDIR(cattr->la_mode); + + } + + rc = mdd_unlink_sanity_check(env, mdd_pobj, mdd_cobj, cattr); + if (rc) + GOTO(cleanup, rc); + + if (likely(no_name == 0)) { + rc = __mdd_index_delete(env, mdd_pobj, name, is_dir, handle, + mdd_object_capa(env, mdd_pobj)); + if (rc) + GOTO(cleanup, rc); + } + + if (likely(mdd_cobj != NULL)) { + rc = mdo_ref_del(env, mdd_cobj, handle); + if (rc != 0) { + __mdd_index_insert_only(env, mdd_pobj, + mdo2fid(mdd_cobj), + name, handle, + mdd_object_capa(env, mdd_pobj)); + GOTO(cleanup, rc); + } + + if (is_dir) + /* unlink dot */ + mdo_ref_del(env, mdd_cobj, handle); + + /* fetch updated nlink */ + rc = mdd_la_get(env, mdd_cobj, cattr, + mdd_object_capa(env, mdd_cobj)); + if (rc) + GOTO(cleanup, rc); + } + + LASSERT(ma->ma_attr.la_valid & LA_CTIME); + la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; + + la->la_valid = LA_CTIME | LA_MTIME; + rc = mdd_attr_check_set_internal(env, mdd_pobj, la, handle, 0); + if (rc) + GOTO(cleanup, rc); + + /* Enough for only unlink the entry */ + if (unlikely(mdd_cobj == NULL)) { + mdd_pdo_write_unlock(env, mdd_pobj, dlh); + GOTO(stop, rc); + } + + if (cattr->la_nlink > 0 || mdd_cobj->mod_count > 0) { + /* update ctime of an unlinked file only if it is still + * opened or a link still exists */ + la->la_valid = LA_CTIME; + rc = mdd_attr_check_set_internal(env, mdd_cobj, la, handle, 0); + if (rc) + GOTO(cleanup, rc); + } - dlh = mdd_pdo_write_lock(env, mdd_pobj, name, MOR_TGT_PARENT); - if (dlh == NULL) - GOTO(out_trans, rc = -ENOMEM); - mdd_write_lock(env, mdd_cobj, MOR_TGT_CHILD); - - is_dir = S_ISDIR(ma->ma_attr.la_mode); - rc = mdd_unlink_sanity_check(env, mdd_pobj, mdd_cobj, ma); - if (rc) - GOTO(cleanup, rc); + /* XXX: this transfer to ma will be removed with LOD/OSP */ + ma->ma_attr = *cattr; + ma->ma_valid |= MA_INODE; + rc = mdd_finish_unlink(env, mdd_cobj, ma, handle); - rc = __mdd_index_delete(env, mdd_pobj, name, is_dir, handle, - mdd_object_capa(env, mdd_pobj)); - if (rc) - GOTO(cleanup, rc); + /* fetch updated nlink */ + if (rc == 0) + rc = mdd_la_get(env, mdd_cobj, cattr, + mdd_object_capa(env, mdd_cobj)); - __mdd_ref_del(env, mdd_cobj, handle, 0); - if (is_dir) - /* unlink dot */ - __mdd_ref_del(env, mdd_cobj, handle, 1); + if (!is_dir) + /* old files may not have link ea; ignore errors */ + mdd_links_del(env, mdd_cobj, mdo2fid(mdd_pobj), lname, handle); - LASSERT(ma->ma_attr.la_valid & LA_CTIME); - la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; - - la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_pobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); - - la->la_valid = LA_CTIME; - rc = mdd_attr_check_set_internal(env, mdd_cobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); - - rc = mdd_finish_unlink(env, mdd_cobj, ma, handle); - - if (rc == 0) - obd_set_info_async(mdd2obd_dev(mdd)->u.mds.mds_osc_exp, - sizeof(KEY_UNLINKED), KEY_UNLINKED, 0, - NULL, NULL); + /* if object is removed then we can't get its attrs, use last get */ + if (cattr->la_nlink == 0) { + ma->ma_attr = *cattr; + ma->ma_valid |= MA_INODE; + } EXIT; cleanup: mdd_write_unlock(env, mdd_cobj); mdd_pdo_write_unlock(env, mdd_pobj, dlh); -out_trans: - mdd_trans_stop(env, mdd, rc, handle); - return rc; -} - -/* has not lock on pobj yet */ -static int mdd_ni_sanity_check(const struct lu_env *env, - struct md_object *pobj, - const struct md_attr *ma) -{ - struct mdd_object *obj = md2mdd_obj(pobj); - int rc; - ENTRY; - - if (ma->ma_attr_flags & MDS_PERM_BYPASS) - RETURN(0); - - rc = mdd_may_create(env, obj, NULL, 1, S_ISDIR(ma->ma_attr.la_mode)); - - RETURN(rc); -} - -/* - * Partial operation. - */ -static int mdd_name_insert(const struct lu_env *env, - struct md_object *pobj, - const struct lu_name *lname, - const struct lu_fid *fid, - const struct md_attr *ma) -{ - char *name = lname->ln_name; - struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; - struct mdd_object *mdd_obj = md2mdd_obj(pobj); - struct mdd_device *mdd = mdo2mdd(pobj); - struct dynlock_handle *dlh; - struct thandle *handle; - int is_dir = S_ISDIR(ma->ma_attr.la_mode); - int rc; - ENTRY; - - mdd_txn_param_build(env, mdd, MDD_TXN_INDEX_INSERT_OP); - handle = mdd_trans_start(env, mdo2mdd(pobj)); - if (IS_ERR(handle)) - RETURN(PTR_ERR(handle)); - - dlh = mdd_pdo_write_lock(env, mdd_obj, name, MOR_TGT_PARENT); - if (dlh == NULL) - GOTO(out_trans, rc = -ENOMEM); - - rc = mdd_ni_sanity_check(env, pobj, ma); - if (rc) - GOTO(out_unlock, rc); + if (rc == 0) { + int cl_flags; - rc = __mdd_index_insert(env, mdd_obj, fid, name, is_dir, - handle, BYPASS_CAPA); - if (rc) - GOTO(out_unlock, rc); + cl_flags = (cattr->la_nlink == 0) ? CLF_UNLINK_LAST : 0; + if ((ma->ma_valid & MA_HSM) && + (ma->ma_hsm.mh_flags & HS_EXISTS)) + cl_flags |= CLF_UNLINK_HSM_EXISTS; - /* - * For some case, no need update obj's ctime (LA_CTIME is not set), - * e.g. split_dir. - * For other cases, update obj's ctime (LA_CTIME is set), - * e.g. cmr_link. - */ - if (ma->ma_attr.la_valid & LA_CTIME) { - la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; - la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_obj, la, - handle, 0); + rc = mdd_changelog_ns_store(env, mdd, + is_dir ? CL_RMDIR : CL_UNLINK, cl_flags, + mdd_cobj, mdd_pobj, lname, handle); } - EXIT; -out_unlock: - mdd_pdo_write_unlock(env, mdd_obj, dlh); -out_trans: - mdd_trans_stop(env, mdo2mdd(pobj), rc, handle); - return rc; -} -/* has not lock on pobj yet */ -static int mdd_nr_sanity_check(const struct lu_env *env, - struct md_object *pobj, - const struct md_attr *ma) -{ - struct mdd_object *obj = md2mdd_obj(pobj); - int rc; - ENTRY; - - if (ma->ma_attr_flags & MDS_PERM_BYPASS) - RETURN(0); - - rc = mdd_may_unlink(env, obj, ma); - - RETURN(rc); -} - -/* - * Partial operation. - */ -static int mdd_name_remove(const struct lu_env *env, - struct md_object *pobj, - const struct lu_name *lname, - const struct md_attr *ma) -{ - char *name = lname->ln_name; - struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; - struct mdd_object *mdd_obj = md2mdd_obj(pobj); - struct mdd_device *mdd = mdo2mdd(pobj); - struct dynlock_handle *dlh; - struct thandle *handle; - int is_dir = S_ISDIR(ma->ma_attr.la_mode); - int rc; - ENTRY; - - mdd_txn_param_build(env, mdd, MDD_TXN_INDEX_DELETE_OP); - handle = mdd_trans_start(env, mdd); - if (IS_ERR(handle)) - RETURN(PTR_ERR(handle)); - - dlh = mdd_pdo_write_lock(env, mdd_obj, name, MOR_TGT_PARENT); - if (dlh == NULL) - GOTO(out_trans, rc = -ENOMEM); - - rc = mdd_nr_sanity_check(env, pobj, ma); - if (rc) - GOTO(out_unlock, rc); - - rc = __mdd_index_delete(env, mdd_obj, name, is_dir, - handle, BYPASS_CAPA); - if (rc) - GOTO(out_unlock, rc); - - /* - * For some case, no need update obj's ctime (LA_CTIME is not set), - * e.g. split_dir. - * For other cases, update obj's ctime (LA_CTIME is set), - * e.g. cmr_unlink. - */ - if (ma->ma_attr.la_valid & LA_CTIME) { - la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; - la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_obj, la, - handle, 0); - } - EXIT; -out_unlock: - mdd_pdo_write_unlock(env, mdd_obj, dlh); -out_trans: +stop: mdd_trans_stop(env, mdd, rc, handle); - return rc; -} - -/* - * tobj maybe NULL - * has mdd_write_lock on tobj alreay, but not on tgt_pobj yet - */ -static int mdd_rt_sanity_check(const struct lu_env *env, - struct mdd_object *tgt_pobj, - struct mdd_object *tobj, - struct md_attr *ma) -{ - int rc; - ENTRY; - - if (unlikely(ma->ma_attr_flags & MDS_PERM_BYPASS)) - RETURN(0); - - /* XXX: for mdd_rename_tgt, "tobj == NULL" does not mean tobj not - * exist. In fact, tobj must exist, otherwise the call trace will be: - * mdt_reint_rename_tgt -> mdo_name_insert -> ... -> mdd_name_insert. - * When get here, tobj must be NOT NULL, the other case has been - * processed in cmr_rename_tgt before mdd_rename_tgt and enable - * MDS_PERM_BYPASS. - * So check may_delete, but not check nlink of tgt_pobj. */ - LASSERT(tobj); - rc = mdd_may_delete(env, tgt_pobj, tobj, ma, 1, 1); - - RETURN(rc); -} - -static int mdd_rename_tgt(const struct lu_env *env, - struct md_object *pobj, struct md_object *tobj, - const struct lu_fid *lf, const struct lu_name *lname, - struct md_attr *ma) -{ - char *name = lname->ln_name; - struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; - struct mdd_object *mdd_tpobj = md2mdd_obj(pobj); - struct mdd_object *mdd_tobj = md2mdd_obj(tobj); - struct mdd_device *mdd = mdo2mdd(pobj); - struct dynlock_handle *dlh; - struct thandle *handle; - int rc; - ENTRY; - - mdd_txn_param_build(env, mdd, MDD_TXN_RENAME_TGT_OP); - handle = mdd_trans_start(env, mdd); - if (IS_ERR(handle)) - RETURN(PTR_ERR(handle)); - - dlh = mdd_pdo_write_lock(env, mdd_tpobj, name, MOR_TGT_PARENT); - if (dlh == NULL) - GOTO(out_trans, rc = -ENOMEM); - if (tobj) - mdd_write_lock(env, mdd_tobj, MOR_TGT_CHILD); - - rc = mdd_rt_sanity_check(env, mdd_tpobj, mdd_tobj, ma); - if (rc) - GOTO(cleanup, rc); - - /* - * If rename_tgt is called then we should just re-insert name with - * correct fid, no need to dec/inc parent nlink if obj is dir. - */ - rc = __mdd_index_delete(env, mdd_tpobj, name, 0, handle, BYPASS_CAPA); - if (rc) - GOTO(cleanup, rc); - - rc = __mdd_index_insert_only(env, mdd_tpobj, lf, name, handle, - BYPASS_CAPA); - if (rc) - GOTO(cleanup, rc); - - LASSERT(ma->ma_attr.la_valid & LA_CTIME); - la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; - - la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_tpobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); - - /* - * For tobj is remote case cmm layer has processed - * and pass NULL tobj to here. So when tobj is NOT NULL, - * it must be local one. - */ - if (tobj && mdd_object_exists(mdd_tobj)) { - __mdd_ref_del(env, mdd_tobj, handle, 0); - - /* Remove dot reference. */ - if (S_ISDIR(ma->ma_attr.la_mode)) - __mdd_ref_del(env, mdd_tobj, handle, 1); - - la->la_valid = LA_CTIME; - rc = mdd_attr_check_set_internal(env, mdd_tobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); - rc = mdd_finish_unlink(env, mdd_tobj, ma, handle); - if (rc) - GOTO(cleanup, rc); - } - EXIT; -cleanup: - if (tobj) - mdd_write_unlock(env, mdd_tobj); - mdd_pdo_write_unlock(env, mdd_tpobj, dlh); -out_trans: - mdd_trans_stop(env, mdd, rc, handle); return rc; } @@ -1032,13 +1619,17 @@ static int mdd_create_data(const struct lu_env *env, struct md_object *pobj, struct mdd_device *mdd = mdo2mdd(cobj); struct mdd_object *mdd_pobj = md2mdd_obj(pobj); struct mdd_object *son = md2mdd_obj(cobj); - struct lu_attr *attr = &ma->ma_attr; - struct lov_mds_md *lmm = NULL; - int lmm_size = 0; struct thandle *handle; + const struct lu_buf *buf; + struct lu_attr *attr = &mdd_env_info(env)->mti_cattr; int rc; ENTRY; + /* do not let users to create stripes via .lustre/ + * mdd_obf_setup() sets IMMUTE_OBJ on this directory */ + if (pobj && mdd_pobj->mod_flags & IMMUTE_OBJ) + RETURN(-ENOENT); + rc = mdd_cd_sanity_check(env, son); if (rc) RETURN(rc); @@ -1046,13 +1637,20 @@ static int mdd_create_data(const struct lu_env *env, struct md_object *pobj, if (!md_should_create(spec->sp_cr_flags)) RETURN(0); - rc = mdd_lov_create(env, mdd, mdd_pobj, son, &lmm, &lmm_size, - spec, attr); - if (rc) - RETURN(rc); + /* + * there are following use cases for this function: + * 1) late striping - file was created with MDS_OPEN_DELAY_CREATE + * striping can be specified or not + * 2) CMD? + */ + rc = mdd_la_get(env, son, attr, mdd_object_capa(env, son)); + if (rc) + RETURN(rc); + + /* calling ->ah_make_hint() is used to transfer information from parent */ + mdd_object_make_hint(env, mdd_pobj, son, attr); - mdd_txn_param_build(env, mdd, MDD_TXN_CREATE_DATA_OP); - handle = mdd_trans_start(env, mdd); + handle = mdd_trans_create(env, mdd); if (IS_ERR(handle)) GOTO(out_free, rc = PTR_ERR(handle)); @@ -1060,58 +1658,57 @@ static int mdd_create_data(const struct lu_env *env, struct md_object *pobj, * XXX: Setting the lov ea is not locked but setting the attr is locked? * Should this be fixed? */ - - /* Replay creates has objects already */ -#if 0 - if (spec->u.sp_ea.no_lov_create) { - CDEBUG(D_INFO, "we already have lov ea\n"); - rc = mdd_lov_set_md(env, mdd_pobj, son, - (struct lov_mds_md *)spec->u.sp_ea.eadata, - spec->u.sp_ea.eadatalen, handle, 0); - } else -#endif - /* No need mdd_lsm_sanity_check here */ - rc = mdd_lov_set_md(env, mdd_pobj, son, lmm, - lmm_size, handle, 0); - - if (rc == 0) - rc = mdd_attr_get_internal_locked(env, son, ma); - - /* update lov_objid data, must be before transaction stop! */ - if (rc == 0) - mdd_lov_objid_update(mdd, lmm); - - mdd_trans_stop(env, mdd, rc, handle); + CDEBUG(D_OTHER, "ea %p/%u, cr_flags %Lo, no_create %u\n", + spec->u.sp_ea.eadata, spec->u.sp_ea.eadatalen, + spec->sp_cr_flags, spec->no_create); + + if (spec->no_create || spec->sp_cr_flags & MDS_OPEN_HAS_EA) { + /* replay case or lfs setstripe */ + buf = mdd_buf_get_const(env, spec->u.sp_ea.eadata, + spec->u.sp_ea.eadatalen); + } else { + buf = &LU_BUF_NULL; + } + + rc = dt_declare_xattr_set(env, mdd_object_child(son), buf, + XATTR_NAME_LOV, 0, handle); + if (rc) + GOTO(stop, rc); + + rc = mdd_trans_start(env, mdd, handle); + if (rc) + GOTO(stop, rc); + + rc = dt_xattr_set(env, mdd_object_child(son), buf, XATTR_NAME_LOV, + 0, handle, mdd_object_capa(env, son)); +stop: + mdd_trans_stop(env, mdd, rc, handle); out_free: - /* Finish mdd_lov_create() stuff. */ - mdd_lov_create_finish(env, mdd, lmm, lmm_size, spec); - RETURN(rc); + RETURN(rc); } +/* Get fid from name and parent */ static int __mdd_lookup(const struct lu_env *env, struct md_object *pobj, const struct lu_name *lname, struct lu_fid* fid, int mask) { - char *name = lname->ln_name; + const char *name = lname->ln_name; const struct dt_key *key = (const struct dt_key *)name; struct mdd_object *mdd_obj = md2mdd_obj(pobj); struct mdd_device *m = mdo2mdd(pobj); struct dt_object *dir = mdd_object_child(mdd_obj); - struct lu_fid_pack *pack = &mdd_env_info(env)->mti_pack; int rc; ENTRY; if (unlikely(mdd_is_dead_obj(mdd_obj))) RETURN(-ESTALE); - rc = mdd_object_exists(mdd_obj); - if (unlikely(rc == 0)) - RETURN(-ESTALE); - else if (unlikely(rc < 0)) { - CERROR("Object "DFID" locates on remote server\n", - PFID(mdo2fid(mdd_obj))); - LBUG(); - } + if (mdd_object_remote(mdd_obj)) { + CDEBUG(D_INFO, "%s: Object "DFID" locates on remote server\n", + mdd2obd_dev(m)->obd_name, PFID(mdo2fid(mdd_obj))); + } else if (!mdd_object_exists(mdd_obj)) { + RETURN(-ESTALE); + } /* The common filename length check. */ if (unlikely(lname->ln_namelen > m->mdd_dt_conf.ddp_max_name_len)) @@ -1124,20 +1721,65 @@ __mdd_lookup(const struct lu_env *env, struct md_object *pobj, if (likely(S_ISDIR(mdd_object_type(mdd_obj)) && dt_try_as_dir(env, dir))) { + rc = dir->do_index_ops->dio_lookup(env, dir, - (struct dt_rec *)pack, key, + (struct dt_rec *)fid, key, mdd_object_capa(env, mdd_obj)); - if (rc == 0) - rc = fid_unpack(pack, fid); + if (rc > 0) + rc = 0; + else if (rc == 0) + rc = -ENOENT; } else rc = -ENOTDIR; RETURN(rc); } -int mdd_object_initialize(const struct lu_env *env, const struct lu_fid *pfid, - struct mdd_object *child, struct md_attr *ma, - struct thandle *handle) +static int mdd_declare_object_initialize(const struct lu_env *env, + struct mdd_object *parent, + struct mdd_object *child, + struct lu_attr *attr, + struct thandle *handle, + struct linkea_data *ldata) +{ + int rc; + ENTRY; + + /* + * inode mode has been set in creation time, and it's based on umask, + * la_mode and acl, don't set here again! (which will go wrong + * because below function doesn't consider umask). + * I'd suggest set all object attributes in creation time, see above. + */ + LASSERT(attr->la_valid & (LA_MODE | LA_TYPE)); + attr->la_valid &= ~(LA_MODE | LA_TYPE); + rc = mdo_declare_attr_set(env, child, attr, handle); + attr->la_valid |= LA_MODE | LA_TYPE; + if (rc == 0 && S_ISDIR(attr->la_mode)) { + rc = mdo_declare_index_insert(env, child, mdo2fid(child), + dot, handle); + if (rc == 0) + rc = mdo_declare_ref_add(env, child, handle); + + rc = mdo_declare_index_insert(env, child, mdo2fid(parent), + dotdot, handle); + } + + if (rc == 0 && (fid_is_norm(mdo2fid(child)) || + fid_is_dot_lustre(mdo2fid(child)) || + fid_is_root(mdo2fid(child)))) + mdd_declare_links_add(env, child, handle, ldata); + + RETURN(rc); +} + +static int mdd_object_initialize(const struct lu_env *env, + const struct lu_fid *pfid, + const struct lu_name *lname, + struct mdd_object *child, + struct lu_attr *attr, struct thandle *handle, + const struct md_op_spec *spec, + struct linkea_data *ldata) { int rc; ENTRY; @@ -1150,46 +1792,41 @@ int mdd_object_initialize(const struct lu_env *env, const struct lu_fid *pfid, * (2) maybe, the child attributes should be set in OSD when creation. */ - rc = mdd_attr_set_internal(env, child, &ma->ma_attr, handle, 0); - if (rc != 0) - RETURN(rc); - - if (S_ISDIR(ma->ma_attr.la_mode)) { + rc = mdd_attr_set_internal(env, child, attr, handle, 0); + /* arguments are supposed to stay the same */ + if (S_ISDIR(attr->la_mode)) { /* Add "." and ".." for newly created dir */ - __mdd_ref_add(env, child, handle); + mdo_ref_add(env, child, handle); rc = __mdd_index_insert_only(env, child, mdo2fid(child), dot, handle, BYPASS_CAPA); - if (rc == 0) { + if (rc == 0) rc = __mdd_index_insert_only(env, child, pfid, dotdot, handle, BYPASS_CAPA); - if (rc != 0) { - int rc2; - - rc2 = __mdd_index_delete(env, child, dot, 1, - handle, BYPASS_CAPA); - if (rc2 != 0) - CERROR("Failure to cleanup after dotdot" - " creation: %d (%d)\n", rc2, rc); - } - } + if (rc != 0) + mdo_ref_del(env, child, handle); } - RETURN(rc); + + if (rc == 0 && (fid_is_norm(mdo2fid(child)) || + fid_is_dot_lustre(mdo2fid(child)) || + fid_is_root(mdo2fid(child)))) + mdd_links_add(env, child, pfid, lname, handle, ldata, 1); + + RETURN(rc); } /* has not lock on pobj yet */ static int mdd_create_sanity_check(const struct lu_env *env, struct md_object *pobj, + struct lu_attr *pattr, const struct lu_name *lname, - struct md_attr *ma, + struct lu_attr *cattr, struct md_op_spec *spec) { struct mdd_thread_info *info = mdd_env_info(env); - struct lu_attr *la = &info->mti_la; struct lu_fid *fid = &info->mti_fid; struct mdd_object *obj = md2mdd_obj(pobj); struct mdd_device *m = mdo2mdd(pobj); - int lookup = spec->sp_cr_lookup; int rc; ENTRY; @@ -1197,12 +1834,12 @@ static int mdd_create_sanity_check(const struct lu_env *env, if (mdd_is_dead_obj(obj)) RETURN(-ENOENT); - /* + /* * In some cases this lookup is not needed - we know before if name * exists or not because MDT performs lookup for it. * name length check is done in lookup. */ - if (lookup) { + if (spec->sp_cr_lookup) { /* * Check if the name already exist, though it will be checked in * _index_insert also, for avoiding rolling back if exists @@ -1218,26 +1855,22 @@ static int mdd_create_sanity_check(const struct lu_env *env, * EXEC permission have been checked * when lookup before create already. */ - rc = mdd_permission_internal_locked(env, obj, NULL, MAY_WRITE, - MOR_TGT_PARENT); + rc = mdd_permission_internal_locked(env, obj, pattr, MAY_WRITE, + MOR_TGT_PARENT); if (rc) RETURN(rc); } /* sgid check */ - rc = mdd_la_get(env, obj, la, BYPASS_CAPA); - if (rc != 0) - RETURN(rc); - - if (la->la_mode & S_ISGID) { - ma->ma_attr.la_gid = la->la_gid; - if (S_ISDIR(ma->ma_attr.la_mode)) { - ma->ma_attr.la_mode |= S_ISGID; - ma->ma_attr.la_valid |= LA_MODE; - } - } - - switch (ma->ma_attr.la_mode & S_IFMT) { + if (pattr->la_mode & S_ISGID) { + cattr->la_gid = pattr->la_gid; + if (S_ISDIR(cattr->la_mode)) { + cattr->la_mode |= S_ISGID; + cattr->la_valid |= LA_MODE; + } + } + + switch (cattr->la_mode & S_IFMT) { case S_IFLNK: { unsigned int symlen = strlen(spec->u.sp_symname) + 1; @@ -1261,30 +1894,157 @@ static int mdd_create_sanity_check(const struct lu_env *env, RETURN(rc); } +static int mdd_declare_create(const struct lu_env *env, struct mdd_device *mdd, + struct mdd_object *p, struct mdd_object *c, + const struct lu_name *name, + struct lu_attr *attr, + int got_def_acl, + struct thandle *handle, + const struct md_op_spec *spec, + struct linkea_data *ldata) +{ + int rc; + + rc = mdd_declare_object_create_internal(env, p, c, attr, handle, spec); + if (rc) + GOTO(out, rc); + +#ifdef CONFIG_FS_POSIX_ACL + if (got_def_acl > 0) { + struct lu_buf *acl_buf; + + acl_buf = mdd_buf_get(env, NULL, got_def_acl); + /* if dir, then can inherit default ACl */ + if (S_ISDIR(attr->la_mode)) { + rc = mdo_declare_xattr_set(env, c, acl_buf, + XATTR_NAME_ACL_DEFAULT, + 0, handle); + if (rc) + GOTO(out, rc); + } + + rc = mdo_declare_attr_set(env, c, attr, handle); + if (rc) + GOTO(out, rc); + + rc = mdo_declare_xattr_set(env, c, acl_buf, + XATTR_NAME_ACL_ACCESS, 0, handle); + if (rc) + GOTO(out, rc); + } +#endif + + if (S_ISDIR(attr->la_mode)) { + rc = mdo_declare_ref_add(env, p, handle); + if (rc) + GOTO(out, rc); + } + + rc = mdd_declare_object_initialize(env, p, c, attr, handle, ldata); + if (rc) + GOTO(out, rc); + + if (spec->sp_cr_flags & MDS_OPEN_VOLATILE) + rc = orph_declare_index_insert(env, c, attr->la_mode, handle); + else + rc = mdo_declare_index_insert(env, p, mdo2fid(c), + name->ln_name, handle); + if (rc) + GOTO(out, rc); + + /* replay case, create LOV EA from client data */ + if (spec->no_create || (spec->sp_cr_flags & MDS_OPEN_HAS_EA)) { + const struct lu_buf *buf; + + buf = mdd_buf_get_const(env, spec->u.sp_ea.eadata, + spec->u.sp_ea.eadatalen); + rc = mdo_declare_xattr_set(env, c, buf, XATTR_NAME_LOV, + 0, handle); + if (rc) + GOTO(out, rc); + } + + if (S_ISLNK(attr->la_mode)) { + rc = dt_declare_record_write(env, mdd_object_child(c), + strlen(spec->u.sp_symname), 0, + handle); + if (rc) + GOTO(out, rc); + } + + if (!(spec->sp_cr_flags & MDS_OPEN_VOLATILE)) { + rc = mdo_declare_attr_set(env, p, attr, handle); + if (rc) + return rc; + } + + rc = mdd_declare_changelog_store(env, mdd, name, handle); + if (rc) + return rc; + +out: + return rc; +} + +static int mdd_acl_init(const struct lu_env *env, struct mdd_object *pobj, + struct lu_attr *la, struct lu_buf *acl_buf, + int *got_def_acl, int *reset_acl) +{ + int rc; + ENTRY; + + if (S_ISLNK(la->la_mode)) + RETURN(0); + + mdd_read_lock(env, pobj, MOR_TGT_PARENT); + rc = mdo_xattr_get(env, pobj, acl_buf, + XATTR_NAME_ACL_DEFAULT, BYPASS_CAPA); + mdd_read_unlock(env, pobj); + if (rc > 0) { + /* If there are default ACL, fix mode by default ACL */ + *got_def_acl = rc; + acl_buf->lb_len = rc; + rc = __mdd_fix_mode_acl(env, acl_buf, &la->la_mode); + if (rc < 0) + RETURN(rc); + *reset_acl = rc; + } else if (rc == -ENODATA || rc == -EOPNOTSUPP) { + /* If there are no default ACL, fix mode by mask */ + struct lu_ucred *uc = lu_ucred(env); + + /* The create triggered by MDT internal events, such as + * LFSCK reset, will not contain valid "uc". */ + if (unlikely(uc != NULL)) + la->la_mode &= ~uc->uc_umask; + rc = 0; + } + + RETURN(rc); +} + /* * Create object and insert it into namespace. */ -static int mdd_create(const struct lu_env *env, - struct md_object *pobj, - const struct lu_name *lname, - struct md_object *child, - struct md_op_spec *spec, - struct md_attr* ma) +static int mdd_create(const struct lu_env *env, struct md_object *pobj, + const struct lu_name *lname, struct md_object *child, + struct md_op_spec *spec, struct md_attr* ma) { - struct mdd_thread_info *info = mdd_env_info(env); - struct lu_attr *la = &info->mti_la_for_fix; - struct md_attr *ma_acl = &info->mti_ma; - struct mdd_object *mdd_pobj = md2mdd_obj(pobj); - struct mdd_object *son = md2mdd_obj(child); - struct mdd_device *mdd = mdo2mdd(pobj); - struct lu_attr *attr = &ma->ma_attr; - struct lov_mds_md *lmm = NULL; - struct thandle *handle; - struct dynlock_handle *dlh; - char *name = lname->ln_name; - int rc, created = 0, initialized = 0, inserted = 0, lmm_size = 0; - int got_def_acl = 0; - ENTRY; + struct mdd_thread_info *info = mdd_env_info(env); + struct lu_attr *la = &info->mti_la_for_fix; + struct mdd_object *mdd_pobj = md2mdd_obj(pobj); + struct mdd_object *son = md2mdd_obj(child); + struct mdd_device *mdd = mdo2mdd(pobj); + struct lu_attr *attr = &ma->ma_attr; + struct thandle *handle; + struct lu_attr *pattr = &info->mti_pattr; + struct lu_buf acl_buf; + struct linkea_data *ldata = &info->mti_link_data; + struct dynlock_handle *dlh; + const char *name = lname->ln_name; + int rc, created = 0, initialized = 0, inserted = 0; + int got_def_acl = 0; + int reset_acl = 0; + ENTRY; /* * Two operations have to be performed: @@ -1322,106 +2082,134 @@ static int mdd_create(const struct lu_env *env, * 2. insert (__mdd_index_insert(), lookup again) */ + rc = mdd_la_get(env, mdd_pobj, pattr, BYPASS_CAPA); + if (rc != 0) + RETURN(rc); + /* Sanity checks before big job. */ - rc = mdd_create_sanity_check(env, pobj, lname, ma, spec); + rc = mdd_create_sanity_check(env, pobj, pattr, lname, attr, spec); if (rc) RETURN(rc); - /* - * No RPC inside the transaction, so OST objects should be created at - * first. - */ - if (S_ISREG(attr->la_mode)) { - rc = mdd_lov_create(env, mdd, mdd_pobj, son, &lmm, &lmm_size, - spec, attr); - if (rc) - RETURN(rc); - } + if (OBD_FAIL_CHECK(OBD_FAIL_MDS_DQACQ_NET)) + GOTO(out_free, rc = -EINPROGRESS); - if (!S_ISLNK(attr->la_mode)) { - ma_acl->ma_acl_size = sizeof info->mti_xattr_buf; - ma_acl->ma_acl = info->mti_xattr_buf; - ma_acl->ma_need = MA_ACL_DEF; - ma_acl->ma_valid = 0; + acl_buf.lb_buf = info->mti_xattr_buf; + acl_buf.lb_len = sizeof(info->mti_xattr_buf); + rc = mdd_acl_init(env, mdd_pobj, attr, &acl_buf, &got_def_acl, + &reset_acl); + if (rc < 0) + GOTO(out_free, rc); - mdd_read_lock(env, mdd_pobj, MOR_TGT_PARENT); - rc = mdd_def_acl_get(env, mdd_pobj, ma_acl); - mdd_read_unlock(env, mdd_pobj); - if (rc) - GOTO(out_free, rc); - else if (ma_acl->ma_valid & MA_ACL_DEF) - got_def_acl = 1; - } + mdd_object_make_hint(env, mdd_pobj, son, attr); - mdd_txn_param_build(env, mdd, MDD_TXN_MKDIR_OP); - handle = mdd_trans_start(env, mdd); + handle = mdd_trans_create(env, mdd); if (IS_ERR(handle)) GOTO(out_free, rc = PTR_ERR(handle)); - dlh = mdd_pdo_write_lock(env, mdd_pobj, name, MOR_TGT_PARENT); - if (dlh == NULL) - GOTO(out_trans, rc = -ENOMEM); + memset(ldata, 0, sizeof(*ldata)); + mdd_linkea_prepare(env, son, NULL, NULL, mdd_object_fid(mdd_pobj), + lname, 1, 0, ldata); + rc = mdd_declare_create(env, mdd, mdd_pobj, son, lname, attr, + got_def_acl, handle, spec, ldata); + if (rc) + GOTO(out_stop, rc); - mdd_write_lock(env, son, MOR_TGT_CHILD); - rc = mdd_object_create_internal(env, mdd_pobj, son, ma, handle); - if (rc) { - mdd_write_unlock(env, son); - GOTO(cleanup, rc); - } + rc = mdd_trans_start(env, mdd, handle); + if (rc) + GOTO(out_stop, rc); + + dlh = mdd_pdo_write_lock(env, mdd_pobj, name, MOR_TGT_PARENT); + if (dlh == NULL) + GOTO(out_trans, rc = -ENOMEM); + + mdd_write_lock(env, son, MOR_TGT_CHILD); + rc = mdd_object_create_internal(env, NULL, son, attr, handle, spec); + if (rc) { + mdd_write_unlock(env, son); + GOTO(cleanup, rc); + } - created = 1; + created = 1; #ifdef CONFIG_FS_POSIX_ACL - if (got_def_acl) { - struct lu_buf *acl_buf = &info->mti_buf; - acl_buf->lb_buf = ma_acl->ma_acl; - acl_buf->lb_len = ma_acl->ma_acl_size; - - rc = __mdd_acl_init(env, son, acl_buf, &attr->la_mode, handle); - if (rc) { - mdd_write_unlock(env, son); - GOTO(cleanup, rc); - } else { - ma->ma_attr.la_valid |= LA_MODE; - } - } + if (got_def_acl) { + /* set default acl */ + if (S_ISDIR(attr->la_mode)) { + LASSERTF(acl_buf.lb_len == got_def_acl, + "invalid acl_buf: %p:%d got_def %d\n", + acl_buf.lb_buf, (int)acl_buf.lb_len, + got_def_acl); + rc = mdo_xattr_set(env, son, &acl_buf, + XATTR_NAME_ACL_DEFAULT, 0, + handle, BYPASS_CAPA); + if (rc) { + mdd_write_unlock(env, son); + GOTO(cleanup, rc); + } + } + + /* set its own acl */ + if (reset_acl) { + LASSERTF(acl_buf.lb_buf != NULL && acl_buf.lb_len != 0, + "invalid acl_buf %p:%d\n", acl_buf.lb_buf, + (int)acl_buf.lb_len); + rc = mdo_xattr_set(env, son, &acl_buf, + XATTR_NAME_ACL_ACCESS, + 0, handle, BYPASS_CAPA); + if (rc) { + mdd_write_unlock(env, son); + GOTO(cleanup, rc); + } + } + } #endif - rc = mdd_object_initialize(env, mdo2fid(mdd_pobj), - son, ma, handle); - mdd_write_unlock(env, son); - if (rc) - /* - * Object has no links, so it will be destroyed when last - * reference is released. (XXX not now.) - */ - GOTO(cleanup, rc); + rc = mdd_object_initialize(env, mdo2fid(mdd_pobj), lname, + son, attr, handle, spec, ldata); - initialized = 1; + /* + * in case of replay we just set LOVEA provided by the client + * XXX: I think it would be interesting to try "old" way where + * MDT calls this xattr_set(LOV) in a different transaction. + * probably this way we code can be made better. + */ + if (rc == 0 && (spec->no_create || + (spec->sp_cr_flags & MDS_OPEN_HAS_EA))) { + const struct lu_buf *buf; - rc = __mdd_index_insert(env, mdd_pobj, mdo2fid(son), - name, S_ISDIR(attr->la_mode), handle, - mdd_object_capa(env, mdd_pobj)); + buf = mdd_buf_get_const(env, spec->u.sp_ea.eadata, + spec->u.sp_ea.eadatalen); + rc = mdo_xattr_set(env, son, buf, XATTR_NAME_LOV, 0, handle, + BYPASS_CAPA); + } - if (rc) - GOTO(cleanup, rc); + if (rc == 0 && spec->sp_cr_flags & MDS_OPEN_VOLATILE) + rc = __mdd_orphan_add(env, son, handle); - inserted = 1; + mdd_write_unlock(env, son); - /* No need mdd_lsm_sanity_check here */ - rc = mdd_lov_set_md(env, mdd_pobj, son, lmm, lmm_size, handle, 0); - if (rc) { - CERROR("error on stripe info copy %d \n", rc); - GOTO(cleanup, rc); - } - if (lmm && lmm_size > 0) { - /* Set Lov here, do not get lmm again later */ - memcpy(ma->ma_lmm, lmm, lmm_size); - ma->ma_lmm_size = lmm_size; - ma->ma_valid |= MA_LOV; - } + if (rc != 0) + /* + * Object has no links, so it will be destroyed when last + * reference is released. (XXX not now.) + */ + GOTO(cleanup, rc); + + initialized = 1; + + if (!(spec->sp_cr_flags & MDS_OPEN_VOLATILE)) + rc = __mdd_index_insert(env, mdd_pobj, mdo2fid(son), + name, S_ISDIR(attr->la_mode), handle, + mdd_object_capa(env, mdd_pobj)); + + if (rc != 0) + GOTO(cleanup, rc); + + inserted = 1; if (S_ISLNK(attr->la_mode)) { + struct lu_ucred *uc = lu_ucred_assert(env); struct dt_object *dt = mdd_object_child(son); const char *target_name = spec->u.sp_symname; int sym_len = strlen(target_name); @@ -1429,8 +2217,10 @@ static int mdd_create(const struct lu_env *env, loff_t pos = 0; buf = mdd_buf_get_const(env, target_name, sym_len); - rc = dt->do_body_ops->dbo_write(env, dt, buf, &pos, handle, - mdd_object_capa(env, son)); + rc = dt->do_body_ops->dbo_write(env, dt, buf, &pos, handle, + mdd_object_capa(env, son), + uc->uc_cap & + CFS_CAP_SYS_RESOURCE_MASK); if (rc == sym_len) rc = 0; @@ -1438,47 +2228,73 @@ static int mdd_create(const struct lu_env *env, GOTO(cleanup, rc = -EFAULT); } - *la = ma->ma_attr; - la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_pobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); + /* volatile file creation does not update parent directory times */ + if (spec->sp_cr_flags & MDS_OPEN_VOLATILE) + GOTO(cleanup, rc = 0); + + /* update parent directory mtime/ctime */ + *la = *attr; + la->la_valid = LA_CTIME | LA_MTIME; + rc = mdd_attr_check_set_internal(env, mdd_pobj, la, handle, 0); + if (rc) + GOTO(cleanup, rc); - /* Return attr back. */ - rc = mdd_attr_get_internal_locked(env, son, ma); EXIT; cleanup: - if (rc && created) { - int rc2 = 0; - - if (inserted) { - rc2 = __mdd_index_delete(env, mdd_pobj, name, - S_ISDIR(attr->la_mode), - handle, BYPASS_CAPA); - if (rc2) - CERROR("error can not cleanup destroy %d\n", - rc2); - } - - if (rc2 == 0) { - mdd_write_lock(env, son, MOR_TGT_CHILD); - __mdd_ref_del(env, son, handle, 0); - if (initialized && S_ISDIR(attr->la_mode)) - __mdd_ref_del(env, son, handle, 1); - mdd_write_unlock(env, son); - } + if (rc != 0 && created != 0) { + int rc2; + + if (inserted != 0) { + if (spec->sp_cr_flags & MDS_OPEN_VOLATILE) + rc2 = __mdd_orphan_del(env, son, handle); + else + rc2 = __mdd_index_delete(env, mdd_pobj, name, + S_ISDIR(attr->la_mode), + handle, BYPASS_CAPA); + if (rc2 != 0) + goto out_stop; + } + + mdd_write_lock(env, son, MOR_TGT_CHILD); + if (initialized != 0 && S_ISDIR(attr->la_mode)) { + /* Drop the reference, no need to delete "."/"..", + * because the object to be destroied directly. */ + rc2 = mdo_ref_del(env, son, handle); + if (rc2 != 0) { + mdd_write_unlock(env, son); + goto out_stop; + } + } + + rc2 = mdo_ref_del(env, son, handle); + if (rc2 != 0) { + mdd_write_unlock(env, son); + goto out_stop; + } + + mdo_destroy(env, son, handle); + mdd_write_unlock(env, son); } - /* update lov_objid data, must be before transaction stop! */ - if (rc == 0) - mdd_lov_objid_update(mdd, lmm); - mdd_pdo_write_unlock(env, mdd_pobj, dlh); out_trans: + if (rc == 0 && fid_is_namespace_visible(mdo2fid(son))) + rc = mdd_changelog_ns_store(env, mdd, + S_ISDIR(attr->la_mode) ? CL_MKDIR : + S_ISREG(attr->la_mode) ? CL_CREATE : + S_ISLNK(attr->la_mode) ? CL_SOFTLINK : CL_MKNOD, + 0, son, mdd_pobj, lname, handle); +out_stop: mdd_trans_stop(env, mdd, rc, handle); out_free: - /* finis lov_create stuff, free all temporary data */ - mdd_lov_create_finish(env, mdd, lmm, lmm_size, spec); + if (ldata->ld_buf && ldata->ld_buf->lb_len > OBD_ALLOC_BIG) + /* if we vmalloced a large buffer drop it */ + lu_buf_free(ldata->ld_buf); + + /* The child object shouldn't be cached anymore */ + if (rc) + set_bit(LU_OBJECT_HEARD_BANSHEE, + &child->mo_lu.lo_header->loh_flags); return rc; } @@ -1529,39 +2345,163 @@ static int mdd_rename_sanity_check(const struct lu_env *env, struct mdd_object *tgt_pobj, struct mdd_object *sobj, struct mdd_object *tobj, - struct md_attr *ma) + struct lu_attr *so_attr, + struct lu_attr *tg_attr) { - int rc = 0; - ENTRY; + int rc = 0; + ENTRY; + + /* XXX: when get here, sobj must NOT be NULL, + * the other case has been processed in cld_rename + * before mdd_rename and enable MDS_PERM_BYPASS. */ + LASSERT(sobj); + + rc = mdd_may_delete(env, src_pobj, sobj, so_attr, NULL, 1, 0); + if (rc) + RETURN(rc); + + /* XXX: when get here, "tobj == NULL" means tobj must + * NOT exist (neither on remote MDS, such case has been + * processed in cld_rename before mdd_rename and enable + * MDS_PERM_BYPASS). + * So check may_create, but not check may_unlink. */ + if (!tobj) + rc = mdd_may_create(env, tgt_pobj, NULL, + (src_pobj != tgt_pobj), 0); + else + rc = mdd_may_delete(env, tgt_pobj, tobj, tg_attr, so_attr, + (src_pobj != tgt_pobj), 1); + + if (!rc && !tobj && (src_pobj != tgt_pobj) && + S_ISDIR(so_attr->la_mode)) + rc = __mdd_may_link(env, tgt_pobj); + + RETURN(rc); +} - if (unlikely(ma->ma_attr_flags & MDS_PERM_BYPASS)) - RETURN(0); +static int mdd_declare_rename(const struct lu_env *env, + struct mdd_device *mdd, + struct mdd_object *mdd_spobj, + struct mdd_object *mdd_tpobj, + struct mdd_object *mdd_sobj, + struct mdd_object *mdd_tobj, + const struct lu_name *tname, + const struct lu_name *sname, + struct md_attr *ma, + struct linkea_data *ldata, + struct thandle *handle) +{ + struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; + int rc; - /* XXX: when get here, sobj must NOT be NULL, - * the other case has been processed in cml_rename - * before mdd_rename and enable MDS_PERM_BYPASS. */ - LASSERT(sobj); - rc = mdd_may_delete(env, src_pobj, sobj, ma, 1, 0); + LASSERT(ma->ma_attr.la_valid & LA_CTIME); + la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; + + LASSERT(mdd_spobj); + LASSERT(mdd_tpobj); + LASSERT(mdd_sobj); + + /* name from source dir */ + rc = mdo_declare_index_delete(env, mdd_spobj, sname->ln_name, handle); if (rc) - RETURN(rc); + return rc; - /* XXX: when get here, "tobj == NULL" means tobj must - * NOT exist (neither on remote MDS, such case has been - * processed in cml_rename before mdd_rename and enable - * MDS_PERM_BYPASS). - * So check may_create, but not check may_unlink. */ - if (!tobj) - rc = mdd_may_create(env, tgt_pobj, NULL, - (src_pobj != tgt_pobj), 0); - else - rc = mdd_may_delete(env, tgt_pobj, tobj, ma, - (src_pobj != tgt_pobj), 1); + /* .. from source child */ + if (S_ISDIR(mdd_object_type(mdd_sobj))) { + /* source child can be directory, + * counted by source dir's nlink */ + rc = mdo_declare_ref_del(env, mdd_spobj, handle); + if (rc) + return rc; + if (mdd_spobj != mdd_tpobj) { + rc = mdo_declare_index_delete(env, mdd_sobj, dotdot, + handle); + if (rc) + return rc; + + rc = mdo_declare_index_insert(env, mdd_sobj, + mdo2fid(mdd_tpobj), + dotdot, handle); + if (rc) + return rc; + } + /* new target child can be directory, + * counted by target dir's nlink */ + rc = mdo_declare_ref_add(env, mdd_tpobj, handle); + if (rc) + return rc; - if (!rc && !tobj && (src_pobj != tgt_pobj) && - S_ISDIR(ma->ma_attr.la_mode)) - rc = __mdd_may_link(env, tgt_pobj); + } - RETURN(rc); + la->la_valid = LA_CTIME | LA_MTIME; + rc = mdo_declare_attr_set(env, mdd_spobj, la, handle); + if (rc != 0) + return rc; + + rc = mdo_declare_attr_set(env, mdd_tpobj, la, handle); + if (rc != 0) + return rc; + + la->la_valid = LA_CTIME; + rc = mdo_declare_attr_set(env, mdd_sobj, la, handle); + if (rc) + return rc; + + rc = mdd_declare_links_add(env, mdd_sobj, handle, ldata); + if (rc) + return rc; + + /* new name */ + rc = mdo_declare_index_insert(env, mdd_tpobj, mdo2fid(mdd_sobj), + tname->ln_name, handle); + if (rc) + return rc; + + /* name from target dir (old name), we declare it unconditionally + * as mdd_rename() calls delete unconditionally as well. so just + * to balance declarations vs calls to change ... */ + rc = mdo_declare_index_delete(env, mdd_tpobj, tname->ln_name, handle); + if (rc) + return rc; + + if (mdd_tobj && mdd_object_exists(mdd_tobj)) { + /* delete target child in target parent directory */ + rc = mdo_declare_ref_del(env, mdd_tobj, handle); + if (rc) + return rc; + + if (S_ISDIR(mdd_object_type(mdd_tobj))) { + /* target child can be directory, + * delete "." reference in target child directory */ + rc = mdo_declare_ref_del(env, mdd_tobj, handle); + if (rc) + return rc; + + /* delete ".." reference in target parent directory */ + rc = mdo_declare_ref_del(env, mdd_tpobj, handle); + if (rc) + return rc; + } + + la->la_valid = LA_CTIME; + rc = mdo_declare_attr_set(env, mdd_tobj, la, handle); + if (rc) + return rc; + + mdd_declare_links_del(env, mdd_tobj, handle); + if (rc) + return rc; + + rc = mdd_declare_finish_unlink(env, mdd_tobj, ma, handle); + if (rc) + return rc; + } + + rc = mdd_declare_changelog_ext_store(env, mdd, tname, sname, handle); + if (rc) + return rc; + + return rc; } /* src object can be remote that is why we use only fid and type of object */ @@ -1571,30 +2511,65 @@ static int mdd_rename(const struct lu_env *env, struct md_object *tobj, const struct lu_name *ltname, struct md_attr *ma) { - char *sname = lsname->ln_name; - char *tname = ltname->ln_name; - struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; - struct mdd_object *mdd_spobj = md2mdd_obj(src_pobj); - struct mdd_object *mdd_tpobj = md2mdd_obj(tgt_pobj); - struct mdd_device *mdd = mdo2mdd(src_pobj); - struct mdd_object *mdd_sobj = NULL; - struct mdd_object *mdd_tobj = NULL; - struct dynlock_handle *sdlh, *tdlh; - struct thandle *handle; - int is_dir; - int rc; - ENTRY; - - LASSERT(ma->ma_attr.la_mode & S_IFMT); - is_dir = S_ISDIR(ma->ma_attr.la_mode); + const char *sname = lsname->ln_name; + const char *tname = ltname->ln_name; + struct lu_attr *la = &mdd_env_info(env)->mti_la_for_fix; + struct lu_attr *so_attr = &mdd_env_info(env)->mti_cattr; + struct lu_attr *tg_attr = &mdd_env_info(env)->mti_pattr; + struct mdd_object *mdd_spobj = md2mdd_obj(src_pobj); /* source parent */ + struct mdd_object *mdd_tpobj = md2mdd_obj(tgt_pobj); + struct mdd_device *mdd = mdo2mdd(src_pobj); + struct mdd_object *mdd_sobj = NULL; /* source object */ + struct mdd_object *mdd_tobj = NULL; + struct dynlock_handle *sdlh = NULL, *tdlh = NULL; + struct thandle *handle; + struct linkea_data *ldata = &mdd_env_info(env)->mti_link_data; + const struct lu_fid *tpobj_fid = mdo2fid(mdd_tpobj); + const struct lu_fid *spobj_fid = mdo2fid(mdd_spobj); + bool is_dir; + bool tobj_ref = 0; + bool tobj_locked = 0; + unsigned cl_flags = 0; + int rc, rc2; + ENTRY; if (tobj) mdd_tobj = md2mdd_obj(tobj); - mdd_txn_param_build(env, mdd, MDD_TXN_RENAME_OP); - handle = mdd_trans_start(env, mdd); + mdd_sobj = mdd_object_find(env, mdd, lf); + + rc = mdd_la_get(env, mdd_sobj, so_attr, + mdd_object_capa(env, mdd_sobj)); + if (rc) + GOTO(out_pending, rc); + + if (mdd_tobj) { + rc = mdd_la_get(env, mdd_tobj, tg_attr, + mdd_object_capa(env, mdd_tobj)); + if (rc) + GOTO(out_pending, rc); + } + + rc = mdd_rename_sanity_check(env, mdd_spobj, mdd_tpobj, mdd_sobj, + mdd_tobj, so_attr, tg_attr); + if (rc) + GOTO(out_pending, rc); + + handle = mdd_trans_create(env, mdd); if (IS_ERR(handle)) - RETURN(PTR_ERR(handle)); + GOTO(out_pending, rc = PTR_ERR(handle)); + + memset(ldata, 0, sizeof(*ldata)); + mdd_linkea_prepare(env, mdd_sobj, NULL, NULL, mdd_object_fid(mdd_tpobj), + ltname, 1, 0, ldata); + rc = mdd_declare_rename(env, mdd, mdd_spobj, mdd_tpobj, mdd_sobj, + mdd_tobj, lsname, ltname, ma, ldata, handle); + if (rc) + GOTO(stop, rc); + + rc = mdd_trans_start(env, mdd, handle); + if (rc) + GOTO(stop, rc); /* FIXME: Should consider tobj and sobj too in rename_lock. */ rc = mdd_rename_order(env, mdd, mdd_spobj, mdd_tpobj); @@ -1621,30 +2596,47 @@ static int mdd_rename(const struct lu_env *env, if (sdlh == NULL || tdlh == NULL) GOTO(cleanup, rc = -ENOMEM); - mdd_sobj = mdd_object_find(env, mdd, lf); - rc = mdd_rename_sanity_check(env, mdd_spobj, mdd_tpobj, - mdd_sobj, mdd_tobj, ma); - if (rc) - GOTO(cleanup, rc); + is_dir = S_ISDIR(so_attr->la_mode); + /* Remove source name from source directory */ rc = __mdd_index_delete(env, mdd_spobj, sname, is_dir, handle, mdd_object_capa(env, mdd_spobj)); if (rc) GOTO(cleanup, rc); - /* + /* "mv dir1 dir2" needs "dir1/.." link update */ + if (is_dir && mdd_sobj && !lu_fid_eq(spobj_fid, tpobj_fid)) { + rc = __mdd_index_delete_only(env, mdd_sobj, dotdot, handle, + mdd_object_capa(env, mdd_sobj)); + if (rc) + GOTO(fixup_spobj2, rc); + + rc = __mdd_index_insert_only(env, mdd_sobj, tpobj_fid, dotdot, + handle, mdd_object_capa(env, mdd_sobj)); + if (rc) + GOTO(fixup_spobj, rc); + } + + /* Remove target name from target directory * Here tobj can be remote one, so we do index_delete unconditionally * and -ENOENT is allowed. */ rc = __mdd_index_delete(env, mdd_tpobj, tname, is_dir, handle, mdd_object_capa(env, mdd_tpobj)); - if (rc != 0 && rc != -ENOENT) - GOTO(cleanup, rc); + if (rc != 0) { + if (mdd_tobj) { + /* tname might been renamed to something else */ + GOTO(fixup_spobj, rc); + } + if (rc != -ENOENT) + GOTO(fixup_spobj, rc); + } + /* Insert new fid with target name into target dir */ rc = __mdd_index_insert(env, mdd_tpobj, lf, tname, is_dir, handle, mdd_object_capa(env, mdd_tpobj)); if (rc) - GOTO(cleanup, rc); + GOTO(fixup_tpobj, rc); LASSERT(ma->ma_attr.la_valid & LA_CTIME); la->la_ctime = la->la_mtime = ma->ma_attr.la_ctime; @@ -1652,69 +2644,187 @@ static int mdd_rename(const struct lu_env *env, /* XXX: mdd_sobj must be local one if it is NOT NULL. */ if (mdd_sobj) { la->la_valid = LA_CTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_sobj, la, - handle, 0); + rc = mdd_attr_check_set_internal(env, mdd_sobj, la, handle, 0); if (rc) - GOTO(cleanup, rc); + GOTO(fixup_tpobj, rc); } - /* + /* Remove old target object * For tobj is remote case cmm layer has processed * and set tobj to NULL then. So when tobj is NOT NULL, * it must be local one. */ if (tobj && mdd_object_exists(mdd_tobj)) { mdd_write_lock(env, mdd_tobj, MOR_TGT_CHILD); - __mdd_ref_del(env, mdd_tobj, handle, 0); + tobj_locked = 1; + if (mdd_is_dead_obj(mdd_tobj)) { + /* shld not be dead, something is wrong */ + CERROR("tobj is dead, something is wrong\n"); + rc = -EINVAL; + goto cleanup; + } + mdo_ref_del(env, mdd_tobj, handle); /* Remove dot reference. */ - if (is_dir) - __mdd_ref_del(env, mdd_tobj, handle, 1); - - la->la_valid = LA_CTIME; - rc = mdd_attr_check_set_internal(env, mdd_tobj, la, handle, 0); - if (rc) - GOTO(cleanup, rc); - - rc = mdd_finish_unlink(env, mdd_tobj, ma, handle); - mdd_write_unlock(env, mdd_tobj); - if (rc) - GOTO(cleanup, rc); + if (S_ISDIR(tg_attr->la_mode)) + mdo_ref_del(env, mdd_tobj, handle); + tobj_ref = 1; + + /* fetch updated nlink */ + rc = mdd_la_get(env, mdd_tobj, tg_attr, + mdd_object_capa(env, mdd_tobj)); + if (rc != 0) { + CERROR("%s: Failed to get nlink for tobj " + DFID": rc = %d\n", + mdd2obd_dev(mdd)->obd_name, + PFID(tpobj_fid), rc); + GOTO(fixup_tpobj, rc); + } + + la->la_valid = LA_CTIME; + rc = mdd_attr_check_set_internal(env, mdd_tobj, la, handle, 0); + if (rc != 0) { + CERROR("%s: Failed to set ctime for tobj " + DFID": rc = %d\n", + mdd2obd_dev(mdd)->obd_name, + PFID(tpobj_fid), rc); + GOTO(fixup_tpobj, rc); + } + + /* XXX: this transfer to ma will be removed with LOD/OSP */ + ma->ma_attr = *tg_attr; + ma->ma_valid |= MA_INODE; + rc = mdd_finish_unlink(env, mdd_tobj, ma, handle); + if (rc != 0) { + CERROR("%s: Failed to unlink tobj " + DFID": rc = %d\n", + mdd2obd_dev(mdd)->obd_name, + PFID(tpobj_fid), rc); + GOTO(fixup_tpobj, rc); + } + + /* fetch updated nlink */ + rc = mdd_la_get(env, mdd_tobj, tg_attr, + mdd_object_capa(env, mdd_tobj)); + if (rc != 0) { + CERROR("%s: Failed to get nlink for tobj " + DFID": rc = %d\n", + mdd2obd_dev(mdd)->obd_name, + PFID(tpobj_fid), rc); + GOTO(fixup_tpobj, rc); + } + /* XXX: this transfer to ma will be removed with LOD/OSP */ + ma->ma_attr = *tg_attr; + ma->ma_valid |= MA_INODE; + + if (so_attr->la_nlink == 0) + cl_flags |= CLF_RENAME_LAST; } la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_spobj, la, handle, 0); + rc = mdd_attr_check_set_internal(env, mdd_spobj, la, handle, 0); if (rc) - GOTO(cleanup, rc); + GOTO(fixup_tpobj, rc); if (mdd_spobj != mdd_tpobj) { la->la_valid = LA_CTIME | LA_MTIME; - rc = mdd_attr_check_set_internal_locked(env, mdd_tpobj, la, - handle, 0); + rc = mdd_attr_check_set_internal(env, mdd_tpobj, la, + handle, 0); + } + + if (rc == 0 && mdd_sobj) { + mdd_write_lock(env, mdd_sobj, MOR_SRC_CHILD); + rc = mdd_links_rename(env, mdd_sobj, mdo2fid(mdd_spobj), lsname, + mdo2fid(mdd_tpobj), ltname, handle, NULL, + 0, 0); + if (rc == -ENOENT) + /* Old files might not have EA entry */ + mdd_links_add(env, mdd_sobj, mdo2fid(mdd_spobj), + lsname, handle, NULL, 0); + mdd_write_unlock(env, mdd_sobj); + /* We don't fail the transaction if the link ea can't be + updated -- fid2path will use alternate lookup method. */ + rc = 0; } EXIT; + +fixup_tpobj: + if (rc) { + rc2 = __mdd_index_delete(env, mdd_tpobj, tname, is_dir, handle, + BYPASS_CAPA); + if (rc2) + CWARN("tp obj fix error %d\n",rc2); + + if (mdd_tobj && mdd_object_exists(mdd_tobj) && + !mdd_is_dead_obj(mdd_tobj)) { + if (tobj_ref) { + mdo_ref_add(env, mdd_tobj, handle); + if (is_dir) + mdo_ref_add(env, mdd_tobj, handle); + } + + rc2 = __mdd_index_insert(env, mdd_tpobj, + mdo2fid(mdd_tobj), tname, + is_dir, handle, + BYPASS_CAPA); + + if (rc2) + CWARN("tp obj fix error %d\n",rc2); + } + } + +fixup_spobj: + if (rc && is_dir && mdd_sobj && mdd_spobj != mdd_tpobj) { + rc2 = __mdd_index_delete_only(env, mdd_sobj, dotdot, handle, + BYPASS_CAPA); + + if (rc2) + CWARN("%s: sp obj dotdot delete error: rc = %d\n", + mdd2obd_dev(mdd)->obd_name, rc2); + + + rc2 = __mdd_index_insert_only(env, mdd_sobj, spobj_fid, + dotdot, handle, BYPASS_CAPA); + if (rc2) + CWARN("%s: sp obj dotdot insert error: rc = %d\n", + mdd2obd_dev(mdd)->obd_name, rc2); + } + +fixup_spobj2: + if (rc) { + rc2 = __mdd_index_insert(env, mdd_spobj, + lf, sname, is_dir, handle, BYPASS_CAPA); + if (rc2) + CWARN("sp obj fix error %d\n",rc2); + } cleanup: + if (tobj_locked) + mdd_write_unlock(env, mdd_tobj); if (likely(tdlh) && sdlh != tdlh) mdd_pdo_write_unlock(env, mdd_tpobj, tdlh); if (likely(sdlh)) mdd_pdo_write_unlock(env, mdd_spobj, sdlh); cleanup_unlocked: + if (rc == 0) + rc = mdd_changelog_ext_ns_store(env, mdd, CL_RENAME, cl_flags, + mdd_tobj, tpobj_fid, lf, + spobj_fid, ltname, lsname, + handle); + +stop: mdd_trans_stop(env, mdd, rc, handle); - if (mdd_sobj) - mdd_object_put(env, mdd_sobj); - return rc; +out_pending: + mdd_object_put(env, mdd_sobj); + return rc; } -struct md_dir_operations mdd_dir_ops = { +const struct md_dir_operations mdd_dir_ops = { .mdo_is_subdir = mdd_is_subdir, .mdo_lookup = mdd_lookup, .mdo_create = mdd_create, .mdo_rename = mdd_rename, .mdo_link = mdd_link, .mdo_unlink = mdd_unlink, - .mdo_name_insert = mdd_name_insert, - .mdo_name_remove = mdd_name_remove, - .mdo_rename_tgt = mdd_rename_tgt, - .mdo_create_data = mdd_create_data + .mdo_create_data = mdd_create_data, };