Patch "LoongArch: Only fiddle with CHECKFLAGS if `need-compiler'" has been added to the 6.4-stable tree

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

 



This is a note to let you know that I've just added the patch titled

    LoongArch: Only fiddle with CHECKFLAGS if `need-compiler'

to the 6.4-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:
     loongarch-only-fiddle-with-checkflags-if-need-compil.patch
and it can be found in the queue-6.4 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit 5e75bbd7eaa05e46b0fd3ea3c4ea507860be5ac2
Author: Huacai Chen <chenhuacai@xxxxxxxxxx>
Date:   Fri Jul 28 10:30:42 2023 +0800

    LoongArch: Only fiddle with CHECKFLAGS if `need-compiler'
    
    [ Upstream commit 54c2c9df083fe1d4a9df54d9876f32582ce6d77a ]
    
    This is a port of commit 4fe4a6374c4db9ae2b ("MIPS: Only fiddle with
    CHECKFLAGS if `need-compiler'") to LoongArch.
    
    We have originally guarded fiddling with CHECKFLAGS in our arch Makefile
    by checking for the CONFIG_LOONGARCH variable, not set for targets such
    as `distclean', etc. that neither include `.config' nor use the compiler.
    
    Starting from commit 805b2e1d427aab4 ("kbuild: include Makefile.compiler
    only when compiler is needed") we have had a generic `need-compiler'
    variable explicitly telling us if the compiler will be used and thus its
    capabilities need to be checked and expressed in the form of compilation
    flags.  If this variable is not set, then `make' functions such as
    `cc-option' are undefined, causing all kinds of weirdness to happen if
    we expect specific results to be returned.
    
    It doesn't cause problems on LoongArch now. But as a guard we replace
    the check for CONFIG_LOONGARCH with one for `need-compiler' instead, so
    as to prevent the compiler from being ever called for CHECKFLAGS when
    not needed.
    
    Signed-off-by: Huacai Chen <chenhuacai@xxxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/arch/loongarch/Makefile b/arch/loongarch/Makefile
index 63a637fdf6c28..95629322241f5 100644
--- a/arch/loongarch/Makefile
+++ b/arch/loongarch/Makefile
@@ -106,7 +106,7 @@ KBUILD_CFLAGS += -isystem $(shell $(CC) -print-file-name=include)
 
 KBUILD_LDFLAGS	+= -m $(ld-emul)
 
-ifdef CONFIG_LOONGARCH
+ifdef need-compiler
 CHECKFLAGS += $(shell $(CC) $(KBUILD_CPPFLAGS) $(KBUILD_CFLAGS) -dM -E -x c /dev/null | \
 	grep -E -vw '__GNUC_(MINOR_|PATCHLEVEL_)?_' | \
 	sed -e "s/^\#define /-D'/" -e "s/ /'='/" -e "s/$$/'/" -e 's/\$$/&&/g')



[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux