[PATCH] time: Add macros to prevent duplicate inclusion of headers

Qiuer Zhu posted 1 patch 11 months ago
kernel/time/posix-timers.h  | 5 +++++
kernel/time/tick-internal.h | 5 +++++
2 files changed, 10 insertions(+)
[PATCH] time: Add macros to prevent duplicate inclusion of headers
Posted by Qiuer Zhu 11 months ago
Add macros to prevent duplicate inclusion of tick-internal.h
and posix-timers.h.

Signed-off-by: Qiuer Zhu <chaozyhaha@gmail.com>
---
 kernel/time/posix-timers.h  | 5 +++++
 kernel/time/tick-internal.h | 5 +++++
 2 files changed, 10 insertions(+)

diff --git a/kernel/time/posix-timers.h b/kernel/time/posix-timers.h
index 61906f0688c1..5166b832143f 100644
--- a/kernel/time/posix-timers.h
+++ b/kernel/time/posix-timers.h
@@ -1,4 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0 */
+#ifndef _POSIX_TIMERS_H
+#define _POSIX_TIMERS_H
+
 #define TIMER_RETRY 1
 
 enum posix_timer_state {
@@ -50,3 +53,5 @@ int common_timer_set(struct k_itimer *timr, int flags,
 		     struct itimerspec64 *old_setting);
 void posix_timer_set_common(struct k_itimer *timer, struct itimerspec64 *new_setting);
 int common_timer_del(struct k_itimer *timer);
+
+#endif
diff --git a/kernel/time/tick-internal.h b/kernel/time/tick-internal.h
index faac36de35b9..bcccc0975f70 100644
--- a/kernel/time/tick-internal.h
+++ b/kernel/time/tick-internal.h
@@ -1,4 +1,7 @@
 /* SPDX-License-Identifier: GPL-2.0 */
+#ifndef _TICK_INTERNAL_H
+#define _TICK_INTERNAL_H
+
 /*
  * tick internal variable and functions used by low/high res code
  */
@@ -213,3 +216,5 @@ void hrtimers_resume_local(void);
 #endif
 
 extern ssize_t sysfs_get_uname(const char *buf, char *dst, size_t cnt);
+
+#endif
-- 
2.32.0.windows.2
Re: [PATCH] time: Add macros to prevent duplicate inclusion of headers
Posted by Thomas Gleixner 11 months ago
On Sun, Mar 09 2025 at 23:05, Qiuer Zhu wrote:
> Add macros to prevent duplicate inclusion of tick-internal.h
> and posix-timers.h.

That's a problem for public headers, but not for directory local
headers. They don't suffer the problem of accidental recursion.

Thanks,

        tglx