]> granicus.if.org Git - strace/blobdiff - quota.c
Remove XLAT_END
[strace] / quota.c
diff --git a/quota.c b/quota.c
index c28480f4baa980af382117bbe4ccdbbf912273b9..e4073a21a581411ba7dd8f01a53bd1a51e21f8cd 100644 (file)
--- a/quota.c
+++ b/quota.c
@@ -4,32 +4,15 @@
  * Copyright (c) 1993, 1994, 1995, 1996 Rick Sladkey <jrs@world.std.com>
  * Copyright (c) 1996-1999 Wichert Akkerman <wichert@cistron.nl>
  * Copyright (c) 2005-2016 Dmitry V. Levin <ldv@altlinux.org>
+ * Copyright (c) 2006-2018 The strace developers.
  * All rights reserved.
  *
- * Redistribution and use in source and binary forms, with or without
- * modification, are permitted provided that the following conditions
- * are met:
- * 1. Redistributions of source code must retain the above copyright
- *    notice, this list of conditions and the following disclaimer.
- * 2. Redistributions in binary form must reproduce the above copyright
- *    notice, this list of conditions and the following disclaimer in the
- *    documentation and/or other materials provided with the distribution.
- * 3. The name of the author may not be used to endorse or promote products
- *    derived from this software without specific prior written permission.
- *
- * THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS OR
- * IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED WARRANTIES
- * OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE DISCLAIMED.
- * IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY DIRECT, INDIRECT,
- * INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT
- * NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- * DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- * THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
- * THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ * SPDX-License-Identifier: LGPL-2.1-or-later
  */
 
 #include "defs.h"
+#include "print_fields.h"
+#include "xfs_quota_stat.h"
 
 #define SUBCMDMASK  0x00ff
 #define SUBCMDSHIFT 8
@@ -55,8 +38,7 @@
  * leads to problems when it is used on 32-bit tracee which does not have such
  * padding.
  */
-struct if_dqblk
-{
+struct if_dqblk {
        uint64_t dqb_bhardlimit;
        uint64_t dqb_bsoftlimit;
        uint64_t dqb_curspace;
@@ -81,10 +63,9 @@ struct if_nextdqblk {
        uint32_t dqb_id;
 };
 
-struct xfs_dqblk
-{
+struct xfs_dqblk {
        int8_t  d_version;              /* version of this structure */
-       int8_t  d_flags;                /* XFS_{USER,PROJ,GROUP}_QUOTA */
+       uint8_t  d_flags;               /* XFS_{USER,PROJ,GROUP}_QUOTA */
        uint16_t d_fieldmask;           /* field specifier */
        uint32_t d_id;                  /* user, project, or group ID */
        uint64_t d_blk_hardlimit;       /* absolute limit on disk blks */
@@ -107,36 +88,13 @@ struct xfs_dqblk
        char    d_padding4[8];          /* yet more padding */
 };
 
-struct if_dqinfo
-{
+struct if_dqinfo {
        uint64_t dqi_bgrace;
        uint64_t dqi_igrace;
        uint32_t dqi_flags;
        uint32_t dqi_valid;
 };
 
-typedef struct fs_qfilestat
-{
-       uint64_t qfs_ino;       /* inode number */
-       uint64_t qfs_nblks;     /* number of BBs 512-byte-blks */
-       uint32_t qfs_nextents;  /* number of extents */
-} fs_qfilestat_t;
-
-struct xfs_dqstats
-{
-       int8_t  qs_version;             /* version number for future changes */
-       uint16_t qs_flags;              /* XFS_QUOTA_{U,P,G}DQ_{ACCT,ENFD} */
-       int8_t  qs_pad;                 /* unused */
-       fs_qfilestat_t qs_uquota;       /* user quota storage information */
-       fs_qfilestat_t qs_gquota;       /* group quota storage information */
-       uint32_t qs_incoredqs;          /* number of dquots incore */
-       int32_t qs_btimelimit;          /* limit for blks timer */
-       int32_t qs_itimelimit;          /* limit for inodes timer */
-       int32_t qs_rtbtimelimit;        /* limit for rt blks timer */
-       uint16_t qs_bwarnlimit;         /* limit for num warnings */
-       uint16_t qs_iwarnlimit;         /* limit for num warnings */
-};
-
 struct fs_qfilestatv {
        uint64_t qfs_ino, qfs_nblks;
        uint32_t qfs_nextents, qfs_pad;
@@ -159,13 +117,19 @@ struct fs_quota_statv {
 };
 
 static int
-decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
+decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, kernel_ulong_t data)
 {
        switch (cmd) {
        case Q_QUOTAOFF:
        case Q_SYNC:
        case Q_XQUOTASYNC:
                break;
+       case Q_QUOTAON:
+               tprints(", ");
+               printxval(quota_formats, id, "QFMT_VFS_???");
+               tprints(", ");
+               printpath(tcp, data);
+               break;
        case Q_GETQUOTA:
                if (entering(tcp)) {
                        printuid(", ", id);
@@ -174,7 +138,7 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
                        return 0;
                }
 
-               /* Fall-through */
+               ATTRIBUTE_FALLTHROUGH;
        case Q_SETQUOTA:
        {
                struct if_dqblk dq;
@@ -186,21 +150,21 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                if (umove_or_printaddr(tcp, data, &dq))
                        break;
-               tprintf("{dqb_bhardlimit=%" PRIu64 ", ", dq.dqb_bhardlimit);
-               tprintf("dqb_bsoftlimit=%" PRIu64 ", ", dq.dqb_bsoftlimit);
-               tprintf("dqb_curspace=%" PRIu64 ", ", dq.dqb_curspace);
-               tprintf("dqb_ihardlimit=%" PRIu64 ", ", dq.dqb_ihardlimit);
-               tprintf("dqb_isoftlimit=%" PRIu64 ", ", dq.dqb_isoftlimit);
-               tprintf("dqb_curinodes=%" PRIu64 ", ", dq.dqb_curinodes);
+               PRINT_FIELD_U("{", dq, dqb_bhardlimit);
+               PRINT_FIELD_U(", ", dq, dqb_bsoftlimit);
+               PRINT_FIELD_U(", ", dq, dqb_curspace);
+               PRINT_FIELD_U(", ", dq, dqb_ihardlimit);
+               PRINT_FIELD_U(", ", dq, dqb_isoftlimit);
+               PRINT_FIELD_U(", ", dq, dqb_curinodes);
                if (!abbrev(tcp)) {
-                       tprintf("dqb_btime=%" PRIu64 ", ", dq.dqb_btime);
-                       tprintf("dqb_itime=%" PRIu64 ", ", dq.dqb_itime);
-                       tprints("dqb_valid=");
-                       printflags(if_dqblk_valid,
-                                  dq.dqb_valid, "QIF_???");
-                       tprints("}");
-               } else
-                       tprints("...}");
+                       PRINT_FIELD_U(", ", dq, dqb_btime);
+                       PRINT_FIELD_U(", ", dq, dqb_itime);
+                       PRINT_FIELD_FLAGS(", ", dq, dqb_valid,
+                                         if_dqblk_valid, "QIF_???");
+               } else {
+                       tprints(", ...");
+               }
+               tprints("}");
                break;
        }
        case Q_GETNEXTQUOTA:
@@ -216,21 +180,23 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                if (umove_or_printaddr(tcp, data, &dq))
                        break;
-               tprintf("{dqb_bhardlimit=%" PRIu64 ", ", dq.dqb_bhardlimit);
-               tprintf("dqb_bsoftlimit=%" PRIu64 ", ", dq.dqb_bsoftlimit);
-               tprintf("dqb_curspace=%" PRIu64 ", ", dq.dqb_curspace);
-               tprintf("dqb_ihardlimit=%" PRIu64 ", ", dq.dqb_ihardlimit);
-               tprintf("dqb_isoftlimit=%" PRIu64 ", ", dq.dqb_isoftlimit);
-               tprintf("dqb_curinodes=%" PRIu64 ", ", dq.dqb_curinodes);
+               PRINT_FIELD_U("{", dq, dqb_bhardlimit);
+               PRINT_FIELD_U(", ", dq, dqb_bsoftlimit);
+               PRINT_FIELD_U(", ", dq, dqb_curspace);
+               PRINT_FIELD_U(", ", dq, dqb_ihardlimit);
+               PRINT_FIELD_U(", ", dq, dqb_isoftlimit);
+               PRINT_FIELD_U(", ", dq, dqb_curinodes);
                if (!abbrev(tcp)) {
-                       tprintf("dqb_btime=%" PRIu64 ", ", dq.dqb_btime);
-                       tprintf("dqb_itime=%" PRIu64 ", ", dq.dqb_itime);
-                       tprints("dqb_valid=");
-                       printflags(if_dqblk_valid,
-                                  dq.dqb_valid, "QIF_???");
-                       tprintf(", dqb_id=%u}", dq.dqb_id);
-               } else
-                       tprintf("dqb_id=%u, ...}", dq.dqb_id);
+                       PRINT_FIELD_U(", ", dq, dqb_btime);
+                       PRINT_FIELD_U(", ", dq, dqb_itime);
+                       PRINT_FIELD_FLAGS(", ", dq, dqb_valid,
+                                         if_dqblk_valid, "QIF_???");
+                       PRINT_FIELD_U(", ", dq, dqb_id);
+               } else {
+                       PRINT_FIELD_U(", ", dq, dqb_id);
+                       tprints(", ...");
+               }
+               tprints("}");
                break;
        }
        case Q_XGETQUOTA:
@@ -242,7 +208,7 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
                        return 0;
                }
 
-               /* Fall-through */
+               ATTRIBUTE_FALLTHROUGH;
        case Q_XSETQLIM:
        {
                struct xfs_dqblk dq;
@@ -254,32 +220,31 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                if (umove_or_printaddr(tcp, data, &dq))
                        break;
-               tprintf("{d_version=%d, ", dq.d_version);
-               tprints("d_flags=");
-               printflags(xfs_dqblk_flags,
-                          (uint8_t) dq.d_flags, "XFS_???_QUOTA");
-               tprintf(", d_fieldmask=%#x, ", dq.d_fieldmask);
-               tprintf("d_id=%u, ", dq.d_id);
-               tprintf("d_blk_hardlimit=%" PRIu64 ", ", dq.d_blk_hardlimit);
-               tprintf("d_blk_softlimit=%" PRIu64 ", ", dq.d_blk_softlimit);
-               tprintf("d_ino_hardlimit=%" PRIu64 ", ", dq.d_ino_hardlimit);
-               tprintf("d_ino_softlimit=%" PRIu64 ", ", dq.d_ino_softlimit);
-               tprintf("d_bcount=%" PRIu64 ", ", dq.d_bcount);
-               tprintf("d_icount=%" PRIu64 ", ", dq.d_icount);
+               PRINT_FIELD_D("{", dq, d_version);
+               PRINT_FIELD_FLAGS(", ", dq, d_flags,
+                                 xfs_dqblk_flags, "XFS_???_QUOTA");
+               PRINT_FIELD_X(", ", dq, d_fieldmask);
+               PRINT_FIELD_U(", ", dq, d_id);
+               PRINT_FIELD_U(", ", dq, d_blk_hardlimit);
+               PRINT_FIELD_U(", ", dq, d_blk_softlimit);
+               PRINT_FIELD_U(", ", dq, d_ino_hardlimit);
+               PRINT_FIELD_U(", ", dq, d_ino_softlimit);
+               PRINT_FIELD_U(", ", dq, d_bcount);
+               PRINT_FIELD_U(", ", dq, d_icount);
                if (!abbrev(tcp)) {
-                       tprintf("d_itimer=%d, ", dq.d_itimer);
-                       tprintf("d_btimer=%d, ", dq.d_btimer);
-                       tprintf("d_iwarns=%u, ", dq.d_iwarns);
-                       tprintf("d_bwarns=%u, ", dq.d_bwarns);
-                       tprintf("d_rtb_hardlimit=%" PRIu64 ", ",
-                               dq.d_rtb_hardlimit);
-                       tprintf("d_rtb_softlimit=%" PRIu64 ", ",
-                               dq.d_rtb_softlimit);
-                       tprintf("d_rtbcount=%" PRIu64 ", ", dq.d_rtbcount);
-                       tprintf("d_rtbtimer=%d, ", dq.d_rtbtimer);
-                       tprintf("d_rtbwarns=%u}", dq.d_rtbwarns);
-               } else
-                       tprints("...}");
+                       PRINT_FIELD_D(", ", dq, d_itimer);
+                       PRINT_FIELD_D(", ", dq, d_btimer);
+                       PRINT_FIELD_U(", ", dq, d_iwarns);
+                       PRINT_FIELD_U(", ", dq, d_bwarns);
+                       PRINT_FIELD_U(", ", dq, d_rtb_hardlimit);
+                       PRINT_FIELD_U(", ", dq, d_rtb_softlimit);
+                       PRINT_FIELD_U(", ", dq, d_rtbcount);
+                       PRINT_FIELD_D(", ", dq, d_rtbtimer);
+                       PRINT_FIELD_U(", ", dq, d_rtbwarns);
+               } else {
+                       tprints(", ...");
+               }
+               tprints("}");
                break;
        }
        case Q_GETFMT:
@@ -306,7 +271,7 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
                        return 0;
                }
 
-               /* Fall-through */
+               ATTRIBUTE_FALLTHROUGH;
        case Q_SETINFO:
        {
                struct if_dqinfo dq;
@@ -316,12 +281,10 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                if (umove_or_printaddr(tcp, data, &dq))
                        break;
-               tprintf("{dqi_bgrace=%" PRIu64 ", ", dq.dqi_bgrace);
-               tprintf("dqi_igrace=%" PRIu64 ", ", dq.dqi_igrace);
-               tprints("dqi_flags=");
-               printflags(if_dqinfo_flags, dq.dqi_flags, "DQF_???");
-               tprints(", dqi_valid=");
-               printflags(if_dqinfo_valid, dq.dqi_valid, "IIF_???");
+               PRINT_FIELD_U("{", dq, dqi_bgrace);
+               PRINT_FIELD_U(", ", dq, dqi_igrace);
+               PRINT_FIELD_FLAGS(", ", dq, dqi_flags, if_dqinfo_flags, "DQF_???");
+               PRINT_FIELD_FLAGS(", ", dq, dqi_valid, if_dqinfo_valid, "IIF_???");
                tprints("}");
                break;
        }
@@ -334,31 +297,28 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                        return 0;
                }
-
-               if (umove_or_printaddr(tcp, data, &dq))
-                       break;
-               tprintf("{qs_version=%d, ", dq.qs_version);
-               if (abbrev(tcp)) {
-                       tprints("...}");
-                       break;
+               if (fetch_struct_quotastat(tcp, data, &dq)) {
+                       PRINT_FIELD_D("{", dq, qs_version);
+                       if (!abbrev(tcp)) {
+                               PRINT_FIELD_FLAGS(", ", dq, qs_flags,
+                                                 xfs_quota_flags, "XFS_QUOTA_???");
+                               PRINT_FIELD_U(", qs_uquota={", dq.qs_uquota, qfs_ino);
+                               PRINT_FIELD_U(", ", dq.qs_uquota, qfs_nblks);
+                               PRINT_FIELD_U(", ", dq.qs_uquota, qfs_nextents);
+                               PRINT_FIELD_U("}, qs_gquota={", dq.qs_gquota, qfs_ino);
+                               PRINT_FIELD_U(", ", dq.qs_gquota, qfs_nblks);
+                               PRINT_FIELD_U(", ", dq.qs_gquota, qfs_nextents);
+                               PRINT_FIELD_U("}, ", dq, qs_incoredqs);
+                               PRINT_FIELD_D(", ", dq, qs_btimelimit);
+                               PRINT_FIELD_D(", ", dq, qs_itimelimit);
+                               PRINT_FIELD_D(", ", dq, qs_rtbtimelimit);
+                               PRINT_FIELD_U(", ", dq, qs_bwarnlimit);
+                               PRINT_FIELD_U(", ", dq, qs_iwarnlimit);
+                       } else {
+                               tprints(", ...");
+                       }
+                       tprints("}");
                }
-               tprints("qs_flags=");
-               printflags(xfs_quota_flags,
-                          dq.qs_flags, "XFS_QUOTA_???");
-               tprintf(", qs_incoredqs=%u, ", dq.qs_incoredqs);
-               tprints("qs_uquota={");
-               tprintf("qfs_ino=%" PRIu64 ", ", dq.qs_uquota.qfs_ino);
-               tprintf("qfs_nblks=%" PRIu64 ", ", dq.qs_uquota.qfs_nblks);
-               tprintf("qfs_nextents=%u}, ", dq.qs_uquota.qfs_nextents);
-               tprints("qs_gquota={");
-               tprintf("qfs_ino=%" PRIu64 ", ", dq.qs_gquota.qfs_ino);
-               tprintf("qfs_nblks=%" PRIu64 ", ", dq.qs_gquota.qfs_nblks);
-               tprintf("qfs_nextents=%u}, ", dq.qs_gquota.qfs_nextents);
-               tprintf("qs_btimelimit=%d, ", dq.qs_btimelimit);
-               tprintf("qs_itimelimit=%d, ", dq.qs_itimelimit);
-               tprintf("qs_rtbtimelimit=%d, ", dq.qs_rtbtimelimit);
-               tprintf("qs_bwarnlimit=%u, ", dq.qs_bwarnlimit);
-               tprintf("qs_iwarnlimit=%u}", dq.qs_iwarnlimit);
                break;
        }
        case Q_XGETQSTATV:
@@ -373,32 +333,29 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
 
                if (umove_or_printaddr(tcp, data, &dq))
                        break;
-               tprintf("{qs_version=%d, ", dq.qs_version);
-               if (abbrev(tcp)) {
-                       tprints("...}");
-                       break;
+               PRINT_FIELD_D("{", dq, qs_version);
+               if (!abbrev(tcp)) {
+                       PRINT_FIELD_FLAGS(", ", dq, qs_flags,
+                                         xfs_quota_flags, "XFS_QUOTA_???");
+                       PRINT_FIELD_U(", ", dq, qs_incoredqs);
+                       PRINT_FIELD_U(", qs_uquota={", dq.qs_uquota, qfs_ino);
+                       PRINT_FIELD_U(", ", dq.qs_uquota, qfs_nblks);
+                       PRINT_FIELD_U(", ", dq.qs_uquota, qfs_nextents);
+                       PRINT_FIELD_U("}, qs_gquota={", dq.qs_gquota, qfs_ino);
+                       PRINT_FIELD_U(", ", dq.qs_gquota, qfs_nblks);
+                       PRINT_FIELD_U(", ", dq.qs_gquota, qfs_nextents);
+                       PRINT_FIELD_U("}, qs_pquota={", dq.qs_pquota, qfs_ino);
+                       PRINT_FIELD_U(", ", dq.qs_pquota, qfs_nblks);
+                       PRINT_FIELD_U(", ", dq.qs_pquota, qfs_nextents);
+                       PRINT_FIELD_D("}, ", dq, qs_btimelimit);
+                       PRINT_FIELD_D(", ", dq, qs_itimelimit);
+                       PRINT_FIELD_D(", ", dq, qs_rtbtimelimit);
+                       PRINT_FIELD_U(", ", dq, qs_bwarnlimit);
+                       PRINT_FIELD_U(", ", dq, qs_iwarnlimit);
+               } else {
+                       tprints(", ...");
                }
-               tprints("qs_flags=");
-               printflags(xfs_quota_flags,
-                          dq.qs_flags, "XFS_QUOTA_???");
-               tprintf(", qs_incoredqs=%u, ", dq.qs_incoredqs);
-               tprints("qs_uquota={");
-               tprintf("qfs_ino=%" PRIu64 ", ", dq.qs_uquota.qfs_ino);
-               tprintf("qfs_nblks=%" PRIu64 ", ", dq.qs_uquota.qfs_nblks);
-               tprintf("qfs_nextents=%u}, ", dq.qs_uquota.qfs_nextents);
-               tprints("qs_gquota={");
-               tprintf("qfs_ino=%" PRIu64 ", ", dq.qs_gquota.qfs_ino);
-               tprintf("qfs_nblks=%" PRIu64 ", ", dq.qs_gquota.qfs_nblks);
-               tprintf("qfs_nextents=%u}, ", dq.qs_gquota.qfs_nextents);
-               tprints("qs_pquota={");
-               tprintf("qfs_ino=%" PRIu64 ", ", dq.qs_pquota.qfs_ino);
-               tprintf("qfs_nblks=%" PRIu64 ", ", dq.qs_pquota.qfs_nblks);
-               tprintf("qfs_nextents=%u}, ", dq.qs_pquota.qfs_nextents);
-               tprintf("qs_btimelimit=%d, ", dq.qs_btimelimit);
-               tprintf("qs_itimelimit=%d, ", dq.qs_itimelimit);
-               tprintf("qs_rtbtimelimit=%d, ", dq.qs_rtbtimelimit);
-               tprintf("qs_bwarnlimit=%u, ", dq.qs_bwarnlimit);
-               tprintf("qs_iwarnlimit=%u}", dq.qs_iwarnlimit);
+               tprints("}");
                break;
        }
        case Q_XQUOTAON:
@@ -437,6 +394,31 @@ decode_cmd_data(struct tcb *tcp, uint32_t id, uint32_t cmd, unsigned long data)
        return RVAL_DECODED;
 }
 
+static void
+print_qcmd(const uint32_t qcmd)
+{
+       const uint32_t cmd = QCMD_CMD(qcmd);
+       const uint32_t type = QCMD_TYPE(qcmd);
+
+       if (xlat_verbose(xlat_verbosity) != XLAT_STYLE_ABBREV)
+               tprintf("%u", qcmd);
+
+       if (xlat_verbose(xlat_verbosity) == XLAT_STYLE_RAW)
+               return;
+
+       if (xlat_verbose(xlat_verbosity) == XLAT_STYLE_VERBOSE)
+               tprints(" /* ");
+
+       tprints("QCMD(");
+       printxvals_ex(cmd, "Q_???", XLAT_STYLE_ABBREV, quotacmds, NULL);
+       tprints(", ");
+       printxvals_ex(type, "???QUOTA", XLAT_STYLE_ABBREV, quotatypes, NULL);
+       tprints(")");
+
+       if (xlat_verbose(xlat_verbosity) == XLAT_STYLE_VERBOSE)
+               tprints(" */");
+}
+
 SYS_FUNC(quotactl)
 {
        /*
@@ -447,24 +429,12 @@ SYS_FUNC(quotactl)
         */
        uint32_t qcmd = tcp->u_arg[0];
        uint32_t cmd = QCMD_CMD(qcmd);
-       uint32_t type = QCMD_TYPE(qcmd);
        uint32_t id = tcp->u_arg[2];
 
        if (entering(tcp)) {
-               tprints("QCMD(");
-               printxval(quotacmds, cmd, "Q_???");
+               print_qcmd(qcmd);
                tprints(", ");
-               printxval(quotatypes, type, "???QUOTA");
-               tprints("), ");
                printpath(tcp, tcp->u_arg[1]);
-               switch (cmd) {
-                       case Q_QUOTAON:
-                               tprints(", ");
-                               printxval(quota_formats, id, "QFMT_VFS_???");
-                               tprints(", ");
-                               printpath(tcp, tcp->u_arg[3]);
-                               return RVAL_DECODED;
-               }
        }
        return decode_cmd_data(tcp, id, cmd, tcp->u_arg[3]);
 }