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

Optimize check for routing feature flag

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.14.0
    • None
    • None
    • 9223372036854775807

    Description

      We can perform a single check for the LNET_PING_FEAT_RTE_DISABLED flag in lnet_router_discovery_ping_reply() rather than repeatedly check it in the loop. Specifically, we can move this check to earlier in the function where we check for LNET_PEER_PING_FAILED.

      Attachments

        Activity

          [LU-12942] Optimize check for routing feature flag
          pjones Peter Jones added a comment -

          Landed for 2.14

          pjones Peter Jones added a comment - Landed for 2.14
          gerrit Gerrit Updater added a comment -

          Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36679/
          Subject: LU-12942 lnet: Optimize check for routing feature flag
          Project: fs/lustre-release
          Branch: master
          Current Patch Set:
          Commit: 7a99dc0b2f27132bea0d823ba9d2ce35ade49cb1

          gerrit Gerrit Updater added a comment - Oleg Drokin (green@whamcloud.com) merged in patch https://review.whamcloud.com/36679/ Subject: LU-12942 lnet: Optimize check for routing feature flag Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7a99dc0b2f27132bea0d823ba9d2ce35ade49cb1
          gerrit Gerrit Updater added a comment -

          Chris Horn (hornc@cray.com) uploaded a new patch: https://review.whamcloud.com/36679
          Subject: LU-12942 lnet: Optimize check for routing feature flag
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 4c033851b5ba7ba8f3013464e9e26efefb8648a8

          gerrit Gerrit Updater added a comment - Chris Horn (hornc@cray.com) uploaded a new patch: https://review.whamcloud.com/36679 Subject: LU-12942 lnet: Optimize check for routing feature flag Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 4c033851b5ba7ba8f3013464e9e26efefb8648a8

          People

            hornc Chris Horn
            hornc Chris Horn
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: