Re: Patch "modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS" has been added to the 6.6-stable tree

Re: Patch "modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS" has been added to the 6.6-stable tree

[Nathan Chancellor]

Hi Greg,

On Mon, Feb 19, 2024 at 05:28:33PM +0100, gregkh@linuxfoundation.org wrote:
> 
> This is a note to let you know that I've just added the patch titled
> 
>     modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS
> 
> to the 6.6-stable tree which can be found at:
>     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> 
> The filename of the patch is:
>      modpost-add-.ltext-and-.ltext.-to-text_sections.patch
> and it can be found in the queue-6.6 subdirectory.
> 
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@vger.kernel.org> know about it.
> 
> 
> From 397586506c3da005b9333ce5947ad01e8018a3be Mon Sep 17 00:00:00 2001
> From: Nathan Chancellor <nathan@kernel.org>
> Date: Tue, 23 Jan 2024 15:59:55 -0700
> Subject: modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS
> 
> From: Nathan Chancellor <nathan@kernel.org>
> 
> commit 397586506c3da005b9333ce5947ad01e8018a3be upstream.

Please apply upstream commit 6a4e59eeedc3 ("linux/init: remove __memexit*
annotations") before this change, as there is a warning in modpost
without it:

  scripts/mod/modpost.c:916:37: warning: excess elements in array initializer [-Wexcess-initializers]
    916 |         .good_tosec = {ALL_TEXT_SECTIONS , NULL},
        |                                            ^~~~
  .../lib/clang/19/include/__stddef_null.h:26:14: note: expanded from macro 'NULL'
     26 | #define NULL ((void*)0)
        |              ^~~~~~~~~~
  1 warning generated.

It applies cleanly with 'git format-patch | patch' for me and I don't
think it is unreasonable as a fix for this issue in stable.

This will be needed in 6.1 as well but that backport is a little more
involved, I will reply there with an mbox series.

Cheers,
Nathan

Re: Patch "modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS" has been added to the 6.6-stable tree

[Greg KH]

On Mon, Feb 19, 2024 at 12:04:09PM -0700, Nathan Chancellor wrote:
> Hi Greg,
> 
> On Mon, Feb 19, 2024 at 05:28:33PM +0100, gregkh@linuxfoundation.org wrote:
> > 
> > This is a note to let you know that I've just added the patch titled
> > 
> >     modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS
> > 
> > to the 6.6-stable tree which can be found at:
> >     http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary
> > 
> > The filename of the patch is:
> >      modpost-add-.ltext-and-.ltext.-to-text_sections.patch
> > and it can be found in the queue-6.6 subdirectory.
> > 
> > If you, or anyone else, feels it should not be added to the stable tree,
> > please let <stable@vger.kernel.org> know about it.
> > 
> > 
> > From 397586506c3da005b9333ce5947ad01e8018a3be Mon Sep 17 00:00:00 2001
> > From: Nathan Chancellor <nathan@kernel.org>
> > Date: Tue, 23 Jan 2024 15:59:55 -0700
> > Subject: modpost: Add '.ltext' and '.ltext.*' to TEXT_SECTIONS
> > 
> > From: Nathan Chancellor <nathan@kernel.org>
> > 
> > commit 397586506c3da005b9333ce5947ad01e8018a3be upstream.
> 
> Please apply upstream commit 6a4e59eeedc3 ("linux/init: remove __memexit*
> annotations") before this change, as there is a warning in modpost
> without it:
> 
>   scripts/mod/modpost.c:916:37: warning: excess elements in array initializer [-Wexcess-initializers]
>     916 |         .good_tosec = {ALL_TEXT_SECTIONS , NULL},
>         |                                            ^~~~
>   .../lib/clang/19/include/__stddef_null.h:26:14: note: expanded from macro 'NULL'
>      26 | #define NULL ((void*)0)
>         |              ^~~~~~~~~~
>   1 warning generated.
> 
> It applies cleanly with 'git format-patch | patch' for me and I don't
> think it is unreasonable as a fix for this issue in stable.
> 
> This will be needed in 6.1 as well but that backport is a little more
> involved, I will reply there with an mbox series.

Thanks, that fixed the build warning!

greg k-h