| Message ID | 20240823070049.3499625-3-lilingfeng3@huawei.com (mailing list archive) |
|---|---|
| State | New |
| Headers | show |
| Series | nfsd: fix some comments and code cleanup | expand |
On Fri, 2024-08-23 at 15:00 +0800, Li Lingfeng wrote: > Fix spelling errors in comments of nfsd4_release_lockowner and > nfs4_set_delegation. > > Signed-off-by: Li Lingfeng <lilingfeng3@huawei.com> > --- > fs/nfsd/nfs4state.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c > index a20c2c9d7d45..66a0c76850f3 100644 > --- a/fs/nfsd/nfs4state.c > +++ b/fs/nfsd/nfs4state.c > @@ -5856,7 +5856,7 @@ nfs4_set_delegation(struct nfsd4_open *open, struct nfs4_ol_stateid *stp, > > /* > * Now that the deleg is set, check again to ensure that nothing > - * raced in and changed the mode while we weren't lookng. > + * raced in and changed the mode while we weren't looking. > */ > status = nfsd4_verify_setuid_write(open, fp->fi_deleg_file); > if (status) > @@ -8335,7 +8335,7 @@ check_for_locks(struct nfs4_file *fp, struct nfs4_lockowner *lowner) > * @cstate: NFSv4 COMPOUND state > * @u: RELEASE_LOCKOWNER arguments > * > - * Check if theree are any locks still held and if not - free the lockowner > + * Check if there are any locks still held and if not - free the lockowner This is probably better grammatically: Check if there are any locks still held and if not, free the lockowner > * and any lock state that is owned. > * > * Return values:
在 2024/8/23 20:07, Jeff Layton 写道: > On Fri, 2024-08-23 at 15:00 +0800, Li Lingfeng wrote: >> Fix spelling errors in comments of nfsd4_release_lockowner and >> nfs4_set_delegation. >> >> Signed-off-by: Li Lingfeng <lilingfeng3@huawei.com> >> --- >> fs/nfsd/nfs4state.c | 4 ++-- >> 1 file changed, 2 insertions(+), 2 deletions(-) >> >> diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c >> index a20c2c9d7d45..66a0c76850f3 100644 >> --- a/fs/nfsd/nfs4state.c >> +++ b/fs/nfsd/nfs4state.c >> @@ -5856,7 +5856,7 @@ nfs4_set_delegation(struct nfsd4_open *open, struct nfs4_ol_stateid *stp, >> >> /* >> * Now that the deleg is set, check again to ensure that nothing >> - * raced in and changed the mode while we weren't lookng. >> + * raced in and changed the mode while we weren't looking. >> */ >> status = nfsd4_verify_setuid_write(open, fp->fi_deleg_file); >> if (status) >> @@ -8335,7 +8335,7 @@ check_for_locks(struct nfs4_file *fp, struct nfs4_lockowner *lowner) >> * @cstate: NFSv4 COMPOUND state >> * @u: RELEASE_LOCKOWNER arguments >> * >> - * Check if theree are any locks still held and if not - free the lockowner >> + * Check if there are any locks still held and if not - free the lockowner > This is probably better grammatically: > > Check if there are any locks still held and if not, free the lockowner Thanks for your advice, it does look better. >> * and any lock state that is owned. >> * >> * Return values:
diff --git a/fs/nfsd/nfs4state.c b/fs/nfsd/nfs4state.c index a20c2c9d7d45..66a0c76850f3 100644 --- a/fs/nfsd/nfs4state.c +++ b/fs/nfsd/nfs4state.c @@ -5856,7 +5856,7 @@ nfs4_set_delegation(struct nfsd4_open *open, struct nfs4_ol_stateid *stp, /* * Now that the deleg is set, check again to ensure that nothing - * raced in and changed the mode while we weren't lookng. + * raced in and changed the mode while we weren't looking. */ status = nfsd4_verify_setuid_write(open, fp->fi_deleg_file); if (status) @@ -8335,7 +8335,7 @@ check_for_locks(struct nfs4_file *fp, struct nfs4_lockowner *lowner) * @cstate: NFSv4 COMPOUND state * @u: RELEASE_LOCKOWNER arguments * - * Check if theree are any locks still held and if not - free the lockowner + * Check if there are any locks still held and if not - free the lockowner * and any lock state that is owned. * * Return values:
Fix spelling errors in comments of nfsd4_release_lockowner and nfs4_set_delegation. Signed-off-by: Li Lingfeng <lilingfeng3@huawei.com> --- fs/nfsd/nfs4state.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)