[LU-3572] build failure seen in SLES SP2 server build

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.5.0
• Severity: 3
• Assignee: Bob Glossman (Inactive)
• Reporter: Bob Glossman (Inactive)

Description

The recent large flood of landings in master have introduced a build failure in SLES11 SP2 server builds. errors look like:

CC [M] /home/bogl/lustre-release/lustre/mgs/mgs_handler.o
cc1: warnings being treated as errors
/home/bogl/lustre-release/lustre/mgs/mgs_handler.c: In function ‘mgs_init0’:
/home/bogl/lustre-release/lustre/mgs/mgs_handler.c:1066: error: ‘obd’ may be used uninitialized in this function
make[6]: *** [/home/bogl/lustre-release/lustre/mgs/mgs_handler.o] Error 1
make[5]: *** [/home/bogl/lustre-release/lustre/mgs] 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/rpmbuild/usr/src/linux-3.0.80-0.7.1'
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

Strongly suspect the only reason this hasn't been seen in regular review and master builds is that only client builds are done there for SLES, not server builds.

I believe this would hit any server builds done with newer gcc and kernel versions.

Rather then spend the time to precisely identify and back out a specific commit, I plan to submit a tiny fixup patch.

Comments

[Andreas Dilger]

Patch is landed - we should enable this building automatically ASAP.

[Bob Glossman (Inactive)]

Just for reference I believe the commit that introduced this compiler error was 10b256a65b8fd746ca7952b40d9026a957b171bc, LU-3101 mgs: deregister the mount for failed case

[Bob Glossman (Inactive)]

http://review.whamcloud.com/6937