Module Name:    src
Committed By:   dholland
Date:           Sat Oct 10 22:34:34 UTC 2015

Modified Files:
        src/sys/ufs/lfs: lfs_alloc.c lfs_segment.c lfs_syscalls.c

Log Message:
Fix minor bitrot in #if 0 or otherwise disabled code.


To generate a diff of this commit:
cvs rdiff -u -r1.130 -r1.131 src/sys/ufs/lfs/lfs_alloc.c
cvs rdiff -u -r1.261 -r1.262 src/sys/ufs/lfs/lfs_segment.c
cvs rdiff -u -r1.170 -r1.171 src/sys/ufs/lfs/lfs_syscalls.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/ufs/lfs/lfs_alloc.c
diff -u src/sys/ufs/lfs/lfs_alloc.c:1.130 src/sys/ufs/lfs/lfs_alloc.c:1.131
--- src/sys/ufs/lfs/lfs_alloc.c:1.130	Sun Sep 13 07:53:37 2015
+++ src/sys/ufs/lfs/lfs_alloc.c	Sat Oct 10 22:34:33 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: lfs_alloc.c,v 1.130 2015/09/13 07:53:37 dholland Exp $	*/
+/*	$NetBSD: lfs_alloc.c,v 1.131 2015/10/10 22:34:33 dholland Exp $	*/
 
 /*-
  * Copyright (c) 1999, 2000, 2001, 2002, 2003, 2007 The NetBSD Foundation, Inc.
@@ -60,7 +60,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: lfs_alloc.c,v 1.130 2015/09/13 07:53:37 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: lfs_alloc.c,v 1.131 2015/10/10 22:34:33 dholland Exp $");
 
 #if defined(_KERNEL_OPT)
 #include "opt_quota.h"
@@ -327,7 +327,8 @@ lfs_last_alloc_ino(struct lfs *fs)
 	ino_t ino, maxino;
 
 	maxino = ((fs->lfs_ivnode->v_size >> lfs_sb_getbshift(fs)) -
-		  lfs_sb_getcleansz(fs) - lfs_sb_getsegtabsz(fs)) * fs->lfs_ifpb;
+		  lfs_sb_getcleansz(fs) - lfs_sb_getsegtabsz(fs)) *
+		lfs_sb_getifpb(fs);
 	for (ino = maxino - 1; ino > LFS_UNUSED_INUM; --ino) {
 		if (ISSET_BITMAP_FREE(fs, ino) == 0)
 			break;
@@ -602,8 +603,8 @@ lfs_order_freelist(struct lfs *fs)
 			lfs_truncate(vp, 0, 0, NOCRED);
 			vput(vp);
 			LFS_SEGENTRY(sup, fs, segno, bp);
-			KASSERT(sup->su_nbytes >= LFS_DINODE1_SIZE);
-			sup->su_nbytes -= LFS_DINODE1_SIZE;
+			KASSERT(sup->su_nbytes >= DINOSIZE(fs));
+			sup->su_nbytes -= DINOSIZE(fs);
 			LFS_WRITESEGENTRY(sup, fs, segno, bp);
 
 			/* Set up to fall through to next section */

Index: src/sys/ufs/lfs/lfs_segment.c
diff -u src/sys/ufs/lfs/lfs_segment.c:1.261 src/sys/ufs/lfs/lfs_segment.c:1.262
--- src/sys/ufs/lfs/lfs_segment.c:1.261	Sat Oct 10 22:33:31 2015
+++ src/sys/ufs/lfs/lfs_segment.c	Sat Oct 10 22:34:33 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: lfs_segment.c,v 1.261 2015/10/10 22:33:31 dholland Exp $	*/
+/*	$NetBSD: lfs_segment.c,v 1.262 2015/10/10 22:34:33 dholland Exp $	*/
 
 /*-
  * Copyright (c) 1999, 2000, 2001, 2002, 2003 The NetBSD Foundation, Inc.
@@ -60,7 +60,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: lfs_segment.c,v 1.261 2015/10/10 22:33:31 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: lfs_segment.c,v 1.262 2015/10/10 22:34:33 dholland Exp $");
 
 #ifdef DEBUG
 # define vndebug(vp, str) do {						\
@@ -2232,7 +2232,7 @@ lfs_writeseg(struct lfs *fs, struct segm
 				if (copyin((void *)(*bpp)->b_saveaddr +
 					   byteoffset, dp, el_size)) {
 					panic("lfs_writeseg: copyin failed [1]:"
-						" ino %d blk %" PRId64,
+						" ino %" PRIu64 " blk %" PRId64,
 						VTOI((*bpp)->b_vp)->i_number,
 						(*bpp)->b_lblkno);
 				}

Index: src/sys/ufs/lfs/lfs_syscalls.c
diff -u src/sys/ufs/lfs/lfs_syscalls.c:1.170 src/sys/ufs/lfs/lfs_syscalls.c:1.171
--- src/sys/ufs/lfs/lfs_syscalls.c:1.170	Tue Sep  1 06:08:37 2015
+++ src/sys/ufs/lfs/lfs_syscalls.c	Sat Oct 10 22:34:33 2015
@@ -1,4 +1,4 @@
-/*	$NetBSD: lfs_syscalls.c,v 1.170 2015/09/01 06:08:37 dholland Exp $	*/
+/*	$NetBSD: lfs_syscalls.c,v 1.171 2015/10/10 22:34:33 dholland Exp $	*/
 
 /*-
  * Copyright (c) 1999, 2000, 2001, 2002, 2003, 2007, 2007, 2008
@@ -61,7 +61,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: lfs_syscalls.c,v 1.170 2015/09/01 06:08:37 dholland Exp $");
+__KERNEL_RCSID(0, "$NetBSD: lfs_syscalls.c,v 1.171 2015/10/10 22:34:33 dholland Exp $");
 
 #ifndef LFS
 # define LFS		/* for prototypes in syscallargs.h */
@@ -122,7 +122,7 @@ sys_lfs_markv(struct lwp *l, const struc
 	if ((error = copyin(SCARG(uap, fsidp), &fsid, sizeof(fsid_t))) != 0)
 		return (error);
 
-	if ((mntp = vfs_getvfs(fsidp)) == NULL) 
+	if ((mntp = vfs_getvfs(&fsid)) == NULL) 
 		return (ENOENT);
 	fs = VFSTOULFS(mntp)->um_lfs;
 
@@ -136,7 +136,7 @@ sys_lfs_markv(struct lwp *l, const struc
 			    blkcnt * sizeof(BLOCK_INFO))) != 0)
 		goto out;
 
-	if ((error = lfs_markv(p, &fsid, blkiov, blkcnt)) == 0)
+	if ((error = lfs_markv(l, &fsid, blkiov, blkcnt)) == 0)
 		copyout(blkiov, SCARG(uap, blkiov),
 			blkcnt * sizeof(BLOCK_INFO));
     out:
@@ -551,15 +551,17 @@ sys_lfs_bmapv(struct lwp *l, const struc
 	fs = VFSTOULFS(mntp)->um_lfs;
 
 	blkcnt = SCARG(uap, blkcnt);
+#if SIZE_T_MAX <= UINT_MAX
 	if ((u_int) blkcnt > SIZE_T_MAX / sizeof(BLOCK_INFO))
 		return (EINVAL);
+#endif
 	KERNEL_LOCK(1, NULL);
 	blkiov = lfs_malloc(fs, blkcnt * sizeof(BLOCK_INFO), LFS_NB_BLKIOV);
 	if ((error = copyin(SCARG(uap, blkiov), blkiov,
 			    blkcnt * sizeof(BLOCK_INFO))) != 0)
 		goto out;
 
-	if ((error = lfs_bmapv(p, &fsid, blkiov, blkcnt)) == 0)
+	if ((error = lfs_bmapv(l, &fsid, blkiov, blkcnt)) == 0)
 		copyout(blkiov, SCARG(uap, blkiov),
 			blkcnt * sizeof(BLOCK_INFO));
     out:

Reply via email to