* [PATCH] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros
@ 2013-11-08 3:47 Chen Gang
2013-11-14 2:41 ` Chen Gang
2013-11-14 2:42 ` [PATCH v2] " Chen Gang
0 siblings, 2 replies; 5+ messages in thread
From: Chen Gang @ 2013-11-08 3:47 UTC (permalink / raw)
To: Will Deacon, Catalin Marinas, Russell King - ARM Linux,
Christoffer Dall, Uwe Kleine-König, Jonathan Austin,
Al Viro
Cc: linux-arm-kernel, linux-kernel, dhowells, kvmarm
For all uapi headers, need use "_UAPI" prefix for its guard macro
(which will be stripped by "scripts/headers_installer.sh").
Additional completions:
- be sure that all "endif" need be appended related comment, also it
has and only has one empty line above and no any lines below either.
- be sure that all normal uapi header files need content guard macro.
- remove the default "kvm_para.h" which was added in Kbuild.
- remove the detail address of Free Software Foundation (or can not
pass "scripts/checkpatch.pl").
Cc: David Howells <dhowells@redhat.com>
Signed-off-by: Chen Gang <gang.chen@asianux.com>
---
arch/arm/include/uapi/asm/byteorder.h | 7 +++----
arch/arm/include/uapi/asm/fcntl.h | 6 +++---
arch/arm/include/uapi/asm/ioctls.h | 6 +++---
arch/arm/include/uapi/asm/kvm.h | 8 ++++----
arch/arm/include/uapi/asm/kvm_para.h | 1 -
arch/arm/include/uapi/asm/mman.h | 5 +++++
arch/arm/include/uapi/asm/perf_regs.h | 7 ++++---
arch/arm/include/uapi/asm/posix_types.h | 6 +++---
arch/arm/include/uapi/asm/ptrace.h | 1 -
arch/arm/include/uapi/asm/setup.h | 1 -
arch/arm/include/uapi/asm/sigcontext.h | 7 +++----
arch/arm/include/uapi/asm/signal.h | 1 -
arch/arm/include/uapi/asm/stat.h | 6 +++---
arch/arm/include/uapi/asm/statfs.h | 7 ++++---
14 files changed, 35 insertions(+), 34 deletions(-)
delete mode 100644 arch/arm/include/uapi/asm/kvm_para.h
diff --git a/arch/arm/include/uapi/asm/byteorder.h b/arch/arm/include/uapi/asm/byteorder.h
index 7737974..ffd1e93 100644
--- a/arch/arm/include/uapi/asm/byteorder.h
+++ b/arch/arm/include/uapi/asm/byteorder.h
@@ -12,8 +12,8 @@
* and word accesses (data or instruction) appear as:
* d0...d31
*/
-#ifndef __ASM_ARM_BYTEORDER_H
-#define __ASM_ARM_BYTEORDER_H
+#ifndef _UAPI__ASM_ARM_BYTEORDER_H
+#define _UAPI__ASM_ARM_BYTEORDER_H
#ifdef __ARMEB__
#include <linux/byteorder/big_endian.h>
@@ -21,5 +21,4 @@
#include <linux/byteorder/little_endian.h>
#endif
-#endif
-
+#endif /* _UAPI__ASM_ARM_BYTEORDER_H */
diff --git a/arch/arm/include/uapi/asm/fcntl.h b/arch/arm/include/uapi/asm/fcntl.h
index a80b660..4cde9c4 100644
--- a/arch/arm/include/uapi/asm/fcntl.h
+++ b/arch/arm/include/uapi/asm/fcntl.h
@@ -1,5 +1,5 @@
-#ifndef _ARM_FCNTL_H
-#define _ARM_FCNTL_H
+#ifndef _UAPI_ARM_FCNTL_H
+#define _UAPI_ARM_FCNTL_H
#define O_DIRECTORY 040000 /* must be a directory */
#define O_NOFOLLOW 0100000 /* don't follow links */
@@ -8,4 +8,4 @@
#include <asm-generic/fcntl.h>
-#endif
+#endif /* _UAPI_ARM_FCNTL_H */
diff --git a/arch/arm/include/uapi/asm/ioctls.h b/arch/arm/include/uapi/asm/ioctls.h
index 9c96298..a9d8008 100644
--- a/arch/arm/include/uapi/asm/ioctls.h
+++ b/arch/arm/include/uapi/asm/ioctls.h
@@ -1,8 +1,8 @@
-#ifndef __ASM_ARM_IOCTLS_H
-#define __ASM_ARM_IOCTLS_H
+#ifndef _UAPI__ASM_ARM_IOCTLS_H
+#define _UAPI__ASM_ARM_IOCTLS_H
#define FIOQSIZE 0x545E
#include <asm-generic/ioctls.h>
-#endif
+#endif /* _UAPI__ASM_ARM_IOCTLS_H */
diff --git a/arch/arm/include/uapi/asm/kvm.h b/arch/arm/include/uapi/asm/kvm.h
index c498b60..f44fad1 100644
--- a/arch/arm/include/uapi/asm/kvm.h
+++ b/arch/arm/include/uapi/asm/kvm.h
@@ -13,11 +13,11 @@
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
- * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * Foundation (the address is in a copy of the GPL).
*/
-#ifndef __ARM_KVM_H__
-#define __ARM_KVM_H__
+#ifndef _UAPI__ARM_KVM_H__
+#define _UAPI__ARM_KVM_H__
#include <linux/types.h>
#include <asm/ptrace.h>
@@ -178,4 +178,4 @@ struct kvm_arch_memory_slot {
#define KVM_PSCI_RET_INVAL ((unsigned long)-2)
#define KVM_PSCI_RET_DENIED ((unsigned long)-3)
-#endif /* __ARM_KVM_H__ */
+#endif /* _UAPI__ARM_KVM_H__ */
diff --git a/arch/arm/include/uapi/asm/kvm_para.h b/arch/arm/include/uapi/asm/kvm_para.h
deleted file mode 100644
index 14fab8f..0000000
--- a/arch/arm/include/uapi/asm/kvm_para.h
+++ /dev/null
@@ -1 +0,0 @@
-#include <asm-generic/kvm_para.h>
diff --git a/arch/arm/include/uapi/asm/mman.h b/arch/arm/include/uapi/asm/mman.h
index 41f99c5..bf6ac7c 100644
--- a/arch/arm/include/uapi/asm/mman.h
+++ b/arch/arm/include/uapi/asm/mman.h
@@ -1,4 +1,9 @@
+#ifndef _UAPI_ARM_MMAN_H
+#define _UAPI_ARM_MMAN_H
+
#include <asm-generic/mman.h>
#define arch_mmap_check(addr, len, flags) \
(((flags) & MAP_FIXED && (addr) < FIRST_USER_ADDRESS) ? -EINVAL : 0)
+
+#endif /* _UAPI_ARM_MMAN_H */
diff --git a/arch/arm/include/uapi/asm/perf_regs.h b/arch/arm/include/uapi/asm/perf_regs.h
index ce59448..61e86fb 100644
--- a/arch/arm/include/uapi/asm/perf_regs.h
+++ b/arch/arm/include/uapi/asm/perf_regs.h
@@ -1,5 +1,5 @@
-#ifndef _ASM_ARM_PERF_REGS_H
-#define _ASM_ARM_PERF_REGS_H
+#ifndef _UAPI_ASM_ARM_PERF_REGS_H
+#define _UAPI_ASM_ARM_PERF_REGS_H
enum perf_event_arm_regs {
PERF_REG_ARM_R0,
@@ -20,4 +20,5 @@ enum perf_event_arm_regs {
PERF_REG_ARM_PC,
PERF_REG_ARM_MAX,
};
-#endif /* _ASM_ARM_PERF_REGS_H */
+
+#endif /* _UAPI_ASM_ARM_PERF_REGS_H */
diff --git a/arch/arm/include/uapi/asm/posix_types.h b/arch/arm/include/uapi/asm/posix_types.h
index d2de9cb..d0e3856 100644
--- a/arch/arm/include/uapi/asm/posix_types.h
+++ b/arch/arm/include/uapi/asm/posix_types.h
@@ -10,8 +10,8 @@
* Changelog:
* 27-06-1996 RMK Created
*/
-#ifndef __ARCH_ARM_POSIX_TYPES_H
-#define __ARCH_ARM_POSIX_TYPES_H
+#ifndef _UAPI__ARCH_ARM_POSIX_TYPES_H
+#define _UAPI__ARCH_ARM_POSIX_TYPES_H
/*
* This file is generally used by user-level software, so you need to
@@ -34,4 +34,4 @@ typedef unsigned short __kernel_old_dev_t;
#include <asm-generic/posix_types.h>
-#endif
+#endif /* _UAPI__ARCH_ARM_POSIX_TYPES_H */
diff --git a/arch/arm/include/uapi/asm/ptrace.h b/arch/arm/include/uapi/asm/ptrace.h
index 5af0ed1..9fd79dd 100644
--- a/arch/arm/include/uapi/asm/ptrace.h
+++ b/arch/arm/include/uapi/asm/ptrace.h
@@ -151,7 +151,6 @@ struct pt_regs {
*/
#define ARM_VFPREGS_SIZE ( 32 * 8 /*fpregs*/ + 4 /*fpscr*/ )
-
#endif /* __ASSEMBLY__ */
#endif /* _UAPI__ASM_ARM_PTRACE_H */
diff --git a/arch/arm/include/uapi/asm/setup.h b/arch/arm/include/uapi/asm/setup.h
index 979ff40..be889f3 100644
--- a/arch/arm/include/uapi/asm/setup.h
+++ b/arch/arm/include/uapi/asm/setup.h
@@ -183,5 +183,4 @@ struct tagtable {
#define for_each_tag(t,base) \
for (t = base; t->hdr.size; t = tag_next(t))
-
#endif /* _UAPI__ASMARM_SETUP_H */
diff --git a/arch/arm/include/uapi/asm/sigcontext.h b/arch/arm/include/uapi/asm/sigcontext.h
index fc0b80b..f6aa372 100644
--- a/arch/arm/include/uapi/asm/sigcontext.h
+++ b/arch/arm/include/uapi/asm/sigcontext.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_SIGCONTEXT_H
-#define _ASMARM_SIGCONTEXT_H
+#ifndef _UAPI_ASMARM_SIGCONTEXT_H
+#define _UAPI_ASMARM_SIGCONTEXT_H
/*
* Signal context structure - contains all info to do with the state
@@ -30,5 +30,4 @@ struct sigcontext {
unsigned long fault_address;
};
-
-#endif
+#endif /* _UAPI_ASMARM_SIGCONTEXT_H */
diff --git a/arch/arm/include/uapi/asm/signal.h b/arch/arm/include/uapi/asm/signal.h
index 33073bd..8954e9a 100644
--- a/arch/arm/include/uapi/asm/signal.h
+++ b/arch/arm/include/uapi/asm/signal.h
@@ -116,5 +116,4 @@ typedef struct sigaltstack {
size_t ss_size;
} stack_t;
-
#endif /* _UAPI_ASMARM_SIGNAL_H */
diff --git a/arch/arm/include/uapi/asm/stat.h b/arch/arm/include/uapi/asm/stat.h
index 42c0c13..57addf5 100644
--- a/arch/arm/include/uapi/asm/stat.h
+++ b/arch/arm/include/uapi/asm/stat.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_STAT_H
-#define _ASMARM_STAT_H
+#ifndef _UAPI_ASMARM_STAT_H
+#define _UAPI_ASMARM_STAT_H
struct __old_kernel_stat {
unsigned short st_dev;
@@ -84,4 +84,4 @@ struct stat64 {
unsigned long long st_ino;
};
-#endif
+#endif /* _UAPI_ASMARM_STAT_H */
diff --git a/arch/arm/include/uapi/asm/statfs.h b/arch/arm/include/uapi/asm/statfs.h
index 079447c..ebcb391 100644
--- a/arch/arm/include/uapi/asm/statfs.h
+++ b/arch/arm/include/uapi/asm/statfs.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_STATFS_H
-#define _ASMARM_STATFS_H
+#ifndef _UAPI_ASMARM_STATFS_H
+#define _UAPI_ASMARM_STATFS_H
/*
* With EABI there is 4 bytes of padding added to this structure.
@@ -9,4 +9,5 @@
#define ARCH_PACK_STATFS64 __attribute__((packed,aligned(4)))
#include <asm-generic/statfs.h>
-#endif
+
+#endif /* _UAPI_ASMARM_STATFS_H */
--
1.7.7.6
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros
2013-11-08 3:47 [PATCH] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros Chen Gang
@ 2013-11-14 2:41 ` Chen Gang
2013-11-14 2:42 ` [PATCH v2] " Chen Gang
1 sibling, 0 replies; 5+ messages in thread
From: Chen Gang @ 2013-11-14 2:41 UTC (permalink / raw)
To: Will Deacon, Catalin Marinas, Russell King - ARM Linux,
Christoffer Dall, Uwe Kleine-König, Jonathan Austin,
Al Viro
Cc: linux-arm-kernel, linux-kernel, dhowells, kvmarm, Hans-Christian Egtvedt
On 11/08/2013 11:47 AM, Chen Gang wrote:
> For all uapi headers, need use "_UAPI" prefix for its guard macro
> (which will be stripped by "scripts/headers_installer.sh").
>
> Additional completions:
>
> - be sure that all "endif" need be appended related comment, also it
> has and only has one empty line above and no any lines below either.
>
> - be sure that all normal uapi header files need content guard macro.
>
> - remove the default "kvm_para.h" which was added in Kbuild.
Oh, sorry, it should use 'generic-y' instead of 'header-y' in Kbuild. I
will send patch v2 for it.
>
> - remove the detail address of Free Software Foundation (or can not
> pass "scripts/checkpatch.pl").
>
Thanks.
--
Chen Gang
^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH v2] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros
2013-11-08 3:47 [PATCH] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros Chen Gang
2013-11-14 2:41 ` Chen Gang
@ 2013-11-14 2:42 ` Chen Gang
2013-11-14 18:28 ` Christoffer Dall
1 sibling, 1 reply; 5+ messages in thread
From: Chen Gang @ 2013-11-14 2:42 UTC (permalink / raw)
To: Will Deacon, Catalin Marinas, Russell King - ARM Linux,
Christoffer Dall, Uwe Kleine-König, Jonathan Austin,
Al Viro
Cc: linux-arm-kernel, linux-kernel, dhowells, kvmarm
For all uapi headers, need use "_UAPI" prefix for its guard macro
(which will be stripped by "scripts/headers_installer.sh").
Additional completions:
- be sure that all "endif" need be appended related comment, also it
has and only has one empty line above and no any lines below either.
- be sure that all normal uapi header files need content guard macro.
- remove the default "kvm_para.h" and add 'generic-y' in Kbuild.
- remove the detail address of Free Software Foundation (or can not
pass "scripts/checkpatch.pl").
Signed-off-by: Chen Gang <gang.chen@asianux.com>
---
arch/arm/include/uapi/asm/Kbuild | 3 ++-
arch/arm/include/uapi/asm/byteorder.h | 7 +++----
arch/arm/include/uapi/asm/fcntl.h | 6 +++---
arch/arm/include/uapi/asm/ioctls.h | 6 +++---
arch/arm/include/uapi/asm/kvm.h | 8 ++++----
arch/arm/include/uapi/asm/kvm_para.h | 1 -
arch/arm/include/uapi/asm/mman.h | 5 +++++
arch/arm/include/uapi/asm/perf_regs.h | 7 ++++---
arch/arm/include/uapi/asm/posix_types.h | 6 +++---
arch/arm/include/uapi/asm/ptrace.h | 1 -
arch/arm/include/uapi/asm/setup.h | 1 -
arch/arm/include/uapi/asm/sigcontext.h | 7 +++----
arch/arm/include/uapi/asm/signal.h | 1 -
arch/arm/include/uapi/asm/stat.h | 6 +++---
arch/arm/include/uapi/asm/statfs.h | 7 ++++---
15 files changed, 37 insertions(+), 35 deletions(-)
delete mode 100644 arch/arm/include/uapi/asm/kvm_para.h
diff --git a/arch/arm/include/uapi/asm/Kbuild b/arch/arm/include/uapi/asm/Kbuild
index 70a1c9d..12a65af 100644
--- a/arch/arm/include/uapi/asm/Kbuild
+++ b/arch/arm/include/uapi/asm/Kbuild
@@ -1,11 +1,12 @@
# UAPI Header export list
include include/uapi/asm-generic/Kbuild.asm
+generic-y += kvm_para.h
+
header-y += byteorder.h
header-y += fcntl.h
header-y += hwcap.h
header-y += ioctls.h
-header-y += kvm_para.h
header-y += mman.h
header-y += perf_regs.h
header-y += posix_types.h
diff --git a/arch/arm/include/uapi/asm/byteorder.h b/arch/arm/include/uapi/asm/byteorder.h
index 7737974..ffd1e93 100644
--- a/arch/arm/include/uapi/asm/byteorder.h
+++ b/arch/arm/include/uapi/asm/byteorder.h
@@ -12,8 +12,8 @@
* and word accesses (data or instruction) appear as:
* d0...d31
*/
-#ifndef __ASM_ARM_BYTEORDER_H
-#define __ASM_ARM_BYTEORDER_H
+#ifndef _UAPI__ASM_ARM_BYTEORDER_H
+#define _UAPI__ASM_ARM_BYTEORDER_H
#ifdef __ARMEB__
#include <linux/byteorder/big_endian.h>
@@ -21,5 +21,4 @@
#include <linux/byteorder/little_endian.h>
#endif
-#endif
-
+#endif /* _UAPI__ASM_ARM_BYTEORDER_H */
diff --git a/arch/arm/include/uapi/asm/fcntl.h b/arch/arm/include/uapi/asm/fcntl.h
index a80b660..4cde9c4 100644
--- a/arch/arm/include/uapi/asm/fcntl.h
+++ b/arch/arm/include/uapi/asm/fcntl.h
@@ -1,5 +1,5 @@
-#ifndef _ARM_FCNTL_H
-#define _ARM_FCNTL_H
+#ifndef _UAPI_ARM_FCNTL_H
+#define _UAPI_ARM_FCNTL_H
#define O_DIRECTORY 040000 /* must be a directory */
#define O_NOFOLLOW 0100000 /* don't follow links */
@@ -8,4 +8,4 @@
#include <asm-generic/fcntl.h>
-#endif
+#endif /* _UAPI_ARM_FCNTL_H */
diff --git a/arch/arm/include/uapi/asm/ioctls.h b/arch/arm/include/uapi/asm/ioctls.h
index 9c96298..a9d8008 100644
--- a/arch/arm/include/uapi/asm/ioctls.h
+++ b/arch/arm/include/uapi/asm/ioctls.h
@@ -1,8 +1,8 @@
-#ifndef __ASM_ARM_IOCTLS_H
-#define __ASM_ARM_IOCTLS_H
+#ifndef _UAPI__ASM_ARM_IOCTLS_H
+#define _UAPI__ASM_ARM_IOCTLS_H
#define FIOQSIZE 0x545E
#include <asm-generic/ioctls.h>
-#endif
+#endif /* _UAPI__ASM_ARM_IOCTLS_H */
diff --git a/arch/arm/include/uapi/asm/kvm.h b/arch/arm/include/uapi/asm/kvm.h
index c498b60..f44fad1 100644
--- a/arch/arm/include/uapi/asm/kvm.h
+++ b/arch/arm/include/uapi/asm/kvm.h
@@ -13,11 +13,11 @@
*
* You should have received a copy of the GNU General Public License
* along with this program; if not, write to the Free Software
- * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * Foundation (the address is in a copy of the GPL).
*/
-#ifndef __ARM_KVM_H__
-#define __ARM_KVM_H__
+#ifndef _UAPI__ARM_KVM_H__
+#define _UAPI__ARM_KVM_H__
#include <linux/types.h>
#include <asm/ptrace.h>
@@ -178,4 +178,4 @@ struct kvm_arch_memory_slot {
#define KVM_PSCI_RET_INVAL ((unsigned long)-2)
#define KVM_PSCI_RET_DENIED ((unsigned long)-3)
-#endif /* __ARM_KVM_H__ */
+#endif /* _UAPI__ARM_KVM_H__ */
diff --git a/arch/arm/include/uapi/asm/kvm_para.h b/arch/arm/include/uapi/asm/kvm_para.h
deleted file mode 100644
index 14fab8f..0000000
--- a/arch/arm/include/uapi/asm/kvm_para.h
+++ /dev/null
@@ -1 +0,0 @@
-#include <asm-generic/kvm_para.h>
diff --git a/arch/arm/include/uapi/asm/mman.h b/arch/arm/include/uapi/asm/mman.h
index 41f99c5..bf6ac7c 100644
--- a/arch/arm/include/uapi/asm/mman.h
+++ b/arch/arm/include/uapi/asm/mman.h
@@ -1,4 +1,9 @@
+#ifndef _UAPI_ARM_MMAN_H
+#define _UAPI_ARM_MMAN_H
+
#include <asm-generic/mman.h>
#define arch_mmap_check(addr, len, flags) \
(((flags) & MAP_FIXED && (addr) < FIRST_USER_ADDRESS) ? -EINVAL : 0)
+
+#endif /* _UAPI_ARM_MMAN_H */
diff --git a/arch/arm/include/uapi/asm/perf_regs.h b/arch/arm/include/uapi/asm/perf_regs.h
index ce59448..61e86fb 100644
--- a/arch/arm/include/uapi/asm/perf_regs.h
+++ b/arch/arm/include/uapi/asm/perf_regs.h
@@ -1,5 +1,5 @@
-#ifndef _ASM_ARM_PERF_REGS_H
-#define _ASM_ARM_PERF_REGS_H
+#ifndef _UAPI_ASM_ARM_PERF_REGS_H
+#define _UAPI_ASM_ARM_PERF_REGS_H
enum perf_event_arm_regs {
PERF_REG_ARM_R0,
@@ -20,4 +20,5 @@ enum perf_event_arm_regs {
PERF_REG_ARM_PC,
PERF_REG_ARM_MAX,
};
-#endif /* _ASM_ARM_PERF_REGS_H */
+
+#endif /* _UAPI_ASM_ARM_PERF_REGS_H */
diff --git a/arch/arm/include/uapi/asm/posix_types.h b/arch/arm/include/uapi/asm/posix_types.h
index d2de9cb..d0e3856 100644
--- a/arch/arm/include/uapi/asm/posix_types.h
+++ b/arch/arm/include/uapi/asm/posix_types.h
@@ -10,8 +10,8 @@
* Changelog:
* 27-06-1996 RMK Created
*/
-#ifndef __ARCH_ARM_POSIX_TYPES_H
-#define __ARCH_ARM_POSIX_TYPES_H
+#ifndef _UAPI__ARCH_ARM_POSIX_TYPES_H
+#define _UAPI__ARCH_ARM_POSIX_TYPES_H
/*
* This file is generally used by user-level software, so you need to
@@ -34,4 +34,4 @@ typedef unsigned short __kernel_old_dev_t;
#include <asm-generic/posix_types.h>
-#endif
+#endif /* _UAPI__ARCH_ARM_POSIX_TYPES_H */
diff --git a/arch/arm/include/uapi/asm/ptrace.h b/arch/arm/include/uapi/asm/ptrace.h
index 5af0ed1..9fd79dd 100644
--- a/arch/arm/include/uapi/asm/ptrace.h
+++ b/arch/arm/include/uapi/asm/ptrace.h
@@ -151,7 +151,6 @@ struct pt_regs {
*/
#define ARM_VFPREGS_SIZE ( 32 * 8 /*fpregs*/ + 4 /*fpscr*/ )
-
#endif /* __ASSEMBLY__ */
#endif /* _UAPI__ASM_ARM_PTRACE_H */
diff --git a/arch/arm/include/uapi/asm/setup.h b/arch/arm/include/uapi/asm/setup.h
index 979ff40..be889f3 100644
--- a/arch/arm/include/uapi/asm/setup.h
+++ b/arch/arm/include/uapi/asm/setup.h
@@ -183,5 +183,4 @@ struct tagtable {
#define for_each_tag(t,base) \
for (t = base; t->hdr.size; t = tag_next(t))
-
#endif /* _UAPI__ASMARM_SETUP_H */
diff --git a/arch/arm/include/uapi/asm/sigcontext.h b/arch/arm/include/uapi/asm/sigcontext.h
index fc0b80b..f6aa372 100644
--- a/arch/arm/include/uapi/asm/sigcontext.h
+++ b/arch/arm/include/uapi/asm/sigcontext.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_SIGCONTEXT_H
-#define _ASMARM_SIGCONTEXT_H
+#ifndef _UAPI_ASMARM_SIGCONTEXT_H
+#define _UAPI_ASMARM_SIGCONTEXT_H
/*
* Signal context structure - contains all info to do with the state
@@ -30,5 +30,4 @@ struct sigcontext {
unsigned long fault_address;
};
-
-#endif
+#endif /* _UAPI_ASMARM_SIGCONTEXT_H */
diff --git a/arch/arm/include/uapi/asm/signal.h b/arch/arm/include/uapi/asm/signal.h
index 33073bd..8954e9a 100644
--- a/arch/arm/include/uapi/asm/signal.h
+++ b/arch/arm/include/uapi/asm/signal.h
@@ -116,5 +116,4 @@ typedef struct sigaltstack {
size_t ss_size;
} stack_t;
-
#endif /* _UAPI_ASMARM_SIGNAL_H */
diff --git a/arch/arm/include/uapi/asm/stat.h b/arch/arm/include/uapi/asm/stat.h
index 42c0c13..57addf5 100644
--- a/arch/arm/include/uapi/asm/stat.h
+++ b/arch/arm/include/uapi/asm/stat.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_STAT_H
-#define _ASMARM_STAT_H
+#ifndef _UAPI_ASMARM_STAT_H
+#define _UAPI_ASMARM_STAT_H
struct __old_kernel_stat {
unsigned short st_dev;
@@ -84,4 +84,4 @@ struct stat64 {
unsigned long long st_ino;
};
-#endif
+#endif /* _UAPI_ASMARM_STAT_H */
diff --git a/arch/arm/include/uapi/asm/statfs.h b/arch/arm/include/uapi/asm/statfs.h
index 079447c..ebcb391 100644
--- a/arch/arm/include/uapi/asm/statfs.h
+++ b/arch/arm/include/uapi/asm/statfs.h
@@ -1,5 +1,5 @@
-#ifndef _ASMARM_STATFS_H
-#define _ASMARM_STATFS_H
+#ifndef _UAPI_ASMARM_STATFS_H
+#define _UAPI_ASMARM_STATFS_H
/*
* With EABI there is 4 bytes of padding added to this structure.
@@ -9,4 +9,5 @@
#define ARCH_PACK_STATFS64 __attribute__((packed,aligned(4)))
#include <asm-generic/statfs.h>
-#endif
+
+#endif /* _UAPI_ASMARM_STATFS_H */
--
1.7.7.6
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH v2] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros
2013-11-14 2:42 ` [PATCH v2] " Chen Gang
@ 2013-11-14 18:28 ` Christoffer Dall
2013-11-15 1:33 ` Chen Gang
0 siblings, 1 reply; 5+ messages in thread
From: Christoffer Dall @ 2013-11-14 18:28 UTC (permalink / raw)
To: Chen Gang
Cc: Will Deacon, Catalin Marinas, Russell King - ARM Linux,
Uwe Kleine-König, Jonathan Austin, Al Viro,
linux-arm-kernel, linux-kernel, dhowells, kvmarm
On Thu, Nov 14, 2013 at 10:42:31AM +0800, Chen Gang wrote:
> For all uapi headers, need use "_UAPI" prefix for its guard macro
> (which will be stripped by "scripts/headers_installer.sh").
>
> Additional completions:
>
> - be sure that all "endif" need be appended related comment, also it
> has and only has one empty line above and no any lines below either.
>
> - be sure that all normal uapi header files need content guard macro.
>
> - remove the default "kvm_para.h" and add 'generic-y' in Kbuild.
>
> - remove the detail address of Free Software Foundation (or can not
> pass "scripts/checkpatch.pl").
>
for the kvm/arm part I guess it looks fine.
-Christoffer
>
>
> Signed-off-by: Chen Gang <gang.chen@asianux.com>
> ---
> arch/arm/include/uapi/asm/Kbuild | 3 ++-
> arch/arm/include/uapi/asm/byteorder.h | 7 +++----
> arch/arm/include/uapi/asm/fcntl.h | 6 +++---
> arch/arm/include/uapi/asm/ioctls.h | 6 +++---
> arch/arm/include/uapi/asm/kvm.h | 8 ++++----
> arch/arm/include/uapi/asm/kvm_para.h | 1 -
> arch/arm/include/uapi/asm/mman.h | 5 +++++
> arch/arm/include/uapi/asm/perf_regs.h | 7 ++++---
> arch/arm/include/uapi/asm/posix_types.h | 6 +++---
> arch/arm/include/uapi/asm/ptrace.h | 1 -
> arch/arm/include/uapi/asm/setup.h | 1 -
> arch/arm/include/uapi/asm/sigcontext.h | 7 +++----
> arch/arm/include/uapi/asm/signal.h | 1 -
> arch/arm/include/uapi/asm/stat.h | 6 +++---
> arch/arm/include/uapi/asm/statfs.h | 7 ++++---
> 15 files changed, 37 insertions(+), 35 deletions(-)
> delete mode 100644 arch/arm/include/uapi/asm/kvm_para.h
>
> diff --git a/arch/arm/include/uapi/asm/Kbuild b/arch/arm/include/uapi/asm/Kbuild
> index 70a1c9d..12a65af 100644
> --- a/arch/arm/include/uapi/asm/Kbuild
> +++ b/arch/arm/include/uapi/asm/Kbuild
> @@ -1,11 +1,12 @@
> # UAPI Header export list
> include include/uapi/asm-generic/Kbuild.asm
>
> +generic-y += kvm_para.h
> +
> header-y += byteorder.h
> header-y += fcntl.h
> header-y += hwcap.h
> header-y += ioctls.h
> -header-y += kvm_para.h
> header-y += mman.h
> header-y += perf_regs.h
> header-y += posix_types.h
> diff --git a/arch/arm/include/uapi/asm/byteorder.h b/arch/arm/include/uapi/asm/byteorder.h
> index 7737974..ffd1e93 100644
> --- a/arch/arm/include/uapi/asm/byteorder.h
> +++ b/arch/arm/include/uapi/asm/byteorder.h
> @@ -12,8 +12,8 @@
> * and word accesses (data or instruction) appear as:
> * d0...d31
> */
> -#ifndef __ASM_ARM_BYTEORDER_H
> -#define __ASM_ARM_BYTEORDER_H
> +#ifndef _UAPI__ASM_ARM_BYTEORDER_H
> +#define _UAPI__ASM_ARM_BYTEORDER_H
>
> #ifdef __ARMEB__
> #include <linux/byteorder/big_endian.h>
> @@ -21,5 +21,4 @@
> #include <linux/byteorder/little_endian.h>
> #endif
>
> -#endif
> -
> +#endif /* _UAPI__ASM_ARM_BYTEORDER_H */
> diff --git a/arch/arm/include/uapi/asm/fcntl.h b/arch/arm/include/uapi/asm/fcntl.h
> index a80b660..4cde9c4 100644
> --- a/arch/arm/include/uapi/asm/fcntl.h
> +++ b/arch/arm/include/uapi/asm/fcntl.h
> @@ -1,5 +1,5 @@
> -#ifndef _ARM_FCNTL_H
> -#define _ARM_FCNTL_H
> +#ifndef _UAPI_ARM_FCNTL_H
> +#define _UAPI_ARM_FCNTL_H
>
> #define O_DIRECTORY 040000 /* must be a directory */
> #define O_NOFOLLOW 0100000 /* don't follow links */
> @@ -8,4 +8,4 @@
>
> #include <asm-generic/fcntl.h>
>
> -#endif
> +#endif /* _UAPI_ARM_FCNTL_H */
> diff --git a/arch/arm/include/uapi/asm/ioctls.h b/arch/arm/include/uapi/asm/ioctls.h
> index 9c96298..a9d8008 100644
> --- a/arch/arm/include/uapi/asm/ioctls.h
> +++ b/arch/arm/include/uapi/asm/ioctls.h
> @@ -1,8 +1,8 @@
> -#ifndef __ASM_ARM_IOCTLS_H
> -#define __ASM_ARM_IOCTLS_H
> +#ifndef _UAPI__ASM_ARM_IOCTLS_H
> +#define _UAPI__ASM_ARM_IOCTLS_H
>
> #define FIOQSIZE 0x545E
>
> #include <asm-generic/ioctls.h>
>
> -#endif
> +#endif /* _UAPI__ASM_ARM_IOCTLS_H */
> diff --git a/arch/arm/include/uapi/asm/kvm.h b/arch/arm/include/uapi/asm/kvm.h
> index c498b60..f44fad1 100644
> --- a/arch/arm/include/uapi/asm/kvm.h
> +++ b/arch/arm/include/uapi/asm/kvm.h
> @@ -13,11 +13,11 @@
> *
> * You should have received a copy of the GNU General Public License
> * along with this program; if not, write to the Free Software
> - * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
> + * Foundation (the address is in a copy of the GPL).
> */
>
> -#ifndef __ARM_KVM_H__
> -#define __ARM_KVM_H__
> +#ifndef _UAPI__ARM_KVM_H__
> +#define _UAPI__ARM_KVM_H__
>
> #include <linux/types.h>
> #include <asm/ptrace.h>
> @@ -178,4 +178,4 @@ struct kvm_arch_memory_slot {
> #define KVM_PSCI_RET_INVAL ((unsigned long)-2)
> #define KVM_PSCI_RET_DENIED ((unsigned long)-3)
>
> -#endif /* __ARM_KVM_H__ */
> +#endif /* _UAPI__ARM_KVM_H__ */
> diff --git a/arch/arm/include/uapi/asm/kvm_para.h b/arch/arm/include/uapi/asm/kvm_para.h
> deleted file mode 100644
> index 14fab8f..0000000
> --- a/arch/arm/include/uapi/asm/kvm_para.h
> +++ /dev/null
> @@ -1 +0,0 @@
> -#include <asm-generic/kvm_para.h>
> diff --git a/arch/arm/include/uapi/asm/mman.h b/arch/arm/include/uapi/asm/mman.h
> index 41f99c5..bf6ac7c 100644
> --- a/arch/arm/include/uapi/asm/mman.h
> +++ b/arch/arm/include/uapi/asm/mman.h
> @@ -1,4 +1,9 @@
> +#ifndef _UAPI_ARM_MMAN_H
> +#define _UAPI_ARM_MMAN_H
> +
> #include <asm-generic/mman.h>
>
> #define arch_mmap_check(addr, len, flags) \
> (((flags) & MAP_FIXED && (addr) < FIRST_USER_ADDRESS) ? -EINVAL : 0)
> +
> +#endif /* _UAPI_ARM_MMAN_H */
> diff --git a/arch/arm/include/uapi/asm/perf_regs.h b/arch/arm/include/uapi/asm/perf_regs.h
> index ce59448..61e86fb 100644
> --- a/arch/arm/include/uapi/asm/perf_regs.h
> +++ b/arch/arm/include/uapi/asm/perf_regs.h
> @@ -1,5 +1,5 @@
> -#ifndef _ASM_ARM_PERF_REGS_H
> -#define _ASM_ARM_PERF_REGS_H
> +#ifndef _UAPI_ASM_ARM_PERF_REGS_H
> +#define _UAPI_ASM_ARM_PERF_REGS_H
>
> enum perf_event_arm_regs {
> PERF_REG_ARM_R0,
> @@ -20,4 +20,5 @@ enum perf_event_arm_regs {
> PERF_REG_ARM_PC,
> PERF_REG_ARM_MAX,
> };
> -#endif /* _ASM_ARM_PERF_REGS_H */
> +
> +#endif /* _UAPI_ASM_ARM_PERF_REGS_H */
> diff --git a/arch/arm/include/uapi/asm/posix_types.h b/arch/arm/include/uapi/asm/posix_types.h
> index d2de9cb..d0e3856 100644
> --- a/arch/arm/include/uapi/asm/posix_types.h
> +++ b/arch/arm/include/uapi/asm/posix_types.h
> @@ -10,8 +10,8 @@
> * Changelog:
> * 27-06-1996 RMK Created
> */
> -#ifndef __ARCH_ARM_POSIX_TYPES_H
> -#define __ARCH_ARM_POSIX_TYPES_H
> +#ifndef _UAPI__ARCH_ARM_POSIX_TYPES_H
> +#define _UAPI__ARCH_ARM_POSIX_TYPES_H
>
> /*
> * This file is generally used by user-level software, so you need to
> @@ -34,4 +34,4 @@ typedef unsigned short __kernel_old_dev_t;
>
> #include <asm-generic/posix_types.h>
>
> -#endif
> +#endif /* _UAPI__ARCH_ARM_POSIX_TYPES_H */
> diff --git a/arch/arm/include/uapi/asm/ptrace.h b/arch/arm/include/uapi/asm/ptrace.h
> index 5af0ed1..9fd79dd 100644
> --- a/arch/arm/include/uapi/asm/ptrace.h
> +++ b/arch/arm/include/uapi/asm/ptrace.h
> @@ -151,7 +151,6 @@ struct pt_regs {
> */
> #define ARM_VFPREGS_SIZE ( 32 * 8 /*fpregs*/ + 4 /*fpscr*/ )
>
> -
> #endif /* __ASSEMBLY__ */
>
> #endif /* _UAPI__ASM_ARM_PTRACE_H */
> diff --git a/arch/arm/include/uapi/asm/setup.h b/arch/arm/include/uapi/asm/setup.h
> index 979ff40..be889f3 100644
> --- a/arch/arm/include/uapi/asm/setup.h
> +++ b/arch/arm/include/uapi/asm/setup.h
> @@ -183,5 +183,4 @@ struct tagtable {
> #define for_each_tag(t,base) \
> for (t = base; t->hdr.size; t = tag_next(t))
>
> -
> #endif /* _UAPI__ASMARM_SETUP_H */
> diff --git a/arch/arm/include/uapi/asm/sigcontext.h b/arch/arm/include/uapi/asm/sigcontext.h
> index fc0b80b..f6aa372 100644
> --- a/arch/arm/include/uapi/asm/sigcontext.h
> +++ b/arch/arm/include/uapi/asm/sigcontext.h
> @@ -1,5 +1,5 @@
> -#ifndef _ASMARM_SIGCONTEXT_H
> -#define _ASMARM_SIGCONTEXT_H
> +#ifndef _UAPI_ASMARM_SIGCONTEXT_H
> +#define _UAPI_ASMARM_SIGCONTEXT_H
>
> /*
> * Signal context structure - contains all info to do with the state
> @@ -30,5 +30,4 @@ struct sigcontext {
> unsigned long fault_address;
> };
>
> -
> -#endif
> +#endif /* _UAPI_ASMARM_SIGCONTEXT_H */
> diff --git a/arch/arm/include/uapi/asm/signal.h b/arch/arm/include/uapi/asm/signal.h
> index 33073bd..8954e9a 100644
> --- a/arch/arm/include/uapi/asm/signal.h
> +++ b/arch/arm/include/uapi/asm/signal.h
> @@ -116,5 +116,4 @@ typedef struct sigaltstack {
> size_t ss_size;
> } stack_t;
>
> -
> #endif /* _UAPI_ASMARM_SIGNAL_H */
> diff --git a/arch/arm/include/uapi/asm/stat.h b/arch/arm/include/uapi/asm/stat.h
> index 42c0c13..57addf5 100644
> --- a/arch/arm/include/uapi/asm/stat.h
> +++ b/arch/arm/include/uapi/asm/stat.h
> @@ -1,5 +1,5 @@
> -#ifndef _ASMARM_STAT_H
> -#define _ASMARM_STAT_H
> +#ifndef _UAPI_ASMARM_STAT_H
> +#define _UAPI_ASMARM_STAT_H
>
> struct __old_kernel_stat {
> unsigned short st_dev;
> @@ -84,4 +84,4 @@ struct stat64 {
> unsigned long long st_ino;
> };
>
> -#endif
> +#endif /* _UAPI_ASMARM_STAT_H */
> diff --git a/arch/arm/include/uapi/asm/statfs.h b/arch/arm/include/uapi/asm/statfs.h
> index 079447c..ebcb391 100644
> --- a/arch/arm/include/uapi/asm/statfs.h
> +++ b/arch/arm/include/uapi/asm/statfs.h
> @@ -1,5 +1,5 @@
> -#ifndef _ASMARM_STATFS_H
> -#define _ASMARM_STATFS_H
> +#ifndef _UAPI_ASMARM_STATFS_H
> +#define _UAPI_ASMARM_STATFS_H
>
> /*
> * With EABI there is 4 bytes of padding added to this structure.
> @@ -9,4 +9,5 @@
> #define ARCH_PACK_STATFS64 __attribute__((packed,aligned(4)))
>
> #include <asm-generic/statfs.h>
> -#endif
> +
> +#endif /* _UAPI_ASMARM_STATFS_H */
> --
> 1.7.7.6
--
Christoffer
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH v2] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros
2013-11-14 18:28 ` Christoffer Dall
@ 2013-11-15 1:33 ` Chen Gang
0 siblings, 0 replies; 5+ messages in thread
From: Chen Gang @ 2013-11-15 1:33 UTC (permalink / raw)
To: Christoffer Dall
Cc: Will Deacon, Catalin Marinas, Russell King - ARM Linux,
Uwe Kleine-König, Jonathan Austin, Al Viro,
linux-arm-kernel, linux-kernel, dhowells, kvmarm
On 11/15/2013 02:28 AM, Christoffer Dall wrote:
> On Thu, Nov 14, 2013 at 10:42:31AM +0800, Chen Gang wrote:
>> For all uapi headers, need use "_UAPI" prefix for its guard macro
>> (which will be stripped by "scripts/headers_installer.sh").
>>
>> Additional completions:
>>
>> - be sure that all "endif" need be appended related comment, also it
>> has and only has one empty line above and no any lines below either.
>>
>> - be sure that all normal uapi header files need content guard macro.
>>
>> - remove the default "kvm_para.h" and add 'generic-y' in Kbuild.
>>
>> - remove the detail address of Free Software Foundation (or can not
>> pass "scripts/checkpatch.pl").
>>
>
> for the kvm/arm part I guess it looks fine.
>
OK, thanks.
> -Christoffer
>
>>
>>
>> Signed-off-by: Chen Gang <gang.chen@asianux.com>
>> ---
>> arch/arm/include/uapi/asm/Kbuild | 3 ++-
>> arch/arm/include/uapi/asm/byteorder.h | 7 +++----
>> arch/arm/include/uapi/asm/fcntl.h | 6 +++---
>> arch/arm/include/uapi/asm/ioctls.h | 6 +++---
>> arch/arm/include/uapi/asm/kvm.h | 8 ++++----
>> arch/arm/include/uapi/asm/kvm_para.h | 1 -
>> arch/arm/include/uapi/asm/mman.h | 5 +++++
>> arch/arm/include/uapi/asm/perf_regs.h | 7 ++++---
>> arch/arm/include/uapi/asm/posix_types.h | 6 +++---
>> arch/arm/include/uapi/asm/ptrace.h | 1 -
>> arch/arm/include/uapi/asm/setup.h | 1 -
>> arch/arm/include/uapi/asm/sigcontext.h | 7 +++----
>> arch/arm/include/uapi/asm/signal.h | 1 -
>> arch/arm/include/uapi/asm/stat.h | 6 +++---
>> arch/arm/include/uapi/asm/statfs.h | 7 ++++---
>> 15 files changed, 37 insertions(+), 35 deletions(-)
>> delete mode 100644 arch/arm/include/uapi/asm/kvm_para.h
>>
>> diff --git a/arch/arm/include/uapi/asm/Kbuild b/arch/arm/include/uapi/asm/Kbuild
>> index 70a1c9d..12a65af 100644
>> --- a/arch/arm/include/uapi/asm/Kbuild
>> +++ b/arch/arm/include/uapi/asm/Kbuild
>> @@ -1,11 +1,12 @@
>> # UAPI Header export list
>> include include/uapi/asm-generic/Kbuild.asm
>>
>> +generic-y += kvm_para.h
>> +
>> header-y += byteorder.h
>> header-y += fcntl.h
>> header-y += hwcap.h
>> header-y += ioctls.h
>> -header-y += kvm_para.h
>> header-y += mman.h
>> header-y += perf_regs.h
>> header-y += posix_types.h
>> diff --git a/arch/arm/include/uapi/asm/byteorder.h b/arch/arm/include/uapi/asm/byteorder.h
>> index 7737974..ffd1e93 100644
>> --- a/arch/arm/include/uapi/asm/byteorder.h
>> +++ b/arch/arm/include/uapi/asm/byteorder.h
>> @@ -12,8 +12,8 @@
>> * and word accesses (data or instruction) appear as:
>> * d0...d31
>> */
>> -#ifndef __ASM_ARM_BYTEORDER_H
>> -#define __ASM_ARM_BYTEORDER_H
>> +#ifndef _UAPI__ASM_ARM_BYTEORDER_H
>> +#define _UAPI__ASM_ARM_BYTEORDER_H
>>
>> #ifdef __ARMEB__
>> #include <linux/byteorder/big_endian.h>
>> @@ -21,5 +21,4 @@
>> #include <linux/byteorder/little_endian.h>
>> #endif
>>
>> -#endif
>> -
>> +#endif /* _UAPI__ASM_ARM_BYTEORDER_H */
>> diff --git a/arch/arm/include/uapi/asm/fcntl.h b/arch/arm/include/uapi/asm/fcntl.h
>> index a80b660..4cde9c4 100644
>> --- a/arch/arm/include/uapi/asm/fcntl.h
>> +++ b/arch/arm/include/uapi/asm/fcntl.h
>> @@ -1,5 +1,5 @@
>> -#ifndef _ARM_FCNTL_H
>> -#define _ARM_FCNTL_H
>> +#ifndef _UAPI_ARM_FCNTL_H
>> +#define _UAPI_ARM_FCNTL_H
>>
>> #define O_DIRECTORY 040000 /* must be a directory */
>> #define O_NOFOLLOW 0100000 /* don't follow links */
>> @@ -8,4 +8,4 @@
>>
>> #include <asm-generic/fcntl.h>
>>
>> -#endif
>> +#endif /* _UAPI_ARM_FCNTL_H */
>> diff --git a/arch/arm/include/uapi/asm/ioctls.h b/arch/arm/include/uapi/asm/ioctls.h
>> index 9c96298..a9d8008 100644
>> --- a/arch/arm/include/uapi/asm/ioctls.h
>> +++ b/arch/arm/include/uapi/asm/ioctls.h
>> @@ -1,8 +1,8 @@
>> -#ifndef __ASM_ARM_IOCTLS_H
>> -#define __ASM_ARM_IOCTLS_H
>> +#ifndef _UAPI__ASM_ARM_IOCTLS_H
>> +#define _UAPI__ASM_ARM_IOCTLS_H
>>
>> #define FIOQSIZE 0x545E
>>
>> #include <asm-generic/ioctls.h>
>>
>> -#endif
>> +#endif /* _UAPI__ASM_ARM_IOCTLS_H */
>> diff --git a/arch/arm/include/uapi/asm/kvm.h b/arch/arm/include/uapi/asm/kvm.h
>> index c498b60..f44fad1 100644
>> --- a/arch/arm/include/uapi/asm/kvm.h
>> +++ b/arch/arm/include/uapi/asm/kvm.h
>> @@ -13,11 +13,11 @@
>> *
>> * You should have received a copy of the GNU General Public License
>> * along with this program; if not, write to the Free Software
>> - * Foundation, 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
>> + * Foundation (the address is in a copy of the GPL).
>> */
>>
>> -#ifndef __ARM_KVM_H__
>> -#define __ARM_KVM_H__
>> +#ifndef _UAPI__ARM_KVM_H__
>> +#define _UAPI__ARM_KVM_H__
>>
>> #include <linux/types.h>
>> #include <asm/ptrace.h>
>> @@ -178,4 +178,4 @@ struct kvm_arch_memory_slot {
>> #define KVM_PSCI_RET_INVAL ((unsigned long)-2)
>> #define KVM_PSCI_RET_DENIED ((unsigned long)-3)
>>
>> -#endif /* __ARM_KVM_H__ */
>> +#endif /* _UAPI__ARM_KVM_H__ */
>> diff --git a/arch/arm/include/uapi/asm/kvm_para.h b/arch/arm/include/uapi/asm/kvm_para.h
>> deleted file mode 100644
>> index 14fab8f..0000000
>> --- a/arch/arm/include/uapi/asm/kvm_para.h
>> +++ /dev/null
>> @@ -1 +0,0 @@
>> -#include <asm-generic/kvm_para.h>
>> diff --git a/arch/arm/include/uapi/asm/mman.h b/arch/arm/include/uapi/asm/mman.h
>> index 41f99c5..bf6ac7c 100644
>> --- a/arch/arm/include/uapi/asm/mman.h
>> +++ b/arch/arm/include/uapi/asm/mman.h
>> @@ -1,4 +1,9 @@
>> +#ifndef _UAPI_ARM_MMAN_H
>> +#define _UAPI_ARM_MMAN_H
>> +
>> #include <asm-generic/mman.h>
>>
>> #define arch_mmap_check(addr, len, flags) \
>> (((flags) & MAP_FIXED && (addr) < FIRST_USER_ADDRESS) ? -EINVAL : 0)
>> +
>> +#endif /* _UAPI_ARM_MMAN_H */
>> diff --git a/arch/arm/include/uapi/asm/perf_regs.h b/arch/arm/include/uapi/asm/perf_regs.h
>> index ce59448..61e86fb 100644
>> --- a/arch/arm/include/uapi/asm/perf_regs.h
>> +++ b/arch/arm/include/uapi/asm/perf_regs.h
>> @@ -1,5 +1,5 @@
>> -#ifndef _ASM_ARM_PERF_REGS_H
>> -#define _ASM_ARM_PERF_REGS_H
>> +#ifndef _UAPI_ASM_ARM_PERF_REGS_H
>> +#define _UAPI_ASM_ARM_PERF_REGS_H
>>
>> enum perf_event_arm_regs {
>> PERF_REG_ARM_R0,
>> @@ -20,4 +20,5 @@ enum perf_event_arm_regs {
>> PERF_REG_ARM_PC,
>> PERF_REG_ARM_MAX,
>> };
>> -#endif /* _ASM_ARM_PERF_REGS_H */
>> +
>> +#endif /* _UAPI_ASM_ARM_PERF_REGS_H */
>> diff --git a/arch/arm/include/uapi/asm/posix_types.h b/arch/arm/include/uapi/asm/posix_types.h
>> index d2de9cb..d0e3856 100644
>> --- a/arch/arm/include/uapi/asm/posix_types.h
>> +++ b/arch/arm/include/uapi/asm/posix_types.h
>> @@ -10,8 +10,8 @@
>> * Changelog:
>> * 27-06-1996 RMK Created
>> */
>> -#ifndef __ARCH_ARM_POSIX_TYPES_H
>> -#define __ARCH_ARM_POSIX_TYPES_H
>> +#ifndef _UAPI__ARCH_ARM_POSIX_TYPES_H
>> +#define _UAPI__ARCH_ARM_POSIX_TYPES_H
>>
>> /*
>> * This file is generally used by user-level software, so you need to
>> @@ -34,4 +34,4 @@ typedef unsigned short __kernel_old_dev_t;
>>
>> #include <asm-generic/posix_types.h>
>>
>> -#endif
>> +#endif /* _UAPI__ARCH_ARM_POSIX_TYPES_H */
>> diff --git a/arch/arm/include/uapi/asm/ptrace.h b/arch/arm/include/uapi/asm/ptrace.h
>> index 5af0ed1..9fd79dd 100644
>> --- a/arch/arm/include/uapi/asm/ptrace.h
>> +++ b/arch/arm/include/uapi/asm/ptrace.h
>> @@ -151,7 +151,6 @@ struct pt_regs {
>> */
>> #define ARM_VFPREGS_SIZE ( 32 * 8 /*fpregs*/ + 4 /*fpscr*/ )
>>
>> -
>> #endif /* __ASSEMBLY__ */
>>
>> #endif /* _UAPI__ASM_ARM_PTRACE_H */
>> diff --git a/arch/arm/include/uapi/asm/setup.h b/arch/arm/include/uapi/asm/setup.h
>> index 979ff40..be889f3 100644
>> --- a/arch/arm/include/uapi/asm/setup.h
>> +++ b/arch/arm/include/uapi/asm/setup.h
>> @@ -183,5 +183,4 @@ struct tagtable {
>> #define for_each_tag(t,base) \
>> for (t = base; t->hdr.size; t = tag_next(t))
>>
>> -
>> #endif /* _UAPI__ASMARM_SETUP_H */
>> diff --git a/arch/arm/include/uapi/asm/sigcontext.h b/arch/arm/include/uapi/asm/sigcontext.h
>> index fc0b80b..f6aa372 100644
>> --- a/arch/arm/include/uapi/asm/sigcontext.h
>> +++ b/arch/arm/include/uapi/asm/sigcontext.h
>> @@ -1,5 +1,5 @@
>> -#ifndef _ASMARM_SIGCONTEXT_H
>> -#define _ASMARM_SIGCONTEXT_H
>> +#ifndef _UAPI_ASMARM_SIGCONTEXT_H
>> +#define _UAPI_ASMARM_SIGCONTEXT_H
>>
>> /*
>> * Signal context structure - contains all info to do with the state
>> @@ -30,5 +30,4 @@ struct sigcontext {
>> unsigned long fault_address;
>> };
>>
>> -
>> -#endif
>> +#endif /* _UAPI_ASMARM_SIGCONTEXT_H */
>> diff --git a/arch/arm/include/uapi/asm/signal.h b/arch/arm/include/uapi/asm/signal.h
>> index 33073bd..8954e9a 100644
>> --- a/arch/arm/include/uapi/asm/signal.h
>> +++ b/arch/arm/include/uapi/asm/signal.h
>> @@ -116,5 +116,4 @@ typedef struct sigaltstack {
>> size_t ss_size;
>> } stack_t;
>>
>> -
>> #endif /* _UAPI_ASMARM_SIGNAL_H */
>> diff --git a/arch/arm/include/uapi/asm/stat.h b/arch/arm/include/uapi/asm/stat.h
>> index 42c0c13..57addf5 100644
>> --- a/arch/arm/include/uapi/asm/stat.h
>> +++ b/arch/arm/include/uapi/asm/stat.h
>> @@ -1,5 +1,5 @@
>> -#ifndef _ASMARM_STAT_H
>> -#define _ASMARM_STAT_H
>> +#ifndef _UAPI_ASMARM_STAT_H
>> +#define _UAPI_ASMARM_STAT_H
>>
>> struct __old_kernel_stat {
>> unsigned short st_dev;
>> @@ -84,4 +84,4 @@ struct stat64 {
>> unsigned long long st_ino;
>> };
>>
>> -#endif
>> +#endif /* _UAPI_ASMARM_STAT_H */
>> diff --git a/arch/arm/include/uapi/asm/statfs.h b/arch/arm/include/uapi/asm/statfs.h
>> index 079447c..ebcb391 100644
>> --- a/arch/arm/include/uapi/asm/statfs.h
>> +++ b/arch/arm/include/uapi/asm/statfs.h
>> @@ -1,5 +1,5 @@
>> -#ifndef _ASMARM_STATFS_H
>> -#define _ASMARM_STATFS_H
>> +#ifndef _UAPI_ASMARM_STATFS_H
>> +#define _UAPI_ASMARM_STATFS_H
>>
>> /*
>> * With EABI there is 4 bytes of padding added to this structure.
>> @@ -9,4 +9,5 @@
>> #define ARCH_PACK_STATFS64 __attribute__((packed,aligned(4)))
>>
>> #include <asm-generic/statfs.h>
>> -#endif
>> +
>> +#endif /* _UAPI_ASMARM_STATFS_H */
>> --
>> 1.7.7.6
>
--
Chen Gang
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2013-11-15 1:31 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-11-08 3:47 [PATCH] arch: arm: uapi: be sure of "_UAPI" prefix for all guard macros Chen Gang
2013-11-14 2:41 ` Chen Gang
2013-11-14 2:42 ` [PATCH v2] " Chen Gang
2013-11-14 18:28 ` Christoffer Dall
2013-11-15 1:33 ` Chen Gang
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).