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

clean up technical debt for proc_dir_entry changes

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.7.0
    • Lustre 2.7.0
    • 3
    • 14723

    Description

      This is to address the last comment made in LU-3319 to cleanup the technical debt.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-5983 builds broken by LU-5275 fix

          • Blocker - Blocks development and/or testing work, production could not run.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-4416 support for 3.12 linux kernel

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-3319 Adapt to 3.10 upstream kernel proc_dir_entry change

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Activity

            [LU-5275] clean up technical debt for proc_dir_entry changes
            gerrit Gerrit Updater added a comment -

            Johann Lombardi (johann.lombardi@intel.com) merged in patch http://review.whamcloud.com/12953/
            Subject: Revert "LU-5275 lprocfs: remove last of non seq data structs and functions."
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: b1e595c09e1b07a6840142b3ae015b8a5a8affeb

            gerrit Gerrit Updater added a comment - Johann Lombardi (johann.lombardi@intel.com) merged in patch http://review.whamcloud.com/12953/ Subject: Revert " LU-5275 lprocfs: remove last of non seq data structs and functions." Project: fs/lustre-release Branch: master Current Patch Set: Commit: b1e595c09e1b07a6840142b3ae015b8a5a8affeb
            gerrit Gerrit Updater added a comment -

            Johann Lombardi (johann.lombardi@intel.com) uploaded a new patch: http://review.whamcloud.com/12953
            Subject: Revert "LU-5275 lprocfs: remove last of non seq data structs and functions."
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 80e25c90a3d45ee0ec55b0a83282168b0583303a

            gerrit Gerrit Updater added a comment - Johann Lombardi (johann.lombardi@intel.com) uploaded a new patch: http://review.whamcloud.com/12953 Subject: Revert " LU-5275 lprocfs: remove last of non seq data structs and functions." Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 80e25c90a3d45ee0ec55b0a83282168b0583303a
            simmonsja James A Simmons added a comment -

            Update patch http://review.whamcloud.com/#/c/12235. This should be the last patch in the series.

            simmonsja James A Simmons added a comment - Update patch http://review.whamcloud.com/#/c/12235 . This should be the last patch in the series.
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12298/
            Subject: LU-5275 lprocfs: remove last of non seq data structs and functions.
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 0ad4f8a4227ed7dd93fec99d33c6bb25056473fc

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/12298/ Subject: LU-5275 lprocfs: remove last of non seq data structs and functions. Project: fs/lustre-release Branch: master Current Patch Set: Commit: 0ad4f8a4227ed7dd93fec99d33c6bb25056473fc
            simmonsja James A Simmons added a comment -

            Yes I plan to update the second patch. The problem is the first patch is ready to land but it is based a earlier master revision. That current master has changed enough that I have to rebase the second patch I have after the first patch lands to master.

            simmonsja James A Simmons added a comment - Yes I plan to update the second patch. The problem is the first patch is ready to land but it is based a earlier master revision. That current master has changed enough that I have to rebase the second patch I have after the first patch lands to master.
            jlevi Jodi Levi (Inactive) added a comment -

            James,
            Are you planning to submit the patch for cleaning up lock handling for the 2.7 release?

            jlevi Jodi Levi (Inactive) added a comment - James, Are you planning to submit the patch for cleaning up lock handling for the 2.7 release?
            simmonsja James A Simmons added a comment -

            New cleanup patch at

            http://review.whamcloud.com/#/c/12298

            One patch left after this to cleanup lock handling that needs to be worked on.

            simmonsja James A Simmons added a comment - New cleanup patch at http://review.whamcloud.com/#/c/12298 One patch left after this to cleanup lock handling that needs to be worked on.
            simmonsja James A Simmons added a comment -

            I say two, maybe three more patches.

            simmonsja James A Simmons added a comment - I say two, maybe three more patches.
            pjones Peter Jones added a comment -

            Do you have a rough idea how many James?

            pjones Peter Jones added a comment - Do you have a rough idea how many James?
            simmonsja James A Simmons added a comment -

            Sorry about the confusion but more patches are coming. Please reopen this ticket.

            simmonsja James A Simmons added a comment - Sorry about the confusion but more patches are coming. Please reopen this ticket.
            jlevi Jodi Levi (Inactive) added a comment -

            Patches landed to Master. Please reopen ticket if more is expected for this ticket.

            jlevi Jodi Levi (Inactive) added a comment - Patches landed to Master. Please reopen ticket if more is expected for this ticket.

            People

              jhammond John Hammond
              jlevi Jodi Levi (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: