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

Remove dependency for lu_env structure in lu_cdebug_printer

Details

    • Improvement
    • Resolution: Low Priority
    • Minor
    • None
    • Lustre 2.4.0
    • 7402

    Description

      I was investigating the possibility of removing the need for an lu_env structure from ll_releasepage (as a result of LU-2622), and the first hurdle seems to be the fact that the debug infrastructure relies on this. In particular, it seems lu_cdebug_printer uses this structure to provide a temporary buffer so consecutive, non-newline terminated, calls to the print functions can be concatenated into a single message.

      Thus, unless the debug log infrastructure is reworked, I wont be able to remove the lu_env dependency. Since anything modifying the debug log code is likely to touch a lot of lines, I wanted to get some opinions on whether restructuring this would be advantageous.

      Is the debug logging infrastructure something that can/should be reworked to be more straight forward and simpler?

      Attachments

        Activity

          [LU-3033] Remove dependency for lu_env structure in lu_cdebug_printer
          tappro Mikhail Pershin made changes -
          Resolution New: Low Priority [ 10100 ]
          Status Original: Open [ 1 ] New: Closed [ 6 ]
          pjones Peter Jones made changes -
          End date New: 26/Jun/14
          Start date New: 26/Mar/13
          morrone Christopher Morrone (Inactive) made changes -
          Labels New: llnl
          pjones Peter Jones made changes -
          Assignee Original: WC Triage [ wc-triage ] New: Mikhail Pershin [ tappro ]
          prakash Prakash Surya (Inactive) created issue -

          People

            tappro Mikhail Pershin
            prakash Prakash Surya (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: