Linux Modules
 help / color / mirror / Atom feed
* [PATCH] modules: Drop the .export_symbol section from the final modules
@ 2024-04-17  5:35 wangyao
  2024-04-22  2:02 ` Ainux Wang
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: wangyao @ 2024-04-17  5:35 UTC (permalink / raw)
  To: mcgrof, masahiroy, ainux.wang; +Cc: linux-modules, linux-kernel, Wang Yao

From: Wang Yao <wangyao@lemote.com>

Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
forget drop the .export_symbol section from the final modules.

Signed-off-by: Wang Yao <wangyao@lemote.com>
---
 scripts/module.lds.S | 1 +
 1 file changed, 1 insertion(+)

diff --git a/scripts/module.lds.S b/scripts/module.lds.S
index bf5bcf2836d8..89ff01a22634 100644
--- a/scripts/module.lds.S
+++ b/scripts/module.lds.S
@@ -13,6 +13,7 @@ SECTIONS {
 	/DISCARD/ : {
 		*(.discard)
 		*(.discard.*)
+		*(.export_symbol)
 	}
 
 	__ksymtab		0 : { *(SORT(___ksymtab+*)) }
-- 
2.27.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] modules: Drop the .export_symbol section from the final modules
  2024-04-17  5:35 [PATCH] modules: Drop the .export_symbol section from the final modules wangyao
@ 2024-04-22  2:02 ` Ainux Wang
  2024-05-07  5:15 ` Ainux Wang
  2024-05-11 22:41 ` Luis Chamberlain
  2 siblings, 0 replies; 6+ messages in thread
From: Ainux Wang @ 2024-04-22  2:02 UTC (permalink / raw)
  To: wangyao; +Cc: mcgrof, masahiroy, linux-modules, linux-kernel

Hi, everyone:
Could someone review this patch?
Best regards,
Ainux Wang.

<wangyao@lemote.com> 于2024年4月17日周三 13:35写道:
>
> From: Wang Yao <wangyao@lemote.com>
>
> Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
> forget drop the .export_symbol section from the final modules.
>
> Signed-off-by: Wang Yao <wangyao@lemote.com>
> ---
>  scripts/module.lds.S | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/scripts/module.lds.S b/scripts/module.lds.S
> index bf5bcf2836d8..89ff01a22634 100644
> --- a/scripts/module.lds.S
> +++ b/scripts/module.lds.S
> @@ -13,6 +13,7 @@ SECTIONS {
>         /DISCARD/ : {
>                 *(.discard)
>                 *(.discard.*)
> +               *(.export_symbol)
>         }
>
>         __ksymtab               0 : { *(SORT(___ksymtab+*)) }
> --
> 2.27.0
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] modules: Drop the .export_symbol section from the final modules
  2024-04-17  5:35 [PATCH] modules: Drop the .export_symbol section from the final modules wangyao
  2024-04-22  2:02 ` Ainux Wang
@ 2024-05-07  5:15 ` Ainux Wang
  2024-05-11  1:03   ` Ainux Wang
  2024-05-11 22:41 ` Luis Chamberlain
  2 siblings, 1 reply; 6+ messages in thread
From: Ainux Wang @ 2024-05-07  5:15 UTC (permalink / raw)
  To: mcgrof, masahiroy, Wang Yao, linux-modules; +Cc: linux-kernel

Hi, everyone:

Could someone review this patch?

Best regards,
Ainux Wang.

<wangyao@lemote.com> 于2024年4月17日周三 13:35写道:

>
> From: Wang Yao <wangyao@lemote.com>
>
> Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
> forget drop the .export_symbol section from the final modules.
>
> Signed-off-by: Wang Yao <wangyao@lemote.com>
> ---
>  scripts/module.lds.S | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/scripts/module.lds.S b/scripts/module.lds.S
> index bf5bcf2836d8..89ff01a22634 100644
> --- a/scripts/module.lds.S
> +++ b/scripts/module.lds.S
> @@ -13,6 +13,7 @@ SECTIONS {
>         /DISCARD/ : {
>                 *(.discard)
>                 *(.discard.*)
> +               *(.export_symbol)
>         }
>
>         __ksymtab               0 : { *(SORT(___ksymtab+*)) }
> --
> 2.27.0
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] modules: Drop the .export_symbol section from the final modules
  2024-05-07  5:15 ` Ainux Wang
@ 2024-05-11  1:03   ` Ainux Wang
  0 siblings, 0 replies; 6+ messages in thread
From: Ainux Wang @ 2024-05-11  1:03 UTC (permalink / raw)
  To: mcgrof, masahiroy, Wang Yao, linux-modules, linux-kernel

Hi, everyone:

Could someone help to review this patch?

Best regards,
 Ainux Wang.


Ainux Wang <ainux.wang@gmail.com> 于2024年5月7日周二 13:15写道:
>
> Hi, everyone:
>
> Could someone review this patch?
>
> Best regards,
> Ainux Wang.
>
> <wangyao@lemote.com> 于2024年4月17日周三 13:35写道:
>
> >
> > From: Wang Yao <wangyao@lemote.com>
> >
> > Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
> > forget drop the .export_symbol section from the final modules.
> >
> > Signed-off-by: Wang Yao <wangyao@lemote.com>
> > ---
> >  scripts/module.lds.S | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/scripts/module.lds.S b/scripts/module.lds.S
> > index bf5bcf2836d8..89ff01a22634 100644
> > --- a/scripts/module.lds.S
> > +++ b/scripts/module.lds.S
> > @@ -13,6 +13,7 @@ SECTIONS {
> >         /DISCARD/ : {
> >                 *(.discard)
> >                 *(.discard.*)
> > +               *(.export_symbol)
> >         }
> >
> >         __ksymtab               0 : { *(SORT(___ksymtab+*)) }
> > --
> > 2.27.0
> >

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] modules: Drop the .export_symbol section from the final modules
  2024-04-17  5:35 [PATCH] modules: Drop the .export_symbol section from the final modules wangyao
  2024-04-22  2:02 ` Ainux Wang
  2024-05-07  5:15 ` Ainux Wang
@ 2024-05-11 22:41 ` Luis Chamberlain
  2024-05-13 23:10   ` Masahiro Yamada
  2 siblings, 1 reply; 6+ messages in thread
From: Luis Chamberlain @ 2024-05-11 22:41 UTC (permalink / raw)
  To: wangyao, Masahiro Yamada
  Cc: masahiroy, ainux.wang, linux-modules, linux-kernel

On Wed, Apr 17, 2024 at 01:35:30PM +0800, wangyao@lemote.com wrote:
> From: Wang Yao <wangyao@lemote.com>
> 
> Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
> forget drop the .export_symbol section from the final modules.
> 
> Signed-off-by: Wang Yao <wangyao@lemote.com>

Masahiro, commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by
modpost") was your change, wanna address / take it through your
tree? It makes sense to me though.

  Luis

> ---
>  scripts/module.lds.S | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/scripts/module.lds.S b/scripts/module.lds.S
> index bf5bcf2836d8..89ff01a22634 100644
> --- a/scripts/module.lds.S
> +++ b/scripts/module.lds.S
> @@ -13,6 +13,7 @@ SECTIONS {
>  	/DISCARD/ : {
>  		*(.discard)
>  		*(.discard.*)
> +		*(.export_symbol)
>  	}
>  
>  	__ksymtab		0 : { *(SORT(___ksymtab+*)) }
> -- 
> 2.27.0
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] modules: Drop the .export_symbol section from the final modules
  2024-05-11 22:41 ` Luis Chamberlain
@ 2024-05-13 23:10   ` Masahiro Yamada
  0 siblings, 0 replies; 6+ messages in thread
From: Masahiro Yamada @ 2024-05-13 23:10 UTC (permalink / raw)
  To: Luis Chamberlain; +Cc: wangyao, ainux.wang, linux-modules, linux-kernel

On Sun, May 12, 2024 at 7:42 AM Luis Chamberlain <mcgrof@kernel.org> wrote:
>
> On Wed, Apr 17, 2024 at 01:35:30PM +0800, wangyao@lemote.com wrote:
> > From: Wang Yao <wangyao@lemote.com>
> >
> > Commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by modpost")
> > forget drop the .export_symbol section from the final modules.
> >
> > Signed-off-by: Wang Yao <wangyao@lemote.com>
>
> Masahiro, commit ddb5cdbafaaa ("kbuild: generate KSYMTAB entries by
> modpost") was your change, wanna address / take it through your
> tree? It makes sense to me though.


Yes, applied now.

Thanks for the reminder.





>
>   Luis
>
> > ---
> >  scripts/module.lds.S | 1 +
> >  1 file changed, 1 insertion(+)
> >
> > diff --git a/scripts/module.lds.S b/scripts/module.lds.S
> > index bf5bcf2836d8..89ff01a22634 100644
> > --- a/scripts/module.lds.S
> > +++ b/scripts/module.lds.S
> > @@ -13,6 +13,7 @@ SECTIONS {
> >       /DISCARD/ : {
> >               *(.discard)
> >               *(.discard.*)
> > +             *(.export_symbol)
> >       }
> >
> >       __ksymtab               0 : { *(SORT(___ksymtab+*)) }
> > --
> > 2.27.0
> >



-- 
Best Regards
Masahiro Yamada

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2024-05-13 23:11 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-17  5:35 [PATCH] modules: Drop the .export_symbol section from the final modules wangyao
2024-04-22  2:02 ` Ainux Wang
2024-05-07  5:15 ` Ainux Wang
2024-05-11  1:03   ` Ainux Wang
2024-05-11 22:41 ` Luis Chamberlain
2024-05-13 23:10   ` Masahiro Yamada

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox