openkylin
/
linux
mirror of https://gitee.com/openkylin/linux.git
9
0
Fork 0
Code Issues Projects Releases Wiki Activity

staging: lustre: fid: remove seq_fid_alloc_fini() and simplify 

seq_fid_alloc_fini() is tiny and only called
from two places in the one function.  We can move
both those calls earlier and merge them so only
one call is needed.  At that point, there is no
value added by having a separate function.

Also instead of using ++ and -- on ->lcs_update to
toggle between 0 and 1, explicitly set to 0 or 1
as appropriate.

Moving the locking earlier means that the code which updates
seq->lcs_fid is now protected, so
ldebugfs_fid_fid_seq_show() now cannot see a torn value.

Signed-off-by: NeilBrown <neilb@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
NeilBrown 2018-02-20 13:23:38 +11:00 committed by Greg Kroah-Hartman
parent 40f5bd3501
commit 8ede253133
1 changed files with 6 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
drivers/staging/lustre/lustre/fid/fid_request.c
View File

@ -192,14 +192,6 @@ static int seq_client_alloc_seq(const struct lu_env *env,
return rc;
}
static void seq_fid_alloc_fini(struct lu_client_seq *seq)
{
LASSERT(seq->lcs_update == 1);
spin_lock(&seq->lcs_lock);
--seq->lcs_update;
wake_up(&seq->lcs_waitq);
}
/* Allocate new fid on passed client @seq and save it to @fid. */
int seq_client_alloc_fid(const struct lu_env *env,
struct lu_client_seq *seq, struct lu_fid *fid)
@ -230,14 +222,18 @@ int seq_client_alloc_fid(const struct lu_env *env,
u64 seqnr;
LASSERT(seq->lcs_update == 0);
++seq->lcs_update;
seq->lcs_update = 1;
spin_unlock(&seq->lcs_lock);
rc = seq_client_alloc_seq(env, seq, &seqnr);
spin_lock(&seq->lcs_lock);
seq->lcs_update = 0;
wake_up(&seq->lcs_waitq);
if (rc) {
CERROR("%s: Can't allocate new sequence, rc %d\n",
seq->lcs_name, rc);
seq_fid_alloc_fini(seq);
spin_unlock(&seq->lcs_lock);
return rc;
}
@ -254,8 +250,6 @@ int seq_client_alloc_fid(const struct lu_env *env,
* to setup FLD for it.
*/
rc = 1;
seq_fid_alloc_fini(seq);
}
*fid = seq->lcs_fid;