Re: [PATCH] Revert "scripts/mod/modpost.c: permit '.cranges' secton for sh64 architecture."

[Nick Desaulniers]

On Thu, Jun 30, 2022 at 1:19 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
>
> This reverts commit 4d10c223baab8be8f717df3625cfece5be26dead.
>
> Commit 37744feebc08 ("sh: remove sh5 support") removed the sh64 support
> entirely.
>
> If ".cranges" section is used for other architectures, it is easy to
> get it back.

I don't think that will be an issue. BFD has a comment about this
being a sh64-only section.  I couldn't find any other reference to
such a section.
Reviewed-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>

>
> Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>
> ---
>
>  scripts/mod/modpost.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/scripts/mod/modpost.c b/scripts/mod/modpost.c
> index 620dc8c4c814..e15227ee58fc 100644
> --- a/scripts/mod/modpost.c
> +++ b/scripts/mod/modpost.c
> @@ -742,7 +742,6 @@ static const char *const section_white_list[] =
>  {
>         ".comment*",
>         ".debug*",
> -       ".cranges",             /* sh64 */
>         ".zdebug*",             /* Compressed debug sections. */
>         ".GCC.command.line",    /* record-gcc-switches */
>         ".mdebug*",        /* alpha, score, mips etc. */
> --
> 2.32.0
>


-- 
Thanks,
~Nick Desaulniers