mirror of
https://github.com/freebsd/freebsd-src
synced 2024-10-17 22:04:40 +00:00
Modify the return value for nfscl_mustflush() from boolean_t,
which I mistakenly thought was correct w.r.t. style(9), back to int and add the checks for != 0. This is just a stylistic modification. MFC after: 1 week
This commit is contained in:
parent
fae1602944
commit
e3649d5a2f
Notes:
svn2git
2020-12-20 02:59:44 +00:00
svn path=/head/; revision=210786
|
@ -467,7 +467,7 @@ void nfscl_docb(struct nfsrv_descript *, NFSPROC_T *);
|
|||
void nfscl_releasealllocks(struct nfsclclient *, vnode_t, NFSPROC_T *);
|
||||
int nfscl_lockt(vnode_t, struct nfsclclient *, u_int64_t,
|
||||
u_int64_t, struct flock *, NFSPROC_T *);
|
||||
boolean_t nfscl_mustflush(vnode_t);
|
||||
int nfscl_mustflush(vnode_t);
|
||||
int nfscl_nodeleg(vnode_t, int);
|
||||
int nfscl_removedeleg(vnode_t, NFSPROC_T *, nfsv4stateid_t *);
|
||||
int nfscl_getref(struct nfsmount *);
|
||||
|
|
|
@ -3735,7 +3735,7 @@ nfscl_tryclose(struct nfsclopen *op, struct ucred *cred,
|
|||
* to the server. This might be a big performance win in some environments.
|
||||
* (Not useful until the client does caching on local stable storage.)
|
||||
*/
|
||||
APPLESTATIC boolean_t
|
||||
APPLESTATIC int
|
||||
nfscl_mustflush(vnode_t vp)
|
||||
{
|
||||
struct nfsclclient *clp;
|
||||
|
@ -3746,12 +3746,12 @@ nfscl_mustflush(vnode_t vp)
|
|||
np = VTONFS(vp);
|
||||
nmp = VFSTONFS(vnode_mount(vp));
|
||||
if (!NFSHASNFSV4(nmp))
|
||||
return (TRUE);
|
||||
return (1);
|
||||
NFSLOCKCLSTATE();
|
||||
clp = nfscl_findcl(nmp);
|
||||
if (clp == NULL) {
|
||||
NFSUNLOCKCLSTATE();
|
||||
return (TRUE);
|
||||
return (1);
|
||||
}
|
||||
dp = nfscl_finddeleg(clp, np->n_fhp->nfh_fh, np->n_fhp->nfh_len);
|
||||
if (dp != NULL && (dp->nfsdl_flags & (NFSCLDL_WRITE | NFSCLDL_RECALL))
|
||||
|
@ -3759,10 +3759,10 @@ nfscl_mustflush(vnode_t vp)
|
|||
(dp->nfsdl_sizelimit >= np->n_size ||
|
||||
!NFSHASSTRICT3530(nmp))) {
|
||||
NFSUNLOCKCLSTATE();
|
||||
return (FALSE);
|
||||
return (0);
|
||||
}
|
||||
NFSUNLOCKCLSTATE();
|
||||
return (TRUE);
|
||||
return (1);
|
||||
}
|
||||
|
||||
/*
|
||||
|
|
|
@ -187,8 +187,7 @@ ncl_getattrcache(struct vnode *vp, struct vattr *vaper)
|
|||
struct nfsnode *np;
|
||||
struct vattr *vap;
|
||||
struct nfsmount *nmp;
|
||||
int timeo;
|
||||
boolean_t mustflush;
|
||||
int timeo, mustflush;
|
||||
|
||||
np = VTONFS(vp);
|
||||
vap = &np->n_vattr.na_vattr;
|
||||
|
@ -230,7 +229,7 @@ ncl_getattrcache(struct vnode *vp, struct vattr *vaper)
|
|||
#endif
|
||||
|
||||
if ((time_second - np->n_attrstamp) >= timeo &&
|
||||
(mustflush || np->n_attrstamp == 0)) {
|
||||
(mustflush != 0 || np->n_attrstamp == 0)) {
|
||||
newnfsstats.attrcache_misses++;
|
||||
mtx_unlock(&np->n_mtx);
|
||||
#ifdef NFS_ACDEBUG
|
||||
|
|
|
@ -495,7 +495,8 @@ nfs_open(struct vop_open_args *ap)
|
|||
* Now, if this Open will be doing reading, re-validate/flush the
|
||||
* cache, so that Close/Open coherency is maintained.
|
||||
*/
|
||||
if ((fmode & FREAD) && (!NFS_ISV4(vp) || nfscl_mustflush(vp))) {
|
||||
if ((fmode & FREAD) != 0 &&
|
||||
(!NFS_ISV4(vp) || nfscl_mustflush(vp) != 0)) {
|
||||
mtx_lock(&np->n_mtx);
|
||||
if (np->n_flag & NMODIFIED) {
|
||||
mtx_unlock(&np->n_mtx);
|
||||
|
@ -667,7 +668,7 @@ nfs_close(struct vop_close_args *ap)
|
|||
error = ncl_flush(vp, MNT_WAIT, cred, ap->a_td, cm, 0);
|
||||
/* np->n_flag &= ~NMODIFIED; */
|
||||
} else if (NFS_ISV4(vp)) {
|
||||
if (nfscl_mustflush(vp)) {
|
||||
if (nfscl_mustflush(vp) != 0) {
|
||||
int cm = newnfs_commit_on_close ? 1 : 0;
|
||||
error = ncl_flush(vp, MNT_WAIT, cred, ap->a_td,
|
||||
cm, 0);
|
||||
|
@ -709,7 +710,7 @@ nfs_close(struct vop_close_args *ap)
|
|||
/*
|
||||
* Get attributes so "change" is up to date.
|
||||
*/
|
||||
if (error == 0 && nfscl_mustflush(vp)) {
|
||||
if (error == 0 && nfscl_mustflush(vp) != 0) {
|
||||
ret = nfsrpc_getattr(vp, cred, ap->a_td, &nfsva,
|
||||
NULL);
|
||||
if (!ret) {
|
||||
|
|
Loading…
Reference in a new issue