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

libcfs simplification

Details

    • Improvement
    • Resolution: Unresolved
    • Minor
    • Upstream
    • Lustre 2.11.0
    • 9223372036854775807

    Description

      Now that libcfs has been sorted out with user land we can look to simplify the code and use kernel function that already exist. Good example are prng.c or the cfs_alloc_array_* functions. Also only place in the linux directory only code that is needed to support various distros.

      Attachments

        Issue Links

          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-17053 t-f's log() needs lnet

          • Trivial - Cosmetic problem like misspelt words or misaligned text.
          • Open

          Bug - A problem which impairs or prevents the functions of the product. LU-12838 ptlrpc watchdog ratelimiting is broken

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-14463 stack traces not being printed to console on RHEL8

          • Critical - Crashes, loss of data, severe memory leak.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-17788 restore ptlrpc watchdog timer message

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. LU-8304 libcfs_debug_dumplog calls potentially sleeping function in wrong state

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. LU-12952 oops in cfs_trace_copyin_string

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Closed

          Technical task - A technical task. LU-12923 replace CLASSERT() with BUILD_BUG_ON()

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-12511 Prepare lustre for adoption into the linux kernel

          • Major - Major loss of function.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. LU-14290 Convert fault-injection framework to match the model using in linux

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. LU-14428 Convert tracefile to use ring_buffer from linux

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. LU-16796 Replace LASSERT_ATOMIC_* (atomic_t) with reference counters (refcount)

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. LU-17242 Clean up and Improve Lustre Debugging

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Improvement - An improvement or enhancement to an existing feature or task. LU-9019 Migrate lustre to standard 64 bit time kernel API

          • Major - Major loss of function.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-4922 Fix cfs_str2mask to better deal with garbage input

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-13258 Bind linux workqueues to specific core

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-14289 Restrict libcfs to compatibility code only

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-8703 rework CPU partition code

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. LU-14680 remove strict dependency between user tools and kernel RPMs

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Open

          Activity

            [LU-9859] libcfs simplification
            pjones Peter Jones added a comment -

            Still more?

            pjones Peter Jones added a comment - Still more?
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55187/
            Subject: LU-9859 libcfs: move crypto wrappers to lnet
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: a4d358a94496ffb4cd628f3dc10efa712c416f64

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/55187/ Subject: LU-9859 libcfs: move crypto wrappers to lnet Project: fs/lustre-release Branch: master Current Patch Set: Commit: a4d358a94496ffb4cd628f3dc10efa712c416f64
            gerrit Gerrit Updater added a comment -

            "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55187
            Subject: LU-9859 libcfs: move crypto wrappers to lnet
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 7d9f12e442b01aa097d5a3abe048212ea05b4539

            gerrit Gerrit Updater added a comment - "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/55187 Subject: LU-9859 libcfs: move crypto wrappers to lnet Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 7d9f12e442b01aa097d5a3abe048212ea05b4539
            simmonsja James A Simmons added a comment -

            Yes but it is mostly done.

            simmonsja James A Simmons added a comment - Yes but it is mostly done.
            pjones Peter Jones added a comment -

            James

            Is there still more to come here?

            Peter

            pjones Peter Jones added a comment - James Is there still more to come here? Peter
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50845/
            Subject: LU-9859 lnet: move expr parsing from libcfs
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 84d3585890c4758178cedbef0e4f032bf0eeaa03

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/50845/ Subject: LU-9859 lnet: move expr parsing from libcfs Project: fs/lustre-release Branch: master Current Patch Set: Commit: 84d3585890c4758178cedbef0e4f032bf0eeaa03
            gerrit Gerrit Updater added a comment -

            "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52923/
            Subject: LU-9859 lnet: move CPT handling to LNet
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 7f8cde3b77ada95e8b96dee1996f8d40bd17a538

            gerrit Gerrit Updater added a comment - "Oleg Drokin <green@whamcloud.com>" merged in patch https://review.whamcloud.com/c/fs/lustre-release/+/52923/ Subject: LU-9859 lnet: move CPT handling to LNet Project: fs/lustre-release Branch: master Current Patch Set: Commit: 7f8cde3b77ada95e8b96dee1996f8d40bd17a538
            gerrit Gerrit Updater added a comment -

            "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54178
            Subject: LU-9859 libcfs: move libcfs hash to obdclass
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 44901b543321d443a5c83a363fa28c5cc9eee2a1

            gerrit Gerrit Updater added a comment - "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54178 Subject: LU-9859 libcfs: move libcfs hash to obdclass Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 44901b543321d443a5c83a363fa28c5cc9eee2a1
            gerrit Gerrit Updater added a comment -

            "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54177
            Subject: LU-9859 libcfs: move libcfs hash to obdclass
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 2da0309e3481134f25bd72a57c5e15c1e808e8e9

            gerrit Gerrit Updater added a comment - "James Simmons <jsimmons@infradead.org>" uploaded a new patch: https://review.whamcloud.com/c/fs/lustre-release/+/54177 Subject: LU-9859 libcfs: move libcfs hash to obdclass Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 2da0309e3481134f25bd72a57c5e15c1e808e8e9
            simmonsja James A Simmons added a comment -

            Yes I can reproduce it. Also patch https://review.whamcloud.com/c/fs/lustre-release/+/41664 resolves this

            simmonsja James A Simmons added a comment - Yes I can reproduce it. Also patch https://review.whamcloud.com/c/fs/lustre-release/+/41664 resolves this
            simmonsja James A Simmons added a comment -

            Thanks. Let me try to reproduce this problem.

            simmonsja James A Simmons added a comment - Thanks. Let me try to reproduce this problem.

            People

              simmonsja James A Simmons
              simmonsja James A Simmons
              Votes:
              0 Vote for this issue
              Watchers:
              10 Start watching this issue

              Dates

                Created:
                Updated: