* [PATCH 10/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in uapi header
[not found] <20250314071013.1575167-1-thuth@redhat.com>
@ 2025-03-14 7:09 ` Thomas Huth
2025-03-14 7:09 ` [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers Thomas Huth
1 sibling, 0 replies; 8+ messages in thread
From: Thomas Huth @ 2025-03-14 7:09 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnd Bergmann, linux-arch, Thomas Huth, Guo Ren, linux-csky
__ASSEMBLY__ is only defined by the Makefile of the kernel, so
this is not really useful for uapi headers (unless the userspace
Makefile defines it, too). Let's switch to __ASSEMBLER__ which
gets set automatically by the compiler when compiling assembly
code.
Cc: Guo Ren <guoren@kernel.org>
Cc: linux-csky@vger.kernel.org
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
arch/csky/include/uapi/asm/ptrace.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/arch/csky/include/uapi/asm/ptrace.h b/arch/csky/include/uapi/asm/ptrace.h
index 3be9c14334a6e..90a5c36e4345d 100644
--- a/arch/csky/include/uapi/asm/ptrace.h
+++ b/arch/csky/include/uapi/asm/ptrace.h
@@ -3,7 +3,7 @@
#ifndef _CSKY_PTRACE_H
#define _CSKY_PTRACE_H
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
struct pt_regs {
unsigned long tls;
@@ -47,5 +47,5 @@ struct user_fp {
unsigned long reserved;
};
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* _CSKY_PTRACE_H */
--
2.48.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
[not found] <20250314071013.1575167-1-thuth@redhat.com>
2025-03-14 7:09 ` [PATCH 10/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in uapi header Thomas Huth
@ 2025-03-14 7:09 ` Thomas Huth
2025-06-05 18:28 ` Thomas Huth
1 sibling, 1 reply; 8+ messages in thread
From: Thomas Huth @ 2025-03-14 7:09 UTC (permalink / raw)
To: linux-kernel; +Cc: Arnd Bergmann, linux-arch, Thomas Huth, Guo Ren, linux-csky
While the GCC and Clang compilers already define __ASSEMBLER__
automatically when compiling assembly code, __ASSEMBLY__ is a
macro that only gets defined by the Makefiles in the kernel.
This can be very confusing when switching between userspace
and kernelspace coding, or when dealing with uapi headers that
rather should use __ASSEMBLER__ instead. So let's standardize on
the __ASSEMBLER__ macro that is provided by the compilers now.
This is a completely mechanical patch (done with a simple "sed -i"
statement).
Cc: Guo Ren <guoren@kernel.org>
Cc: linux-csky@vger.kernel.org
Signed-off-by: Thomas Huth <thuth@redhat.com>
---
arch/csky/abiv1/inc/abi/regdef.h | 2 +-
arch/csky/abiv2/inc/abi/regdef.h | 2 +-
arch/csky/include/asm/barrier.h | 4 ++--
arch/csky/include/asm/cache.h | 2 +-
arch/csky/include/asm/ftrace.h | 4 ++--
arch/csky/include/asm/jump_label.h | 4 ++--
arch/csky/include/asm/page.h | 4 ++--
arch/csky/include/asm/ptrace.h | 4 ++--
arch/csky/include/asm/string.h | 2 +-
arch/csky/include/asm/thread_info.h | 4 ++--
10 files changed, 16 insertions(+), 16 deletions(-)
diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
index 7b386fd670702..c75ecf2cafd7c 100644
--- a/arch/csky/abiv1/inc/abi/regdef.h
+++ b/arch/csky/abiv1/inc/abi/regdef.h
@@ -3,7 +3,7 @@
#ifndef __ASM_CSKY_REGDEF_H
#define __ASM_CSKY_REGDEF_H
-#ifdef __ASSEMBLY__
+#ifdef __ASSEMBLER__
#define syscallid r1
#else
#define syscallid "r1"
diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
index 0933addbc27b7..fc08d56ccdbe1 100644
--- a/arch/csky/abiv2/inc/abi/regdef.h
+++ b/arch/csky/abiv2/inc/abi/regdef.h
@@ -3,7 +3,7 @@
#ifndef __ASM_CSKY_REGDEF_H
#define __ASM_CSKY_REGDEF_H
-#ifdef __ASSEMBLY__
+#ifdef __ASSEMBLER__
#define syscallid r7
#else
#define syscallid "r7"
diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
index 15de58b10aece..c33fdcfe3770c 100644
--- a/arch/csky/include/asm/barrier.h
+++ b/arch/csky/include/asm/barrier.h
@@ -3,7 +3,7 @@
#ifndef __ASM_CSKY_BARRIER_H
#define __ASM_CSKY_BARRIER_H
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#define nop() asm volatile ("nop\n":::"memory")
@@ -84,5 +84,5 @@
#include <asm-generic/barrier.h>
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* __ASM_CSKY_BARRIER_H */
diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
index 4b5c09bf1d25e..d575482e0fcec 100644
--- a/arch/csky/include/asm/cache.h
+++ b/arch/csky/include/asm/cache.h
@@ -10,7 +10,7 @@
#define ARCH_DMA_MINALIGN L1_CACHE_BYTES
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
void dcache_wb_line(unsigned long start);
diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
index 00f9f7647e3f3..21532f2180587 100644
--- a/arch/csky/include/asm/ftrace.h
+++ b/arch/csky/include/asm/ftrace.h
@@ -11,7 +11,7 @@
#define MCOUNT_ADDR ((unsigned long)_mcount)
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
extern void _mcount(unsigned long);
@@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
unsigned long frame_pointer);
-#endif /* !__ASSEMBLY__ */
+#endif /* !__ASSEMBLER__ */
#endif /* __ASM_CSKY_FTRACE_H */
diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
index ef2e37a10a0fe..603b0caa80249 100644
--- a/arch/csky/include/asm/jump_label.h
+++ b/arch/csky/include/asm/jump_label.h
@@ -3,7 +3,7 @@
#ifndef __ASM_CSKY_JUMP_LABEL_H
#define __ASM_CSKY_JUMP_LABEL_H
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#include <linux/types.h>
@@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
enum jump_label_type type);
#define arch_jump_label_transform_static arch_jump_label_transform_static
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* __ASM_CSKY_JUMP_LABEL_H */
diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
index 4911d0892b71d..76774dbce8697 100644
--- a/arch/csky/include/asm/page.h
+++ b/arch/csky/include/asm/page.h
@@ -26,7 +26,7 @@
#define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#include <linux/pfn.h>
@@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
#include <asm-generic/memory_model.h>
#include <asm-generic/getorder.h>
-#endif /* !__ASSEMBLY__ */
+#endif /* !__ASSEMBLER__ */
#endif /* __ASM_CSKY_PAGE_H */
diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
index 0634b7895d81d..5f01839e11843 100644
--- a/arch/csky/include/asm/ptrace.h
+++ b/arch/csky/include/asm/ptrace.h
@@ -8,7 +8,7 @@
#include <linux/types.h>
#include <linux/compiler.h>
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#define PS_S 0x80000000 /* Supervisor Mode */
@@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
asmlinkage int syscall_trace_enter(struct pt_regs *regs);
asmlinkage void syscall_trace_exit(struct pt_regs *regs);
-#endif /* __ASSEMBLY__ */
+#endif /* __ASSEMBLER__ */
#endif /* __ASM_CSKY_PTRACE_H */
diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
index a0d81e9d6b8f6..82e99f52b547c 100644
--- a/arch/csky/include/asm/string.h
+++ b/arch/csky/include/asm/string.h
@@ -3,7 +3,7 @@
#ifndef _CSKY_STRING_MM_H_
#define _CSKY_STRING_MM_H_
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#include <linux/types.h>
#include <linux/compiler.h>
#include <abi/string.h>
diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
index b5ed788f0c681..fdd4f8ad45acf 100644
--- a/arch/csky/include/asm/thread_info.h
+++ b/arch/csky/include/asm/thread_info.h
@@ -3,7 +3,7 @@
#ifndef _ASM_CSKY_THREAD_INFO_H
#define _ASM_CSKY_THREAD_INFO_H
-#ifndef __ASSEMBLY__
+#ifndef __ASSEMBLER__
#include <asm/types.h>
#include <asm/page.h>
@@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
}
-#endif /* !__ASSEMBLY__ */
+#endif /* !__ASSEMBLER__ */
#define TIF_SIGPENDING 0 /* signal pending */
#define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
--
2.48.1
^ permalink raw reply related [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-03-14 7:09 ` [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers Thomas Huth
@ 2025-06-05 18:28 ` Thomas Huth
2025-06-06 2:23 ` Guo Ren
0 siblings, 1 reply; 8+ messages in thread
From: Thomas Huth @ 2025-06-05 18:28 UTC (permalink / raw)
To: Guo Ren; +Cc: linux-csky
On 14/03/2025 08.09, Thomas Huth wrote:
> While the GCC and Clang compilers already define __ASSEMBLER__
> automatically when compiling assembly code, __ASSEMBLY__ is a
> macro that only gets defined by the Makefiles in the kernel.
> This can be very confusing when switching between userspace
> and kernelspace coding, or when dealing with uapi headers that
> rather should use __ASSEMBLER__ instead. So let's standardize on
> the __ASSEMBLER__ macro that is provided by the compilers now.
>
> This is a completely mechanical patch (done with a simple "sed -i"
> statement).
>
> Cc: Guo Ren <guoren@kernel.org>
> Cc: linux-csky@vger.kernel.org
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
> arch/csky/abiv1/inc/abi/regdef.h | 2 +-
> arch/csky/abiv2/inc/abi/regdef.h | 2 +-
> arch/csky/include/asm/barrier.h | 4 ++--
> arch/csky/include/asm/cache.h | 2 +-
> arch/csky/include/asm/ftrace.h | 4 ++--
> arch/csky/include/asm/jump_label.h | 4 ++--
> arch/csky/include/asm/page.h | 4 ++--
> arch/csky/include/asm/ptrace.h | 4 ++--
> arch/csky/include/asm/string.h | 2 +-
> arch/csky/include/asm/thread_info.h | 4 ++--
> 10 files changed, 16 insertions(+), 16 deletions(-)
>
> diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
> index 7b386fd670702..c75ecf2cafd7c 100644
> --- a/arch/csky/abiv1/inc/abi/regdef.h
> +++ b/arch/csky/abiv1/inc/abi/regdef.h
> @@ -3,7 +3,7 @@
> #ifndef __ASM_CSKY_REGDEF_H
> #define __ASM_CSKY_REGDEF_H
>
> -#ifdef __ASSEMBLY__
> +#ifdef __ASSEMBLER__
> #define syscallid r1
> #else
> #define syscallid "r1"
> diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
> index 0933addbc27b7..fc08d56ccdbe1 100644
> --- a/arch/csky/abiv2/inc/abi/regdef.h
> +++ b/arch/csky/abiv2/inc/abi/regdef.h
> @@ -3,7 +3,7 @@
> #ifndef __ASM_CSKY_REGDEF_H
> #define __ASM_CSKY_REGDEF_H
>
> -#ifdef __ASSEMBLY__
> +#ifdef __ASSEMBLER__
> #define syscallid r7
> #else
> #define syscallid "r7"
> diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
> index 15de58b10aece..c33fdcfe3770c 100644
> --- a/arch/csky/include/asm/barrier.h
> +++ b/arch/csky/include/asm/barrier.h
> @@ -3,7 +3,7 @@
> #ifndef __ASM_CSKY_BARRIER_H
> #define __ASM_CSKY_BARRIER_H
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> #define nop() asm volatile ("nop\n":::"memory")
>
> @@ -84,5 +84,5 @@
>
> #include <asm-generic/barrier.h>
>
> -#endif /* __ASSEMBLY__ */
> +#endif /* __ASSEMBLER__ */
> #endif /* __ASM_CSKY_BARRIER_H */
> diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
> index 4b5c09bf1d25e..d575482e0fcec 100644
> --- a/arch/csky/include/asm/cache.h
> +++ b/arch/csky/include/asm/cache.h
> @@ -10,7 +10,7 @@
>
> #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> void dcache_wb_line(unsigned long start);
>
> diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
> index 00f9f7647e3f3..21532f2180587 100644
> --- a/arch/csky/include/asm/ftrace.h
> +++ b/arch/csky/include/asm/ftrace.h
> @@ -11,7 +11,7 @@
>
> #define MCOUNT_ADDR ((unsigned long)_mcount)
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> extern void _mcount(unsigned long);
>
> @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
> void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
> unsigned long frame_pointer);
>
> -#endif /* !__ASSEMBLY__ */
> +#endif /* !__ASSEMBLER__ */
> #endif /* __ASM_CSKY_FTRACE_H */
> diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
> index ef2e37a10a0fe..603b0caa80249 100644
> --- a/arch/csky/include/asm/jump_label.h
> +++ b/arch/csky/include/asm/jump_label.h
> @@ -3,7 +3,7 @@
> #ifndef __ASM_CSKY_JUMP_LABEL_H
> #define __ASM_CSKY_JUMP_LABEL_H
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> #include <linux/types.h>
>
> @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
> enum jump_label_type type);
> #define arch_jump_label_transform_static arch_jump_label_transform_static
>
> -#endif /* __ASSEMBLY__ */
> +#endif /* __ASSEMBLER__ */
> #endif /* __ASM_CSKY_JUMP_LABEL_H */
> diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
> index 4911d0892b71d..76774dbce8697 100644
> --- a/arch/csky/include/asm/page.h
> +++ b/arch/csky/include/asm/page.h
> @@ -26,7 +26,7 @@
>
> #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> #include <linux/pfn.h>
>
> @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
> #include <asm-generic/memory_model.h>
> #include <asm-generic/getorder.h>
>
> -#endif /* !__ASSEMBLY__ */
> +#endif /* !__ASSEMBLER__ */
> #endif /* __ASM_CSKY_PAGE_H */
> diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
> index 0634b7895d81d..5f01839e11843 100644
> --- a/arch/csky/include/asm/ptrace.h
> +++ b/arch/csky/include/asm/ptrace.h
> @@ -8,7 +8,7 @@
> #include <linux/types.h>
> #include <linux/compiler.h>
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> #define PS_S 0x80000000 /* Supervisor Mode */
>
> @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
>
> asmlinkage int syscall_trace_enter(struct pt_regs *regs);
> asmlinkage void syscall_trace_exit(struct pt_regs *regs);
> -#endif /* __ASSEMBLY__ */
> +#endif /* __ASSEMBLER__ */
> #endif /* __ASM_CSKY_PTRACE_H */
> diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
> index a0d81e9d6b8f6..82e99f52b547c 100644
> --- a/arch/csky/include/asm/string.h
> +++ b/arch/csky/include/asm/string.h
> @@ -3,7 +3,7 @@
> #ifndef _CSKY_STRING_MM_H_
> #define _CSKY_STRING_MM_H_
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
> #include <linux/types.h>
> #include <linux/compiler.h>
> #include <abi/string.h>
> diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
> index b5ed788f0c681..fdd4f8ad45acf 100644
> --- a/arch/csky/include/asm/thread_info.h
> +++ b/arch/csky/include/asm/thread_info.h
> @@ -3,7 +3,7 @@
> #ifndef _ASM_CSKY_THREAD_INFO_H
> #define _ASM_CSKY_THREAD_INFO_H
>
> -#ifndef __ASSEMBLY__
> +#ifndef __ASSEMBLER__
>
> #include <asm/types.h>
> #include <asm/page.h>
> @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
> return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
> }
>
> -#endif /* !__ASSEMBLY__ */
> +#endif /* !__ASSEMBLER__ */
>
> #define TIF_SIGPENDING 0 /* signal pending */
> #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
Friendly ping!
Guo Ren, could you maybe pick this (and the previous patch) up via your csky
tree? (x86 and parisc already got merge via their respective architecture
trees, so I guess the same should happen for this patch here, too).
Thanks,
Thomas
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-06-05 18:28 ` Thomas Huth
@ 2025-06-06 2:23 ` Guo Ren
2025-06-06 5:31 ` Thomas Huth
0 siblings, 1 reply; 8+ messages in thread
From: Guo Ren @ 2025-06-06 2:23 UTC (permalink / raw)
To: Thomas Huth; +Cc: linux-csky
On Fri, Jun 6, 2025 at 2:28 AM Thomas Huth <thuth@redhat.com> wrote:
>
> On 14/03/2025 08.09, Thomas Huth wrote:
> > While the GCC and Clang compilers already define __ASSEMBLER__
> > automatically when compiling assembly code, __ASSEMBLY__ is a
> > macro that only gets defined by the Makefiles in the kernel.
> > This can be very confusing when switching between userspace
> > and kernelspace coding, or when dealing with uapi headers that
> > rather should use __ASSEMBLER__ instead. So let's standardize on
> > the __ASSEMBLER__ macro that is provided by the compilers now.
> >
> > This is a completely mechanical patch (done with a simple "sed -i"
> > statement).
> >
> > Cc: Guo Ren <guoren@kernel.org>
> > Cc: linux-csky@vger.kernel.org
> > Signed-off-by: Thomas Huth <thuth@redhat.com>
> > ---
> > arch/csky/abiv1/inc/abi/regdef.h | 2 +-
> > arch/csky/abiv2/inc/abi/regdef.h | 2 +-
> > arch/csky/include/asm/barrier.h | 4 ++--
> > arch/csky/include/asm/cache.h | 2 +-
> > arch/csky/include/asm/ftrace.h | 4 ++--
> > arch/csky/include/asm/jump_label.h | 4 ++--
> > arch/csky/include/asm/page.h | 4 ++--
> > arch/csky/include/asm/ptrace.h | 4 ++--
> > arch/csky/include/asm/string.h | 2 +-
> > arch/csky/include/asm/thread_info.h | 4 ++--
> > 10 files changed, 16 insertions(+), 16 deletions(-)
> >
> > diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
> > index 7b386fd670702..c75ecf2cafd7c 100644
> > --- a/arch/csky/abiv1/inc/abi/regdef.h
> > +++ b/arch/csky/abiv1/inc/abi/regdef.h
> > @@ -3,7 +3,7 @@
> > #ifndef __ASM_CSKY_REGDEF_H
> > #define __ASM_CSKY_REGDEF_H
> >
> > -#ifdef __ASSEMBLY__
> > +#ifdef __ASSEMBLER__
> > #define syscallid r1
> > #else
> > #define syscallid "r1"
> > diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
> > index 0933addbc27b7..fc08d56ccdbe1 100644
> > --- a/arch/csky/abiv2/inc/abi/regdef.h
> > +++ b/arch/csky/abiv2/inc/abi/regdef.h
> > @@ -3,7 +3,7 @@
> > #ifndef __ASM_CSKY_REGDEF_H
> > #define __ASM_CSKY_REGDEF_H
> >
> > -#ifdef __ASSEMBLY__
> > +#ifdef __ASSEMBLER__
> > #define syscallid r7
> > #else
> > #define syscallid "r7"
> > diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
> > index 15de58b10aece..c33fdcfe3770c 100644
> > --- a/arch/csky/include/asm/barrier.h
> > +++ b/arch/csky/include/asm/barrier.h
> > @@ -3,7 +3,7 @@
> > #ifndef __ASM_CSKY_BARRIER_H
> > #define __ASM_CSKY_BARRIER_H
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > #define nop() asm volatile ("nop\n":::"memory")
> >
> > @@ -84,5 +84,5 @@
> >
> > #include <asm-generic/barrier.h>
> >
> > -#endif /* __ASSEMBLY__ */
> > +#endif /* __ASSEMBLER__ */
> > #endif /* __ASM_CSKY_BARRIER_H */
> > diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
> > index 4b5c09bf1d25e..d575482e0fcec 100644
> > --- a/arch/csky/include/asm/cache.h
> > +++ b/arch/csky/include/asm/cache.h
> > @@ -10,7 +10,7 @@
> >
> > #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > void dcache_wb_line(unsigned long start);
> >
> > diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
> > index 00f9f7647e3f3..21532f2180587 100644
> > --- a/arch/csky/include/asm/ftrace.h
> > +++ b/arch/csky/include/asm/ftrace.h
> > @@ -11,7 +11,7 @@
> >
> > #define MCOUNT_ADDR ((unsigned long)_mcount)
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > extern void _mcount(unsigned long);
> >
> > @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
> > void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
> > unsigned long frame_pointer);
> >
> > -#endif /* !__ASSEMBLY__ */
> > +#endif /* !__ASSEMBLER__ */
> > #endif /* __ASM_CSKY_FTRACE_H */
> > diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
> > index ef2e37a10a0fe..603b0caa80249 100644
> > --- a/arch/csky/include/asm/jump_label.h
> > +++ b/arch/csky/include/asm/jump_label.h
> > @@ -3,7 +3,7 @@
> > #ifndef __ASM_CSKY_JUMP_LABEL_H
> > #define __ASM_CSKY_JUMP_LABEL_H
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > #include <linux/types.h>
> >
> > @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
> > enum jump_label_type type);
> > #define arch_jump_label_transform_static arch_jump_label_transform_static
> >
> > -#endif /* __ASSEMBLY__ */
> > +#endif /* __ASSEMBLER__ */
> > #endif /* __ASM_CSKY_JUMP_LABEL_H */
> > diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
> > index 4911d0892b71d..76774dbce8697 100644
> > --- a/arch/csky/include/asm/page.h
> > +++ b/arch/csky/include/asm/page.h
> > @@ -26,7 +26,7 @@
> >
> > #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > #include <linux/pfn.h>
> >
> > @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
> > #include <asm-generic/memory_model.h>
> > #include <asm-generic/getorder.h>
> >
> > -#endif /* !__ASSEMBLY__ */
> > +#endif /* !__ASSEMBLER__ */
> > #endif /* __ASM_CSKY_PAGE_H */
> > diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
> > index 0634b7895d81d..5f01839e11843 100644
> > --- a/arch/csky/include/asm/ptrace.h
> > +++ b/arch/csky/include/asm/ptrace.h
> > @@ -8,7 +8,7 @@
> > #include <linux/types.h>
> > #include <linux/compiler.h>
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > #define PS_S 0x80000000 /* Supervisor Mode */
> >
> > @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
> >
> > asmlinkage int syscall_trace_enter(struct pt_regs *regs);
> > asmlinkage void syscall_trace_exit(struct pt_regs *regs);
> > -#endif /* __ASSEMBLY__ */
> > +#endif /* __ASSEMBLER__ */
> > #endif /* __ASM_CSKY_PTRACE_H */
> > diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
> > index a0d81e9d6b8f6..82e99f52b547c 100644
> > --- a/arch/csky/include/asm/string.h
> > +++ b/arch/csky/include/asm/string.h
> > @@ -3,7 +3,7 @@
> > #ifndef _CSKY_STRING_MM_H_
> > #define _CSKY_STRING_MM_H_
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> > #include <linux/types.h>
> > #include <linux/compiler.h>
> > #include <abi/string.h>
> > diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
> > index b5ed788f0c681..fdd4f8ad45acf 100644
> > --- a/arch/csky/include/asm/thread_info.h
> > +++ b/arch/csky/include/asm/thread_info.h
> > @@ -3,7 +3,7 @@
> > #ifndef _ASM_CSKY_THREAD_INFO_H
> > #define _ASM_CSKY_THREAD_INFO_H
> >
> > -#ifndef __ASSEMBLY__
> > +#ifndef __ASSEMBLER__
> >
> > #include <asm/types.h>
> > #include <asm/page.h>
> > @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
> > return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
> > }
> >
> > -#endif /* !__ASSEMBLY__ */
> > +#endif /* !__ASSEMBLER__ */
> >
> > #define TIF_SIGPENDING 0 /* signal pending */
> > #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
>
> Friendly ping!
>
> Guo Ren, could you maybe pick this (and the previous patch) up via your csky
> tree? (x86 and parisc already got merge via their respective architecture
> trees, so I guess the same should happen for this patch here, too).
>
> Thanks,
> Thomas
>
How about riscv?
--
Best Regards
Guo Ren
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-06-06 2:23 ` Guo Ren
@ 2025-06-06 5:31 ` Thomas Huth
2025-06-06 7:20 ` Guo Ren
0 siblings, 1 reply; 8+ messages in thread
From: Thomas Huth @ 2025-06-06 5:31 UTC (permalink / raw)
To: Guo Ren; +Cc: linux-csky
On 06/06/2025 04.23, Guo Ren wrote:
> On Fri, Jun 6, 2025 at 2:28 AM Thomas Huth <thuth@redhat.com> wrote:
>>
>> On 14/03/2025 08.09, Thomas Huth wrote:
>>> While the GCC and Clang compilers already define __ASSEMBLER__
>>> automatically when compiling assembly code, __ASSEMBLY__ is a
>>> macro that only gets defined by the Makefiles in the kernel.
>>> This can be very confusing when switching between userspace
>>> and kernelspace coding, or when dealing with uapi headers that
>>> rather should use __ASSEMBLER__ instead. So let's standardize on
>>> the __ASSEMBLER__ macro that is provided by the compilers now.
>>>
>>> This is a completely mechanical patch (done with a simple "sed -i"
>>> statement).
>>>
>>> Cc: Guo Ren <guoren@kernel.org>
>>> Cc: linux-csky@vger.kernel.org
>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>> ---
>>> arch/csky/abiv1/inc/abi/regdef.h | 2 +-
>>> arch/csky/abiv2/inc/abi/regdef.h | 2 +-
>>> arch/csky/include/asm/barrier.h | 4 ++--
>>> arch/csky/include/asm/cache.h | 2 +-
>>> arch/csky/include/asm/ftrace.h | 4 ++--
>>> arch/csky/include/asm/jump_label.h | 4 ++--
>>> arch/csky/include/asm/page.h | 4 ++--
>>> arch/csky/include/asm/ptrace.h | 4 ++--
>>> arch/csky/include/asm/string.h | 2 +-
>>> arch/csky/include/asm/thread_info.h | 4 ++--
>>> 10 files changed, 16 insertions(+), 16 deletions(-)
>>>
>>> diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
>>> index 7b386fd670702..c75ecf2cafd7c 100644
>>> --- a/arch/csky/abiv1/inc/abi/regdef.h
>>> +++ b/arch/csky/abiv1/inc/abi/regdef.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef __ASM_CSKY_REGDEF_H
>>> #define __ASM_CSKY_REGDEF_H
>>>
>>> -#ifdef __ASSEMBLY__
>>> +#ifdef __ASSEMBLER__
>>> #define syscallid r1
>>> #else
>>> #define syscallid "r1"
>>> diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
>>> index 0933addbc27b7..fc08d56ccdbe1 100644
>>> --- a/arch/csky/abiv2/inc/abi/regdef.h
>>> +++ b/arch/csky/abiv2/inc/abi/regdef.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef __ASM_CSKY_REGDEF_H
>>> #define __ASM_CSKY_REGDEF_H
>>>
>>> -#ifdef __ASSEMBLY__
>>> +#ifdef __ASSEMBLER__
>>> #define syscallid r7
>>> #else
>>> #define syscallid "r7"
>>> diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
>>> index 15de58b10aece..c33fdcfe3770c 100644
>>> --- a/arch/csky/include/asm/barrier.h
>>> +++ b/arch/csky/include/asm/barrier.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef __ASM_CSKY_BARRIER_H
>>> #define __ASM_CSKY_BARRIER_H
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> #define nop() asm volatile ("nop\n":::"memory")
>>>
>>> @@ -84,5 +84,5 @@
>>>
>>> #include <asm-generic/barrier.h>
>>>
>>> -#endif /* __ASSEMBLY__ */
>>> +#endif /* __ASSEMBLER__ */
>>> #endif /* __ASM_CSKY_BARRIER_H */
>>> diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
>>> index 4b5c09bf1d25e..d575482e0fcec 100644
>>> --- a/arch/csky/include/asm/cache.h
>>> +++ b/arch/csky/include/asm/cache.h
>>> @@ -10,7 +10,7 @@
>>>
>>> #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> void dcache_wb_line(unsigned long start);
>>>
>>> diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
>>> index 00f9f7647e3f3..21532f2180587 100644
>>> --- a/arch/csky/include/asm/ftrace.h
>>> +++ b/arch/csky/include/asm/ftrace.h
>>> @@ -11,7 +11,7 @@
>>>
>>> #define MCOUNT_ADDR ((unsigned long)_mcount)
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> extern void _mcount(unsigned long);
>>>
>>> @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
>>> void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
>>> unsigned long frame_pointer);
>>>
>>> -#endif /* !__ASSEMBLY__ */
>>> +#endif /* !__ASSEMBLER__ */
>>> #endif /* __ASM_CSKY_FTRACE_H */
>>> diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
>>> index ef2e37a10a0fe..603b0caa80249 100644
>>> --- a/arch/csky/include/asm/jump_label.h
>>> +++ b/arch/csky/include/asm/jump_label.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef __ASM_CSKY_JUMP_LABEL_H
>>> #define __ASM_CSKY_JUMP_LABEL_H
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> #include <linux/types.h>
>>>
>>> @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
>>> enum jump_label_type type);
>>> #define arch_jump_label_transform_static arch_jump_label_transform_static
>>>
>>> -#endif /* __ASSEMBLY__ */
>>> +#endif /* __ASSEMBLER__ */
>>> #endif /* __ASM_CSKY_JUMP_LABEL_H */
>>> diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
>>> index 4911d0892b71d..76774dbce8697 100644
>>> --- a/arch/csky/include/asm/page.h
>>> +++ b/arch/csky/include/asm/page.h
>>> @@ -26,7 +26,7 @@
>>>
>>> #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> #include <linux/pfn.h>
>>>
>>> @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
>>> #include <asm-generic/memory_model.h>
>>> #include <asm-generic/getorder.h>
>>>
>>> -#endif /* !__ASSEMBLY__ */
>>> +#endif /* !__ASSEMBLER__ */
>>> #endif /* __ASM_CSKY_PAGE_H */
>>> diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
>>> index 0634b7895d81d..5f01839e11843 100644
>>> --- a/arch/csky/include/asm/ptrace.h
>>> +++ b/arch/csky/include/asm/ptrace.h
>>> @@ -8,7 +8,7 @@
>>> #include <linux/types.h>
>>> #include <linux/compiler.h>
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> #define PS_S 0x80000000 /* Supervisor Mode */
>>>
>>> @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
>>>
>>> asmlinkage int syscall_trace_enter(struct pt_regs *regs);
>>> asmlinkage void syscall_trace_exit(struct pt_regs *regs);
>>> -#endif /* __ASSEMBLY__ */
>>> +#endif /* __ASSEMBLER__ */
>>> #endif /* __ASM_CSKY_PTRACE_H */
>>> diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
>>> index a0d81e9d6b8f6..82e99f52b547c 100644
>>> --- a/arch/csky/include/asm/string.h
>>> +++ b/arch/csky/include/asm/string.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef _CSKY_STRING_MM_H_
>>> #define _CSKY_STRING_MM_H_
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>> #include <linux/types.h>
>>> #include <linux/compiler.h>
>>> #include <abi/string.h>
>>> diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
>>> index b5ed788f0c681..fdd4f8ad45acf 100644
>>> --- a/arch/csky/include/asm/thread_info.h
>>> +++ b/arch/csky/include/asm/thread_info.h
>>> @@ -3,7 +3,7 @@
>>> #ifndef _ASM_CSKY_THREAD_INFO_H
>>> #define _ASM_CSKY_THREAD_INFO_H
>>>
>>> -#ifndef __ASSEMBLY__
>>> +#ifndef __ASSEMBLER__
>>>
>>> #include <asm/types.h>
>>> #include <asm/page.h>
>>> @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
>>> return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
>>> }
>>>
>>> -#endif /* !__ASSEMBLY__ */
>>> +#endif /* !__ASSEMBLER__ */
>>>
>>> #define TIF_SIGPENDING 0 /* signal pending */
>>> #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
>>
>> Friendly ping!
>>
>> Guo Ren, could you maybe pick this (and the previous patch) up via your csky
>> tree? (x86 and parisc already got merge via their respective architecture
>> trees, so I guess the same should happen for this patch here, too).
>>
>> Thanks,
>> Thomas
>>
> How about riscv?
The riscv patch
(https://lore.kernel.org/all/20250314071013.1575167-30-thuth@redhat.com/)
needs a rebase since it does not apply cleanly anymore. I'll try to do that
in the next couple of days.
Thomas
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-06-06 5:31 ` Thomas Huth
@ 2025-06-06 7:20 ` Guo Ren
2025-09-30 4:27 ` Thomas Huth
0 siblings, 1 reply; 8+ messages in thread
From: Guo Ren @ 2025-06-06 7:20 UTC (permalink / raw)
To: Thomas Huth; +Cc: linux-csky
On Fri, Jun 6, 2025 at 1:31 PM Thomas Huth <thuth@redhat.com> wrote:
>
> On 06/06/2025 04.23, Guo Ren wrote:
> > On Fri, Jun 6, 2025 at 2:28 AM Thomas Huth <thuth@redhat.com> wrote:
> >>
> >> On 14/03/2025 08.09, Thomas Huth wrote:
> >>> While the GCC and Clang compilers already define __ASSEMBLER__
> >>> automatically when compiling assembly code, __ASSEMBLY__ is a
> >>> macro that only gets defined by the Makefiles in the kernel.
> >>> This can be very confusing when switching between userspace
> >>> and kernelspace coding, or when dealing with uapi headers that
> >>> rather should use __ASSEMBLER__ instead. So let's standardize on
> >>> the __ASSEMBLER__ macro that is provided by the compilers now.
> >>>
> >>> This is a completely mechanical patch (done with a simple "sed -i"
> >>> statement).
> >>>
> >>> Cc: Guo Ren <guoren@kernel.org>
> >>> Cc: linux-csky@vger.kernel.org
> >>> Signed-off-by: Thomas Huth <thuth@redhat.com>
> >>> ---
> >>> arch/csky/abiv1/inc/abi/regdef.h | 2 +-
> >>> arch/csky/abiv2/inc/abi/regdef.h | 2 +-
> >>> arch/csky/include/asm/barrier.h | 4 ++--
> >>> arch/csky/include/asm/cache.h | 2 +-
> >>> arch/csky/include/asm/ftrace.h | 4 ++--
> >>> arch/csky/include/asm/jump_label.h | 4 ++--
> >>> arch/csky/include/asm/page.h | 4 ++--
> >>> arch/csky/include/asm/ptrace.h | 4 ++--
> >>> arch/csky/include/asm/string.h | 2 +-
> >>> arch/csky/include/asm/thread_info.h | 4 ++--
> >>> 10 files changed, 16 insertions(+), 16 deletions(-)
> >>>
> >>> diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
> >>> index 7b386fd670702..c75ecf2cafd7c 100644
> >>> --- a/arch/csky/abiv1/inc/abi/regdef.h
> >>> +++ b/arch/csky/abiv1/inc/abi/regdef.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef __ASM_CSKY_REGDEF_H
> >>> #define __ASM_CSKY_REGDEF_H
> >>>
> >>> -#ifdef __ASSEMBLY__
> >>> +#ifdef __ASSEMBLER__
> >>> #define syscallid r1
> >>> #else
> >>> #define syscallid "r1"
> >>> diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
> >>> index 0933addbc27b7..fc08d56ccdbe1 100644
> >>> --- a/arch/csky/abiv2/inc/abi/regdef.h
> >>> +++ b/arch/csky/abiv2/inc/abi/regdef.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef __ASM_CSKY_REGDEF_H
> >>> #define __ASM_CSKY_REGDEF_H
> >>>
> >>> -#ifdef __ASSEMBLY__
> >>> +#ifdef __ASSEMBLER__
> >>> #define syscallid r7
> >>> #else
> >>> #define syscallid "r7"
> >>> diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
> >>> index 15de58b10aece..c33fdcfe3770c 100644
> >>> --- a/arch/csky/include/asm/barrier.h
> >>> +++ b/arch/csky/include/asm/barrier.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef __ASM_CSKY_BARRIER_H
> >>> #define __ASM_CSKY_BARRIER_H
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> #define nop() asm volatile ("nop\n":::"memory")
> >>>
> >>> @@ -84,5 +84,5 @@
> >>>
> >>> #include <asm-generic/barrier.h>
> >>>
> >>> -#endif /* __ASSEMBLY__ */
> >>> +#endif /* __ASSEMBLER__ */
> >>> #endif /* __ASM_CSKY_BARRIER_H */
> >>> diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
> >>> index 4b5c09bf1d25e..d575482e0fcec 100644
> >>> --- a/arch/csky/include/asm/cache.h
> >>> +++ b/arch/csky/include/asm/cache.h
> >>> @@ -10,7 +10,7 @@
> >>>
> >>> #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> void dcache_wb_line(unsigned long start);
> >>>
> >>> diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
> >>> index 00f9f7647e3f3..21532f2180587 100644
> >>> --- a/arch/csky/include/asm/ftrace.h
> >>> +++ b/arch/csky/include/asm/ftrace.h
> >>> @@ -11,7 +11,7 @@
> >>>
> >>> #define MCOUNT_ADDR ((unsigned long)_mcount)
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> extern void _mcount(unsigned long);
> >>>
> >>> @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
> >>> void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
> >>> unsigned long frame_pointer);
> >>>
> >>> -#endif /* !__ASSEMBLY__ */
> >>> +#endif /* !__ASSEMBLER__ */
> >>> #endif /* __ASM_CSKY_FTRACE_H */
> >>> diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
> >>> index ef2e37a10a0fe..603b0caa80249 100644
> >>> --- a/arch/csky/include/asm/jump_label.h
> >>> +++ b/arch/csky/include/asm/jump_label.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef __ASM_CSKY_JUMP_LABEL_H
> >>> #define __ASM_CSKY_JUMP_LABEL_H
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> #include <linux/types.h>
> >>>
> >>> @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
> >>> enum jump_label_type type);
> >>> #define arch_jump_label_transform_static arch_jump_label_transform_static
> >>>
> >>> -#endif /* __ASSEMBLY__ */
> >>> +#endif /* __ASSEMBLER__ */
> >>> #endif /* __ASM_CSKY_JUMP_LABEL_H */
> >>> diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
> >>> index 4911d0892b71d..76774dbce8697 100644
> >>> --- a/arch/csky/include/asm/page.h
> >>> +++ b/arch/csky/include/asm/page.h
> >>> @@ -26,7 +26,7 @@
> >>>
> >>> #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> #include <linux/pfn.h>
> >>>
> >>> @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
> >>> #include <asm-generic/memory_model.h>
> >>> #include <asm-generic/getorder.h>
> >>>
> >>> -#endif /* !__ASSEMBLY__ */
> >>> +#endif /* !__ASSEMBLER__ */
> >>> #endif /* __ASM_CSKY_PAGE_H */
> >>> diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
> >>> index 0634b7895d81d..5f01839e11843 100644
> >>> --- a/arch/csky/include/asm/ptrace.h
> >>> +++ b/arch/csky/include/asm/ptrace.h
> >>> @@ -8,7 +8,7 @@
> >>> #include <linux/types.h>
> >>> #include <linux/compiler.h>
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> #define PS_S 0x80000000 /* Supervisor Mode */
> >>>
> >>> @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
> >>>
> >>> asmlinkage int syscall_trace_enter(struct pt_regs *regs);
> >>> asmlinkage void syscall_trace_exit(struct pt_regs *regs);
> >>> -#endif /* __ASSEMBLY__ */
> >>> +#endif /* __ASSEMBLER__ */
> >>> #endif /* __ASM_CSKY_PTRACE_H */
> >>> diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
> >>> index a0d81e9d6b8f6..82e99f52b547c 100644
> >>> --- a/arch/csky/include/asm/string.h
> >>> +++ b/arch/csky/include/asm/string.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef _CSKY_STRING_MM_H_
> >>> #define _CSKY_STRING_MM_H_
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>> #include <linux/types.h>
> >>> #include <linux/compiler.h>
> >>> #include <abi/string.h>
> >>> diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
> >>> index b5ed788f0c681..fdd4f8ad45acf 100644
> >>> --- a/arch/csky/include/asm/thread_info.h
> >>> +++ b/arch/csky/include/asm/thread_info.h
> >>> @@ -3,7 +3,7 @@
> >>> #ifndef _ASM_CSKY_THREAD_INFO_H
> >>> #define _ASM_CSKY_THREAD_INFO_H
> >>>
> >>> -#ifndef __ASSEMBLY__
> >>> +#ifndef __ASSEMBLER__
> >>>
> >>> #include <asm/types.h>
> >>> #include <asm/page.h>
> >>> @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
> >>> return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
> >>> }
> >>>
> >>> -#endif /* !__ASSEMBLY__ */
> >>> +#endif /* !__ASSEMBLER__ */
> >>>
> >>> #define TIF_SIGPENDING 0 /* signal pending */
> >>> #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
> >>
> >> Friendly ping!
> >>
> >> Guo Ren, could you maybe pick this (and the previous patch) up via your csky
> >> tree? (x86 and parisc already got merge via their respective architecture
> >> trees, so I guess the same should happen for this patch here, too).
> >>
> >> Thanks,
> >> Thomas
> >>
> > How about riscv?
>
> The riscv patch
> (https://lore.kernel.org/all/20250314071013.1575167-30-thuth@redhat.com/)
> needs a rebase since it does not apply cleanly anymore. I'll try to do that
> in the next couple of days.
Thanks for the info, I would take the patch after the other
architecture is approved. It's not a rush job.
>
> Thomas
>
--
Best Regards
Guo Ren
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-06-06 7:20 ` Guo Ren
@ 2025-09-30 4:27 ` Thomas Huth
2025-09-30 6:57 ` Guo Ren
0 siblings, 1 reply; 8+ messages in thread
From: Thomas Huth @ 2025-09-30 4:27 UTC (permalink / raw)
To: Guo Ren; +Cc: linux-csky
On 06/06/2025 09.20, Guo Ren wrote:
> On Fri, Jun 6, 2025 at 1:31 PM Thomas Huth <thuth@redhat.com> wrote:
>>
>> On 06/06/2025 04.23, Guo Ren wrote:
>>> On Fri, Jun 6, 2025 at 2:28 AM Thomas Huth <thuth@redhat.com> wrote:
>>>>
>>>> On 14/03/2025 08.09, Thomas Huth wrote:
>>>>> While the GCC and Clang compilers already define __ASSEMBLER__
>>>>> automatically when compiling assembly code, __ASSEMBLY__ is a
>>>>> macro that only gets defined by the Makefiles in the kernel.
>>>>> This can be very confusing when switching between userspace
>>>>> and kernelspace coding, or when dealing with uapi headers that
>>>>> rather should use __ASSEMBLER__ instead. So let's standardize on
>>>>> the __ASSEMBLER__ macro that is provided by the compilers now.
>>>>>
>>>>> This is a completely mechanical patch (done with a simple "sed -i"
>>>>> statement).
>>>>>
>>>>> Cc: Guo Ren <guoren@kernel.org>
>>>>> Cc: linux-csky@vger.kernel.org
>>>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
>>>>> ---
>>>>> arch/csky/abiv1/inc/abi/regdef.h | 2 +-
>>>>> arch/csky/abiv2/inc/abi/regdef.h | 2 +-
>>>>> arch/csky/include/asm/barrier.h | 4 ++--
>>>>> arch/csky/include/asm/cache.h | 2 +-
>>>>> arch/csky/include/asm/ftrace.h | 4 ++--
>>>>> arch/csky/include/asm/jump_label.h | 4 ++--
>>>>> arch/csky/include/asm/page.h | 4 ++--
>>>>> arch/csky/include/asm/ptrace.h | 4 ++--
>>>>> arch/csky/include/asm/string.h | 2 +-
>>>>> arch/csky/include/asm/thread_info.h | 4 ++--
>>>>> 10 files changed, 16 insertions(+), 16 deletions(-)
>>>>>
>>>>> diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
>>>>> index 7b386fd670702..c75ecf2cafd7c 100644
>>>>> --- a/arch/csky/abiv1/inc/abi/regdef.h
>>>>> +++ b/arch/csky/abiv1/inc/abi/regdef.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef __ASM_CSKY_REGDEF_H
>>>>> #define __ASM_CSKY_REGDEF_H
>>>>>
>>>>> -#ifdef __ASSEMBLY__
>>>>> +#ifdef __ASSEMBLER__
>>>>> #define syscallid r1
>>>>> #else
>>>>> #define syscallid "r1"
>>>>> diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
>>>>> index 0933addbc27b7..fc08d56ccdbe1 100644
>>>>> --- a/arch/csky/abiv2/inc/abi/regdef.h
>>>>> +++ b/arch/csky/abiv2/inc/abi/regdef.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef __ASM_CSKY_REGDEF_H
>>>>> #define __ASM_CSKY_REGDEF_H
>>>>>
>>>>> -#ifdef __ASSEMBLY__
>>>>> +#ifdef __ASSEMBLER__
>>>>> #define syscallid r7
>>>>> #else
>>>>> #define syscallid "r7"
>>>>> diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
>>>>> index 15de58b10aece..c33fdcfe3770c 100644
>>>>> --- a/arch/csky/include/asm/barrier.h
>>>>> +++ b/arch/csky/include/asm/barrier.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef __ASM_CSKY_BARRIER_H
>>>>> #define __ASM_CSKY_BARRIER_H
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> #define nop() asm volatile ("nop\n":::"memory")
>>>>>
>>>>> @@ -84,5 +84,5 @@
>>>>>
>>>>> #include <asm-generic/barrier.h>
>>>>>
>>>>> -#endif /* __ASSEMBLY__ */
>>>>> +#endif /* __ASSEMBLER__ */
>>>>> #endif /* __ASM_CSKY_BARRIER_H */
>>>>> diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
>>>>> index 4b5c09bf1d25e..d575482e0fcec 100644
>>>>> --- a/arch/csky/include/asm/cache.h
>>>>> +++ b/arch/csky/include/asm/cache.h
>>>>> @@ -10,7 +10,7 @@
>>>>>
>>>>> #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> void dcache_wb_line(unsigned long start);
>>>>>
>>>>> diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
>>>>> index 00f9f7647e3f3..21532f2180587 100644
>>>>> --- a/arch/csky/include/asm/ftrace.h
>>>>> +++ b/arch/csky/include/asm/ftrace.h
>>>>> @@ -11,7 +11,7 @@
>>>>>
>>>>> #define MCOUNT_ADDR ((unsigned long)_mcount)
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> extern void _mcount(unsigned long);
>>>>>
>>>>> @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
>>>>> void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
>>>>> unsigned long frame_pointer);
>>>>>
>>>>> -#endif /* !__ASSEMBLY__ */
>>>>> +#endif /* !__ASSEMBLER__ */
>>>>> #endif /* __ASM_CSKY_FTRACE_H */
>>>>> diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
>>>>> index ef2e37a10a0fe..603b0caa80249 100644
>>>>> --- a/arch/csky/include/asm/jump_label.h
>>>>> +++ b/arch/csky/include/asm/jump_label.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef __ASM_CSKY_JUMP_LABEL_H
>>>>> #define __ASM_CSKY_JUMP_LABEL_H
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> #include <linux/types.h>
>>>>>
>>>>> @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
>>>>> enum jump_label_type type);
>>>>> #define arch_jump_label_transform_static arch_jump_label_transform_static
>>>>>
>>>>> -#endif /* __ASSEMBLY__ */
>>>>> +#endif /* __ASSEMBLER__ */
>>>>> #endif /* __ASM_CSKY_JUMP_LABEL_H */
>>>>> diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
>>>>> index 4911d0892b71d..76774dbce8697 100644
>>>>> --- a/arch/csky/include/asm/page.h
>>>>> +++ b/arch/csky/include/asm/page.h
>>>>> @@ -26,7 +26,7 @@
>>>>>
>>>>> #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> #include <linux/pfn.h>
>>>>>
>>>>> @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
>>>>> #include <asm-generic/memory_model.h>
>>>>> #include <asm-generic/getorder.h>
>>>>>
>>>>> -#endif /* !__ASSEMBLY__ */
>>>>> +#endif /* !__ASSEMBLER__ */
>>>>> #endif /* __ASM_CSKY_PAGE_H */
>>>>> diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
>>>>> index 0634b7895d81d..5f01839e11843 100644
>>>>> --- a/arch/csky/include/asm/ptrace.h
>>>>> +++ b/arch/csky/include/asm/ptrace.h
>>>>> @@ -8,7 +8,7 @@
>>>>> #include <linux/types.h>
>>>>> #include <linux/compiler.h>
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> #define PS_S 0x80000000 /* Supervisor Mode */
>>>>>
>>>>> @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
>>>>>
>>>>> asmlinkage int syscall_trace_enter(struct pt_regs *regs);
>>>>> asmlinkage void syscall_trace_exit(struct pt_regs *regs);
>>>>> -#endif /* __ASSEMBLY__ */
>>>>> +#endif /* __ASSEMBLER__ */
>>>>> #endif /* __ASM_CSKY_PTRACE_H */
>>>>> diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
>>>>> index a0d81e9d6b8f6..82e99f52b547c 100644
>>>>> --- a/arch/csky/include/asm/string.h
>>>>> +++ b/arch/csky/include/asm/string.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef _CSKY_STRING_MM_H_
>>>>> #define _CSKY_STRING_MM_H_
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>> #include <linux/types.h>
>>>>> #include <linux/compiler.h>
>>>>> #include <abi/string.h>
>>>>> diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
>>>>> index b5ed788f0c681..fdd4f8ad45acf 100644
>>>>> --- a/arch/csky/include/asm/thread_info.h
>>>>> +++ b/arch/csky/include/asm/thread_info.h
>>>>> @@ -3,7 +3,7 @@
>>>>> #ifndef _ASM_CSKY_THREAD_INFO_H
>>>>> #define _ASM_CSKY_THREAD_INFO_H
>>>>>
>>>>> -#ifndef __ASSEMBLY__
>>>>> +#ifndef __ASSEMBLER__
>>>>>
>>>>> #include <asm/types.h>
>>>>> #include <asm/page.h>
>>>>> @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
>>>>> return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
>>>>> }
>>>>>
>>>>> -#endif /* !__ASSEMBLY__ */
>>>>> +#endif /* !__ASSEMBLER__ */
>>>>>
>>>>> #define TIF_SIGPENDING 0 /* signal pending */
>>>>> #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
>>>>
>>>> Friendly ping!
>>>>
>>>> Guo Ren, could you maybe pick this (and the previous patch) up via your csky
>>>> tree? (x86 and parisc already got merge via their respective architecture
>>>> trees, so I guess the same should happen for this patch here, too).
>>>>
>>>> Thanks,
>>>> Thomas
>>>>
>>> How about riscv?
>>
>> The riscv patch
>> (https://lore.kernel.org/all/20250314071013.1575167-30-thuth@redhat.com/)
>> needs a rebase since it does not apply cleanly anymore. I'll try to do that
>> in the next couple of days.
> Thanks for the info, I would take the patch after the other
> architecture is approved. It's not a rush job.
The riscv patch has now been merged. Could you please pick up the csky patch
now?
Thanks,
Thomas
^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers
2025-09-30 4:27 ` Thomas Huth
@ 2025-09-30 6:57 ` Guo Ren
0 siblings, 0 replies; 8+ messages in thread
From: Guo Ren @ 2025-09-30 6:57 UTC (permalink / raw)
To: Thomas Huth; +Cc: linux-csky
On Tue, Sep 30, 2025 at 12:27 PM Thomas Huth <thuth@redhat.com> wrote:
>
> On 06/06/2025 09.20, Guo Ren wrote:
> > On Fri, Jun 6, 2025 at 1:31 PM Thomas Huth <thuth@redhat.com> wrote:
> >>
> >> On 06/06/2025 04.23, Guo Ren wrote:
> >>> On Fri, Jun 6, 2025 at 2:28 AM Thomas Huth <thuth@redhat.com> wrote:
> >>>>
> >>>> On 14/03/2025 08.09, Thomas Huth wrote:
> >>>>> While the GCC and Clang compilers already define __ASSEMBLER__
> >>>>> automatically when compiling assembly code, __ASSEMBLY__ is a
> >>>>> macro that only gets defined by the Makefiles in the kernel.
> >>>>> This can be very confusing when switching between userspace
> >>>>> and kernelspace coding, or when dealing with uapi headers that
> >>>>> rather should use __ASSEMBLER__ instead. So let's standardize on
> >>>>> the __ASSEMBLER__ macro that is provided by the compilers now.
> >>>>>
> >>>>> This is a completely mechanical patch (done with a simple "sed -i"
> >>>>> statement).
> >>>>>
> >>>>> Cc: Guo Ren <guoren@kernel.org>
> >>>>> Cc: linux-csky@vger.kernel.org
> >>>>> Signed-off-by: Thomas Huth <thuth@redhat.com>
> >>>>> ---
> >>>>> arch/csky/abiv1/inc/abi/regdef.h | 2 +-
> >>>>> arch/csky/abiv2/inc/abi/regdef.h | 2 +-
> >>>>> arch/csky/include/asm/barrier.h | 4 ++--
> >>>>> arch/csky/include/asm/cache.h | 2 +-
> >>>>> arch/csky/include/asm/ftrace.h | 4 ++--
> >>>>> arch/csky/include/asm/jump_label.h | 4 ++--
> >>>>> arch/csky/include/asm/page.h | 4 ++--
> >>>>> arch/csky/include/asm/ptrace.h | 4 ++--
> >>>>> arch/csky/include/asm/string.h | 2 +-
> >>>>> arch/csky/include/asm/thread_info.h | 4 ++--
> >>>>> 10 files changed, 16 insertions(+), 16 deletions(-)
> >>>>>
> >>>>> diff --git a/arch/csky/abiv1/inc/abi/regdef.h b/arch/csky/abiv1/inc/abi/regdef.h
> >>>>> index 7b386fd670702..c75ecf2cafd7c 100644
> >>>>> --- a/arch/csky/abiv1/inc/abi/regdef.h
> >>>>> +++ b/arch/csky/abiv1/inc/abi/regdef.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef __ASM_CSKY_REGDEF_H
> >>>>> #define __ASM_CSKY_REGDEF_H
> >>>>>
> >>>>> -#ifdef __ASSEMBLY__
> >>>>> +#ifdef __ASSEMBLER__
> >>>>> #define syscallid r1
> >>>>> #else
> >>>>> #define syscallid "r1"
> >>>>> diff --git a/arch/csky/abiv2/inc/abi/regdef.h b/arch/csky/abiv2/inc/abi/regdef.h
> >>>>> index 0933addbc27b7..fc08d56ccdbe1 100644
> >>>>> --- a/arch/csky/abiv2/inc/abi/regdef.h
> >>>>> +++ b/arch/csky/abiv2/inc/abi/regdef.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef __ASM_CSKY_REGDEF_H
> >>>>> #define __ASM_CSKY_REGDEF_H
> >>>>>
> >>>>> -#ifdef __ASSEMBLY__
> >>>>> +#ifdef __ASSEMBLER__
> >>>>> #define syscallid r7
> >>>>> #else
> >>>>> #define syscallid "r7"
> >>>>> diff --git a/arch/csky/include/asm/barrier.h b/arch/csky/include/asm/barrier.h
> >>>>> index 15de58b10aece..c33fdcfe3770c 100644
> >>>>> --- a/arch/csky/include/asm/barrier.h
> >>>>> +++ b/arch/csky/include/asm/barrier.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef __ASM_CSKY_BARRIER_H
> >>>>> #define __ASM_CSKY_BARRIER_H
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> #define nop() asm volatile ("nop\n":::"memory")
> >>>>>
> >>>>> @@ -84,5 +84,5 @@
> >>>>>
> >>>>> #include <asm-generic/barrier.h>
> >>>>>
> >>>>> -#endif /* __ASSEMBLY__ */
> >>>>> +#endif /* __ASSEMBLER__ */
> >>>>> #endif /* __ASM_CSKY_BARRIER_H */
> >>>>> diff --git a/arch/csky/include/asm/cache.h b/arch/csky/include/asm/cache.h
> >>>>> index 4b5c09bf1d25e..d575482e0fcec 100644
> >>>>> --- a/arch/csky/include/asm/cache.h
> >>>>> +++ b/arch/csky/include/asm/cache.h
> >>>>> @@ -10,7 +10,7 @@
> >>>>>
> >>>>> #define ARCH_DMA_MINALIGN L1_CACHE_BYTES
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> void dcache_wb_line(unsigned long start);
> >>>>>
> >>>>> diff --git a/arch/csky/include/asm/ftrace.h b/arch/csky/include/asm/ftrace.h
> >>>>> index 00f9f7647e3f3..21532f2180587 100644
> >>>>> --- a/arch/csky/include/asm/ftrace.h
> >>>>> +++ b/arch/csky/include/asm/ftrace.h
> >>>>> @@ -11,7 +11,7 @@
> >>>>>
> >>>>> #define MCOUNT_ADDR ((unsigned long)_mcount)
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> extern void _mcount(unsigned long);
> >>>>>
> >>>>> @@ -28,5 +28,5 @@ struct dyn_arch_ftrace {
> >>>>> void prepare_ftrace_return(unsigned long *parent, unsigned long self_addr,
> >>>>> unsigned long frame_pointer);
> >>>>>
> >>>>> -#endif /* !__ASSEMBLY__ */
> >>>>> +#endif /* !__ASSEMBLER__ */
> >>>>> #endif /* __ASM_CSKY_FTRACE_H */
> >>>>> diff --git a/arch/csky/include/asm/jump_label.h b/arch/csky/include/asm/jump_label.h
> >>>>> index ef2e37a10a0fe..603b0caa80249 100644
> >>>>> --- a/arch/csky/include/asm/jump_label.h
> >>>>> +++ b/arch/csky/include/asm/jump_label.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef __ASM_CSKY_JUMP_LABEL_H
> >>>>> #define __ASM_CSKY_JUMP_LABEL_H
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> #include <linux/types.h>
> >>>>>
> >>>>> @@ -48,5 +48,5 @@ void arch_jump_label_transform_static(struct jump_entry *entry,
> >>>>> enum jump_label_type type);
> >>>>> #define arch_jump_label_transform_static arch_jump_label_transform_static
> >>>>>
> >>>>> -#endif /* __ASSEMBLY__ */
> >>>>> +#endif /* __ASSEMBLER__ */
> >>>>> #endif /* __ASM_CSKY_JUMP_LABEL_H */
> >>>>> diff --git a/arch/csky/include/asm/page.h b/arch/csky/include/asm/page.h
> >>>>> index 4911d0892b71d..76774dbce8697 100644
> >>>>> --- a/arch/csky/include/asm/page.h
> >>>>> +++ b/arch/csky/include/asm/page.h
> >>>>> @@ -26,7 +26,7 @@
> >>>>>
> >>>>> #define PHYS_OFFSET_OFFSET (CONFIG_DRAM_BASE & (SSEG_SIZE - 1))
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> #include <linux/pfn.h>
> >>>>>
> >>>>> @@ -84,5 +84,5 @@ static inline unsigned long virt_to_pfn(const void *kaddr)
> >>>>> #include <asm-generic/memory_model.h>
> >>>>> #include <asm-generic/getorder.h>
> >>>>>
> >>>>> -#endif /* !__ASSEMBLY__ */
> >>>>> +#endif /* !__ASSEMBLER__ */
> >>>>> #endif /* __ASM_CSKY_PAGE_H */
> >>>>> diff --git a/arch/csky/include/asm/ptrace.h b/arch/csky/include/asm/ptrace.h
> >>>>> index 0634b7895d81d..5f01839e11843 100644
> >>>>> --- a/arch/csky/include/asm/ptrace.h
> >>>>> +++ b/arch/csky/include/asm/ptrace.h
> >>>>> @@ -8,7 +8,7 @@
> >>>>> #include <linux/types.h>
> >>>>> #include <linux/compiler.h>
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> #define PS_S 0x80000000 /* Supervisor Mode */
> >>>>>
> >>>>> @@ -98,5 +98,5 @@ static inline unsigned long regs_get_register(struct pt_regs *regs,
> >>>>>
> >>>>> asmlinkage int syscall_trace_enter(struct pt_regs *regs);
> >>>>> asmlinkage void syscall_trace_exit(struct pt_regs *regs);
> >>>>> -#endif /* __ASSEMBLY__ */
> >>>>> +#endif /* __ASSEMBLER__ */
> >>>>> #endif /* __ASM_CSKY_PTRACE_H */
> >>>>> diff --git a/arch/csky/include/asm/string.h b/arch/csky/include/asm/string.h
> >>>>> index a0d81e9d6b8f6..82e99f52b547c 100644
> >>>>> --- a/arch/csky/include/asm/string.h
> >>>>> +++ b/arch/csky/include/asm/string.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef _CSKY_STRING_MM_H_
> >>>>> #define _CSKY_STRING_MM_H_
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>> #include <linux/types.h>
> >>>>> #include <linux/compiler.h>
> >>>>> #include <abi/string.h>
> >>>>> diff --git a/arch/csky/include/asm/thread_info.h b/arch/csky/include/asm/thread_info.h
> >>>>> index b5ed788f0c681..fdd4f8ad45acf 100644
> >>>>> --- a/arch/csky/include/asm/thread_info.h
> >>>>> +++ b/arch/csky/include/asm/thread_info.h
> >>>>> @@ -3,7 +3,7 @@
> >>>>> #ifndef _ASM_CSKY_THREAD_INFO_H
> >>>>> #define _ASM_CSKY_THREAD_INFO_H
> >>>>>
> >>>>> -#ifndef __ASSEMBLY__
> >>>>> +#ifndef __ASSEMBLER__
> >>>>>
> >>>>> #include <asm/types.h>
> >>>>> #include <asm/page.h>
> >>>>> @@ -51,7 +51,7 @@ static inline struct thread_info *current_thread_info(void)
> >>>>> return (struct thread_info *)(sp & ~(THREAD_SIZE - 1));
> >>>>> }
> >>>>>
> >>>>> -#endif /* !__ASSEMBLY__ */
> >>>>> +#endif /* !__ASSEMBLER__ */
> >>>>>
> >>>>> #define TIF_SIGPENDING 0 /* signal pending */
> >>>>> #define TIF_NOTIFY_RESUME 1 /* callback before returning to user */
> >>>>
> >>>> Friendly ping!
> >>>>
> >>>> Guo Ren, could you maybe pick this (and the previous patch) up via your csky
> >>>> tree? (x86 and parisc already got merge via their respective architecture
> >>>> trees, so I guess the same should happen for this patch here, too).
> >>>>
> >>>> Thanks,
> >>>> Thomas
> >>>>
> >>> How about riscv?
> >>
> >> The riscv patch
> >> (https://lore.kernel.org/all/20250314071013.1575167-30-thuth@redhat.com/)
> >> needs a rebase since it does not apply cleanly anymore. I'll try to do that
> >> in the next couple of days.
> > Thanks for the info, I would take the patch after the other
> > architecture is approved. It's not a rush job.
>
> The riscv patch has now been merged. Could you please pick up the csky patch
> now?
Yes, I'll review and pick it up.
--
Best Regards
Guo Ren
^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2025-09-30 6:57 UTC | newest]
Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20250314071013.1575167-1-thuth@redhat.com>
2025-03-14 7:09 ` [PATCH 10/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in uapi header Thomas Huth
2025-03-14 7:09 ` [PATCH 11/41] csky: Replace __ASSEMBLY__ with __ASSEMBLER__ in non-uapi headers Thomas Huth
2025-06-05 18:28 ` Thomas Huth
2025-06-06 2:23 ` Guo Ren
2025-06-06 5:31 ` Thomas Huth
2025-06-06 7:20 ` Guo Ren
2025-09-30 4:27 ` Thomas Huth
2025-09-30 6:57 ` Guo Ren
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox