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

Untangle lustre userland and kernel headers

Details

    • Improvement
    • Resolution: Fixed
    • Minor
    • Lustre 2.11.0
    • Lustre 2.8.0
    • 9223372036854775807

    Description

      Much like the current situation with libcfs the headers for lustre being used internally for the kernel and for userland utilites is heavly intertwined. This work will look to unravel this mess and create a clear separation between lustre kernel space and lustre user land.

      Attachments

        Issue Links

          is blocked by

          Improvement - An improvement or enhancement to an existing feature or task. LU-6245 Untangle userland and kernel space support for libcfs

          • Major - Major loss of function.
          • Resolved
          is related to

          Bug - A problem which impairs or prevents the functions of the product. LU-6202 clean up ioctl handling

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

          Bug - A problem which impairs or prevents the functions of the product. LU-6746 move IT_* definitions to lustre_idl.h

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

          Bug - A problem which impairs or prevents the functions of the product. LU-5000 lustre_idl.h compilation regression from ORNL-10

          • 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-6450 most MAY_* constants defined in lustre_idl.h are unused

          • 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-9301 Ubuntu1404 client sanity test_400a/b FAIL: fatal error: libcfs/types.h: No such file or directory

          • 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-5011 lustre_idl.h again does not compile in user space

          • Major - Major loss of function.
          • Closed

          Bug - A problem which impairs or prevents the functions of the product. LU-9128 remove OBD_OCD_VERSION-ed out code

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

          Improvement - An improvement or enhancement to an existing feature or task. LU-8945 Stop building ptlrpc/layout.c in userland

          • 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-9767 FS name checking got removed

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

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

          • Major - Major loss of function.
          • Resolved

          Activity

            [LU-6401] Untangle lustre userland and kernel headers
            simmonsja James A Simmons added a comment -

            Oops. Forgot to close it out.

            simmonsja James A Simmons added a comment - Oops. Forgot to close it out.
            cfaber#1 Colin Faber [X] (Inactive) added a comment -

            And nearly 3 years later it's still not closed =)

            cfaber#1 Colin Faber [X] (Inactive) added a comment - And nearly 3 years later it's still not closed =)
            simmonsja James A Simmons added a comment -

            After 3 years of work this is complete.

            simmonsja James A Simmons added a comment - After 3 years of work this is complete.
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25246/
            Subject: LU-6401 uapi: migrate remaining uapi headers to uapi directory
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 6712478e79588e73e28c7ccac3afc7ac2368a4f3

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/25246/ Subject: LU-6401 uapi: migrate remaining uapi headers to uapi directory Project: fs/lustre-release Branch: master Current Patch Set: Commit: 6712478e79588e73e28c7ccac3afc7ac2368a4f3
            spitzcor Cory Spitz added a comment -

            8 vs 16 length issue reported at LU-9767.

            spitzcor Cory Spitz added a comment - 8 vs 16 length issue reported at LU-9767 .
            hornc Chris Horn added a comment -

            If that change was intentional then there's a bug with mkfs.lustre rename option that restricts a rename to 8 characters.

            hornc Chris Horn added a comment - If that change was intentional then there's a bug with mkfs.lustre rename option that restricts a rename to 8 characters.
            hornc Chris Horn added a comment - - edited

            Was it intentional to remove the 8 character filesystem name limit in commit 424e56abd685c5eabc6a276154dec1c031cf5044 ?

             

            And if so, is there an LUDOC for that change?

            hornc Chris Horn added a comment - - edited Was it intentional to remove the 8 character filesystem name limit in commit 424e56abd685c5eabc6a276154dec1c031cf5044 ?   And if so, is there an LUDOC for that change?
            gerrit Gerrit Updater added a comment -

            Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26966/
            Subject: LU-6401 uapi: change lustre_cfg.h into a proper UAPI header
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: 4fedbc879571305a0d14a2caa989c8c306efc54c

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch https://review.whamcloud.com/26966/ Subject: LU-6401 uapi: change lustre_cfg.h into a proper UAPI header Project: fs/lustre-release Branch: master Current Patch Set: Commit: 4fedbc879571305a0d14a2caa989c8c306efc54c
            yujian Jian Yu added a comment -

            Thank you James!

            yujian Jian Yu added a comment - Thank you James!
            simmonsja James A Simmons added a comment -

            Last two patches have been updated that will finish this ticket !!!!

            simmonsja James A Simmons added a comment - Last two patches have been updated that will finish this ticket !!!!

            People

              bevans Ben Evans (Inactive)
              simmonsja James A Simmons
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: