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

Modify existing CDEBUG/CERROR messages in LNet to provide more useful information

Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • None
    • None
    • 9223372036854775807

    Description

      There are a large number of CDEBUG and CERROR messages that could provide more useful information when debugging problems. In particular, any message that references an lnet_msg would be vastly more helpful if it included the address of the lnet_msg pointer.

      There are similar opportunities with other important LNet data structures such as the lnet_peer, lnet_peer_ni, etc.

      As a first pass we can make some incremental improvements to existing messages by just including pointer addresses, but we may also want to investigate whether it would be useful to include a standardized set of information and create a macro, similar to the DEBUG_REQ or LDLM_DEBUG macros, and convert appropriate CDEBUG messages over to the macro.

      Attachments

        Activity

          [LU-12257] Modify existing CDEBUG/CERROR messages in LNet to provide more useful information

          Chris Horn (hornc@cray.com) uploaded a new patch: https://review.whamcloud.com/34881
          Subject: LU-12257 lnet: Add msg pointer address to debug messages
          Project: fs/lustre-release
          Branch: master
          Current Patch Set: 1
          Commit: 2faec8db420cb4fb660874467f54f3c4a111c338

          gerrit Gerrit Updater added a comment - Chris Horn (hornc@cray.com) uploaded a new patch: https://review.whamcloud.com/34881 Subject: LU-12257 lnet: Add msg pointer address to debug messages Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 2faec8db420cb4fb660874467f54f3c4a111c338

          People

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

            Dates

              Created:
              Updated: