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

Don't LASSERT on received data

Details

    • Improvement
    • Resolution: Unresolved
    • Critical
    • None
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      If a server receives corrupted data from a bad client, this could trigger an LASSERT (for instance in server_bulk_callback) and lead to the crash of the server.
      However, it is possible to manage such errors in a softer way that does not end with a server failure.

      Attachments

        Activity

          [LU-17532] Don't LASSERT on received data
          cbordage Cyril Bordage added a comment -

          More changes will come, but after some test "framework" will be implemented to generate bad data on network and check everything.

          cbordage Cyril Bordage added a comment - More changes will come, but after some test "framework" will be implemented to generate bad data on network and check everything.
          gerrit Gerrit Updater added a comment -

          "Cyril Bordage <cbordage@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58356
          Subject: LU-17532 ptlrpc: LASSERT replaced by CERROR
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 8ea9cd59edb0fd2dc33b498fc09edef660561651

          gerrit Gerrit Updater added a comment - "Cyril Bordage <cbordage@whamcloud.com>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/58356 Subject: LU-17532 ptlrpc: LASSERT replaced by CERROR Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 8ea9cd59edb0fd2dc33b498fc09edef660561651
          cbordage Cyril Bordage added a comment -

          Just a quick update. I have replaced several LASSERTs (including the one leading to the error in the related ticket) but I need to work on others and be able to simulate network errors properly.

          cbordage Cyril Bordage added a comment - Just a quick update. I have replaced several LASSERTs (including the one leading to the error in the related ticket) but I need to work on others and be able to simulate network errors properly.

          People

            cbordage Cyril Bordage
            cbordage Cyril Bordage
            Votes:
            0 Vote for this issue
            Watchers:
            9 Start watching this issue

            Dates

              Created:
              Updated: