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

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

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]

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

[John Hammond]

Patch landed to master.