Details

    • Bug
    • Resolution: Fixed
    • Minor
    • Lustre 2.6.0
    • Lustre 2.7.0
    • None
    • 3
    • 14837

    Description

      server build now breaks in Centos 7/el7. client-only build still works OK.
      example errors:

        CC [M]  /home/bogl/lustre-release/lustre/mdd/mdd_dir.o
      /home/bogl/lustre-release/lustre/mdd/mdd_dir.c: In function ‘mdd_migrate_create’:
      /home/bogl/lustre-release/lustre/mdd/mdd_dir.c:3396:26: error: argument to ‘sizeof’ in ‘memset’ call is the same expression as the destination; did you mean to dereference it? [-Werror=sizeof-pointer-memaccess]
        memset(mgr_ea, 0, sizeof(mgr_ea));
                                ^
      cc1: all warnings being treated as errors
      make[6]: *** [/home/bogl/lustre-release/lustre/mdd/mdd_dir.o] Error 1
      make[5]: *** [/home/bogl/lustre-release/lustre/mdd] Error 2
      make[4]: *** [/home/bogl/lustre-release/lustre] Error 2
      make[3]: *** [_module_/home/bogl/lustre-release] Error 2
      make[3]: Leaving directory `/home/bogl/rb/BUILD/kernel-3.10.0_123.4.2.l0708'
      make[2]: *** [modules] Error 2
      make[2]: Leaving directory `/home/bogl/lustre-release'
      make[1]: *** [all-recursive] Error 1
      make[1]: Leaving directory `/home/bogl/lustre-release'
      make: *** [all] Error 2
      

      I believe this is due to the recent landing in master of commit de2d5808bd2987f76d2486272e1a9c192ba277d4; LU-5223 lmv: build master LMV EA dynamically build via readdir

      It appears the gcc v4.8.2 in el7 is a bit pickier about some things than earlier gcc versions.

      Attachments

        Issue Links

          Activity

            [LU-5310] build error in Centos 7
            jlevi Jodi Levi (Inactive) made changes -
            Fix Version/s New: Lustre 2.6.0 [ 10595 ]
            Resolution New: Fixed [ 1 ]
            Status Original: Open [ 1 ] New: Resolved [ 5 ]

            Patch landed to Master.

            jlevi Jodi Levi (Inactive) added a comment - Patch landed to Master.

            Have verified the build failure isn't specific to el7. It will happen with any gcc v4.8.x. I've seen it now with both v4.8.2 and v4.8.3. The fix is good for all cases so far.

            bogl Bob Glossman (Inactive) added a comment - Have verified the build failure isn't specific to el7. It will happen with any gcc v4.8.x. I've seen it now with both v4.8.2 and v4.8.3. The fix is good for all cases so far.
            adilger Andreas Dilger made changes -
            Link New: This issue is related to LU-5223 [ LU-5223 ]
            bogl Bob Glossman (Inactive) made changes -
            Assignee Original: WC Triage [ wc-triage ] New: Bob Glossman [ bogl ]
            bogl Bob Glossman (Inactive) made changes -
            Affects Version/s New: Lustre 2.7.0 [ 10631 ]

            I think the 1 line correction I commented on above & pushed as a patch is functionally correct as well as repairing the build, but I will appreciate good reviews on it to make sure I didn't mess up.

            bogl Bob Glossman (Inactive) added a comment - I think the 1 line correction I commented on above & pushed as a patch is functionally correct as well as repairing the build, but I will appreciate good reviews on it to make sure I didn't mess up.
            bogl Bob Glossman (Inactive) made changes -
            Link New: This issue is related to LU-5022 [ LU-5022 ]

            At least it is a nice simple patch.

            simmonsja James A Simmons added a comment - At least it is a nice simple patch.

            People

              bogl Bob Glossman (Inactive)
              bogl Bob Glossman (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: