* [PATCH v4 1/9] alpha: Replace __ASSEMBLY__ with __ASSEMBLER__ in the alpha headers [not found] <20251121100044.282684-1-thuth@redhat.com> @ 2025-11-21 10:00 ` Thomas Huth 2025-11-29 23:00 ` Magnus Lindholm 0 siblings, 1 reply; 2+ messages in thread From: Thomas Huth @ 2025-11-21 10:00 UTC (permalink / raw) To: Arnd Bergmann, linux-arch, linux-kernel Cc: linux-arm-kernel, linux-kbuild, Thomas Huth, Richard Henderson, Matt Turner, linux-alpha From: Thomas Huth <thuth@redhat.com> 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 now on the __ASSEMBLER__ macro that is provided by the compilers. This is a completely mechanical patch (done with a simple "sed -i" statement). Cc: Richard Henderson <richard.henderson@linaro.org> Cc: Matt Turner <mattst88@gmail.com> Cc: linux-alpha@vger.kernel.org Signed-off-by: Thomas Huth <thuth@redhat.com> --- arch/alpha/include/asm/console.h | 4 ++-- arch/alpha/include/asm/page.h | 4 ++-- arch/alpha/include/asm/pal.h | 4 ++-- arch/alpha/include/asm/thread_info.h | 8 ++++---- 4 files changed, 10 insertions(+), 10 deletions(-) diff --git a/arch/alpha/include/asm/console.h b/arch/alpha/include/asm/console.h index 088b7b9eb15ae..1cabdb6064bbe 100644 --- a/arch/alpha/include/asm/console.h +++ b/arch/alpha/include/asm/console.h @@ -4,7 +4,7 @@ #include <uapi/asm/console.h> -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ extern long callback_puts(long unit, const char *s, long length); extern long callback_getc(long unit); extern long callback_open_console(void); @@ -26,5 +26,5 @@ struct crb_struct; struct hwrpb_struct; extern int callback_init_done; extern void * callback_init(void *); -#endif /* __ASSEMBLY__ */ +#endif /* __ASSEMBLER__ */ #endif /* __AXP_CONSOLE_H */ diff --git a/arch/alpha/include/asm/page.h b/arch/alpha/include/asm/page.h index 5ec4c77e432e0..d2c6667d73e9e 100644 --- a/arch/alpha/include/asm/page.h +++ b/arch/alpha/include/asm/page.h @@ -6,7 +6,7 @@ #include <asm/pal.h> #include <vdso/page.h> -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ #define STRICT_MM_TYPECHECKS @@ -74,7 +74,7 @@ typedef struct page *pgtable_t; #define PAGE_OFFSET 0xfffffc0000000000 #endif -#endif /* !__ASSEMBLY__ */ +#endif /* !__ASSEMBLER__ */ #define __pa(x) ((unsigned long) (x) - PAGE_OFFSET) #define __va(x) ((void *)((unsigned long) (x) + PAGE_OFFSET)) diff --git a/arch/alpha/include/asm/pal.h b/arch/alpha/include/asm/pal.h index db2b3b18b34c7..799a64c051984 100644 --- a/arch/alpha/include/asm/pal.h +++ b/arch/alpha/include/asm/pal.h @@ -4,7 +4,7 @@ #include <uapi/asm/pal.h> -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ extern void halt(void) __attribute__((noreturn)); #define __halt() __asm__ __volatile__ ("call_pal %0 #halt" : : "i" (PAL_halt)) @@ -183,5 +183,5 @@ qemu_get_vmtime(void) return v0; } -#endif /* !__ASSEMBLY__ */ +#endif /* !__ASSEMBLER__ */ #endif /* __ALPHA_PAL_H */ diff --git a/arch/alpha/include/asm/thread_info.h b/arch/alpha/include/asm/thread_info.h index 4a4d00b37986e..98ccbca64984c 100644 --- a/arch/alpha/include/asm/thread_info.h +++ b/arch/alpha/include/asm/thread_info.h @@ -4,14 +4,14 @@ #ifdef __KERNEL__ -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ #include <asm/processor.h> #include <asm/types.h> #include <asm/hwrpb.h> #include <asm/sysinfo.h> #endif -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ struct thread_info { struct pcb_struct pcb; /* palcode state */ @@ -44,7 +44,7 @@ register struct thread_info *__current_thread_info __asm__("$8"); register unsigned long *current_stack_pointer __asm__ ("$30"); -#endif /* __ASSEMBLY__ */ +#endif /* __ASSEMBLER__ */ /* Thread information allocation. */ #define THREAD_SIZE_ORDER 1 @@ -110,7 +110,7 @@ register unsigned long *current_stack_pointer __asm__ ("$30"); put_user(res, (int __user *)(value)); \ }) -#ifndef __ASSEMBLY__ +#ifndef __ASSEMBLER__ extern void __save_fpu(void); static inline void save_fpu(void) -- 2.51.1 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH v4 1/9] alpha: Replace __ASSEMBLY__ with __ASSEMBLER__ in the alpha headers 2025-11-21 10:00 ` [PATCH v4 1/9] alpha: Replace __ASSEMBLY__ with __ASSEMBLER__ in the alpha headers Thomas Huth @ 2025-11-29 23:00 ` Magnus Lindholm 0 siblings, 0 replies; 2+ messages in thread From: Magnus Lindholm @ 2025-11-29 23:00 UTC (permalink / raw) To: Thomas Huth Cc: Arnd Bergmann, linux-arch, linux-kernel, linux-arm-kernel, linux-kbuild, Richard Henderson, Matt Turner, linux-alpha On Fri, Nov 21, 2025 at 11:10 AM Thomas Huth <thuth@redhat.com> wrote: > > From: Thomas Huth <thuth@redhat.com> > > 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 now > on the __ASSEMBLER__ macro that is provided by the compilers. > > This is a completely mechanical patch (done with a simple "sed -i" > statement). > > Cc: Richard Henderson <richard.henderson@linaro.org> > Cc: Matt Turner <mattst88@gmail.com> > Cc: linux-alpha@vger.kernel.org > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > arch/alpha/include/asm/console.h | 4 ++-- > arch/alpha/include/asm/page.h | 4 ++-- > arch/alpha/include/asm/pal.h | 4 ++-- > arch/alpha/include/asm/thread_info.h | 8 ++++---- > 4 files changed, 10 insertions(+), 10 deletions(-) > > diff --git a/arch/alpha/include/asm/console.h b/arch/alpha/include/asm/console.h > index 088b7b9eb15ae..1cabdb6064bbe 100644 > --- a/arch/alpha/include/asm/console.h > +++ b/arch/alpha/include/asm/console.h > @@ -4,7 +4,7 @@ > > #include <uapi/asm/console.h> > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > extern long callback_puts(long unit, const char *s, long length); > extern long callback_getc(long unit); > extern long callback_open_console(void); > @@ -26,5 +26,5 @@ struct crb_struct; > struct hwrpb_struct; > extern int callback_init_done; > extern void * callback_init(void *); > -#endif /* __ASSEMBLY__ */ > +#endif /* __ASSEMBLER__ */ > #endif /* __AXP_CONSOLE_H */ > diff --git a/arch/alpha/include/asm/page.h b/arch/alpha/include/asm/page.h > index 5ec4c77e432e0..d2c6667d73e9e 100644 > --- a/arch/alpha/include/asm/page.h > +++ b/arch/alpha/include/asm/page.h > @@ -6,7 +6,7 @@ > #include <asm/pal.h> > #include <vdso/page.h> > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > > #define STRICT_MM_TYPECHECKS > > @@ -74,7 +74,7 @@ typedef struct page *pgtable_t; > #define PAGE_OFFSET 0xfffffc0000000000 > #endif > > -#endif /* !__ASSEMBLY__ */ > +#endif /* !__ASSEMBLER__ */ > > #define __pa(x) ((unsigned long) (x) - PAGE_OFFSET) > #define __va(x) ((void *)((unsigned long) (x) + PAGE_OFFSET)) > diff --git a/arch/alpha/include/asm/pal.h b/arch/alpha/include/asm/pal.h > index db2b3b18b34c7..799a64c051984 100644 > --- a/arch/alpha/include/asm/pal.h > +++ b/arch/alpha/include/asm/pal.h > @@ -4,7 +4,7 @@ > > #include <uapi/asm/pal.h> > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > > extern void halt(void) __attribute__((noreturn)); > #define __halt() __asm__ __volatile__ ("call_pal %0 #halt" : : "i" (PAL_halt)) > @@ -183,5 +183,5 @@ qemu_get_vmtime(void) > return v0; > } > > -#endif /* !__ASSEMBLY__ */ > +#endif /* !__ASSEMBLER__ */ > #endif /* __ALPHA_PAL_H */ > diff --git a/arch/alpha/include/asm/thread_info.h b/arch/alpha/include/asm/thread_info.h > index 4a4d00b37986e..98ccbca64984c 100644 > --- a/arch/alpha/include/asm/thread_info.h > +++ b/arch/alpha/include/asm/thread_info.h > @@ -4,14 +4,14 @@ > > #ifdef __KERNEL__ > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > #include <asm/processor.h> > #include <asm/types.h> > #include <asm/hwrpb.h> > #include <asm/sysinfo.h> > #endif > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > struct thread_info { > struct pcb_struct pcb; /* palcode state */ > > @@ -44,7 +44,7 @@ register struct thread_info *__current_thread_info __asm__("$8"); > > register unsigned long *current_stack_pointer __asm__ ("$30"); > > -#endif /* __ASSEMBLY__ */ > +#endif /* __ASSEMBLER__ */ > > /* Thread information allocation. */ > #define THREAD_SIZE_ORDER 1 > @@ -110,7 +110,7 @@ register unsigned long *current_stack_pointer __asm__ ("$30"); > put_user(res, (int __user *)(value)); \ > }) > > -#ifndef __ASSEMBLY__ > +#ifndef __ASSEMBLER__ > extern void __save_fpu(void); > > static inline void save_fpu(void) > -- > 2.51.1 > Reviewed-by: Magnus Lindholm <linmag7@gmail.com> ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2025-11-29 23:00 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20251121100044.282684-1-thuth@redhat.com>
2025-11-21 10:00 ` [PATCH v4 1/9] alpha: Replace __ASSEMBLY__ with __ASSEMBLER__ in the alpha headers Thomas Huth
2025-11-29 23:00 ` Magnus Lindholm
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).