Re: Patch "purgatory: fix disabling debug info" has been added to the 5.15-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, Apr 17, 2023 at 09:27:22PM -0400, Sasha Levin wrote:
> This is a note to let you know that I've just added the patch titled
>
>     purgatory: fix disabling debug info
>
> to the 5.15-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:
>      purgatory-fix-disabling-debug-info.patch
> and it can be found in the queue-5.15 subdirectory.
>
> If you, or anyone else, feels it should not be added to the stable tree,
> please let <stable@xxxxxxxxxxxxxxx> know about it.

There's no need for this patch on 5.15, as the regression it fixes was
only introduced in 6.0.  It won't do any harm though — is it considered
good practice to keep the code in sync between stable kernels to make
backports of other patches easier?  If so, it would make sense to
backport after all.

> commit 618ea690941689fe28fa9c150f90bb096db5f8a5
> Author: Alyssa Ross <hi@xxxxxxxxx>
> Date:   Sun Mar 26 18:21:21 2023 +0000
>
>     purgatory: fix disabling debug info
>
>     [ Upstream commit d83806c4c0cccc0d6d3c3581a11983a9c186a138 ]
>
>     Since 32ef9e5054ec, -Wa,-gdwarf-2 is no longer used in KBUILD_AFLAGS.
>     Instead, it includes -g, the appropriate -gdwarf-* flag, and also the
>     -Wa versions of both of those if building with Clang and GNU as.  As a
>     result, debug info was being generated for the purgatory objects, even
>     though the intention was that it not be.
>
>     Fixes: 32ef9e5054ec ("Makefile.debug: re-enable debug info for .S files")
>     Signed-off-by: Alyssa Ross <hi@xxxxxxxxx>
>     Cc: stable@xxxxxxxxxxxxxxx
>     Acked-by: Nick Desaulniers <ndesaulniers@xxxxxxxxxx>
>     Signed-off-by: Masahiro Yamada <masahiroy@xxxxxxxxxx>
>     Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
>
> diff --git a/arch/x86/purgatory/Makefile b/arch/x86/purgatory/Makefile
> index 95ea17a9d20cb..1d6ccd4214d5a 100644
> --- a/arch/x86/purgatory/Makefile
> +++ b/arch/x86/purgatory/Makefile
> @@ -64,8 +64,7 @@ CFLAGS_sha256.o			+= $(PURGATORY_CFLAGS)
>  CFLAGS_REMOVE_string.o		+= $(PURGATORY_CFLAGS_REMOVE)
>  CFLAGS_string.o			+= $(PURGATORY_CFLAGS)
>
> -AFLAGS_REMOVE_setup-x86_$(BITS).o	+= -Wa,-gdwarf-2
> -AFLAGS_REMOVE_entry64.o			+= -Wa,-gdwarf-2
> +asflags-remove-y		+= $(foreach x, -g -gdwarf-4 -gdwarf-5, $(x) -Wa,$(x))
>
>  $(obj)/purgatory.ro: $(PURGATORY_OBJS) FORCE
>  		$(call if_changed,ld)

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux