Re: [PATCH v2 0/3] Fix arm_arch_timer clockmode when vDSO disabled
From: Marc Zyngier <maz@kernel.org>
Date: 2020-02-22 10:40:31
Also in:
linux-arch, lkml
On Fri, 21 Feb 2020 18:18:46 +0000 Vincenzo Frascino [off-list ref] wrote:
The arm_arch_timer requires that VDSO_CLOCKMODE_ARCHTIMER to be
defined to compile correctly. On arm the vDSO can be disabled and when
this is the case the compilation ends prematurely with an error:
$ make ARCH=arm multi_v7_defconfig
$ ./scripts/config -d VDSO
$ make
drivers/clocksource/arm_arch_timer.c:73:44: error:
‘VDSO_CLOCKMODE_ARCHTIMER’ undeclared here (not in a function)
static enum vdso_clock_mode vdso_default = VDSO_CLOCKMODE_ARCHTIMER;
^
scripts/Makefile.build:267: recipe for target
'drivers/clocksource/arm_arch_timer.o' failed
make[2]: *** [drivers/clocksource/arm_arch_timer.o] Error 1
make[2]: *** Waiting for unfinished jobs....
scripts/Makefile.build:505: recipe for target 'drivers/clocksource' failed
make[1]: *** [drivers/clocksource] Error 2
make[1]: *** Waiting for unfinished jobs....
Makefile:1683: recipe for target 'drivers' failed
make: *** [drivers] Error 2
This patch series addresses the issue defining a default arch clockmode
for arm and arm64 and using it to initialize the arm_arch_timer.arm only. arm64 is just fine.
Changes: -------- v2: - Addressed Marc Zyngier comments. - Rebased on 5.6-rc2.
This doesn't apply to -rc2, and is rather against next.
Cc: Catalin Marinas <catalin.marinas@arm.com> Cc: Will Deacon <redacted> Cc: Russell King <linux@armlinux.org.uk> Cc: Thomas Gleixner <redacted> Cc: Marc Zyngier <maz@kernel.org> Cc: Mark Rutland <Mark.Rutland@arm.com> Cc: Marek Szyprowski <m.szyprowski@samsung.com> Signed-off-by: Vincenzo Frascino <vincenzo.frascino@arm.com> Vincenzo Frascino (3): arm: clocksource: Add VDSO default clockmode arm64: clocksource: Add VDSO default clockmode clocksource: Fix arm_arch_timer clockmode when vDSO disabled
Please squash the three patches into a single one. There is zero point in having 3 patches for something that small. M. -- Jazz is not dead. It just smells funny... _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel