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

ofd_commitrw() can leak pages due to failed nodemap

Details

    • 3
    • 9223372036854775807

    Description

      	if (cmd == OBD_BRW_WRITE) {
      		struct lu_nodemap *nodemap;
      		__u32 mapped_uid, mapped_gid, mapped_projid;
      
      		nodemap = nodemap_get_from_exp(exp);
      		if (IS_ERR(nodemap))
      			RETURN(PTR_ERR(nodemap));
      

      thus ofd_commitrw_write() (so dt_bufs_put()) won't be called

      Attachments

        Activity

          [LU-15993] ofd_commitrw() can leak pages due to failed nodemap

          "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/48003/
          Subject: LU-15993 ofd: don't leak pages if nodemap fails
          Project: fs/lustre-release
          Branch: b2_15
          Current Patch Set:
          Commit: f543bb685fbcfc2a3325d903c255c7d511bc1569

          gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/48003/ Subject: LU-15993 ofd: don't leak pages if nodemap fails Project: fs/lustre-release Branch: b2_15 Current Patch Set: Commit: f543bb685fbcfc2a3325d903c255c7d511bc1569
          pjones Peter Jones added a comment -

          Landed for 2.16

          pjones Peter Jones added a comment - Landed for 2.16

          "Jian Yu <yujian@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/48003
          Subject: LU-15993 ofd: don't leak pages if nodemap fails
          Project: fs/lustre-release
          Branch: b2_15
          Current Patch Set: 1
          Commit: d07267b1e8277801de5c655df9d307a943891db9

          gerrit Gerrit Updater added a comment - "Jian Yu <yujian@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/48003 Subject: LU-15993 ofd: don't leak pages if nodemap fails Project: fs/lustre-release Branch: b2_15 Current Patch Set: 1 Commit: d07267b1e8277801de5c655df9d307a943891db9

          "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47873/
          Subject: LU-15993 ofd: don't leak pages if nodemap fails
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: ee4b50278e4471b7c64e619e13699c560502b81e

          gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/47873/ Subject: LU-15993 ofd: don't leak pages if nodemap fails Project: fs/lustre-release Branch: master Current Patch Set: Commit: ee4b50278e4471b7c64e619e13699c560502b81e

          "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/47873
          Subject: LU-15993 ofd: don't leak pages if nodemap fails
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: ef0edc8486e4a9db7a9ba08b1c2deb6e5c2d1309

          gerrit Gerrit Updater added a comment - "Alex Zhuravlev <bzzz@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/47873 Subject: LU-15993 ofd: don't leak pages if nodemap fails Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: ef0edc8486e4a9db7a9ba08b1c2deb6e5c2d1309

          People

            bzzz Alex Zhuravlev
            bzzz Alex Zhuravlev
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: