From patchwork Tue Aug 16 03:17:19 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Eric Sandeen X-Patchwork-Id: 9282519 Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id DEC8E600CB for ; Tue, 16 Aug 2016 03:17:32 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id D24DA28C9D for ; Tue, 16 Aug 2016 03:17:32 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id C6E3528CA9; Tue, 16 Aug 2016 03:17:32 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-4.2 required=2.0 tests=BAYES_00, RCVD_IN_DNSWL_MED autolearn=ham version=3.3.1 Received: from oss.sgi.com (oss.sgi.com [192.48.182.195]) (using TLSv1 with cipher DHE-RSA-CAMELLIA256-SHA (256/256 bits)) (No client certificate requested) by mail.wl.linuxfoundation.org (Postfix) with ESMTPS id 2DA5128C9D for ; Tue, 16 Aug 2016 03:17:32 +0000 (UTC) Received: from oss.sgi.com (localhost [IPv6:::1]) by oss.sgi.com (Postfix) with ESMTP id 6EFB97CA3; Mon, 15 Aug 2016 22:17:30 -0500 (CDT) X-Original-To: xfs@oss.sgi.com Delivered-To: xfs@oss.sgi.com Received: from relay.sgi.com (relay3.corp.sgi.com [198.149.34.15]) by oss.sgi.com (Postfix) with ESMTP id 00BC67CA2 for ; Mon, 15 Aug 2016 22:17:28 -0500 (CDT) Received: from cuda.sgi.com (cuda1.sgi.com [192.48.157.11]) by relay3.corp.sgi.com (Postfix) with ESMTP id 793B1AC05A for ; Mon, 15 Aug 2016 20:17:24 -0700 (PDT) X-ASG-Debug-ID: 1471317440-0bf81563e70dc80001-NocioJ Received: from sandeen.net (sandeen.net [63.231.237.45]) by cuda.sgi.com with ESMTP id 8ElwmUSCxXpvhUrj for ; Mon, 15 Aug 2016 20:17:20 -0700 (PDT) X-Barracuda-Envelope-From: sandeen@sandeen.net X-Barracuda-Effective-Source-IP: sandeen.net[63.231.237.45] X-Barracuda-Apparent-Source-IP: 63.231.237.45 Received: from [10.0.0.4] (liberator [10.0.0.4]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by sandeen.net (Postfix) with ESMTPSA id 36DE74454 for ; Mon, 15 Aug 2016 22:17:20 -0500 (CDT) Subject: [PATCH V2] xfs_quota: wire up XFS_GETQSTATV To: xfs@oss.sgi.com X-ASG-Orig-Subj: [PATCH V2] xfs_quota: wire up XFS_GETQSTATV References: From: Eric Sandeen Message-ID: <2ca2dc3b-53d2-0791-644f-c9cd5abef89f@sandeen.net> Date: Mon, 15 Aug 2016 22:17:19 -0500 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.11; rv:45.0) Gecko/20100101 Thunderbird/45.2.0 MIME-Version: 1.0 In-Reply-To: X-Barracuda-Connect: sandeen.net[63.231.237.45] X-Barracuda-Start-Time: 1471317440 X-Barracuda-URL: https://192.48.157.11:443/cgi-mod/mark.cgi X-Barracuda-Scan-Msg-Size: 8274 X-Virus-Scanned: by bsmtpd at sgi.com X-Barracuda-BRTS-Status: 1 X-Barracuda-Spam-Score: 0.00 X-Barracuda-Spam-Status: No, SCORE=0.00 using per-user scores of TAG_LEVEL=1000.0 QUARANTINE_LEVEL=1000.0 KILL_LEVEL=2.7 tests= X-Barracuda-Spam-Report: Code version 3.2, rules version 3.2.3.32042 Rule breakdown below pts rule name description ---- ---------------------- -------------------------------------------------- X-BeenThere: xfs@oss.sgi.com X-Mailman-Version: 2.1.14 Precedence: list List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com X-Virus-Scanned: ClamAV using ClamSMTP The new XFS_GETQSTATV quotactl, available since kernel v3.12, was never implemented in xfs_quota, and the "state" command continues to use XFS_GETQSTAT, which cannot report both group & project quota on newer formats. The new call has room for all 3 quota types (user, group, and quota), vs just two, where previously project and quota overlapped. So: First, try XFS_GETQSTATV. If it passes, we have all the information we need, and we print it. state_qfilestat() is modified to take the newer structure. If it fails, try XFS_GETQSTAT. If that passes, we are on an older kernel with neither XFS_GETQSTATV nor the on-disk project quota inode. We copy the available information into the newer statv structure, carefully determining wither group or project (or neither) is actually active, and print it with the same state_qfilestat routine. Signed-off-by: Eric Sandeen Reviewed-by: Zorro Lang --- I probably could have done some memcpy()'s in state_stat_to_statv(), but opted for the explicit copy-out; the structures aren't identical, although the newer one only differs by padding on the end. If memcpy() is preferable I could send a V2... V2: set sv.qs_version = FS_QSTATV_VERSION1; before calling the quotactl (thanks Zorro!) diff --git a/include/xqm.h b/include/xqm.h index c084b2d..5b6934a 100644 --- a/include/xqm.h +++ b/include/xqm.h @@ -32,6 +32,7 @@ #define Q_XGETQSTAT XQM_CMD(5) /* get quota subsystem status */ #define Q_XQUOTARM XQM_CMD(6) /* free disk space used by dquots */ #define Q_XQUOTASYNC XQM_CMD(7) /* delalloc flush, updates dquots */ +#define Q_XGETQSTATV XQM_CMD(8) /* newer version of get quota */ #define Q_XGETNEXTQUOTA XQM_CMD(9) /* get disk limits and usage */ /* @@ -149,4 +150,35 @@ typedef struct fs_quota_stat { __u16 qs_iwarnlimit; /* limit for num warnings */ } fs_quota_stat_t; + +#ifndef FS_QSTATV_VERSION1 +#define FS_QSTATV_VERSION1 1 /* fs_quota_statv.qs_version */ +#endif + +/* + * Some basic information about 'quota files' for Q_XGETQSTATV command + */ +struct fs_qfilestatv { + __u64 qfs_ino; /* inode number */ + __u64 qfs_nblks; /* number of BBs 512-byte-blks */ + __u32 qfs_nextents; /* number of extents */ + __u32 qfs_pad; /* pad for 8-byte alignment */ +}; + +struct fs_quota_statv { + __s8 qs_version; /* version for future changes */ + __u8 qs_pad1; /* pad for 16bit alignment */ + __u16 qs_flags; /* FS_QUOTA_.* flags */ + __u32 qs_incoredqs; /* number of dquots incore */ + struct fs_qfilestatv qs_uquota; /* user quota information */ + struct fs_qfilestatv qs_gquota; /* group quota information */ + struct fs_qfilestatv qs_pquota; /* project quota information */ + __s32 qs_btimelimit; /* limit for blks timer */ + __s32 qs_itimelimit; /* limit for inodes timer */ + __s32 qs_rtbtimelimit;/* limit for rt blks timer */ + __u16 qs_bwarnlimit; /* limit for num warnings */ + __u16 qs_iwarnlimit; /* limit for num warnings */ + __u64 qs_pad2[8]; /* for future proofing */ +}; + #endif /* __XQM_H__ */ diff --git a/quota/linux.c b/quota/linux.c index 74dba01..4f1f3c4 100644 --- a/quota/linux.c +++ b/quota/linux.c @@ -55,6 +55,8 @@ xcommand_to_qcommand( return Q_XSETQLIM; case XFS_GETQSTAT: return Q_XGETQSTAT; + case XFS_GETQSTATV: + return Q_XGETQSTATV; case XFS_QUOTARM: return Q_XQUOTARM; case XFS_QSYNC: diff --git a/quota/state.c b/quota/state.c index 8186762..9f6616e 100644 --- a/quota/state.c +++ b/quota/state.c @@ -111,12 +111,12 @@ remove_help(void) static void state_qfilestat( - FILE *fp, - fs_path_t *mount, - uint type, - fs_qfilestat_t *qfs, - int accounting, - int enforcing) + FILE *fp, + struct fs_path *mount, + uint type, + struct fs_qfilestatv *qfs, + int accounting, + int enforcing) { fprintf(fp, _("%s quota state on %s (%s)\n"), type_to_string(type), mount->fs_dir, mount->fs_name); @@ -142,39 +142,96 @@ state_timelimit( time_to_string(timelimit, VERBOSE_FLAG | ABSOLUTE_FLAG)); } +/* + * fs_quota_stat holds a subset of fs_quota_statv; this copies + * the smaller into the larger, leaving any not-present fields + * empty. This is so the same reporting function can be used + * for both XFS_GETQSTAT and XFS_GETQSTATV results. + */ static void -state_quotafile_mount( - FILE *fp, - uint type, - fs_path_t *mount, - uint flags) +state_stat_to_statv( + struct fs_quota_stat *s, + struct fs_quota_statv *sv) { - fs_quota_stat_t s; - char *dev = mount->fs_name; + memset(sv, 0, sizeof(struct fs_quota_statv)); + + /* shared information */ + sv->qs_version = s->qs_version; + sv->qs_flags = s->qs_flags; + sv->qs_incoredqs = s->qs_incoredqs; + sv->qs_btimelimit = s->qs_btimelimit; + sv->qs_itimelimit = s->qs_itimelimit; + sv->qs_rtbtimelimit = s->qs_rtbtimelimit; + sv->qs_bwarnlimit = s->qs_bwarnlimit; + sv->qs_iwarnlimit = s->qs_iwarnlimit; + + /* Always room for uquota */ + sv->qs_uquota.qfs_ino = s->qs_uquota.qfs_ino; + sv->qs_uquota.qfs_nblks = s->qs_uquota.qfs_nblks; + sv->qs_uquota.qfs_nextents = s->qs_uquota.qfs_nextents; + + /* + * If we are here, XFS_GETQSTATV failed and XFS_GETQSTAT passed; + * that is a very strong hint that we're on a kernel which predates + * the on-disk pquota inode; both were added in v3.12. So, we do + * some tricksy determination here. + * gs_gquota may hold either group quota inode info, or project + * quota if that is used instead; which one it actually holds depends + * on the quota flags. (If neither is set, neither is used) + */ + if (s->qs_flags & XFS_QUOTA_GDQ_ACCT) { + /* gs_gquota holds group quota info */ + sv->qs_gquota.qfs_ino = s->qs_gquota.qfs_ino; + sv->qs_gquota.qfs_nblks = s->qs_gquota.qfs_nblks; + sv->qs_gquota.qfs_nextents = s->qs_gquota.qfs_nextents; + } else if (s->qs_flags & XFS_QUOTA_PDQ_ACCT) { + /* gs_gquota actually holds project quota info */ + sv->qs_pquota.qfs_ino = s->qs_gquota.qfs_ino; + sv->qs_pquota.qfs_nblks = s->qs_gquota.qfs_nblks; + sv->qs_pquota.qfs_nextents = s->qs_gquota.qfs_nextents; + } +} - if (xfsquotactl(XFS_GETQSTAT, dev, type, 0, (void *)&s) < 0) { - if (flags & VERBOSE_FLAG) - fprintf(fp, _("%s quota are not enabled on %s\n"), - type_to_string(type), dev); - return; +static void +state_quotafile_mount( + FILE *fp, + uint type, + struct fs_path *mount, + uint flags) +{ + struct fs_quota_stat s; + struct fs_quota_statv sv; + char *dev = mount->fs_name; + + sv.qs_version = FS_QSTATV_VERSION1; + + if (xfsquotactl(XFS_GETQSTATV, dev, type, 0, (void *)&sv) < 0) { + if (xfsquotactl(XFS_GETQSTAT, dev, type, 0, (void *)&s) < 0) { + if (flags & VERBOSE_FLAG) + fprintf(fp, + _("%s quota are not enabled on %s\n"), + type_to_string(type), dev); + return; + } + state_stat_to_statv(&s, &sv); } if (type & XFS_USER_QUOTA) - state_qfilestat(fp, mount, XFS_USER_QUOTA, &s.qs_uquota, - s.qs_flags & XFS_QUOTA_UDQ_ACCT, - s.qs_flags & XFS_QUOTA_UDQ_ENFD); + state_qfilestat(fp, mount, XFS_USER_QUOTA, &sv.qs_uquota, + sv.qs_flags & XFS_QUOTA_UDQ_ACCT, + sv.qs_flags & XFS_QUOTA_UDQ_ENFD); if (type & XFS_GROUP_QUOTA) - state_qfilestat(fp, mount, XFS_GROUP_QUOTA, &s.qs_gquota, - s.qs_flags & XFS_QUOTA_GDQ_ACCT, - s.qs_flags & XFS_QUOTA_GDQ_ENFD); + state_qfilestat(fp, mount, XFS_GROUP_QUOTA, &sv.qs_gquota, + sv.qs_flags & XFS_QUOTA_GDQ_ACCT, + sv.qs_flags & XFS_QUOTA_GDQ_ENFD); if (type & XFS_PROJ_QUOTA) - state_qfilestat(fp, mount, XFS_PROJ_QUOTA, &s.qs_gquota, - s.qs_flags & XFS_QUOTA_PDQ_ACCT, - s.qs_flags & XFS_QUOTA_PDQ_ENFD); + state_qfilestat(fp, mount, XFS_PROJ_QUOTA, &sv.qs_pquota, + sv.qs_flags & XFS_QUOTA_PDQ_ACCT, + sv.qs_flags & XFS_QUOTA_PDQ_ENFD); - state_timelimit(fp, XFS_BLOCK_QUOTA, s.qs_btimelimit); - state_timelimit(fp, XFS_INODE_QUOTA, s.qs_itimelimit); - state_timelimit(fp, XFS_RTBLOCK_QUOTA, s.qs_rtbtimelimit); + state_timelimit(fp, XFS_BLOCK_QUOTA, sv.qs_btimelimit); + state_timelimit(fp, XFS_INODE_QUOTA, sv.qs_itimelimit); + state_timelimit(fp, XFS_RTBLOCK_QUOTA, sv.qs_rtbtimelimit); } static void