Uploaded image for project: 'Lustre'
  1. Lustre
  2. LU-3953 lustre build system improvments
  3. LU-4314

no dependency from ldiskfs sources

Details

    • Technical task
    • Resolution: Fixed
    • Major
    • Lustre 2.6.0
    • None
    • None
    • 11810

    Description

      During the parallel build (with -j <n> option) we can have errors because of the build of modules begins early than actual sources prepared in ldiskfs directory. This caused by absence dependency modules from ldiskfs sources.

      Attachments

        Activity

          [LU-4314] no dependency from ldiskfs sources
          dmiter Dmitry Eremin (Inactive) added a comment -

          landed to master

          dmiter Dmitry Eremin (Inactive) added a comment - landed to master
          dmiter Dmitry Eremin (Inactive) added a comment -

          It's separate minor issue.

          dmiter Dmitry Eremin (Inactive) added a comment - It's separate minor issue.
          simmonsja James A Simmons added a comment - - edited

          Yep you are right. The patch from LU-4266 fixes my issues. Is this ticket a duplicate of LU-4266?

          simmonsja James A Simmons added a comment - - edited Yep you are right. The patch from LU-4266 fixes my issues. Is this ticket a duplicate of LU-4266 ?
          mdiep Minh Diep added a comment -

          James, the patch to fix the ldiskfs issue above is in LU-4266

          mdiep Minh Diep added a comment - James, the patch to fix the ldiskfs issue above is in LU-4266
          simmonsja James A Simmons added a comment -

          Just tested your patch and it didn't resolve this issue. Since this is a cray lustre build it uses it own spec file. I found that for their spec file they use %{_make} instead of just plain make. When I changed that out to plain make it built properly. With your patch it still didn't address this issue. I will look to see what the _make macro expands out to.

          simmonsja James A Simmons added a comment - Just tested your patch and it didn't resolve this issue. Since this is a cray lustre build it uses it own spec file. I found that for their spec file they use %{_make} instead of just plain make. When I changed that out to plain make it built properly. With your patch it still didn't address this issue. I will look to see what the _make macro expands out to.
          dmiter Dmitry Eremin (Inactive) added a comment -

          The patch is http://review.whamcloud.com/8404

          dmiter Dmitry Eremin (Inactive) added a comment - The patch is http://review.whamcloud.com/8404

          People

            dmiter Dmitry Eremin (Inactive)
            dmiter Dmitry Eremin (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: