Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.3.0
    • None
    • None
    • 4572

    Description

      Lustre use crypto hashes algo in two ways: PTLRPC (ptlrpc/sec_bulk.c), OST(ost/ost_handler.c)/OSC (osc/osc_request.c).
      OST,OSC use crc32, crc32c, adler for checksumming (compute_checksum() function)
      PTLRPC uses crc32, adler, md5, sha1-512 ( kernel crypto api for all, excluding crc32 adler)
      All subsystems go through bulk pages and update checksum.
      To resolve conflicts with different implementation of checksumming, a new crypto hash interface is needed at libcfs. It should use kernel crypto api for hash calculation for kernel modules, and lustre implementation for user mode. Previus checksum calculation should be changed to the new libcfs crypto hash api. And adding new hash algo would be a simple task.

      Attachments

        Issue Links

          Activity

            [LU-1201] Lustre crypto hash cleanup
            pjones Peter Jones made changes -
            Resolution New: Fixed [ 1 ]
            Status Original: Reopened [ 4 ] New: Resolved [ 5 ]
            pjones Peter Jones made changes -
            Resolution Original: Fixed [ 1 ]
            Status Original: Resolved [ 5 ] New: Reopened [ 4 ]
            pjones Peter Jones made changes -
            Fix Version/s New: Lustre 2.3.0 [ 10117 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]
            ihara Shuichi Ihara (Inactive) made changes -
            Attachment New: lustre-LU1201.xlsx [ 11346 ]
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-744 [ LU-744 ]
            ihara Shuichi Ihara (Inactive) made changes -
            Attachment New: lustre-singleclient-comparison.xlsx [ 11303 ]
            aboyko Alexander Boyko created issue -

            People

              wc-triage WC Triage
              aboyko Alexander Boyko
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: