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

'lfs find --size ...' fails to detect striped directories above a given size

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.8.0
    • Lustre 2.7.0
    • 3
    • 16576

    Description

      The supporting code for lfs find implicitly assumes a regular file in a few places. See the LL_IOC_LOV_GETSTRIPE, LL_IOC_MDC_GETINFO, IOC_MDC_GETFILEINFO, IOC_MDC_GETFILESTRIPE case of ll_dir_ioctl() where we return the attributes of the master inode

      q:lustre# mkdir d0
      q:lustre# lfs mkdir -c4 d1
      q:lustre# touch d0/f{0..4095} d1/f{0..4095}
      q:lustre# stat d0 d1
        File: `d0'
        Size: 184320    	Blocks: 368        IO Block: 4096   directory
      Device: 2c54f966h/743766374d	Inode: 180144002274699275  Links: 2
      Access: (0755/drwxr-xr-x)  Uid: (    0/    root)   Gid: (    0/    root)
      Access: 2014-11-19 11:19:46.000000000 -0600
      Modify: 2014-11-19 11:20:24.000000000 -0600
      Change: 2014-11-19 11:20:24.000000000 -0600
        File: `d1'
        Size: 212992    	Blocks: 8          IO Block: 4096   directory
      Device: 2c54f966h/743766374d	Inode: 180144002274699276  Links: 2
      Access: (0755/drwxr-xr-x)  Uid: (    0/    root)   Gid: (    0/    root)
      Access: 2014-11-19 11:19:53.000000000 -0600
      Modify: 2014-11-19 11:20:37.000000000 -0600
      Change: 2014-11-19 11:20:37.000000000 -0600
      q:lustre# lfs find --size +1K .
      .
      ./d0
      ./d1
      q:lustre# lfs find --size +4K .
      ./d0
      

      Attachments

        Issue Links

          Activity

            [LU-5937] 'lfs find --size ...' fails to detect striped directories above a given size
            pjones Peter Jones added a comment -

            Landed for 2.8

            pjones Peter Jones added a comment - Landed for 2.8

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13456/
            Subject: LU-5937 lfs: ensure a valid directory size in lfs find
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 7d9c948c6016b10ed0d07e20cbd7925e108b3450

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13456/ Subject: LU-5937 lfs: ensure a valid directory size in lfs find Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7d9c948c6016b10ed0d07e20cbd7925e108b3450

            John L. Hammond (john.hammond@intel.com) uploaded a new patch: http://review.whamcloud.com/13456
            Subject: LU-5937 lfs: ensure a valid directory size in lfs find
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: ef16d531da708af9eb82b0b7f70a0320d65e937c

            gerrit Gerrit Updater added a comment - John L. Hammond (john.hammond@intel.com) uploaded a new patch: http://review.whamcloud.com/13456 Subject: LU-5937 lfs: ensure a valid directory size in lfs find Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: ef16d531da708af9eb82b0b7f70a0320d65e937c

            wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/12826
            Subject: LU-5937 llite: merge attr for striped dir in dir_ioctl
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 47ea9738135f8e4227834094be3af65ee948854e

            gerrit Gerrit Updater added a comment - wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/12826 Subject: LU-5937 llite: merge attr for striped dir in dir_ioctl Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 47ea9738135f8e4227834094be3af65ee948854e

            Di,
            Can you please have a look at this one?
            Thank you!

            jlevi Jodi Levi (Inactive) added a comment - Di, Can you please have a look at this one? Thank you!

            People

              jhammond John Hammond
              jhammond John Hammond
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: