git: f0c9847a6c47 - main - vfs: Add "ioflag" and "cred" arguments to VOP_ALLOCATE
- Go to: [ bottom of page ] [ top of archives ] [ this month ]
Date: Sat, 06 Nov 2021 20:30:43 UTC
The branch main has been updated by rmacklem:
URL: https://cgit.FreeBSD.org/src/commit/?id=f0c9847a6c477430d6fff647b12e9e9e2b461f2a
commit f0c9847a6c477430d6fff647b12e9e9e2b461f2a
Author: Rick Macklem <rmacklem@FreeBSD.org>
AuthorDate: 2021-11-06 20:26:43 +0000
Commit: Rick Macklem <rmacklem@FreeBSD.org>
CommitDate: 2021-11-06 20:26:43 +0000
vfs: Add "ioflag" and "cred" arguments to VOP_ALLOCATE
When the NFSv4.2 server does a VOP_ALLOCATE(), it needs
the operation to be done for the RPC's credential and not
td_ucred. It also needs the writing to be done synchronously.
This patch adds "ioflag" and "cred" arguments to VOP_ALLOCATE()
and modifies vop_stdallocate() to use these arguments.
The VOP_ALLOCATE.9 man page will be patched separately.
Reviewed by: khng, kib
Differential Revision: https://reviews.freebsd.org/D32865
---
sys/fs/nfsclient/nfs_clvnops.c | 2 +-
sys/fs/nfsserver/nfs_nfsdport.c | 2 +-
sys/kern/vfs_default.c | 10 +++++-----
sys/kern/vfs_vnops.c | 3 ++-
sys/kern/vnode_if.src | 2 ++
5 files changed, 11 insertions(+), 8 deletions(-)
diff --git a/sys/fs/nfsclient/nfs_clvnops.c b/sys/fs/nfsclient/nfs_clvnops.c
index abccf82e3ff0..d4545cf2d840 100644
--- a/sys/fs/nfsclient/nfs_clvnops.c
+++ b/sys/fs/nfsclient/nfs_clvnops.c
@@ -3731,7 +3731,7 @@ nfs_allocate(struct vop_allocate_args *ap)
if ((uint64_t)alen > nfs_maxalloclen)
alen = nfs_maxalloclen;
error = nfsrpc_allocate(vp, *ap->a_offset, alen,
- &nfsva, &attrflag, td->td_ucred, td, NULL);
+ &nfsva, &attrflag, ap->a_cred, td, NULL);
}
if (error == 0) {
*ap->a_offset += alen;
diff --git a/sys/fs/nfsserver/nfs_nfsdport.c b/sys/fs/nfsserver/nfs_nfsdport.c
index 5fa6853446ae..806f4a8545b8 100644
--- a/sys/fs/nfsserver/nfs_nfsdport.c
+++ b/sys/fs/nfsserver/nfs_nfsdport.c
@@ -6598,7 +6598,7 @@ nfsvno_allocate(struct vnode *vp, off_t off, off_t len, struct ucred *cred,
*/
do {
olen = len;
- error = VOP_ALLOCATE(vp, &off, &len);
+ error = VOP_ALLOCATE(vp, &off, &len, IO_SYNC, cred);
if (error == 0 && len > 0 && olen > len)
maybe_yield();
} while (error == 0 && len > 0 && olen > len);
diff --git a/sys/kern/vfs_default.c b/sys/kern/vfs_default.c
index d66443e6d6f0..02fae41e3404 100644
--- a/sys/kern/vfs_default.c
+++ b/sys/kern/vfs_default.c
@@ -963,7 +963,7 @@ vop_stdallocate(struct vop_allocate_args *ap)
len = *ap->a_len;
offset = *ap->a_offset;
- error = VOP_GETATTR(vp, vap, td->td_ucred);
+ error = VOP_GETATTR(vp, vap, ap->a_cred);
if (error != 0)
goto out;
fsize = vap->va_size;
@@ -1000,12 +1000,12 @@ vop_stdallocate(struct vop_allocate_args *ap)
*/
VATTR_NULL(vap);
vap->va_size = offset + len;
- error = VOP_SETATTR(vp, vap, td->td_ucred);
+ error = VOP_SETATTR(vp, vap, ap->a_cred);
if (error != 0)
goto out;
VATTR_NULL(vap);
vap->va_size = fsize;
- error = VOP_SETATTR(vp, vap, td->td_ucred);
+ error = VOP_SETATTR(vp, vap, ap->a_cred);
if (error != 0)
goto out;
}
@@ -1031,7 +1031,7 @@ vop_stdallocate(struct vop_allocate_args *ap)
auio.uio_segflg = UIO_SYSSPACE;
auio.uio_rw = UIO_READ;
auio.uio_td = td;
- error = VOP_READ(vp, &auio, 0, td->td_ucred);
+ error = VOP_READ(vp, &auio, ap->a_ioflag, ap->a_cred);
if (error != 0)
break;
if (auio.uio_resid > 0) {
@@ -1052,7 +1052,7 @@ vop_stdallocate(struct vop_allocate_args *ap)
auio.uio_rw = UIO_WRITE;
auio.uio_td = td;
- error = VOP_WRITE(vp, &auio, 0, td->td_ucred);
+ error = VOP_WRITE(vp, &auio, ap->a_ioflag, ap->a_cred);
if (error != 0)
break;
diff --git a/sys/kern/vfs_vnops.c b/sys/kern/vfs_vnops.c
index d6c472995489..79d422aacfef 100644
--- a/sys/kern/vfs_vnops.c
+++ b/sys/kern/vfs_vnops.c
@@ -3466,7 +3466,8 @@ vn_fallocate(struct file *fp, off_t offset, off_t len, struct thread *td)
error = mac_vnode_check_write(td->td_ucred, fp->f_cred, vp);
if (error == 0)
#endif
- error = VOP_ALLOCATE(vp, &offset, &len);
+ error = VOP_ALLOCATE(vp, &offset, &len, 0,
+ td->td_ucred);
VOP_UNLOCK(vp);
vn_finished_write(mp);
diff --git a/sys/kern/vnode_if.src b/sys/kern/vnode_if.src
index d10758019b87..276382738504 100644
--- a/sys/kern/vnode_if.src
+++ b/sys/kern/vnode_if.src
@@ -702,6 +702,8 @@ vop_allocate {
IN struct vnode *vp;
INOUT off_t *offset;
INOUT off_t *len;
+ IN int ioflag;
+ IN struct ucred *cred;
};