Module Name: src
Committed By: bouyer
Date: Sat Aug 27 15:18:05 UTC 2016
Modified Files:
src/sbin/fsdb [netbsd-7-0]: fsdb.c
src/sys/ufs/ffs [netbsd-7-0]: ffs_vfsops.c
src/usr.sbin/quot [netbsd-7-0]: quot.c
Log Message:
Pull up following revision(s) (requested by martin in ticket #1210):
sys/ufs/ffs/ffs_vfsops.c: revision 1.340
usr.sbin/quot/quot.c: revision 1.34
sbin/fsdb/fsdb.c: revision 1.49
>From Michael Plass:
The superblock field that distinguishes between 4.2BSD and 4.4BSD
inodes is really only relevant on a UFS1 file system. Make sure that
it is a UFS1 fs before using fs_old_inodefmt.
Note that the NetBSD newfs and mkfs utilities initialize fs_old_inodefmt
even for UFS2, so problems were apparent only on file systems created
by other operating systems, for example, FreeBSD.
To generate a diff of this commit:
cvs rdiff -u -r1.48 -r1.48.8.1 src/sbin/fsdb/fsdb.c
cvs rdiff -u -r1.299.2.3 -r1.299.2.3.2.1 src/sys/ufs/ffs/ffs_vfsops.c
cvs rdiff -u -r1.33 -r1.33.6.1 src/usr.sbin/quot/quot.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/sbin/fsdb/fsdb.c
diff -u src/sbin/fsdb/fsdb.c:1.48 src/sbin/fsdb/fsdb.c:1.48.8.1
--- src/sbin/fsdb/fsdb.c:1.48 Sun Jun 23 02:06:04 2013
+++ src/sbin/fsdb/fsdb.c Sat Aug 27 15:18:05 2016
@@ -1,4 +1,4 @@
-/* $NetBSD: fsdb.c,v 1.48 2013/06/23 02:06:04 dholland Exp $ */
+/* $NetBSD: fsdb.c,v 1.48.8.1 2016/08/27 15:18:05 bouyer Exp $ */
/*-
* Copyright (c) 1996 The NetBSD Foundation, Inc.
@@ -31,7 +31,7 @@
#include <sys/cdefs.h>
#ifndef lint
-__RCSID("$NetBSD: fsdb.c,v 1.48 2013/06/23 02:06:04 dholland Exp $");
+__RCSID("$NetBSD: fsdb.c,v 1.48.8.1 2016/08/27 15:18:05 bouyer Exp $");
#endif /* not lint */
#include <sys/types.h>
@@ -1232,7 +1232,7 @@ CMDFUNC(chgroup)
return 1;
}
}
- if (sblock->fs_old_inodefmt < FS_44INODEFMT)
+ if (!is_ufs2 && sblock->fs_old_inodefmt < FS_44INODEFMT)
curinode->dp1.di_ogid = iswap32(gid);
else
DIP_SET(curinode, gid, iswap32(gid));
Index: src/sys/ufs/ffs/ffs_vfsops.c
diff -u src/sys/ufs/ffs/ffs_vfsops.c:1.299.2.3 src/sys/ufs/ffs/ffs_vfsops.c:1.299.2.3.2.1
--- src/sys/ufs/ffs/ffs_vfsops.c:1.299.2.3 Wed Jan 28 18:34:11 2015
+++ src/sys/ufs/ffs/ffs_vfsops.c Sat Aug 27 15:18:05 2016
@@ -1,4 +1,4 @@
-/* $NetBSD: ffs_vfsops.c,v 1.299.2.3 2015/01/28 18:34:11 martin Exp $ */
+/* $NetBSD: ffs_vfsops.c,v 1.299.2.3.2.1 2016/08/27 15:18:05 bouyer Exp $ */
/*-
* Copyright (c) 2008, 2009 The NetBSD Foundation, Inc.
@@ -61,7 +61,7 @@
*/
#include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: ffs_vfsops.c,v 1.299.2.3 2015/01/28 18:34:11 martin Exp $");
+__KERNEL_RCSID(0, "$NetBSD: ffs_vfsops.c,v 1.299.2.3.2.1 2016/08/27 15:18:05 bouyer Exp $");
#if defined(_KERNEL_OPT)
#include "opt_ffs.h"
@@ -1450,7 +1450,8 @@ ffs_oldfscompat_read(struct fs *fs, stru
fs->fs_old_trackskew = 0;
}
- if (fs->fs_old_inodefmt < FS_44INODEFMT) {
+ if (fs->fs_magic == FS_UFS1_MAGIC &&
+ fs->fs_old_inodefmt < FS_44INODEFMT) {
fs->fs_maxfilesize = (u_quad_t) 1LL << 39;
fs->fs_qbmask = ~fs->fs_bmask;
fs->fs_qfmask = ~fs->fs_fmask;
@@ -1895,7 +1896,8 @@ ffs_loadvnode(struct mount *mp, struct v
* fix until fsck has been changed to do the update.
*/
- if (fs->fs_old_inodefmt < FS_44INODEFMT) { /* XXX */
+ if (fs->fs_magic == FS_UFS1_MAGIC && /* XXX */
+ fs->fs_old_inodefmt < FS_44INODEFMT) { /* XXX */
ip->i_uid = ip->i_ffs1_ouid; /* XXX */
ip->i_gid = ip->i_ffs1_ogid; /* XXX */
} /* XXX */
Index: src/usr.sbin/quot/quot.c
diff -u src/usr.sbin/quot/quot.c:1.33 src/usr.sbin/quot/quot.c:1.33.6.1
--- src/usr.sbin/quot/quot.c:1.33 Sat Oct 19 17:16:38 2013
+++ src/usr.sbin/quot/quot.c Sat Aug 27 15:18:05 2016
@@ -1,4 +1,4 @@
-/* $NetBSD: quot.c,v 1.33 2013/10/19 17:16:38 christos Exp $ */
+/* $NetBSD: quot.c,v 1.33.6.1 2016/08/27 15:18:05 bouyer Exp $ */
/*
* Copyright (C) 1991, 1994 Wolfgang Solfrank.
@@ -33,7 +33,7 @@
#include <sys/cdefs.h>
#ifndef lint
-__RCSID("$NetBSD: quot.c,v 1.33 2013/10/19 17:16:38 christos Exp $");
+__RCSID("$NetBSD: quot.c,v 1.33.6.1 2016/08/27 15:18:05 bouyer Exp $");
#endif /* not lint */
#include <sys/param.h>
@@ -521,7 +521,8 @@ ffs_oldfscompat(struct fs *fs)
{
int i;
- if (fs->fs_old_inodefmt < FS_44INODEFMT) {
+ if (fs->fs_magic == FS_UFS1_MAGIC &&
+ fs->fs_old_inodefmt < FS_44INODEFMT) {
quad_t sizepb = fs->fs_bsize;
fs->fs_maxfilesize = fs->fs_bsize * UFS_NDADDR - 1;