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

Support checksum at the LNet layer

Details

    • New Feature
    • Resolution: Unresolved
    • Critical
    • None
    • Lustre 2.8.0
    • 16823

    Description

      Currently Lustre only supports check summing at the ptlrpc layer and only form bulk data. At the LNet layer check sums is done at the LND level if at all. Also when the LND driver do some level of check sum they use different non-standard algorithms from the other LND drivers. Because of this check sums need to be performed at the client, router and server destination. The router level check summing shouldn't be needed.

      This project aims to create a check summing layer for LNET that is above the LND drivers so it can cover all the network data instead of just bulk data. Secondly this will be handled independent of the LND drivers thus
      allowing us to avoid check summing on the routers.

      Attachments

        Activity

          [LU-6034] Support checksum at the LNet layer

          James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/15713
          Subject: LU-6034 lnet: Create a checksum value for LNet package data
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: c062805a4987aa613c792dac86ed8ff0bbc68889

          gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@yahoo.com) uploaded a new patch: http://review.whamcloud.com/15713 Subject: LU-6034 lnet: Create a checksum value for LNet package data Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: c062805a4987aa613c792dac86ed8ff0bbc68889

          Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13092/
          Subject: LU-6034 lnet: add default case for check summing
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: ee51e8ebc96baf37d59ab771e61f8fc9f8908266

          gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/13092/ Subject: LU-6034 lnet: add default case for check summing Project: fs/lustre-release Branch: master Current Patch Set: Commit: ee51e8ebc96baf37d59ab771e61f8fc9f8908266

          James Simmons (uja.ornl@gmail.com) uploaded a new patch: http://review.whamcloud.com/13135
          Subject: LU-6034 lnet: Create a checksum value for LNet package data
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 31f1fe9f29e96ef5ce4659708422323fc6ce0516

          gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@gmail.com) uploaded a new patch: http://review.whamcloud.com/13135 Subject: LU-6034 lnet: Create a checksum value for LNet package data Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 31f1fe9f29e96ef5ce4659708422323fc6ce0516

          Thanks for the patch, James.

          jamesanunez James Nunez (Inactive) added a comment - Thanks for the patch, James.
          simmonsja James A Simmons added a comment - - edited

          Patch http://review.whamcloud.com/#/c/13092 aim is to be included for 2.7 to maintain binary compatibility with the changes coming in Lustre 2.8

          simmonsja James A Simmons added a comment - - edited Patch http://review.whamcloud.com/#/c/13092 aim is to be included for 2.7 to maintain binary compatibility with the changes coming in Lustre 2.8

          James Simmons (uja.ornl@gmail.com) uploaded a new patch: http://review.whamcloud.com/13092
          Subject: LU-6034 lnet: add default case for check summing
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 34a070d988cd90d06db35fe2a7972a0ace21d8ef

          gerrit Gerrit Updater added a comment - James Simmons (uja.ornl@gmail.com) uploaded a new patch: http://review.whamcloud.com/13092 Subject: LU-6034 lnet: add default case for check summing Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 34a070d988cd90d06db35fe2a7972a0ace21d8ef

          People

            simmonsja James A Simmons
            simmonsja James A Simmons
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated: