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

Fix Lustre build against installed ZFS/SPL DKMS packages

Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.8.0
    • None
    • 3
    • 9223372036854775807

    Description

      The current method to build against DKMS installs of ZFS and SPL
      assumes directories like this:

         /var/lib/dkms/spl/${VERSION}/build
         /var/lib/dkms/zfs/${VERSION}/build
      

      Unfortunately, that "build" directory was really just a temporary
      step in the DKMS build process. Older versions of DKMS forgot to
      clean up that directory, but in RHEL7 at least, DKMS no longer
      leaves that around.

      Instead we need to look in these places for source code:

        /var/lib/dkms/spl/${VERSION}/source
        /var/lib/dkms/zfs/${VERSION}/source
      

      and in these places for the per-kernel object info:

        /var/lib/dkms/spl/${VERSION}/${LINUXRELEASE}/${target_cpu}
        /var/lib/dkms/zfs/${VERSION}/${LINUXRELEASE}/${target_cpu}
      

      The changes in this patch update the Lustre build system to
      look in those corrected directories for DKMS builds of SPL
      and ZFS.

      Attachments

        Issue Links

          Activity

            [LU-7534] Fix Lustre build against installed ZFS/SPL DKMS packages

            Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17536/
            Subject: LU-7534 build: Allow lustre to build against ZFS/SPL DKMS install
            Project: fs/lustre-release
            Branch: master
            Current Patch Set:
            Commit: f9602738d023e17e25a004e166e0f41c000d97dc

            gerrit Gerrit Updater added a comment - Oleg Drokin (oleg.drokin@intel.com) merged in patch http://review.whamcloud.com/17536/ Subject: LU-7534 build: Allow lustre to build against ZFS/SPL DKMS install Project: fs/lustre-release Branch: master Current Patch Set: Commit: f9602738d023e17e25a004e166e0f41c000d97dc

            Patch looks good on it's face. Will be testing build of #17536

            utopiabound Nathaniel Clark added a comment - Patch looks good on it's face. Will be testing build of #17536

            I presume you refer to

            I didn't bother to look into where they changed the DKMS code, but yes, that does look like the place that it happened.

            morrone Christopher Morrone (Inactive) added a comment - I presume you refer to I didn't bother to look into where they changed the DKMS code, but yes, that does look like the place that it happened.
            pjones Peter Jones added a comment -

            Nathaniel

            Could you please take care of this patch?

            Thanks

            Peter

            pjones Peter Jones added a comment - Nathaniel Could you please take care of this patch? Thanks Peter
            bfaccini Bruno Faccini (Inactive) added a comment - - edited

            Hello Chris,
            I presume you refer to http://linux.dell.com/cgi-bin/cgit.cgi/dkms.git/commit/?id=2ea43f6108558849125cc1d66902d6992ee3fe39 bug+change in DKMS?
            Well, looks like this has been hidden since quite a long time due to specifying the real path, using "--with-[zfs,spl]=<path>" configure options, in our own build/DKMS frameworks usage.

            bfaccini Bruno Faccini (Inactive) added a comment - - edited Hello Chris, I presume you refer to http://linux.dell.com/cgi-bin/cgit.cgi/dkms.git/commit/?id=2ea43f6108558849125cc1d66902d6992ee3fe39 bug+change in DKMS? Well, looks like this has been hidden since quite a long time due to specifying the real path, using "--with- [zfs,spl] =<path>" configure options, in our own build/DKMS frameworks usage.

            Christopher J. Morrone (morrone2@llnl.gov) uploaded a new patch: http://review.whamcloud.com/17536
            Subject: LU-7534 build: Allow lustre to build against ZFS/SPL DKMS install
            Project: fs/lustre-release
            Branch: master
            Current Patch Set: 1
            Commit: 771001b191261234c90a3e3caf9297e137928637

            gerrit Gerrit Updater added a comment - Christopher J. Morrone (morrone2@llnl.gov) uploaded a new patch: http://review.whamcloud.com/17536 Subject: LU-7534 build: Allow lustre to build against ZFS/SPL DKMS install Project: fs/lustre-release Branch: master Current Patch Set: 1 Commit: 771001b191261234c90a3e3caf9297e137928637

            People

              utopiabound Nathaniel Clark
              morrone Christopher Morrone (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: