diff options
author | NeilBrown <neilb@suse.de> | 2021-11-29 07:51:25 +0300 |
---|---|---|
committer | Chuck Lever <chuck.lever@oracle.com> | 2021-12-13 21:42:55 +0300 |
commit | b73a2972041bee70eb0cbbb25fa77828c63c916b (patch) | |
tree | 272471bc5d525d5f0498ef2ab85457ea0a67fa16 /Documentation | |
parent | 5a8a7ff57421b7de3ae72019938ffb5daaee36e7 (diff) | |
download | linux-b73a2972041bee70eb0cbbb25fa77828c63c916b.tar.xz |
lockd: move lockd_start_svc() call into lockd_create_svc()
lockd_start_svc() only needs to be called once, just after the svc is
created. If the start fails, the svc is discarded too.
It thus makes sense to call lockd_start_svc() from lockd_create_svc().
This allows us to remove the test against nlmsvc_rqst at the start of
lockd_start_svc() - it must always be NULL.
lockd_up() only held an extra reference on the svc until a thread was
created - then it dropped it. The thread - and thus the extra reference
- will remain until kthread_stop() is called.
Now that the thread is created in lockd_create_svc(), the extra
reference can be dropped there. So the 'serv' variable is no longer
needed in lockd_up().
Signed-off-by: NeilBrown <neilb@suse.de>
Signed-off-by: Chuck Lever <chuck.lever@oracle.com>
Diffstat (limited to 'Documentation')
0 files changed, 0 insertions, 0 deletions