* [PATCH 1/9] vdso: Remove timekeeper argument of __arch_update_vsyscall()
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-14 13:59 ` Will Deacon
2024-10-10 15:44 ` [PATCH 2/9] arm: vdso: Remove timekeeper includes Thomas Weißschuh
` (7 subsequent siblings)
8 siblings, 1 reply; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
No implementation of this hook uses the passed in timekeeper anymore.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/arm64/include/asm/vdso/vsyscall.h | 3 +--
include/asm-generic/vdso/vsyscall.h | 3 +--
kernel/time/vsyscall.c | 2 +-
3 files changed, 3 insertions(+), 5 deletions(-)
diff --git a/arch/arm64/include/asm/vdso/vsyscall.h b/arch/arm64/include/asm/vdso/vsyscall.h
index 5b6d0dd3cef5483ab8166a24ab3c7ca956927350..eea51946d45a2f8c7eebfff971d74878be53a798 100644
--- a/arch/arm64/include/asm/vdso/vsyscall.h
+++ b/arch/arm64/include/asm/vdso/vsyscall.h
@@ -6,7 +6,6 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
enum vvar_pages {
@@ -37,7 +36,7 @@ struct vdso_rng_data *__arm64_get_k_vdso_rnd_data(void)
#define __arch_get_k_vdso_rng_data __arm64_get_k_vdso_rnd_data
static __always_inline
-void __arm64_update_vsyscall(struct vdso_data *vdata, struct timekeeper *tk)
+void __arm64_update_vsyscall(struct vdso_data *vdata)
{
vdata[CS_HRES_COARSE].mask = VDSO_PRECISION_MASK;
vdata[CS_RAW].mask = VDSO_PRECISION_MASK;
diff --git a/include/asm-generic/vdso/vsyscall.h b/include/asm-generic/vdso/vsyscall.h
index c835607f78ae990e7479878767b05cccd3a99837..01dafd604188fb0512d21c4ce4b027f7da54f5a0 100644
--- a/include/asm-generic/vdso/vsyscall.h
+++ b/include/asm-generic/vdso/vsyscall.h
@@ -12,8 +12,7 @@ static __always_inline struct vdso_data *__arch_get_k_vdso_data(void)
#endif /* __arch_get_k_vdso_data */
#ifndef __arch_update_vsyscall
-static __always_inline void __arch_update_vsyscall(struct vdso_data *vdata,
- struct timekeeper *tk)
+static __always_inline void __arch_update_vsyscall(struct vdso_data *vdata)
{
}
#endif /* __arch_update_vsyscall */
diff --git a/kernel/time/vsyscall.c b/kernel/time/vsyscall.c
index 9193d6133e5d688342be8485b9a0efb7af9ee148..28706a13c222df29635fd22d23f21eacb068855c 100644
--- a/kernel/time/vsyscall.c
+++ b/kernel/time/vsyscall.c
@@ -119,7 +119,7 @@ void update_vsyscall(struct timekeeper *tk)
if (clock_mode != VDSO_CLOCKMODE_NONE)
update_vdso_data(vdata, tk);
- __arch_update_vsyscall(vdata, tk);
+ __arch_update_vsyscall(vdata);
vdso_write_end(vdata);
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 1/9] vdso: Remove timekeeper argument of __arch_update_vsyscall()
2024-10-10 15:44 ` [PATCH 1/9] vdso: Remove timekeeper argument of __arch_update_vsyscall() Thomas Weißschuh
@ 2024-10-14 13:59 ` Will Deacon
0 siblings, 0 replies; 13+ messages in thread
From: Will Deacon @ 2024-10-14 13:59 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Catalin Marinas, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King, linux-arm-kernel, linux-kernel, linux-arch,
linux-mips, linux-s390, loongarch, linuxppc-dev, linux-riscv
On Thu, Oct 10, 2024 at 05:44:44PM +0200, Thomas Weißschuh wrote:
> No implementation of this hook uses the passed in timekeeper anymore.
>
> This avoids including a non-VDSO header while building the VDSO,
> which can lead to compilation errors.
>
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---
> arch/arm64/include/asm/vdso/vsyscall.h | 3 +--
> include/asm-generic/vdso/vsyscall.h | 3 +--
> kernel/time/vsyscall.c | 2 +-
> 3 files changed, 3 insertions(+), 5 deletions(-)
Acked-by: Will Deacon <will@kernel.org>
Will
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/9] arm: vdso: Remove timekeeper includes
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 1/9] vdso: Remove timekeeper argument of __arch_update_vsyscall() Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 3/9] arm64: vdso: Remove timekeeper include Thomas Weißschuh
` (6 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Also drop the comment which is out of date and in the wrong place.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/arm/include/asm/vdso/vsyscall.h | 4 ----
arch/arm/kernel/vdso.c | 1 -
2 files changed, 5 deletions(-)
diff --git a/arch/arm/include/asm/vdso/vsyscall.h b/arch/arm/include/asm/vdso/vsyscall.h
index 47e41ae8ccd0b997b54dda59abea8ae98bbe3c56..705414710dcdbfa9a97c344806bd079c08368801 100644
--- a/arch/arm/include/asm/vdso/vsyscall.h
+++ b/arch/arm/include/asm/vdso/vsyscall.h
@@ -4,16 +4,12 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
#include <asm/cacheflush.h>
extern struct vdso_data *vdso_data;
extern bool cntvct_ok;
-/*
- * Update the vDSO data page to keep in sync with kernel timekeeping.
- */
static __always_inline
struct vdso_data *__arm_get_k_vdso_data(void)
{
diff --git a/arch/arm/kernel/vdso.c b/arch/arm/kernel/vdso.c
index d499ad461b004b05e1f0f13cbedad71b587f8478..29dd2f3c62fec64c7f290468421bfad1e739c667 100644
--- a/arch/arm/kernel/vdso.c
+++ b/arch/arm/kernel/vdso.c
@@ -14,7 +14,6 @@
#include <linux/of.h>
#include <linux/printk.h>
#include <linux/slab.h>
-#include <linux/timekeeper_internal.h>
#include <linux/vmalloc.h>
#include <asm/arch_timer.h>
#include <asm/barrier.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 3/9] arm64: vdso: Remove timekeeper include
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 1/9] vdso: Remove timekeeper argument of __arch_update_vsyscall() Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 2/9] arm: vdso: Remove timekeeper includes Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-14 13:59 ` Will Deacon
2024-10-10 15:44 ` [PATCH 4/9] powerpc/vdso: Remove timekeeper includes Thomas Weißschuh
` (5 subsequent siblings)
8 siblings, 1 reply; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/arm64/kernel/vdso.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/arm64/kernel/vdso.c b/arch/arm64/kernel/vdso.c
index 706c9c3a7a50a4574e77da296e9c83e1e2a9f5ab..8ef20c16bc482e92de8098d55000c9999b89830e 100644
--- a/arch/arm64/kernel/vdso.c
+++ b/arch/arm64/kernel/vdso.c
@@ -19,7 +19,6 @@
#include <linux/signal.h>
#include <linux/slab.h>
#include <linux/time_namespace.h>
-#include <linux/timekeeper_internal.h>
#include <linux/vmalloc.h>
#include <vdso/datapage.h>
#include <vdso/helpers.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 3/9] arm64: vdso: Remove timekeeper include
2024-10-10 15:44 ` [PATCH 3/9] arm64: vdso: Remove timekeeper include Thomas Weißschuh
@ 2024-10-14 13:59 ` Will Deacon
0 siblings, 0 replies; 13+ messages in thread
From: Will Deacon @ 2024-10-14 13:59 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Catalin Marinas, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King, linux-arm-kernel, linux-kernel, linux-arch,
linux-mips, linux-s390, loongarch, linuxppc-dev, linux-riscv
On Thu, Oct 10, 2024 at 05:44:46PM +0200, Thomas Weißschuh wrote:
> Since the generic VDSO clock mode storage is used, this header file is
> unused and can be removed.
>
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---
> arch/arm64/kernel/vdso.c | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/kernel/vdso.c b/arch/arm64/kernel/vdso.c
> index 706c9c3a7a50a4574e77da296e9c83e1e2a9f5ab..8ef20c16bc482e92de8098d55000c9999b89830e 100644
> --- a/arch/arm64/kernel/vdso.c
> +++ b/arch/arm64/kernel/vdso.c
> @@ -19,7 +19,6 @@
> #include <linux/signal.h>
> #include <linux/slab.h>
> #include <linux/time_namespace.h>
> -#include <linux/timekeeper_internal.h>
> #include <linux/vmalloc.h>
> #include <vdso/datapage.h>
> #include <vdso/helpers.h>
Acked-by: Will Deacon <will@kernel.org>
Will
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 4/9] powerpc/vdso: Remove timekeeper includes
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (2 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 3/9] arm64: vdso: Remove timekeeper include Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 5/9] riscv: vdso: Remove timekeeper include Thomas Weißschuh
` (4 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Also drop the comment which is out of date and in the wrong place.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/powerpc/include/asm/vdso/vsyscall.h | 4 ----
arch/powerpc/kernel/time.c | 1 -
2 files changed, 5 deletions(-)
diff --git a/arch/powerpc/include/asm/vdso/vsyscall.h b/arch/powerpc/include/asm/vdso/vsyscall.h
index 92f480d8cc6dcb905d5960185ca8b615cfc541ee..48560a11955956b8fbb59360334a81972723bd57 100644
--- a/arch/powerpc/include/asm/vdso/vsyscall.h
+++ b/arch/powerpc/include/asm/vdso/vsyscall.h
@@ -4,12 +4,8 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <asm/vdso_datapage.h>
-/*
- * Update the vDSO data page to keep in sync with kernel timekeeping.
- */
static __always_inline
struct vdso_data *__arch_get_k_vdso_data(void)
{
diff --git a/arch/powerpc/kernel/time.c b/arch/powerpc/kernel/time.c
index 0ff9f038e800db6dbc910ce581550b457ee5f2db..4a95654c1d36f25d6021284889fdd1510782b408 100644
--- a/arch/powerpc/kernel/time.c
+++ b/arch/powerpc/kernel/time.c
@@ -75,7 +75,6 @@
/* powerpc clocksource/clockevent code */
#include <linux/clockchips.h>
-#include <linux/timekeeper_internal.h>
static u64 timebase_read(struct clocksource *);
static struct clocksource clocksource_timebase = {
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 5/9] riscv: vdso: Remove timekeeper include
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (3 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 4/9] powerpc/vdso: Remove timekeeper includes Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 6/9] s390/vdso: Remove timekeeper includes Thomas Weißschuh
` (3 subsequent siblings)
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Also drop the comment which is out of date and in the wrong place.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/riscv/include/asm/vdso/vsyscall.h | 4 ----
1 file changed, 4 deletions(-)
diff --git a/arch/riscv/include/asm/vdso/vsyscall.h b/arch/riscv/include/asm/vdso/vsyscall.h
index 82fd5d83bd602c72c1688a7dffa292b37b5fa790..e8a9c4b53c0c9f4744196eed800b21f3918d1040 100644
--- a/arch/riscv/include/asm/vdso/vsyscall.h
+++ b/arch/riscv/include/asm/vdso/vsyscall.h
@@ -4,14 +4,10 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
extern struct vdso_data *vdso_data;
-/*
- * Update the vDSO data page to keep in sync with kernel timekeeping.
- */
static __always_inline struct vdso_data *__riscv_get_k_vdso_data(void)
{
return vdso_data;
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 6/9] s390/vdso: Remove timekeeper includes
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (4 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 5/9] riscv: vdso: Remove timekeeper include Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 16:01 ` Heiko Carstens
2024-10-10 15:44 ` [PATCH 7/9] x86/vdso: Remove timekeeper include Thomas Weißschuh
` (2 subsequent siblings)
8 siblings, 1 reply; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Also drop the comment which is out of date and in the wrong place.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/s390/include/asm/vdso/vsyscall.h | 5 -----
arch/s390/kernel/time.c | 1 -
2 files changed, 6 deletions(-)
diff --git a/arch/s390/include/asm/vdso/vsyscall.h b/arch/s390/include/asm/vdso/vsyscall.h
index 3c5d5e47814e16dcfc20f9901481127246e8f348..3eb576ecd3bd998daf9372a52befa8b76d52f2bf 100644
--- a/arch/s390/include/asm/vdso/vsyscall.h
+++ b/arch/s390/include/asm/vdso/vsyscall.h
@@ -7,7 +7,6 @@
#ifndef __ASSEMBLY__
#include <linux/hrtimer.h>
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
#include <asm/vdso.h>
@@ -17,10 +16,6 @@ enum vvar_pages {
VVAR_NR_PAGES
};
-/*
- * Update the vDSO data page to keep in sync with kernel timekeeping.
- */
-
static __always_inline struct vdso_data *__s390_get_k_vdso_data(void)
{
return vdso_data;
diff --git a/arch/s390/kernel/time.c b/arch/s390/kernel/time.c
index b713effe057967623f64da8297314fcc42ff1af2..4fae6c701784746519bab0b295429082529e4501 100644
--- a/arch/s390/kernel/time.c
+++ b/arch/s390/kernel/time.c
@@ -36,7 +36,6 @@
#include <linux/profile.h>
#include <linux/timex.h>
#include <linux/notifier.h>
-#include <linux/timekeeper_internal.h>
#include <linux/clockchips.h>
#include <linux/gfp.h>
#include <linux/kprobes.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* Re: [PATCH 6/9] s390/vdso: Remove timekeeper includes
2024-10-10 15:44 ` [PATCH 6/9] s390/vdso: Remove timekeeper includes Thomas Weißschuh
@ 2024-10-10 16:01 ` Heiko Carstens
0 siblings, 0 replies; 13+ messages in thread
From: Heiko Carstens @ 2024-10-10 16:01 UTC (permalink / raw)
To: Thomas Weißschuh
Cc: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Vasily Gorbik, Alexander Gordeev, Christian Borntraeger,
Sven Schnelle, Huacai Chen, WANG Xuerui, Michael Ellerman,
Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King, linux-arm-kernel, linux-kernel, linux-arch,
linux-mips, linux-s390, loongarch, linuxppc-dev, linux-riscv
On Thu, Oct 10, 2024 at 05:44:49PM +0200, Thomas Weißschuh wrote:
> Since the generic VDSO clock mode storage is used, this header file is
> unused and can be removed.
>
> This avoids including a non-VDSO header while building the VDSO,
> which can lead to compilation errors.
>
> Also drop the comment which is out of date and in the wrong place.
>
> Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
> ---
> arch/s390/include/asm/vdso/vsyscall.h | 5 -----
> arch/s390/kernel/time.c | 1 -
> 2 files changed, 6 deletions(-)
Acked-by: Heiko Carstens <hca@linux.ibm.com>
^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 7/9] x86/vdso: Remove timekeeper include
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (5 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 6/9] s390/vdso: Remove timekeeper includes Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 8/9] LoongArch: vdso: Remove timekeeper includes Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 9/9] MIPS: " Thomas Weißschuh
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/x86/include/asm/vdso/vsyscall.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/arch/x86/include/asm/vdso/vsyscall.h b/arch/x86/include/asm/vdso/vsyscall.h
index 67fedf1698b5e2b710e0504686318949c738bf29..a1f916b18400f65eb91d5ec0e3eb49b070332904 100644
--- a/arch/x86/include/asm/vdso/vsyscall.h
+++ b/arch/x86/include/asm/vdso/vsyscall.h
@@ -4,7 +4,6 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
#include <asm/vgtod.h>
#include <asm/vvar.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 8/9] LoongArch: vdso: Remove timekeeper includes
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (6 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 7/9] x86/vdso: Remove timekeeper include Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
2024-10-10 15:44 ` [PATCH 9/9] MIPS: " Thomas Weißschuh
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Also drop the comment which is out of date and in the wrong place.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/loongarch/include/asm/vdso/vsyscall.h | 4 ----
arch/loongarch/kernel/vdso.c | 1 -
2 files changed, 5 deletions(-)
diff --git a/arch/loongarch/include/asm/vdso/vsyscall.h b/arch/loongarch/include/asm/vdso/vsyscall.h
index b1273ce6f140691ae8104b2be3d1203ebc57fac2..8987e951d0a93c34ca75de676fb9c191ff4ef3c2 100644
--- a/arch/loongarch/include/asm/vdso/vsyscall.h
+++ b/arch/loongarch/include/asm/vdso/vsyscall.h
@@ -4,15 +4,11 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
extern struct vdso_data *vdso_data;
extern struct vdso_rng_data *vdso_rng_data;
-/*
- * Update the vDSO data page to keep in sync with kernel timekeeping.
- */
static __always_inline
struct vdso_data *__loongarch_get_k_vdso_data(void)
{
diff --git a/arch/loongarch/kernel/vdso.c b/arch/loongarch/kernel/vdso.c
index f6fcc52aefae0043e307327b8e7a5872fad0822a..4d7cb9425dc34584dd7abdb80a784a4f2932e1b7 100644
--- a/arch/loongarch/kernel/vdso.c
+++ b/arch/loongarch/kernel/vdso.c
@@ -15,7 +15,6 @@
#include <linux/sched.h>
#include <linux/slab.h>
#include <linux/time_namespace.h>
-#include <linux/timekeeper_internal.h>
#include <asm/page.h>
#include <asm/vdso.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread* [PATCH 9/9] MIPS: vdso: Remove timekeeper includes
2024-10-10 15:44 [PATCH 0/9] vdso: Remove timekeeper argument and includes Thomas Weißschuh
` (7 preceding siblings ...)
2024-10-10 15:44 ` [PATCH 8/9] LoongArch: vdso: Remove timekeeper includes Thomas Weißschuh
@ 2024-10-10 15:44 ` Thomas Weißschuh
8 siblings, 0 replies; 13+ messages in thread
From: Thomas Weißschuh @ 2024-10-10 15:44 UTC (permalink / raw)
To: Catalin Marinas, Will Deacon, Andy Lutomirski, Thomas Gleixner,
Vincenzo Frascino, Arnd Bergmann, Thomas Bogendoerfer,
Ingo Molnar, Borislav Petkov, Dave Hansen, x86, H. Peter Anvin,
Heiko Carstens, Vasily Gorbik, Alexander Gordeev,
Christian Borntraeger, Sven Schnelle, Huacai Chen, WANG Xuerui,
Michael Ellerman, Nicholas Piggin, Christophe Leroy, Naveen N Rao,
Madhavan Srinivasan, Paul Walmsley, Palmer Dabbelt, Albert Ou,
Russell King
Cc: linux-arm-kernel, linux-kernel, linux-arch, linux-mips,
linux-s390, loongarch, linuxppc-dev, linux-riscv,
Thomas Weißschuh
Since the generic VDSO clock mode storage is used, this header file is
unused and can be removed.
This avoids including a non-VDSO header while building the VDSO,
which can lead to compilation errors.
Signed-off-by: Thomas Weißschuh <thomas.weissschuh@linutronix.de>
---
arch/mips/include/asm/vdso/vsyscall.h | 1 -
arch/mips/kernel/vdso.c | 1 -
2 files changed, 2 deletions(-)
diff --git a/arch/mips/include/asm/vdso/vsyscall.h b/arch/mips/include/asm/vdso/vsyscall.h
index 47168aaf1eff051199cf668d584e903b1eb8a3be..a4582870aaea49ac288d62ec4fa1338a98621918 100644
--- a/arch/mips/include/asm/vdso/vsyscall.h
+++ b/arch/mips/include/asm/vdso/vsyscall.h
@@ -4,7 +4,6 @@
#ifndef __ASSEMBLY__
-#include <linux/timekeeper_internal.h>
#include <vdso/datapage.h>
extern struct vdso_data *vdso_data;
diff --git a/arch/mips/kernel/vdso.c b/arch/mips/kernel/vdso.c
index dda36fa26307e27d3de414c811450ed912294a0e..4c8e3c0aa210476d7b8cb349b99e9a5a453aa7ce 100644
--- a/arch/mips/kernel/vdso.c
+++ b/arch/mips/kernel/vdso.c
@@ -14,7 +14,6 @@
#include <linux/random.h>
#include <linux/sched.h>
#include <linux/slab.h>
-#include <linux/timekeeper_internal.h>
#include <asm/abi.h>
#include <asm/mips-cps.h>
--
2.47.0
^ permalink raw reply related [flat|nested] 13+ messages in thread