the !DIAGNOSTIC VATTR_NULL was removed in
$OpenBSD: vnode.h,v 1.84 2007/05/29 16:25:07 thib Exp $

remove the define and directly call the function

diff --git sys/kern/kern_sig.c sys/kern/kern_sig.c
index 6d3800ccdb0..22b0e70e150 100644
--- sys/kern/kern_sig.c
+++ sys/kern/kern_sig.c
@@ -1806,7 +1806,7 @@ coredump(struct proc *p)
 		vn_close(vp, FWRITE, cred, p);
 		goto out;
 	}
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 	vattr.va_size = 0;
 	VOP_SETATTR(vp, &vattr, cred, p);
 	pr->ps_acflag |= ACORE;
diff --git sys/kern/tty_pty.c sys/kern/tty_pty.c
index 2d51d9caf2c..d7fcdf3a7d3 100644
--- sys/kern/tty_pty.c
+++ sys/kern/tty_pty.c
@@ -1025,7 +1025,7 @@ ptm_vn_open(struct nameidata *ndp)
 	error = VOP_OPEN(vp, FREAD|FWRITE, cred, p);
 	if (!error) {
 		/* update atime/mtime */
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		getnanotime(&vattr.va_atime);
 		vattr.va_mtime = vattr.va_atime;
 		vattr.va_vaflags |= VA_UTIMES_NULL;
@@ -1147,7 +1147,7 @@ retry:
 			/* get real uid */
 			uid = p->p_ucred->cr_ruid;
 
-			VATTR_NULL(&vattr);
+			vattr_null(&vattr);
 			vattr.va_uid = uid;
 			vattr.va_gid = gid;
 			vattr.va_mode = (S_IRUSR|S_IWUSR|S_IWGRP) & ALLPERMS;
diff --git sys/kern/uipc_usrreq.c sys/kern/uipc_usrreq.c
index 33e27319ea1..0636906117f 100644
--- sys/kern/uipc_usrreq.c
+++ sys/kern/uipc_usrreq.c
@@ -385,7 +385,7 @@ uipc_bind(struct socket *so, struct mbuf *nam, struct proc *p)
 		solock(unp->unp_socket);
 		goto out;
 	}
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 	vattr.va_type = VSOCK;
 	vattr.va_mode = ACCESSPERMS &~ p->p_fd->fd_cmask;
 	error = VOP_CREATE(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr);
diff --git sys/kern/vfs_syscalls.c sys/kern/vfs_syscalls.c
index c59f16c91e7..6f0aed66097 100644
--- sys/kern/vfs_syscalls.c
+++ sys/kern/vfs_syscalls.c
@@ -1176,7 +1176,7 @@ doopenat(struct proc *p, int fd, const char *path, int oflags, mode_t mode,
 		else if (vp->v_type == VDIR)
 			error = EISDIR;
 		else if ((error = vn_writechk(vp)) == 0) {
-			VATTR_NULL(&vattr);
+			vattr_null(&vattr);
 			vattr.va_size = 0;
 			error = VOP_SETATTR(vp, &vattr, fp->f_cred, p);
 		}
@@ -1405,7 +1405,7 @@ sys_fhopen(struct proc *p, void *v, register_t *retval)
 			goto bad;
 	}
 	if (flags & O_TRUNC) {
-		VATTR_NULL(&va);
+		vattr_null(&va);
 		va.va_size = 0;
 		if ((error = VOP_SETATTR(vp, &va, cred, p)) != 0)
 			goto bad;
@@ -1587,7 +1587,7 @@ domknodat(struct proc *p, int fd, const char *path, mode_t mode, dev_t dev)
 	if (vp != NULL)
 		error = EEXIST;
 	else {
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_mode = (mode & ALLPERMS) &~ p->p_fd->fd_cmask;
 		if ((p->p_p->ps_flags & PS_PLEDGE))
 			vattr.va_mode &= ACCESSPERMS;
@@ -1801,7 +1801,7 @@ dosymlinkat(struct proc *p, const char *upath, int fd, const char *link)
 		error = EEXIST;
 		goto out;
 	}
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 	vattr.va_mode = ACCESSPERMS &~ p->p_fd->fd_cmask;
 	error = VOP_SYMLINK(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr, path);
 out:
@@ -2303,7 +2303,7 @@ dovchflags(struct proc *p, struct vnode *vp, u_int flags)
 				goto out;
 			}
 		}
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_flags = flags;
 		error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
 	}
@@ -2366,7 +2366,7 @@ dofchmodat(struct proc *p, int fd, const char *path, mode_t mode, int flag)
 	if (vp->v_mount->mnt_flag & MNT_RDONLY)
 		error = EROFS;
 	else {
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_mode = mode & ALLPERMS;
 		error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
 	}
@@ -2402,7 +2402,7 @@ sys_fchmod(struct proc *p, void *v, register_t *retval)
 	if (vp->v_mount && vp->v_mount->mnt_flag & MNT_RDONLY)
 		error = EROFS;
 	else {
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_mode = mode & ALLPERMS;
 		error = VOP_SETATTR(vp, &vattr, p->p_ucred, p);
 	}
@@ -2479,7 +2479,7 @@ dofchownat(struct proc *p, int fd, const char *path, uid_t uid, gid_t gid,
 				mode = VNOVAL;
 		} else
 			mode = VNOVAL;
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_uid = uid;
 		vattr.va_gid = gid;
 		vattr.va_mode = mode;
@@ -2532,7 +2532,7 @@ sys_lchown(struct proc *p, void *v, register_t *retval)
 				mode = VNOVAL;
 		} else
 			mode = VNOVAL;
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_uid = uid;
 		vattr.va_gid = gid;
 		vattr.va_mode = mode;
@@ -2582,7 +2582,7 @@ sys_fchown(struct proc *p, void *v, register_t *retval)
 				mode = VNOVAL;
 		} else
 			mode = VNOVAL;
-		VATTR_NULL(&vattr);
+		vattr_null(&vattr);
 		vattr.va_uid = uid;
 		vattr.va_gid = gid;
 		vattr.va_mode = mode;
@@ -2705,7 +2705,7 @@ dovutimens(struct proc *p, struct vnode *vp, struct timespec ts[2])
 	}
 #endif
 
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 
 	/*  make sure ctime is updated even if neither mtime nor atime is */
 	vattr.va_vaflags = VA_UTIMES_CHANGE;
@@ -2845,7 +2845,7 @@ dotruncate(struct proc *p, struct vnode *vp, off_t len)
 			return EFBIG;
 		}
 	}
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 	vattr.va_size = len;
 	return VOP_SETATTR(vp, &vattr, p->p_ucred, p);
 }
@@ -3093,7 +3093,7 @@ domkdirat(struct proc *p, int fd, const char *path, mode_t mode)
 		vrele(vp);
 		return (EEXIST);
 	}
-	VATTR_NULL(&vattr);
+	vattr_null(&vattr);
 	vattr.va_type = VDIR;
 	vattr.va_mode = (mode & ACCESSPERMS) &~ p->p_fd->fd_cmask;
 	error = VOP_MKDIR(nd.ni_dvp, &nd.ni_vp, &nd.ni_cnd, &vattr);
diff --git sys/kern/vfs_vnops.c sys/kern/vfs_vnops.c
index 765b70a0947..c80e03d226c 100644
--- sys/kern/vfs_vnops.c
+++ sys/kern/vfs_vnops.c
@@ -108,7 +108,7 @@ vn_open(struct nameidata *ndp, int fmode, int cmode)
 			return (error);
 
 		if (ndp->ni_vp == NULL) {
-			VATTR_NULL(&va);
+			vattr_null(&va);
 			va.va_type = VREG;
 			va.va_mode = cmode;
 			if (fmode & O_EXCL)
@@ -169,7 +169,7 @@ vn_open(struct nameidata *ndp, int fmode, int cmode)
 		}
 	}
 	if ((fmode & O_TRUNC) && vp->v_type == VREG) {
-		VATTR_NULL(&va);
+		vattr_null(&va);
 		va.va_size = 0;
 		if ((error = VOP_SETATTR(vp, &va, cred, p)) != 0)
 			goto bad;
diff --git sys/nfs/nfs_serv.c sys/nfs/nfs_serv.c
index ea7b596183f..2e30d2f4915 100644
--- sys/nfs/nfs_serv.c
+++ sys/nfs/nfs_serv.c
@@ -296,7 +296,7 @@ nfsrv_setattr(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 	fhp = &nfh.fh_generic;
 	if (nfsm_srvmtofh2(&info, fhp) != 0)
 		goto nfsmout;
-	VATTR_NULL(&va);
+	vattr_null(&va);
 	if (info.nmi_v3) {
 		va.va_vaflags |= VA_UTIMES_NULL;
 		error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep, &info.nmi_dpos);
@@ -1083,7 +1083,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 		return (0);
 	}
 
-	VATTR_NULL(&va);
+	vattr_null(&va);
 	if (info.nmi_v3) {
 		tl = (uint32_t *)nfsm_dissect(&info, NFSX_UNSIGNED);
 		if (tl == NULL)
@@ -1151,7 +1151,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 				pool_put(&namei_pool, nd.ni_cnd.cn_pnbuf);
 				if (exclusive_flag) {
 					exclusive_flag = 0;
-					VATTR_NULL(&va);
+					vattr_null(&va);
 					bcopy(cverf, (caddr_t)&va.va_atime,
 						NFSX_V3CREATEVERF);
 					error = VOP_SETATTR(nd.ni_vp, &va, cred,
@@ -1245,7 +1245,7 @@ nfsrv_create(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 			    (nd.ni_cnd.cn_flags & RDONLY), procp, 0);
 			if (!error) {
 				tempsize = va.va_size;
-				VATTR_NULL(&va);
+				vattr_null(&va);
 				va.va_size = tempsize;
 				error = VOP_SETATTR(vp, &va, cred,
 					 procp);
@@ -1385,7 +1385,7 @@ nfsrv_mknod(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 			vput(nd.ni_vp);
 		goto out;
 	}
-	VATTR_NULL(&va);
+	vattr_null(&va);
 	error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep, &info.nmi_dpos);
 	if (error)
 		goto nfsmout;
@@ -1983,7 +1983,7 @@ nfsrv_symlink(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 	}
 	if (error)
 		goto out;
-	VATTR_NULL(&va);
+	vattr_null(&va);
 	if (info.nmi_v3) {
 		error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep,
 		    &info.nmi_dpos);
@@ -2156,7 +2156,7 @@ nfsrv_mkdir(struct nfsrv_descript *nfsd, struct nfssvc_sock *slp,
 		return (0);
 	}
 
-	VATTR_NULL(&va);
+	vattr_null(&va);
 	if (info.nmi_v3) {
 		error = nfsm_srvsattr(&info.nmi_md, &va, info.nmi_mrep,
 		    &info.nmi_dpos);
diff --git sys/nfs/nfs_vnops.c sys/nfs/nfs_vnops.c
index 2a2b61d4668..75f23a9bb62 100644
--- sys/nfs/nfs_vnops.c
+++ sys/nfs/nfs_vnops.c
@@ -3516,7 +3516,7 @@ nfsspec_close(void *v)
 		np->n_flag |= NCHG;
 		if (vp->v_usecount == 1 &&
 		    (vp->v_mount->mnt_flag & MNT_RDONLY) == 0) {
-			VATTR_NULL(&vattr);
+			vattr_null(&vattr);
 			if (np->n_flag & NACC)
 				vattr.va_atime = np->n_atim;
 			if (np->n_flag & NUPD)
@@ -3585,7 +3585,7 @@ nfsfifo_close(void *v)
 		np->n_flag |= NCHG;
 		if (vp->v_usecount == 1 &&
 		    (vp->v_mount->mnt_flag & MNT_RDONLY) == 0) {
-			VATTR_NULL(&vattr);
+			vattr_null(&vattr);
 			if (np->n_flag & NACC)
 				vattr.va_atime = np->n_atim;
 			if (np->n_flag & NUPD)
diff --git sys/sys/vnode.h sys/sys/vnode.h
index 9a581783bfb..8c2ffc22d21 100644
--- sys/sys/vnode.h
+++ sys/sys/vnode.h
@@ -243,7 +243,6 @@ extern int		vttoif_tab[];
 #define REVOKEALL	0x0001		/* vop_revoke: revoke all aliases */
 
 
-#define	VATTR_NULL(vap)	vattr_null(vap)
 #define	NULLVP	((struct vnode *)NULL)
 #define	VN_KNOTE(vp, b)					\
 	knote_locked(&vp->v_klist, (b))