On Tue, Feb 04, 2025 at 01:47:23PM -0800, Yifei Liu wrote: > From: Kevin Brodsky <kevin.brod...@arm.com> > > [ Upstream commit 46036188ea1f5266df23a6149dea0df1c77cd1c7 ] > > The mm kselftests are currently built with no optimisation (-O0). It's > unclear why, and besides being obviously suboptimal, this also prevents > the pkeys tests from working as intended. Let's build all the tests with > -O2. > > [kevin.brod...@arm.com: silence unused-result warnings] > Link: > https://lkml.kernel.org/r/20250107170110.2819685-1-kevin.brod...@arm.com > Link: https://lkml.kernel.org/r/20241209095019.1732120-6-kevin.brod...@arm.com > Signed-off-by: Kevin Brodsky <kevin.brod...@arm.com> > Cc: Aruna Ramakrishna <aruna.ramakris...@oracle.com> > Cc: Catalin Marinas <catalin.mari...@arm.com> > Cc: Dave Hansen <dave.han...@linux.intel.com> > Cc: Joey Gouly <joey.go...@arm.com> > Cc: Keith Lucas <keith.lu...@oracle.com> > Cc: Ryan Roberts <ryan.robe...@arm.com> > Cc: Shuah Khan <sh...@kernel.org> > Signed-off-by: Andrew Morton <a...@linux-foundation.org> > (cherry picked from commit 46036188ea1f5266df23a6149dea0df1c77cd1c7) > [Yifei: This commit also fix the failure of pkey_sighandler_tests_64, > which is also in linux-6.12.y and linux-6.13.y, thus backport this commit] > Signed-off-by: Yifei Liu <yifei.l....@oracle.com> > --- > tools/testing/selftests/mm/Makefile | 9 ++++++++- > 1 file changed, 8 insertions(+), 1 deletion(-) > > diff --git a/tools/testing/selftests/mm/Makefile > b/tools/testing/selftests/mm/Makefile > index 02e1204971b0..c0138cb19705 100644 > --- a/tools/testing/selftests/mm/Makefile > +++ b/tools/testing/selftests/mm/Makefile > @@ -33,9 +33,16 @@ endif > # LDLIBS. > MAKEFLAGS += --no-builtin-rules > > -CFLAGS = -Wall -I $(top_srcdir) $(EXTRA_CFLAGS) $(KHDR_INCLUDES) > $(TOOLS_INCLUDES) > +CFLAGS = -Wall -O2 -I $(top_srcdir) $(EXTRA_CFLAGS) $(KHDR_INCLUDES) > $(TOOLS_INCLUDES) > LDLIBS = -lrt -lpthread -lm > > +# Some distributions (such as Ubuntu) configure GCC so that _FORTIFY_SOURCE > is > +# automatically enabled at -O1 or above. This triggers various unused-result > +# warnings where functions such as read() or write() are called and their > +# return value is not checked. Disable _FORTIFY_SOURCE to silence those > +# warnings. > +CFLAGS += -U_FORTIFY_SOURCE > + > TEST_GEN_FILES = cow > TEST_GEN_FILES += compaction_test > TEST_GEN_FILES += gup_longterm
This does not apply to 6.13 :(
