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

lower transno may overwrite the bigger one in client last_rcvd slot

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • Lustre 2.8.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      The tgt_last_rcvd_update() may update transaction number in client slot with smaller value. That may cause client to see wrong error message:

      LustreError: 3662:0:(import.c:985:ptlrpc_connect_interpret()) pfs2dat1-OST000a_UUID went back in time (transno 77492683762 was previously committed, server now claims 77492683761)! See https://bugzilla.lustre.org/show_bug.cgi?id=9646
      

      This is possible on OST with several IO requests from one client and safe but it is better to keep transaction value in last_rcvd monotonically increased to avoid confusion on client side.

      Attachments

        Issue Links

          Activity

            [LU-6386] lower transno may overwrite the bigger one in client last_rcvd slot

            Landed for 2.8

            jgmitter Joseph Gmitter (Inactive) added a comment - Landed for 2.8

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/14113/
            Subject: LU-6386 tgt: don't update client data with smaller transno
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 5fa93f6011dc37fba3354137457b80151f46c66f

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/14113/ Subject: LU-6386 tgt: don't update client data with smaller transno Project: fs/lustre-release Branch: master Current Patch Set: Commit: 5fa93f6011dc37fba3354137457b80151f46c66f

            Please could you give us a link where we can download this patch RPM?

            Regards

            lustre.support Supporto Lustre Jnet2000 (Inactive) added a comment - Please could you give us a link where we can download this patch RPM? Regards

            It would also be good to change the error message not to reference bugzilla, but instead an LU ticket.

            adilger Andreas Dilger added a comment - It would also be good to change the error message not to reference bugzilla, but instead an LU ticket.

            Mike Pershin (mike.pershin@intel.com) uploaded a new patch: http://review.whamcloud.com/14113
            Subject: LU-6386 tgt: don't update client data with smaller transno
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: af8cc22454df96abe45e26d7887fd5bc69cc3daf

            gerrit Gerrit Updater added a comment - Mike Pershin (mike.pershin@intel.com) uploaded a new patch: http://review.whamcloud.com/14113 Subject: LU-6386 tgt: don't update client data with smaller transno Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: af8cc22454df96abe45e26d7887fd5bc69cc3daf

            People

              tappro Mikhail Pershin
              tappro Mikhail Pershin
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: