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

make utility functions availalbe to user space

Details

    • Improvement
    • Resolution: Won't Do
    • Minor
    • None
    • None
    • 9143

    Description

      In LU-2973 we decided that user-space callers of lgetxattr() should be responsible for swabbing the value of "lustre.lov" as needed. This issue is to discuss how best to make utility functions (for swabbing, comparing, etc. of data structures like lov_user_md and lu_fid) available to both kernel and user space.

      The swabbing functions are implemented in ptlrpc/pack_generic.c with extern declarations in lustre_idl.h. While these are compiled in a user-space library if liblustre is enabled, it would be nice to make them available in a generic way for liblustreapi (i.e. for LU-2182). That is, liblustre should not be a dependency for liblustreapi. Perhaps such utility functions can be refactored into a generic library that can be linked to from kernel and user code.

      Attachments

        Issue Links

          Activity

            [LU-3602] make utility functions availalbe to user space

            Is this still needed?

            simmonsja James A Simmons added a comment - Is this still needed?

            This should be considered for LU-5969.

            adilger Andreas Dilger added a comment - This should be considered for LU-5969 .
            pjones Peter Jones added a comment -

            Are there any plans to land the above patch to master?

            pjones Peter Jones added a comment - Are there any plans to land the above patch to master?
            laisiyao Lai Siyao added a comment - patch is on http://review.whamcloud.com/#/c/7096/
            pjones Peter Jones added a comment -

            Lai

            Could you please comment on this one?

            Peter

            pjones Peter Jones added a comment - Lai Could you please comment on this one? Peter

            People

              laisiyao Lai Siyao
              nedbass Ned Bass (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: