diff options
| author |   bluhm <bluhm@openbsd.org> | 2017-01-10 19:56:34 +0000 |
|---|---|---|
| committer | bluhm <bluhm@openbsd.org> | 2017-01-10 19:56:34 +0000 |
| commit | 9bf48af2330da1852b6b42f7780347d28fcdf15b (patch) | |
| tree | 6d7944ac8ff9401203ae00f73775ad39c2709c9c /sys/kern/vfs_subr.c | |
| parent | Remove the unused olddp parameter from function dounmount(). (diff) | |
| download | wireguard-openbsd-9bf48af2330da1852b6b42f7780347d28fcdf15b.tar.xz wireguard-openbsd-9bf48af2330da1852b6b42f7780347d28fcdf15b.zip | |
Replace manual for() loops with FOREACH() macro.
OK millert@
Diffstat (limited to 'sys/kern/vfs_subr.c')
| -rw-r--r-- | sys/kern/vfs_subr.c | 11 |
1 files changed, 4 insertions, 7 deletions
diff --git a/sys/kern/vfs_subr.c b/sys/kern/vfs_subr.c index 1072c058346..5304ee32ba1 100644 --- a/sys/kern/vfs_subr.c +++ b/sys/kern/vfs_subr.c @@ -1,4 +1,4 @@ -/* $OpenBSD: vfs_subr.c,v 1.255 2017/01/10 19:48:32 bluhm Exp $ */ +/* $OpenBSD: vfs_subr.c,v 1.256 2017/01/10 19:56:34 bluhm Exp $ */ /* $NetBSD: vfs_subr.c,v 1.53 1996/04/22 01:39:13 christos Exp $ */ /* @@ -380,8 +380,7 @@ getnewvnode(enum vtagtype tag, struct mount *mp, struct vops *vops, TAILQ_INIT(&vp->v_cache_dst); numvnodes++; } else { - for (vp = TAILQ_FIRST(listhd); vp != NULLVP; - vp = TAILQ_NEXT(vp, v_freelist)) { + TAILQ_FOREACH(vp, listhd, v_freelist) { if (VOP_ISLOCKED(vp) == 0) break; } @@ -836,10 +835,9 @@ vfs_mount_foreach_vnode(struct mount *mp, int error = 0; loop: - for (vp = LIST_FIRST(&mp->mnt_vnodelist); vp != NULL; vp = nvp) { + LIST_FOREACH_SAFE(vp , &mp->mnt_vnodelist, v_mntvnodes, nvp) { if (vp->v_mount != mp) goto loop; - nvp = LIST_NEXT(vp, v_mntvnodes); error = func(vp, arg); @@ -1873,8 +1871,7 @@ vflushbuf(struct vnode *vp, int sync) loop: s = splbio(); - for (bp = LIST_FIRST(&vp->v_dirtyblkhd); bp != NULL; bp = nbp) { - nbp = LIST_NEXT(bp, b_vnbufs); + LIST_FOREACH_SAFE(bp, &vp->v_dirtyblkhd, b_vnbufs, nbp) { if ((bp->b_flags & B_BUSY)) continue; if ((bp->b_flags & B_DELWRI) == 0) |