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

lustre_idl.h again does not compile in user space

Details

    • Bug
    • Resolution: Fixed
    • Major
    • Lustre 2.10.0
    • Lustre 2.4.0, Lustre 2.4.1, Lustre 2.5.0, Lustre 2.4.2, Lustre 2.5.1, Lustre 2.4.3
    • 3
    • 13871

    Description

      New ticket for the the recurring problem of lustre_idl.h not compiling in user-space. Conversation began in LU-1606, but we want to have a separate ticket for Lustre 2.6.0.

      Attachments

        Issue Links

          Activity

            [LU-5011] lustre_idl.h again does not compile in user space

            This ticket can be closed since we have test sanity 400b to make sure this doesn't happen again.

            simmonsja James A Simmons added a comment - This ticket can be closed since we have test sanity 400b to make sure this doesn't happen again.
            spitzcor Cory Spitz added a comment -

            Thanks, that makes sense. Since there is activity here, maybe someone can add that definition to the file in the next commit.

            spitzcor Cory Spitz added a comment - Thanks, that makes sense. Since there is activity here, maybe someone can add that definition to the file in the next commit.
            rread Robert Read added a comment - - edited

            Since Lustre is based on an RCP interface, most likely it's a reference to Interface Definition Language, such as the kind used with rpcgen to use SUNRPC and similar RPC implementations.

            rread Robert Read added a comment - - edited Since Lustre is based on an RCP interface, most likely it's a reference to Interface Definition Language, such as the kind used with rpcgen to use SUNRPC and similar RPC implementations.
            spitzcor Cory Spitz added a comment -

            What does IDL even stand for?

            spitzcor Cory Spitz added a comment - What does IDL even stand for?

            With the work of LU-6401 I started to look closely at lustre_idl.h and I noticed its not really needed to user land except for wiretest and the wirechecker hack. It might be possible to make lustre_idl.h a kernel space only header.

            simmonsja James A Simmons added a comment - With the work of LU-6401 I started to look closely at lustre_idl.h and I noticed its not really needed to user land except for wiretest and the wirechecker hack. It might be possible to make lustre_idl.h a kernel space only header.

            That is excellent that some functions for liblustreapi has to be redone. We can move that code to liblustre_util.c and make it LGPL
            Please do send you WIP patch. I will gladly move it forward. I myself started to unwind the libcfs.h mess for the user land tools and utilities.

            simmonsja James A Simmons added a comment - That is excellent that some functions for liblustreapi has to be redone. We can move that code to liblustre_util.c and make it LGPL Please do send you WIP patch. I will gladly move it forward. I myself started to unwind the libcfs.h mess for the user land tools and utilities.

            People

              jhammond John Hammond
              morrone Christopher Morrone (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              12 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: