On Sun, May 21, 2023 at 9:05 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote: > > With the previous refactoring, you can always use EXPORT_SYMBOL*. > > Replace two instances in ia64, then remove EXPORT_DATA_SYMBOL*. > > Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx> Thanks for the patch! Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx> > --- > > arch/ia64/kernel/head.S | 2 +- > arch/ia64/kernel/ivt.S | 2 +- > include/asm-generic/export.h | 3 --- > 3 files changed, 2 insertions(+), 5 deletions(-) > > diff --git a/arch/ia64/kernel/head.S b/arch/ia64/kernel/head.S > index f22469f1c1fc..c096500590e9 100644 > --- a/arch/ia64/kernel/head.S > +++ b/arch/ia64/kernel/head.S > @@ -170,7 +170,7 @@ RestRR: \ > __PAGE_ALIGNED_DATA > > .global empty_zero_page > -EXPORT_DATA_SYMBOL_GPL(empty_zero_page) > +EXPORT_SYMBOL_GPL(empty_zero_page) > empty_zero_page: > .skip PAGE_SIZE > > diff --git a/arch/ia64/kernel/ivt.S b/arch/ia64/kernel/ivt.S > index d6d4229b28db..7a418e324d30 100644 > --- a/arch/ia64/kernel/ivt.S > +++ b/arch/ia64/kernel/ivt.S > @@ -87,7 +87,7 @@ > > .align 32768 // align on 32KB boundary > .global ia64_ivt > - EXPORT_DATA_SYMBOL(ia64_ivt) > + EXPORT_SYMBOL(ia64_ivt) > ia64_ivt: > ///////////////////////////////////////////////////////////////////////////////////////// > // 0x0000 Entry 0 (size 64 bundles) VHPT Translation (8,20,47) > diff --git a/include/asm-generic/export.h b/include/asm-generic/export.h > index 0ae9f38a904c..570cd4da7210 100644 > --- a/include/asm-generic/export.h > +++ b/include/asm-generic/export.h > @@ -8,7 +8,4 @@ > */ > #include <linux/export.h> > > -#define EXPORT_DATA_SYMBOL(name) EXPORT_SYMBOL(name) > -#define EXPORT_DATA_SYMBOL_GPL(name) EXPORT_SYMBOL_GPL(name) > - > #endif > -- > 2.39.2 > -- Thanks, ~Nick Desaulniers