Details

    • 3
    • 9223372036854775807

    Description

      Currently, osd-ldiskfs call __ldiskfs_ioctl_project() to transfer project quota which is user ioctl
      for ext4, it will start transaction and reserve credits, this is not right logic with Lustre.

      Lustre have started a transaction handle and credits have been reserved during declare phase,
      so calling __ldiskfs_ioctl_project() will cause nested handle starting, which is not a problem for jbd2
      because it will attach current thread's handle if transaction have been started, but in this case it will ignore credits reservation.

      Also Lustre don't need inode mutex protection for project transfer, Lustre have reserved enough
      credits for project transfer, we need fix agent inode transfering.

      Attachments

        Activity

          [LU-9874] simplify project transfer codes

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32297/
          Subject: LU-9874 osd-ldiskfs: simplify project transfer codes
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set:
          Commit: 53cde2399719b56817bca26d6895a114a6feed1d

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/32297/ Subject: LU-9874 osd-ldiskfs: simplify project transfer codes Project: fs/lustre-release Branch: b2_10 Current Patch Set: Commit: 53cde2399719b56817bca26d6895a114a6feed1d

          Bob Glossman (bob.glossman@intel.com) uploaded a new patch: https://review.whamcloud.com/32297
          Subject: LU-9874 osd-ldiskfs: simplify project transfer codes
          Project: fs/lustre-release
          Branch: b2_10
          Current Patch Set: 1
          Commit: 67e97c16dc412d453ae372136cb5fd54acb2804e

          gerrit Gerrit Updater added a comment - Bob Glossman (bob.glossman@intel.com) uploaded a new patch: https://review.whamcloud.com/32297 Subject: LU-9874 osd-ldiskfs: simplify project transfer codes Project: fs/lustre-release Branch: b2_10 Current Patch Set: 1 Commit: 67e97c16dc412d453ae372136cb5fd54acb2804e
          pjones Peter Jones added a comment -

          Landed for 2.11

          pjones Peter Jones added a comment - Landed for 2.11

          Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28510/
          Subject: LU-9874 osd-ldiskfs: simplify project transfer codes
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: a9d3c9ba5360f46b2eaa5732a98c0ee836a927df

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/28510/ Subject: LU-9874 osd-ldiskfs: simplify project transfer codes Project: fs/lustre-release Branch: master Current Patch Set: Commit: a9d3c9ba5360f46b2eaa5732a98c0ee836a927df
          pjones Peter Jones added a comment -

          Hongchao

          Can you please review these proposed changes?

          Peter

          pjones Peter Jones added a comment - Hongchao Can you please review these proposed changes? Peter

          Wang Shilong (wshilong@ddn.com) uploaded a new patch: https://review.whamcloud.com/28510
          Subject: LU-9874 osd-ldiskfs: simplify project transfer codes
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: d694df17d5c4f2ed639ef1adbb91b7281037ed3f

          gerrit Gerrit Updater added a comment - Wang Shilong (wshilong@ddn.com) uploaded a new patch: https://review.whamcloud.com/28510 Subject: LU-9874 osd-ldiskfs: simplify project transfer codes Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: d694df17d5c4f2ed639ef1adbb91b7281037ed3f

          People

            hongchao.zhang Hongchao Zhang
            wangshilong Wang Shilong (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: