Skip to content

Commit

Permalink
lockd: Make two symbols static
Browse files Browse the repository at this point in the history
Fix sparse warnings:

fs/lockd/clntproc.c:57:6: warning: symbol 'nlmclnt_put_lockowner' was not declared. Should it be static?
fs/lockd/svclock.c:409:35: warning: symbol 'nlmsvc_lock_ops' was not declared. Should it be static?

Reported-by: Hulk Robot <[email protected]>
Signed-off-by: YueHaibing <[email protected]>
Signed-off-by: J. Bruce Fields <[email protected]>
  • Loading branch information
YueHaibing authored and J. Bruce Fields committed Jul 3, 2019
1 parent f85d933 commit 291adeb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion fs/lockd/clntproc.c
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ nlmclnt_get_lockowner(struct nlm_lockowner *lockowner)
return lockowner;
}

void nlmclnt_put_lockowner(struct nlm_lockowner *lockowner)
static void nlmclnt_put_lockowner(struct nlm_lockowner *lockowner)
{
if (!refcount_dec_and_lock(&lockowner->count, &lockowner->host->h_lock))
return;
Expand Down
2 changes: 1 addition & 1 deletion fs/lockd/svclock.c
Original file line number Diff line number Diff line change
Expand Up @@ -406,7 +406,7 @@ static void nlmsvc_locks_release_private(struct file_lock *fl)
nlmsvc_put_lockowner((struct nlm_lockowner *)fl->fl_owner);
}

const struct file_lock_operations nlmsvc_lock_ops = {
static const struct file_lock_operations nlmsvc_lock_ops = {
.fl_copy_lock = nlmsvc_locks_copy_lock,
.fl_release_private = nlmsvc_locks_release_private,
};
Expand Down

0 comments on commit 291adeb

Please sign in to comment.