[LU-5442] unconditionally include libcfs.h in lustre_idl.h

• Type: Bug
• Resolution: Fixed
• Priority: Minor
• Fix Version/s: Lustre 2.7.0
• Affects Version/s: Lustre 2.7.0
• Severity: 3
• Assignee: John Hammond
• Reporter: John Hammond

Description

lustre_idl.h only includes libcfs.h if LPU64 is not defined:

#if !defined(LPU64)
#include <libcfs/libcfs.h> /* for LPUX64, etc */
#endif

Since the landing of LU-5327 libcfs: add libcfs/types.h it is possible to get definitions for LPU64 without including libcfs.h. (Technically it was always possible, but now it's much easier.)

The outer include guard was added in LU-113: make lustre_

{idl,user}

.h nice to userspace. Since we no longer package lustre_idl.h we should remove this guard.

Comments

[John Hammond]

Patch landed to master.

[John Hammond]

Please see http://review.whamcloud.com/11308.