Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.11.0
    • None
    • None
    • 3
    • 9223372036854775807

    Description

      Concurrent sends tunable was intended to limit the number of in-flight transfers per connection. However queue depth does the exact same job. So for example if the queue depth is negotiated to 16 and concurrent_sends is set to 32, the maximum number of in-flight transfers doesn't exceed 16. There is no need to keep concurrent_sends around since it doesn't add any unique functionality.

      Attachments

        Issue Links

          Activity

            [LU-10291] remove concurrent_sends tunable
            pjones Peter Jones added a comment -

            Landed for 2.11

            pjones Peter Jones added a comment - Landed for 2.11

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30312/
            Subject: LU-10291 lnd: remove concurrent_sends tunable
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 8d35d6c9bd85ed3a282aa124b672e50c02322a7d

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/30312/ Subject: LU-10291 lnd: remove concurrent_sends tunable Project: fs/lustre-release Branch: master Current Patch Set: Commit: 8d35d6c9bd85ed3a282aa124b672e50c02322a7d

            Amir Shehata (amir.shehata@intel.com) uploaded a new patch: https://review.whamcloud.com/30312
            Subject: LU-10291 lnd: remove concurrent_sends tunable
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 53e79a1466aae270787098168ec7db862ce55ddd

            gerrit Gerrit Updater added a comment - Amir Shehata (amir.shehata@intel.com) uploaded a new patch: https://review.whamcloud.com/30312 Subject: LU-10291 lnd: remove concurrent_sends tunable Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 53e79a1466aae270787098168ec7db862ce55ddd

            People

              ashehata Amir Shehata (Inactive)
              ashehata Amir Shehata (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: