* [PATCH] vmlinux.lds.h: drop unused __vermagic
@ 2018-12-30 12:40 Mathias Krause
2019-01-02 20:29 ` Jessica Yu
0 siblings, 1 reply; 6+ messages in thread
From: Mathias Krause @ 2018-12-30 12:40 UTC (permalink / raw)
To: Arnd Bergmann
Cc: linux-arch, linux-kernel, Mathias Krause, Rusty Russell, Jessica Yu
The reference to '__vermagic' is a relict from v2.5 times. And even
there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
("Module Sanity Check") in the historic tree) to v2.5.69 (commit
67ac5b866bda ("[PATCH] complete modinfo section")).
Neither current kernels nor modules contain a '__vermagic' section any
more, so get rid of it.
Signed-off-by: Mathias Krause <minipli@googlemail.com>
Cc: Rusty Russell <rusty@rustcorp.com.au>
Cc: Jessica Yu <jeyu@kernel.org>
---
include/asm-generic/vmlinux.lds.h | 1 -
1 file changed, 1 deletion(-)
diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
index 3d7a6a9c2370..530ce1e7a1ec 100644
--- a/include/asm-generic/vmlinux.lds.h
+++ b/include/asm-generic/vmlinux.lds.h
@@ -332,7 +332,6 @@
__start_rodata = .; \
*(.rodata) *(.rodata.*) \
RO_AFTER_INIT_DATA /* Read only after init */ \
- KEEP(*(__vermagic)) /* Kernel version magic */ \
. = ALIGN(8); \
__start___tracepoints_ptrs = .; \
KEEP(*(__tracepoints_ptrs)) /* Tracepoints: pointer array */ \
--
2.19.2
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] vmlinux.lds.h: drop unused __vermagic
2018-12-30 12:40 [PATCH] vmlinux.lds.h: drop unused __vermagic Mathias Krause
@ 2019-01-02 20:29 ` Jessica Yu
2019-01-21 11:39 ` Mathias Krause
0 siblings, 1 reply; 6+ messages in thread
From: Jessica Yu @ 2019-01-02 20:29 UTC (permalink / raw)
To: Mathias Krause; +Cc: Arnd Bergmann, linux-arch, linux-kernel, Rusty Russell
+++ Mathias Krause [30/12/18 13:40 +0100]:
>The reference to '__vermagic' is a relict from v2.5 times. And even
>there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
>("Module Sanity Check") in the historic tree) to v2.5.69 (commit
>67ac5b866bda ("[PATCH] complete modinfo section")).
>
>Neither current kernels nor modules contain a '__vermagic' section any
>more, so get rid of it.
>
>Signed-off-by: Mathias Krause <minipli@googlemail.com>
>Cc: Rusty Russell <rusty@rustcorp.com.au>
>Cc: Jessica Yu <jeyu@kernel.org>
Thanks for the cleanup.
Reviewed-by: Jessica Yu <jeyu@kernel.org>
>---
> include/asm-generic/vmlinux.lds.h | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
>index 3d7a6a9c2370..530ce1e7a1ec 100644
>--- a/include/asm-generic/vmlinux.lds.h
>+++ b/include/asm-generic/vmlinux.lds.h
>@@ -332,7 +332,6 @@
> __start_rodata = .; \
> *(.rodata) *(.rodata.*) \
> RO_AFTER_INIT_DATA /* Read only after init */ \
>- KEEP(*(__vermagic)) /* Kernel version magic */ \
> . = ALIGN(8); \
> __start___tracepoints_ptrs = .; \
> KEEP(*(__tracepoints_ptrs)) /* Tracepoints: pointer array */ \
>--
>2.19.2
>
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] vmlinux.lds.h: drop unused __vermagic
2019-01-02 20:29 ` Jessica Yu
@ 2019-01-21 11:39 ` Mathias Krause
2019-03-10 19:26 ` Mathias Krause
0 siblings, 1 reply; 6+ messages in thread
From: Mathias Krause @ 2019-01-21 11:39 UTC (permalink / raw)
To: Arnd Bergmann; +Cc: linux-arch, linux-kernel, Rusty Russell, Jessica Yu
On Wed, 2 Jan 2019 at 21:29, Jessica Yu <jeyu@kernel.org> wrote:
>
> +++ Mathias Krause [30/12/18 13:40 +0100]:
> >The reference to '__vermagic' is a relict from v2.5 times. And even
> >there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
> >("Module Sanity Check") in the historic tree) to v2.5.69 (commit
> >67ac5b866bda ("[PATCH] complete modinfo section")).
> >
> >Neither current kernels nor modules contain a '__vermagic' section any
> >more, so get rid of it.
> >
> >Signed-off-by: Mathias Krause <minipli@googlemail.com>
> >Cc: Rusty Russell <rusty@rustcorp.com.au>
> >Cc: Jessica Yu <jeyu@kernel.org>
>
> Thanks for the cleanup.
>
> Reviewed-by: Jessica Yu <jeyu@kernel.org>
>
Arnd, will you carry this patch through your asm-generic tree or did
the MAINTAINERS file mislead me here?
Thanks,
Mathias
> >---
> > include/asm-generic/vmlinux.lds.h | 1 -
> > 1 file changed, 1 deletion(-)
> >
> >diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> >index 3d7a6a9c2370..530ce1e7a1ec 100644
> >--- a/include/asm-generic/vmlinux.lds.h
> >+++ b/include/asm-generic/vmlinux.lds.h
> >@@ -332,7 +332,6 @@
> > __start_rodata = .; \
> > *(.rodata) *(.rodata.*) \
> > RO_AFTER_INIT_DATA /* Read only after init */ \
> >- KEEP(*(__vermagic)) /* Kernel version magic */ \
> > . = ALIGN(8); \
> > __start___tracepoints_ptrs = .; \
> > KEEP(*(__tracepoints_ptrs)) /* Tracepoints: pointer array */ \
> >--
> >2.19.2
> >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] vmlinux.lds.h: drop unused __vermagic
2019-01-21 11:39 ` Mathias Krause
@ 2019-03-10 19:26 ` Mathias Krause
2019-03-11 11:55 ` Jessica Yu
0 siblings, 1 reply; 6+ messages in thread
From: Mathias Krause @ 2019-03-10 19:26 UTC (permalink / raw)
To: Jessica Yu; +Cc: linux-arch, linux-kernel, Rusty Russell, Arnd Bergmann
Hi Jessica,
On Mon, 21 Jan 2019 at 12:39, Mathias Krause <minipli@googlemail.com> wrote:
>
> On Wed, 2 Jan 2019 at 21:29, Jessica Yu <jeyu@kernel.org> wrote:
> >
> > +++ Mathias Krause [30/12/18 13:40 +0100]:
> > >The reference to '__vermagic' is a relict from v2.5 times. And even
> > >there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
> > >("Module Sanity Check") in the historic tree) to v2.5.69 (commit
> > >67ac5b866bda ("[PATCH] complete modinfo section")).
> > >
> > >Neither current kernels nor modules contain a '__vermagic' section any
> > >more, so get rid of it.
> > >
> > >Signed-off-by: Mathias Krause <minipli@googlemail.com>
> > >Cc: Rusty Russell <rusty@rustcorp.com.au>
> > >Cc: Jessica Yu <jeyu@kernel.org>
> >
> > Thanks for the cleanup.
> >
> > Reviewed-by: Jessica Yu <jeyu@kernel.org>
> >
>
> Arnd, will you carry this patch through your asm-generic tree or did
> the MAINTAINERS file mislead me here?
>
looks like Arnd keeps ignoring my Emails. :/ Can you carry this patch
through your module tree? Or do you think I should ask Andrew Morton
instead?
Thanks,
Mathias
> > >---
> > > include/asm-generic/vmlinux.lds.h | 1 -
> > > 1 file changed, 1 deletion(-)
> > >
> > >diff --git a/include/asm-generic/vmlinux.lds.h b/include/asm-generic/vmlinux.lds.h
> > >index 3d7a6a9c2370..530ce1e7a1ec 100644
> > >--- a/include/asm-generic/vmlinux.lds.h
> > >+++ b/include/asm-generic/vmlinux.lds.h
> > >@@ -332,7 +332,6 @@
> > > __start_rodata = .; \
> > > *(.rodata) *(.rodata.*) \
> > > RO_AFTER_INIT_DATA /* Read only after init */ \
> > >- KEEP(*(__vermagic)) /* Kernel version magic */ \
> > > . = ALIGN(8); \
> > > __start___tracepoints_ptrs = .; \
> > > KEEP(*(__tracepoints_ptrs)) /* Tracepoints: pointer array */ \
> > >--
> > >2.19.2
> > >
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] vmlinux.lds.h: drop unused __vermagic
2019-03-10 19:26 ` Mathias Krause
@ 2019-03-11 11:55 ` Jessica Yu
2019-03-11 12:11 ` Arnd Bergmann
0 siblings, 1 reply; 6+ messages in thread
From: Jessica Yu @ 2019-03-11 11:55 UTC (permalink / raw)
To: Mathias Krause; +Cc: linux-arch, linux-kernel, Rusty Russell, Arnd Bergmann
+++ Mathias Krause [10/03/19 20:26 +0100]:
>Hi Jessica,
>
>On Mon, 21 Jan 2019 at 12:39, Mathias Krause <minipli@googlemail.com> wrote:
>>
>> On Wed, 2 Jan 2019 at 21:29, Jessica Yu <jeyu@kernel.org> wrote:
>> >
>> > +++ Mathias Krause [30/12/18 13:40 +0100]:
>> > >The reference to '__vermagic' is a relict from v2.5 times. And even
>> > >there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
>> > >("Module Sanity Check") in the historic tree) to v2.5.69 (commit
>> > >67ac5b866bda ("[PATCH] complete modinfo section")).
>> > >
>> > >Neither current kernels nor modules contain a '__vermagic' section any
>> > >more, so get rid of it.
>> > >
>> > >Signed-off-by: Mathias Krause <minipli@googlemail.com>
>> > >Cc: Rusty Russell <rusty@rustcorp.com.au>
>> > >Cc: Jessica Yu <jeyu@kernel.org>
>> >
>> > Thanks for the cleanup.
>> >
>> > Reviewed-by: Jessica Yu <jeyu@kernel.org>
>> >
>>
>> Arnd, will you carry this patch through your asm-generic tree or did
>> the MAINTAINERS file mislead me here?
>>
>
>looks like Arnd keeps ignoring my Emails. :/ Can you carry this patch
>through your module tree? Or do you think I should ask Andrew Morton
>instead?
Sure, I'll queue this up in modules-next shortly after the merge window.
Thanks,
Jessica
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] vmlinux.lds.h: drop unused __vermagic
2019-03-11 11:55 ` Jessica Yu
@ 2019-03-11 12:11 ` Arnd Bergmann
0 siblings, 0 replies; 6+ messages in thread
From: Arnd Bergmann @ 2019-03-11 12:11 UTC (permalink / raw)
To: Jessica Yu
Cc: Mathias Krause, linux-arch, Linux Kernel Mailing List, Rusty Russell
On Mon, Mar 11, 2019 at 12:55 PM Jessica Yu <jeyu@kernel.org> wrote:
>
> +++ Mathias Krause [10/03/19 20:26 +0100]:
> >Hi Jessica,
> >
> >On Mon, 21 Jan 2019 at 12:39, Mathias Krause <minipli@googlemail.com> wrote:
> >>
> >> On Wed, 2 Jan 2019 at 21:29, Jessica Yu <jeyu@kernel.org> wrote:
> >> >
> >> > +++ Mathias Krause [30/12/18 13:40 +0100]:
> >> > >The reference to '__vermagic' is a relict from v2.5 times. And even
> >> > >there it had a very short life time, from v2.5.59 (commit 1d411b80ee18
> >> > >("Module Sanity Check") in the historic tree) to v2.5.69 (commit
> >> > >67ac5b866bda ("[PATCH] complete modinfo section")).
> >> > >
> >> > >Neither current kernels nor modules contain a '__vermagic' section any
> >> > >more, so get rid of it.
> >> > >
> >> > >Signed-off-by: Mathias Krause <minipli@googlemail.com>
> >> > >Cc: Rusty Russell <rusty@rustcorp.com.au>
> >> > >Cc: Jessica Yu <jeyu@kernel.org>
> >> >
> >> > Thanks for the cleanup.
> >> >
> >> > Reviewed-by: Jessica Yu <jeyu@kernel.org>
> >> >
> >>
> >> Arnd, will you carry this patch through your asm-generic tree or did
> >> the MAINTAINERS file mislead me here?
> >>
> >
> >looks like Arnd keeps ignoring my Emails. :/ Can you carry this patch
> >through your module tree? Or do you think I should ask Andrew Morton
> >instead?
Oops, sorry about this.
> Sure, I'll queue this up in modules-next shortly after the merge window.
Ok, thanks for picking it up!
Arnd
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2019-03-11 12:11 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-12-30 12:40 [PATCH] vmlinux.lds.h: drop unused __vermagic Mathias Krause
2019-01-02 20:29 ` Jessica Yu
2019-01-21 11:39 ` Mathias Krause
2019-03-10 19:26 ` Mathias Krause
2019-03-11 11:55 ` Jessica Yu
2019-03-11 12:11 ` Arnd Bergmann
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).