From a4ac52a7d08540802f899b115c74f7b9cdd0c5d7 Mon Sep 17 00:00:00 2001 From: Andreas Dilger Date: Fri, 2 Feb 2018 08:45:53 -0800 Subject: [PATCH] LU-10560 libcfs: remove extra headers from linux-debug.c Remove headers from linux-debug.c that are no longer (or were possibly never) needed. This avoids a compile problem with 4.14 kernels: CC [M] lustre/libcfs/libcfs/linux/linux-debug.o In file included from lustre/libcfs/libcfs/linux/linux-debug.c:50:0: ./arch/x86/include/asm/uaccess.h:31:9: In function set_fs(): error: dereferencing pointer to incomplete type current->thread.addr_limit = fs; It turns out was included twice but we don't need anything from that header. Same for and a number of other extraneous headers. Sort headers alphabetically so it is easier to avoid duplicates. Test-Parameters: trivial Signed-off-by: Andreas Dilger Change-Id: I751e796913624cd8c9c95052abe4ecbb823ebbe5 Reviewed-on: https://review.whamcloud.com/31007 Tested-by: Jenkins Tested-by: Maloo Reviewed-by: James Simmons Reviewed-by: Dmitry Eremin Reviewed-by: Oleg Drokin --- libcfs/libcfs/linux/linux-debug.c | 21 +++++---------------- 1 file changed, 5 insertions(+), 16 deletions(-) diff --git a/libcfs/libcfs/linux/linux-debug.c b/libcfs/libcfs/linux/linux-debug.c index 81ddd35..2778c62 100644 --- a/libcfs/libcfs/linux/linux-debug.c +++ b/libcfs/libcfs/linux/linux-debug.c @@ -34,26 +34,17 @@ * Author: Phil Schwan */ -#include +#include +#include +#include #include +#include #include -#include -#include -#include -#include -#include #ifdef HAVE_KERNEL_LOCKED #include #endif +#include #include -#include -#include -#include - -#include -#include -#include -#include # define DEBUG_SUBSYSTEM S_LNET @@ -61,8 +52,6 @@ #include "tracefile.h" -#include - char lnet_debug_log_upcall[1024] = "/usr/lib/lustre/lnet_debug_log_upcall"; /** -- 1.8.3.1