X-Git-Url: https://git.whamcloud.com/?p=fs%2Flustre-release.git;a=blobdiff_plain;f=lustre%2Fmdt%2Fmdt_identity.c;h=7d55a0ee3abc31282ac5e4b7416ed7a456142877;hp=9e44e90929634bd1dfd3ff15c8e752a728dfcb7f;hb=e8273a3dd71c4e6ab5ca9de3fbfbc0f7603d6930;hpb=580c1e0017296ea7a25f6f5f8aa8022f713ae762 diff --git a/lustre/mdt/mdt_identity.c b/lustre/mdt/mdt_identity.c index 9e44e90..7d55a0e 100644 --- a/lustre/mdt/mdt_identity.c +++ b/lustre/mdt/mdt_identity.c @@ -57,7 +57,6 @@ #include #include -#include #include #include #include @@ -114,7 +113,7 @@ static int mdt_identity_do_upcall(struct upcall_cache *cache, /* There is race condition: * "uc_upcall" was changed just after "is_identity_get_disabled" check. */ - read_lock(&cache->uc_upcall_rwlock); + down_read(&cache->uc_upcall_rwsem); CDEBUG(D_INFO, "The upcall is: '%s'\n", cache->uc_upcall); if (unlikely(!strcmp(cache->uc_upcall, "NONE"))) { @@ -126,7 +125,7 @@ static int mdt_identity_do_upcall(struct upcall_cache *cache, snprintf(keystr, sizeof(keystr), LPU64, entry->ue_key); do_gettimeofday(&start); - rc = call_usermodehelper(argv[0], argv, envp, 1); + rc = call_usermodehelper(argv[0], argv, envp, UMH_WAIT_EXEC); do_gettimeofday(&end); if (rc < 0) { CERROR("%s: error invoking upcall %s %s %s: rc %d; " @@ -142,8 +141,8 @@ static int mdt_identity_do_upcall(struct upcall_cache *cache, } EXIT; out: - read_unlock(&cache->uc_upcall_rwlock); - return rc; + up_read(&cache->uc_upcall_rwsem); + return rc; } static int mdt_identity_parse_downcall(struct upcall_cache *cache, @@ -246,16 +245,14 @@ void mdt_flush_identity(struct upcall_cache *cache, int uid) * If there is LNET_NID_ANY in perm[i].mp_nid, * it must be perm[0].mp_nid, and act as default perm. */ -__u32 mdt_identity_get_perm(struct md_identity *identity, - __u32 is_rmtclient, lnet_nid_t nid) +__u32 mdt_identity_get_perm(struct md_identity *identity, lnet_nid_t nid) { - struct md_perm *perm; - int i; - if (!identity) { - LASSERT(is_rmtclient == 0); - return CFS_SETGRP_PERM; - } + struct md_perm *perm; + int i; + + if (!identity) + return CFS_SETGRP_PERM; perm = identity->mi_perms; /* check exactly matched nid first */ @@ -271,39 +268,5 @@ __u32 mdt_identity_get_perm(struct md_identity *identity, return perm[0].mp_perm; /* return default last */ - return is_rmtclient ? 0 : CFS_SETGRP_PERM; -} - -int mdt_pack_remote_perm(struct mdt_thread_info *info, struct mdt_object *o, - void *buf) -{ - struct lu_ucred *uc = mdt_ucred_check(info); - struct md_object *next = mdt_object_child(o); - struct mdt_remote_perm *perm = buf; - - ENTRY; - - /* remote client request always pack ptlrpc_user_desc! */ - LASSERT(perm); - - if (!exp_connect_rmtclient(info->mti_exp)) - RETURN(-EBADE); - - if (uc == NULL) - RETURN(-EINVAL); - - perm->rp_uid = uc->uc_o_uid; - perm->rp_gid = uc->uc_o_gid; - perm->rp_fsuid = uc->uc_o_fsuid; - perm->rp_fsgid = uc->uc_o_fsgid; - - perm->rp_access_perm = 0; - if (mo_permission(info->mti_env, NULL, next, NULL, MAY_READ) == 0) - perm->rp_access_perm |= MAY_READ; - if (mo_permission(info->mti_env, NULL, next, NULL, MAY_WRITE) == 0) - perm->rp_access_perm |= MAY_WRITE; - if (mo_permission(info->mti_env, NULL, next, NULL, MAY_EXEC) == 0) - perm->rp_access_perm |= MAY_EXEC; - - RETURN(0); + return CFS_SETGRP_PERM; }