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

too many links to update llogs on zfs

Details

    • Bug
    • Resolution: Won't Fix
    • Minor
    • None
    • None
    • Red Hat Enterprise Linux Server release 7.3
      lustre-2.8.0_5.chaos-1.ch6.x86_64
      zfs-0.7.0-0.5llnl.ch6.x86_64
    • 3
    • 9223372036854775807

    Description

      Looking at an MDT under zfs, found that plain llogs, containing UPDATE_REC type entries, have 3 links. An example is:

      oi.88/0x240019a58:0x6:0x0
      update_log_dir/[0x240019a58:0x6:0x0]
      oi.10/0x20000000a:0x9:0x0/[0x240019a58:0x6:0x0]

      Note that the oi.88, oi.10, and update_log_dir entries all refer to the same underlying object/file.

      Per Di, there should be fewer links.

      Attachments

        Activity

          [LU-8833] too many links to update llogs on zfs
          pjones Peter Jones made changes -
          Resolution New: Won't Fix [ 2 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]
          pjones Peter Jones added a comment -

          ok Olaf

          pjones Peter Jones added a comment - ok Olaf
          ofaaland Olaf Faaland added a comment -

          Di,

          Yes, I agree that this helps find the update log files and we should leave it as it is. This ticket can be closed and the patch abandoned.

          thanks,
          Olaf

          ofaaland Olaf Faaland added a comment - Di, Yes, I agree that this helps find the update log files and we should leave it as it is. This ticket can be closed and the patch abandoned. thanks, Olaf
          di.wang Di Wang (Inactive) added a comment -

          Hi, Olaf,

          Even though I post a patch here, but this seems not an issue. Because putting these update log files under a name directory will help us understanding what these files are when accessing them with ZPL, so we probably want to keep as it is. If you are ok with that, let's close this ticket, and focus on LU-8753. Sorry for the confusion. Thanks.

          di.wang Di Wang (Inactive) added a comment - Hi, Olaf, Even though I post a patch here, but this seems not an issue. Because putting these update log files under a name directory will help us understanding what these files are when accessing them with ZPL, so we probably want to keep as it is. If you are ok with that, let's close this ticket, and focus on LU-8753 . Sorry for the confusion. Thanks.
          gerrit Gerrit Updater added a comment -

          wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/23812
          Subject: LU-8833 osd: only insert reg llog for ldiskfs
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 284b7e58354ed91eef63372adfa68ff9d52c80a3

          gerrit Gerrit Updater added a comment - wangdi (di.wang@intel.com) uploaded a new patch: http://review.whamcloud.com/23812 Subject: LU-8833 osd: only insert reg llog for ldiskfs Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 284b7e58354ed91eef63372adfa68ff9d52c80a3
          di.wang Di Wang (Inactive) added a comment -

          yes, I am working on a fix now.

          di.wang Di Wang (Inactive) added a comment - yes, I am working on a fix now.
          di.wang Di Wang (Inactive) made changes -
          Assignee Original: Mikhail Pershin [ tappro ] New: Di Wang [ di.wang ]
          ofaaland Olaf Faaland added a comment -

          Joe, Mike,

          Di said he was working on a patch for this already, check with him before doing work. This was identified while looking into recovery issues in LU-8753.

          ofaaland Olaf Faaland added a comment - Joe, Mike, Di said he was working on a patch for this already, check with him before doing work. This was identified while looking into recovery issues in LU-8753 .
          jgmitter Joseph Gmitter (Inactive) made changes -
          Assignee Original: WC Triage [ wc-triage ] New: Mikhail Pershin [ tappro ]
          jgmitter Joseph Gmitter (Inactive) added a comment -

          Hi Mike,

          Can you please take a look at this one?

          Thanks.
          Joe

          jgmitter Joseph Gmitter (Inactive) added a comment - Hi Mike, Can you please take a look at this one? Thanks. Joe

          People

            di.wang Di Wang (Inactive)
            ofaaland Olaf Faaland
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: