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

grouplock glimpse callback on mdt do multiple free

Details

    • Bug
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • None
    • 3
    • 9734

    Description

      when mdt takes a grouplock (mdd_file_lock()) it allocates an ma and free it in the glimpse grouplock callback.
      the cb is called for each stripe so the ma is freed multiple time
      the proposed patch (gerrit will follow) is to use a global ma structure allocated out of the lock/unlock sequence

      Attachments

        Activity

          [LU-1072] grouplock glimpse callback on mdt do multiple free

          This patch is very old, I have to check if it still make sens

          jcl jacques-charles lafoucriere added a comment - This patch is very old, I have to check if it still make sens

          This patch is very old, I have to check if it still make sens

          jcl jacques-charles lafoucriere added a comment - This patch is very old, I have to check if it still make sens

          JC – do you have any further plans for the patch you supplied back in February of 2012?
          Many thanks,
          ~ jfc.

          jfc John Fuchs-Chesney (Inactive) added a comment - JC – do you have any further plans for the patch you supplied back in February of 2012? Many thanks, ~ jfc.

          http://review.whamcloud.com/#/c/2103/ shows that the patch is still being reviewed – so I had closed it prematurely.
          ~ jfc.

          jfc John Fuchs-Chesney (Inactive) added a comment - http://review.whamcloud.com/#/c/2103/ shows that the patch is still being reviewed – so I had closed it prematurely. ~ jfc.

          Patch supplied by customer. Marked as resolved.
          ~ jfc.

          jfc John Fuchs-Chesney (Inactive) added a comment - Patch supplied by customer. Marked as resolved. ~ jfc.
          pjones Peter Jones added a comment -

          Thanks for the patch JC!

          pjones Peter Jones added a comment - Thanks for the patch JC!
          jcl jacques-charles lafoucriere added a comment - gerrit is http://review.whamcloud.com/2103

          People

            green Oleg Drokin
            cealustre CEA
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: