Module Name: src Committed By: dholland Date: Wed Feb 1 05:43:55 UTC 2012
Modified Files: src/lib/libquota: quota_kernel.c src/sys/compat/netbsd32: netbsd32.h netbsd32_netbsd.c src/sys/kern: vfs_quotactl.c vfs_syscalls.c src/sys/sys: quotactl.h src/sys/ufs/ufs: ufs_quota.c ufs_quota1.c ufs_quota2.c Log Message: Improve the names of some members of struct quotactl_args. These are effectively function parameter names, but since they need to be described with the same names in the man page the choices do matter. Some. To generate a diff of this commit: cvs rdiff -u -r1.1 -r1.2 src/lib/libquota/quota_kernel.c cvs rdiff -u -r1.90 -r1.91 src/sys/compat/netbsd32/netbsd32.h cvs rdiff -u -r1.178 -r1.179 src/sys/compat/netbsd32/netbsd32_netbsd.c cvs rdiff -u -r1.37 -r1.38 src/sys/kern/vfs_quotactl.c cvs rdiff -u -r1.445 -r1.446 src/sys/kern/vfs_syscalls.c cvs rdiff -u -r1.33 -r1.34 src/sys/sys/quotactl.h cvs rdiff -u -r1.107 -r1.108 src/sys/ufs/ufs/ufs_quota.c cvs rdiff -u -r1.16 -r1.17 src/sys/ufs/ufs/ufs_quota1.c cvs rdiff -u -r1.31 -r1.32 src/sys/ufs/ufs/ufs_quota2.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/lib/libquota/quota_kernel.c diff -u src/lib/libquota/quota_kernel.c:1.1 src/lib/libquota/quota_kernel.c:1.2 --- src/lib/libquota/quota_kernel.c:1.1 Wed Feb 1 05:34:40 2012 +++ src/lib/libquota/quota_kernel.c Wed Feb 1 05:43:53 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: quota_kernel.c,v 1.1 2012/02/01 05:34:40 dholland Exp $ */ +/* $NetBSD: quota_kernel.c,v 1.2 2012/02/01 05:43:53 dholland Exp $ */ /*- * Copyright (c) 2012 The NetBSD Foundation, Inc. * All rights reserved. @@ -29,7 +29,7 @@ */ #include <sys/cdefs.h> -__RCSID("$NetBSD: quota_kernel.c,v 1.1 2012/02/01 05:34:40 dholland Exp $"); +__RCSID("$NetBSD: quota_kernel.c,v 1.2 2012/02/01 05:43:53 dholland Exp $"); #include <stdlib.h> #include <err.h> @@ -52,7 +52,7 @@ __quota_kernel_stat(struct quotahandle * struct quotactl_args args; args.qc_op = QUOTACTL_STAT; - args.u.stat.qc_ret = stat; + args.u.stat.qc_info = stat; return __quotactl(qh->qh_mountpoint, &args); } @@ -199,7 +199,7 @@ __quota_kernel_get(struct quotahandle *q args.qc_op = QUOTACTL_GET; args.u.get.qc_key = qk; - args.u.get.qc_ret = qv; + args.u.get.qc_val = qv; return __quotactl(qh->qh_mountpoint, &args); } Index: src/sys/compat/netbsd32/netbsd32.h diff -u src/sys/compat/netbsd32/netbsd32.h:1.90 src/sys/compat/netbsd32/netbsd32.h:1.91 --- src/sys/compat/netbsd32/netbsd32.h:1.90 Wed Feb 1 05:40:00 2012 +++ src/sys/compat/netbsd32/netbsd32.h Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: netbsd32.h,v 1.90 2012/02/01 05:40:00 dholland Exp $ */ +/* $NetBSD: netbsd32.h,v 1.91 2012/02/01 05:43:54 dholland Exp $ */ /* * Copyright (c) 1998, 2001, 2008 Matthew R. Green @@ -274,7 +274,7 @@ struct netbsd32_quotactlargs { unsigned qc_op; union { struct { - netbsd32_pointer_t qc_ret; + netbsd32_pointer_t qc_info; } stat; struct { int qc_idtype; @@ -286,7 +286,7 @@ struct netbsd32_quotactlargs { } objtypestat; struct { netbsd32_pointer_t qc_key; - netbsd32_pointer_t qc_ret; + netbsd32_pointer_t qc_val; } get; struct { netbsd32_pointer_t qc_key; Index: src/sys/compat/netbsd32/netbsd32_netbsd.c diff -u src/sys/compat/netbsd32/netbsd32_netbsd.c:1.178 src/sys/compat/netbsd32/netbsd32_netbsd.c:1.179 --- src/sys/compat/netbsd32/netbsd32_netbsd.c:1.178 Wed Feb 1 05:40:00 2012 +++ src/sys/compat/netbsd32/netbsd32_netbsd.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: netbsd32_netbsd.c,v 1.178 2012/02/01 05:40:00 dholland Exp $ */ +/* $NetBSD: netbsd32_netbsd.c,v 1.179 2012/02/01 05:43:54 dholland Exp $ */ /* * Copyright (c) 1998, 2001, 2008 Matthew R. Green @@ -27,7 +27,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: netbsd32_netbsd.c,v 1.178 2012/02/01 05:40:00 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: netbsd32_netbsd.c,v 1.179 2012/02/01 05:43:54 dholland Exp $"); #if defined(_KERNEL_OPT) #include "opt_ddb.h" @@ -1204,7 +1204,7 @@ netbsd32___quotactl(struct lwp *l, const args.qc_op = args32.qc_op; switch (args.qc_op) { case QUOTACTL_STAT: - args.u.stat.qc_ret = NETBSD32PTR64(args32.u.stat.qc_ret); + args.u.stat.qc_info = NETBSD32PTR64(args32.u.stat.qc_info); break; case QUOTACTL_IDTYPESTAT: args.u.idtypestat.qc_idtype = args32.u.idtypestat.qc_idtype; @@ -1219,7 +1219,7 @@ netbsd32___quotactl(struct lwp *l, const break; case QUOTACTL_GET: args.u.get.qc_key = NETBSD32PTR64(args32.u.get.qc_key); - args.u.get.qc_ret = NETBSD32PTR64(args32.u.get.qc_ret); + args.u.get.qc_val = NETBSD32PTR64(args32.u.get.qc_val); break; case QUOTACTL_PUT: args.u.put.qc_key = NETBSD32PTR64(args32.u.put.qc_key); Index: src/sys/kern/vfs_quotactl.c diff -u src/sys/kern/vfs_quotactl.c:1.37 src/sys/kern/vfs_quotactl.c:1.38 --- src/sys/kern/vfs_quotactl.c:1.37 Wed Feb 1 05:34:41 2012 +++ src/sys/kern/vfs_quotactl.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: vfs_quotactl.c,v 1.37 2012/02/01 05:34:41 dholland Exp $ */ +/* $NetBSD: vfs_quotactl.c,v 1.38 2012/02/01 05:43:54 dholland Exp $ */ /*- * Copyright (c) 2012 The NetBSD Foundation, Inc. * All rights reserved. @@ -29,7 +29,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: vfs_quotactl.c,v 1.37 2012/02/01 05:34:41 dholland Exp $$"); +__KERNEL_RCSID(0, "$NetBSD: vfs_quotactl.c,v 1.38 2012/02/01 05:43:54 dholland Exp $$"); #include <sys/mount.h> #include <sys/quotactl.h> @@ -40,7 +40,7 @@ vfs_quotactl_stat(struct mount *mp, stru struct quotactl_args args; args.qc_op = QUOTACTL_STAT; - args.u.stat.qc_ret = info; + args.u.stat.qc_info = info; return VFS_QUOTACTL(mp, &args); } @@ -76,7 +76,7 @@ vfs_quotactl_get(struct mount *mp, const args.qc_op = QUOTACTL_GET; args.u.get.qc_key = key; - args.u.get.qc_ret = val; + args.u.get.qc_val = val; return VFS_QUOTACTL(mp, &args); } Index: src/sys/kern/vfs_syscalls.c diff -u src/sys/kern/vfs_syscalls.c:1.445 src/sys/kern/vfs_syscalls.c:1.446 --- src/sys/kern/vfs_syscalls.c:1.445 Wed Feb 1 05:39:28 2012 +++ src/sys/kern/vfs_syscalls.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: vfs_syscalls.c,v 1.445 2012/02/01 05:39:28 dholland Exp $ */ +/* $NetBSD: vfs_syscalls.c,v 1.446 2012/02/01 05:43:54 dholland Exp $ */ /*- * Copyright (c) 2008, 2009 The NetBSD Foundation, Inc. @@ -70,7 +70,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.445 2012/02/01 05:39:28 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: vfs_syscalls.c,v 1.446 2012/02/01 05:43:54 dholland Exp $"); #ifdef _KERNEL_OPT #include "opt_fileassoc.h" @@ -921,7 +921,7 @@ do_sys_quotactl(const char *path_u, cons switch (args->qc_op) { case QUOTACTL_STAT: - error = do_sys_quotactl_stat(mp, args->u.stat.qc_ret); + error = do_sys_quotactl_stat(mp, args->u.stat.qc_info); break; case QUOTACTL_IDTYPESTAT: error = do_sys_quotactl_idtypestat(mp, @@ -936,7 +936,7 @@ do_sys_quotactl(const char *path_u, cons case QUOTACTL_GET: error = do_sys_quotactl_get(mp, args->u.get.qc_key, - args->u.get.qc_ret); + args->u.get.qc_val); break; case QUOTACTL_PUT: error = do_sys_quotactl_put(mp, Index: src/sys/sys/quotactl.h diff -u src/sys/sys/quotactl.h:1.33 src/sys/sys/quotactl.h:1.34 --- src/sys/sys/quotactl.h:1.33 Wed Feb 1 05:34:42 2012 +++ src/sys/sys/quotactl.h Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: quotactl.h,v 1.33 2012/02/01 05:34:42 dholland Exp $ */ +/* $NetBSD: quotactl.h,v 1.34 2012/02/01 05:43:54 dholland Exp $ */ /*- * Copyright (c) 2011 The NetBSD Foundation, Inc. * All rights reserved. @@ -98,7 +98,7 @@ struct quotactl_args { unsigned qc_op; union { struct { - struct quotastat *qc_ret; + struct quotastat *qc_info; } stat; struct { int qc_idtype; @@ -110,7 +110,7 @@ struct quotactl_args { } objtypestat; struct { const struct quotakey *qc_key; - struct quotaval *qc_ret; + struct quotaval *qc_val; } get; struct { const struct quotakey *qc_key; Index: src/sys/ufs/ufs/ufs_quota.c diff -u src/sys/ufs/ufs/ufs_quota.c:1.107 src/sys/ufs/ufs/ufs_quota.c:1.108 --- src/sys/ufs/ufs/ufs_quota.c:1.107 Wed Feb 1 05:34:43 2012 +++ src/sys/ufs/ufs/ufs_quota.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: ufs_quota.c,v 1.107 2012/02/01 05:34:43 dholland Exp $ */ +/* $NetBSD: ufs_quota.c,v 1.108 2012/02/01 05:43:54 dholland Exp $ */ /* * Copyright (c) 1982, 1986, 1990, 1993, 1995 @@ -35,7 +35,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: ufs_quota.c,v 1.107 2012/02/01 05:34:43 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: ufs_quota.c,v 1.108 2012/02/01 05:43:54 dholland Exp $"); #if defined(_KERNEL_OPT) #include "opt_quota.h" @@ -227,32 +227,32 @@ quota_handle_cmd_stat(struct mount *mp, struct quotactl_args *args) { struct ufsmount *ump = VFSTOUFS(mp); - struct quotastat *ret; + struct quotastat *info; KASSERT(args->qc_op == QUOTACTL_STAT); - ret = args->u.stat.qc_ret; + info = args->u.stat.qc_info; if ((ump->um_flags & (UFS_QUOTA|UFS_QUOTA2)) == 0) return EOPNOTSUPP; #ifdef QUOTA if (ump->um_flags & UFS_QUOTA) { - strcpy(ret->qs_implname, "ufs/ffs quota v1"); - ret->qs_numidtypes = MAXQUOTAS; + strcpy(info->qs_implname, "ufs/ffs quota v1"); + info->qs_numidtypes = MAXQUOTAS; /* XXX no define for this */ - ret->qs_numobjtypes = 2; - ret->qs_restrictions = 0; - ret->qs_restrictions |= QUOTA_RESTRICT_NEEDSQUOTACHECK; - ret->qs_restrictions |= QUOTA_RESTRICT_UNIFORMGRACE; - ret->qs_restrictions |= QUOTA_RESTRICT_32BIT; + info->qs_numobjtypes = 2; + info->qs_restrictions = 0; + info->qs_restrictions |= QUOTA_RESTRICT_NEEDSQUOTACHECK; + info->qs_restrictions |= QUOTA_RESTRICT_UNIFORMGRACE; + info->qs_restrictions |= QUOTA_RESTRICT_32BIT; } else #endif #ifdef QUOTA2 if (ump->um_flags & UFS_QUOTA2) { - strcpy(ret->qs_implname, "ufs/ffs quota v2"); - ret->qs_numidtypes = MAXQUOTAS; - ret->qs_numobjtypes = N_QL; - ret->qs_restrictions = 0; + strcpy(info->qs_implname, "ufs/ffs quota v2"); + info->qs_numidtypes = MAXQUOTAS; + info->qs_numobjtypes = N_QL; + info->qs_restrictions = 0; } else #endif return EOPNOTSUPP; @@ -347,11 +347,11 @@ quota_handle_cmd_get(struct mount *mp, s struct ufsmount *ump = VFSTOUFS(mp); int error; const struct quotakey *qk; - struct quotaval *ret; + struct quotaval *qv; KASSERT(args->qc_op == QUOTACTL_GET); qk = args->u.get.qc_key; - ret = args->u.get.qc_ret; + qv = args->u.get.qc_val; if ((ump->um_flags & (UFS_QUOTA|UFS_QUOTA2)) == 0) return EOPNOTSUPP; @@ -361,12 +361,12 @@ quota_handle_cmd_get(struct mount *mp, s return error; #ifdef QUOTA if (ump->um_flags & UFS_QUOTA) { - error = quota1_handle_cmd_get(ump, qk, ret); + error = quota1_handle_cmd_get(ump, qk, qv); } else #endif #ifdef QUOTA2 if (ump->um_flags & UFS_QUOTA2) { - error = quota2_handle_cmd_get(ump, qk, ret); + error = quota2_handle_cmd_get(ump, qk, qv); } else #endif panic("quota_handle_cmd_get: no support ?"); Index: src/sys/ufs/ufs/ufs_quota1.c diff -u src/sys/ufs/ufs/ufs_quota1.c:1.16 src/sys/ufs/ufs/ufs_quota1.c:1.17 --- src/sys/ufs/ufs/ufs_quota1.c:1.16 Sun Jan 29 07:16:53 2012 +++ src/sys/ufs/ufs/ufs_quota1.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: ufs_quota1.c,v 1.16 2012/01/29 07:16:53 dholland Exp $ */ +/* $NetBSD: ufs_quota1.c,v 1.17 2012/02/01 05:43:54 dholland Exp $ */ /* * Copyright (c) 1982, 1986, 1990, 1993, 1995 @@ -35,7 +35,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: ufs_quota1.c,v 1.16 2012/01/29 07:16:53 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: ufs_quota1.c,v 1.17 2012/02/01 05:43:54 dholland Exp $"); #include <sys/param.h> #include <sys/kernel.h> @@ -493,7 +493,7 @@ again: int quota1_handle_cmd_get(struct ufsmount *ump, const struct quotakey *qk, - struct quotaval *ret) + struct quotaval *qv) { struct dquot *dq; int error; @@ -530,10 +530,10 @@ quota1_handle_cmd_get(struct ufsmount *u switch (qk->qk_objtype) { case QUOTA_OBJTYPE_BLOCKS: - *ret = blocks; + *qv = blocks; break; case QUOTA_OBJTYPE_FILES: - *ret = files; + *qv = files; break; default: return EINVAL; Index: src/sys/ufs/ufs/ufs_quota2.c diff -u src/sys/ufs/ufs/ufs_quota2.c:1.31 src/sys/ufs/ufs/ufs_quota2.c:1.32 --- src/sys/ufs/ufs/ufs_quota2.c:1.31 Wed Feb 1 05:10:44 2012 +++ src/sys/ufs/ufs/ufs_quota2.c Wed Feb 1 05:43:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: ufs_quota2.c,v 1.31 2012/02/01 05:10:44 dholland Exp $ */ +/* $NetBSD: ufs_quota2.c,v 1.32 2012/02/01 05:43:54 dholland Exp $ */ /*- * Copyright (c) 2010 Manuel Bouyer * All rights reserved. @@ -26,7 +26,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: ufs_quota2.c,v 1.31 2012/02/01 05:10:44 dholland Exp $"); +__KERNEL_RCSID(0, "$NetBSD: ufs_quota2.c,v 1.32 2012/02/01 05:43:54 dholland Exp $"); #include <sys/buf.h> #include <sys/param.h> @@ -867,7 +867,7 @@ quota2_fetch_quotaval(struct ufsmount *u int quota2_handle_cmd_get(struct ufsmount *ump, const struct quotakey *qk, - struct quotaval *ret) + struct quotaval *qv) { int error; struct quota2_header *q2h; @@ -902,10 +902,10 @@ quota2_handle_cmd_get(struct ufsmount *u mutex_exit(&dqlock); brelse(bp, 0); q2e_to_quotaval(&q2e, qk->qk_id == QUOTA_DEFAULTID, &id2, - qk->qk_objtype, ret); + qk->qk_objtype, qv); (void)id2; } else - error = quota2_fetch_quotaval(ump, qk, ret); + error = quota2_fetch_quotaval(ump, qk, qv); return error; }