Messages in this thread | | | From | Nick Desaulniers <> | Date | Thu, 31 Oct 2019 10:28:43 -0700 | Subject | Re: [PATCH v3 15/17] arm64: vdso: disable Shadow Call Stack |
| |
On Thu, Oct 31, 2019 at 9:47 AM <samitolvanen@google.com> wrote:
I've gotten slapped down before for -ENOCOMMITMSG; maybe include more info if there's a v4? Maintainers can take the safe position of always saying "no," so it is useful to always provide an answer to the implicit question, "why should I take this patch?"
> > Signed-off-by: Sami Tolvanen <samitolvanen@google.com> > --- > arch/arm64/kernel/vdso/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/kernel/vdso/Makefile b/arch/arm64/kernel/vdso/Makefile > index dd2514bb1511..a87a4f11724e 100644 > --- a/arch/arm64/kernel/vdso/Makefile > +++ b/arch/arm64/kernel/vdso/Makefile > @@ -25,7 +25,7 @@ ccflags-y += -DDISABLE_BRANCH_PROFILING > > VDSO_LDFLAGS := -Bsymbolic > > -CFLAGS_REMOVE_vgettimeofday.o = $(CC_FLAGS_FTRACE) -Os > +CFLAGS_REMOVE_vgettimeofday.o = $(CC_FLAGS_FTRACE) -Os $(CC_FLAGS_SCS)
Looks like vgettimeofday is the only remaining source written in C, so we shouldn't need to strip it from other assembly source files. Reviewed-by: Nick Desaulniers <ndesaulniers@google.com>
> KBUILD_CFLAGS += $(DISABLE_LTO) > KASAN_SANITIZE := n > UBSAN_SANITIZE := n > -- > 2.24.0.rc0.303.g954a862665-goog >
-- Thanks, ~Nick Desaulniers
| |