Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-3660

Can't disable ACL support with ZFS MDT

Details

    • 3
    • 9444

    Description

      With ldiskfs, the MDT can be mounted with -o noacl to disable POSIX ACL support. With ZFS, that option seems to have no effect, and clients are still able to set and honor ACLs. It would be desirable to be able to turn off ACLs, as they can be detrimental to performance. For example, the cp -a command tries to write the xattr system.posix_acl_access on every destination file, regardless of whether the source file has that attribute. The ACL update is handled synchronously on the MDT, so the request handler has to wait for a ZFS transaction group to sync. This can introduce significant latency on a busy MDS, effectively limiting the per-file copy rate to the txg sync rate. On ldiskfs, cp -a performance is significantly improved by disabling ACLs.

      (This performance problem is a separate issue that may be helped by integrating ZIL support in Lustre. Turning off ACLs is a temporary workaround.)

      LLNL-bug-ID: TOSS-2207

      Attachments

        Issue Links

          Activity

            [LU-3660] Can't disable ACL support with ZFS MDT
            pjones Peter Jones added a comment -

            Landed for 2.7

            pjones Peter Jones added a comment - Landed for 2.7
            emoly.liu Emoly Liu added a comment -

            Lai's patch for this problem is at http://review.whamcloud.com/#/c/10850/ .

            emoly.liu Emoly Liu added a comment - Lai's patch for this problem is at http://review.whamcloud.com/#/c/10850/ .
            laisiyao Lai Siyao added a comment -

            osd_conf_get() for zfs osd doesn't really get mount options from super_block like ldiskfs, but enables ACL by default. I'll see how to get it work.

            laisiyao Lai Siyao added a comment - osd_conf_get() for zfs osd doesn't really get mount options from super_block like ldiskfs, but enables ACL by default. I'll see how to get it work.

            Lowered priority to minor because disabling ACL support is not really an effective workaround to the performance issue described here. But, we should still decide if this should be fixed and document ACL functionality as it relates to ZFS backends.

            nedbass Ned Bass (Inactive) added a comment - Lowered priority to minor because disabling ACL support is not really an effective workaround to the performance issue described here. But, we should still decide if this should be fixed and document ACL functionality as it relates to ZFS backends.

            Created LU-3671 to discuss the synchronous behavior. This issue is to discuss whether we need a mechanism to disable ACL support with ZFS.

            nedbass Ned Bass (Inactive) added a comment - Created LU-3671 to discuss the synchronous behavior. This issue is to discuss whether we need a mechanism to disable ACL support with ZFS.
            pjones Peter Jones added a comment -

            Lai

            Could you please advise on this one?

            Thanks

            Peter

            pjones Peter Jones added a comment - Lai Could you please advise on this one? Thanks Peter

            People

              laisiyao Lai Siyao
              nedbass Ned Bass (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: