Details

    • 4399

    Description

      Tracker bug for landing orion_quota to master.

      Attachments

        Issue Links

          Activity

            [LU-1842] Quota enforcement landing

            Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33694
            Subject: LU-1842 ptlrpc: clean up ptlrpc_client setup
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: fa1de82769802f5b772ff625e738e15f23b15848

            gerrit Gerrit Updater added a comment - Andreas Dilger (adilger@whamcloud.com) uploaded a new patch: https://review.whamcloud.com/33694 Subject: LU-1842 ptlrpc: clean up ptlrpc_client setup Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: fa1de82769802f5b772ff625e738e15f23b15848

            Looking at the new quota code it appears to be independent of what version of linux quota you are using. From Lustre 2.1.X I see the main dependency was from the lustre quota ioctl in the fsfilt layer which did vary between linux quota versions. Now that code is linux independent.

            simmonsja James A Simmons added a comment - Looking at the new quota code it appears to be independent of what version of linux quota you are using. From Lustre 2.1.X I see the main dependency was from the lustre quota ioctl in the fsfilt layer which did vary between linux quota versions. Now that code is linux independent.

            For the older OS(SLES11 SP1) that does not have QFMT_VFS_V1 support, is there a problem enabling it with QFMT_VFS_V0 instead of QFMT_VFS_V1?

            wang Wally Wang (Inactive) added a comment - For the older OS(SLES11 SP1) that does not have QFMT_VFS_V1 support, is there a problem enabling it with QFMT_VFS_V0 instead of QFMT_VFS_V1?

            how clever. apparently i have to have the quota_v2 module loaded for this feature. however, ldiskfs doesnt depend on this module and doesnt seem to automagically load it either. this doesnt appear to be happening because QFMT_VFS_V1 isn't mapped to any particular module name in linux/quota.h:

            #define INIT_QUOTA_MODULE_NAMES {\

            {QFMT_VFS_OLD, "quota_v1"}

            ,\

            {QFMT_VFS_V0, "quota_v2"}

            ,\
            {0, NULL}}

            chas3 chas williams - CONTRACTOR added a comment - how clever. apparently i have to have the quota_v2 module loaded for this feature. however, ldiskfs doesnt depend on this module and doesnt seem to automagically load it either. this doesnt appear to be happening because QFMT_VFS_V1 isn't mapped to any particular module name in linux/quota.h: #define INIT_QUOTA_MODULE_NAMES {\ {QFMT_VFS_OLD, "quota_v1"} ,\ {QFMT_VFS_V0, "quota_v2"} ,\ {0, NULL}}

            Quota enforcement actually requires accounting to be enabled. The only possible combinations are:

            • no accounting & no limit enforcement
            • space accounting enabled & no limit enforcement
            • space accounting enabled & quota limit enforced
            johann Johann Lombardi (Inactive) added a comment - Quota enforcement actually requires accounting to be enabled. The only possible combinations are: no accounting & no limit enforcement space accounting enabled & no limit enforcement space accounting enabled & quota limit enforced

            i ported the ldiskfs patches to sles11sp2's kernel (basically 3.0). so it is possible to run quota and usage accounting at the same time? i guess i need to look harder at the ldiskfs patches.

            chas3 chas williams - CONTRACTOR added a comment - i ported the ldiskfs patches to sles11sp2's kernel (basically 3.0). so it is possible to run quota and usage accounting at the same time? i guess i need to look harder at the ldiskfs patches.

            This code snippet just enables space accounting automatically at mount time if the quota feature is enabled on the backend ldiskfs filesystem.

            Could you please tell us more about the problem you see with ldiskfs and quota in 2.4?

            johann Johann Lombardi (Inactive) added a comment - This code snippet just enables space accounting automatically at mount time if the quota feature is enabled on the backend ldiskfs filesystem. Could you please tell us more about the problem you see with ldiskfs and quota in 2.4?

            Hello,

            I am seeing a problem with ldiskfs and quotas in 2.4. http://review.whamcloud.com/4098 turned on quotas by default when mounting. However, when my ldiskfs goes to mount, it will fails since quotas and usage accounting are contraindicated. from the "first class accounting" ldiskfs patch:

            +#ifdef CONFIG_QUOTA
            + /* Enable space tracking during mount, enforcement can be enabled/disable
            + * later with quota_on/off */
            + if (EXT4_HAS_RO_COMPAT_FEATURE(sb, EXT4_FEATURE_RO_COMPAT_QUOTA) &&
            + !(sb->s_flags & MS_RDONLY)) {
            + ret = ext4_acct_on(sb);
            + if (ret)

            { + ext4_msg(sb, KERN_ERR, "Can't enable usage tracking on " + "a filesystem with the QUOTA feature set"); + goto failed_mount4; + }

            + }

            am i confused about what is going on here?

            chas3 chas williams - CONTRACTOR added a comment - Hello, I am seeing a problem with ldiskfs and quotas in 2.4. http://review.whamcloud.com/4098 turned on quotas by default when mounting. However, when my ldiskfs goes to mount, it will fails since quotas and usage accounting are contraindicated. from the "first class accounting" ldiskfs patch: +#ifdef CONFIG_QUOTA + /* Enable space tracking during mount, enforcement can be enabled/disable + * later with quota_on/off */ + if (EXT4_HAS_RO_COMPAT_FEATURE(sb, EXT4_FEATURE_RO_COMPAT_QUOTA) && + !(sb->s_flags & MS_RDONLY)) { + ret = ext4_acct_on(sb); + if (ret) { + ext4_msg(sb, KERN_ERR, "Can't enable usage tracking on " + "a filesystem with the QUOTA feature set"); + goto failed_mount4; + } + } am i confused about what is going on here?

            Appears master + http://review.whamcloud.com/#change,4218 fixed my problem.

            simmonsja James A Simmons added a comment - Appears master + http://review.whamcloud.com/#change,4218 fixed my problem.

            For the record, LUDOC-89 is the bug to track changes to the manual.

            johann Johann Lombardi (Inactive) added a comment - For the record, LUDOC-89 is the bug to track changes to the manual.

            People

              johann Johann Lombardi (Inactive)
              johann Johann Lombardi (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              11 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: