Re: [PATCH] arm64: Disable EL2 traps for BRBE instructions executed in EL1
From: kernel test robot
Date: Fri Mar 24 2023 - 15:39:43 EST
Hi Anshuman,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on arm64/for-next/core]
[also build test ERROR on arm/for-next arm/fixes kvmarm/next soc/for-next xilinx-xlnx/master linus/master v6.3-rc3 next-20230324]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Anshuman-Khandual/arm64-Disable-EL2-traps-for-BRBE-instructions-executed-in-EL1/20230324-135312
base: https://git.kernel.org/pub/scm/linux/kernel/git/arm64/linux.git for-next/core
patch link: https://lore.kernel.org/r/20230324055127.2228330-1-anshuman.khandual%40arm.com
patch subject: [PATCH] arm64: Disable EL2 traps for BRBE instructions executed in EL1
config: arm64-allnoconfig (https://download.01.org/0day-ci/archive/20230325/202303250356.K6CxtK2u-lkp@xxxxxxxxx/config)
compiler: aarch64-linux-gcc (GCC) 12.1.0
reproduce (this is a W=1 build):
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
chmod +x ~/bin/make.cross
# https://github.com/intel-lab-lkp/linux/commit/457bbac085c7f83be1a63b54f6caf32b820503eb
git remote add linux-review https://github.com/intel-lab-lkp/linux
git fetch --no-tags linux-review Anshuman-Khandual/arm64-Disable-EL2-traps-for-BRBE-instructions-executed-in-EL1/20230324-135312
git checkout 457bbac085c7f83be1a63b54f6caf32b820503eb
# save the config file
mkdir build_dir && cp config build_dir/.config
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm64 olddefconfig
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash arch/arm64/
If you fix the issue, kindly add following tag where applicable
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Link: https://lore.kernel.org/oe-kbuild-all/202303250356.K6CxtK2u-lkp@xxxxxxxxx/
All errors (new ones prefixed by >>):
In file included from arch/arm64/kernel/head.S:27:
arch/arm64/include/asm/kernel-pgtable.h:126:41: warning: "PUD_SHIFT" is not defined, evaluates to 0 [-Wundef]
126 | #define ARM64_MEMSTART_SHIFT PUD_SHIFT
| ^~~~~~~~~
arch/arm64/include/asm/kernel-pgtable.h:139:5: note: in expansion of macro 'ARM64_MEMSTART_SHIFT'
139 | #if ARM64_MEMSTART_SHIFT < SECTION_SIZE_BITS
| ^~~~~~~~~~~~~~~~~~~~
arch/arm64/kernel/head.S: Assembler messages:
>> arch/arm64/kernel/head.S:577: Error: undefined symbol HFGITR_EL2_nBRBIALL used as an immediate value
>> arch/arm64/kernel/head.S:577: Error: undefined symbol HFGITR_EL2_nBRBINJ used as an immediate value
vim +577 arch/arm64/kernel/head.S
034edabe6cf1d0 Laura Abbott 2014-11-21 526
034edabe6cf1d0 Laura Abbott 2014-11-21 527 /*
034edabe6cf1d0 Laura Abbott 2014-11-21 528 * end early head section, begin head code that is also used for
034edabe6cf1d0 Laura Abbott 2014-11-21 529 * hotplug and needs to have the same protections as the text region
034edabe6cf1d0 Laura Abbott 2014-11-21 530 */
d54170812ef1c8 Mark Rutland 2023-02-20 531 .section ".idmap.text","a"
f80fb3a3d50843 Ard Biesheuvel 2016-01-26 532
9703d9d7f77ce1 Catalin Marinas 2012-03-05 533 /*
ecbb11ab3ebc02 Mark Rutland 2020-11-13 534 * Starting from EL2 or EL1, configure the CPU to execute at the highest
ecbb11ab3ebc02 Mark Rutland 2020-11-13 535 * reachable EL supported by the kernel in a chosen default state. If dropping
ecbb11ab3ebc02 Mark Rutland 2020-11-13 536 * from EL2 to EL1, configure EL2 before configuring EL1.
828e9834e9a5b7 Matthew Leach 2013-10-11 537 *
d87a8e65b51011 Mark Rutland 2020-11-13 538 * Since we cannot always rely on ERET synchronizing writes to sysregs (e.g. if
d87a8e65b51011 Mark Rutland 2020-11-13 539 * SCTLR_ELx.EOS is clear), we place an ISB prior to ERET.
828e9834e9a5b7 Matthew Leach 2013-10-11 540 *
b65e411d6cc2f1 Marc Zyngier 2022-06-30 541 * Returns either BOOT_CPU_MODE_EL1 or BOOT_CPU_MODE_EL2 in x0 if
b65e411d6cc2f1 Marc Zyngier 2022-06-30 542 * booted in EL1 or EL2 respectively, with the top 32 bits containing
b65e411d6cc2f1 Marc Zyngier 2022-06-30 543 * potential context flags. These flags are *not* stored in __boot_cpu_mode.
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 544 *
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 545 * x0: whether we are being called from the primary boot path with the MMU on
9703d9d7f77ce1 Catalin Marinas 2012-03-05 546 */
ecbb11ab3ebc02 Mark Rutland 2020-11-13 547 SYM_FUNC_START(init_kernel_el)
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 548 mrs x1, CurrentEL
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 549 cmp x1, #CurrentEL_EL2
d87a8e65b51011 Mark Rutland 2020-11-13 550 b.eq init_el2
d87a8e65b51011 Mark Rutland 2020-11-13 551
d87a8e65b51011 Mark Rutland 2020-11-13 552 SYM_INNER_LABEL(init_el1, SYM_L_LOCAL)
31a32b49b80f79 Marc Zyngier 2021-04-08 553 mov_q x0, INIT_SCTLR_EL1_MMU_OFF
9d7c13e5dde312 Ard Biesheuvel 2023-01-11 554 pre_disable_mmu_workaround
31a32b49b80f79 Marc Zyngier 2021-04-08 555 msr sctlr_el1, x0
9cf71728931a40 Matthew Leach 2013-10-11 556 isb
d87a8e65b51011 Mark Rutland 2020-11-13 557 mov_q x0, INIT_PSTATE_EL1
d87a8e65b51011 Mark Rutland 2020-11-13 558 msr spsr_el1, x0
d87a8e65b51011 Mark Rutland 2020-11-13 559 msr elr_el1, lr
d87a8e65b51011 Mark Rutland 2020-11-13 560 mov w0, #BOOT_CPU_MODE_EL1
d87a8e65b51011 Mark Rutland 2020-11-13 561 eret
9703d9d7f77ce1 Catalin Marinas 2012-03-05 562
d87a8e65b51011 Mark Rutland 2020-11-13 563 SYM_INNER_LABEL(init_el2, SYM_L_LOCAL)
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 564 msr elr_el2, lr
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 565
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 566 // clean all HYP code to the PoC if we booted at EL2 with the MMU on
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 567 cbz x0, 0f
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 568 adrp x0, __hyp_idmap_text_start
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 569 adr_l x1, __hyp_text_end
d54170812ef1c8 Mark Rutland 2023-02-20 570 adr_l x2, dcache_clean_poc
d54170812ef1c8 Mark Rutland 2023-02-20 571 blr x2
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 572 0:
78869f0f0552d0 David Brazdil 2020-12-02 573 mov_q x0, HCR_HOST_NVHE_FLAGS
78869f0f0552d0 David Brazdil 2020-12-02 574 msr hcr_el2, x0
22043a3c082a58 Dave Martin 2017-10-31 575 isb
78869f0f0552d0 David Brazdil 2020-12-02 576
e2df464173f0b5 Marc Zyngier 2021-02-08 @577 init_el2_state
22043a3c082a58 Dave Martin 2017-10-31 578
712c6ff4dba491 Marc Zyngier 2012-10-19 579 /* Hypervisor stub */
78869f0f0552d0 David Brazdil 2020-12-02 580 adr_l x0, __hyp_stub_vectors
712c6ff4dba491 Marc Zyngier 2012-10-19 581 msr vbar_el2, x0
d87a8e65b51011 Mark Rutland 2020-11-13 582 isb
78869f0f0552d0 David Brazdil 2020-12-02 583
ae4b7e38e9a947 Marc Zyngier 2022-06-30 584 mov_q x1, INIT_SCTLR_EL1_MMU_OFF
ae4b7e38e9a947 Marc Zyngier 2022-06-30 585
31a32b49b80f79 Marc Zyngier 2021-04-08 586 /*
31a32b49b80f79 Marc Zyngier 2021-04-08 587 * Fruity CPUs seem to have HCR_EL2.E2H set to RES1,
31a32b49b80f79 Marc Zyngier 2021-04-08 588 * making it impossible to start in nVHE mode. Is that
31a32b49b80f79 Marc Zyngier 2021-04-08 589 * compliant with the architecture? Absolutely not!
31a32b49b80f79 Marc Zyngier 2021-04-08 590 */
31a32b49b80f79 Marc Zyngier 2021-04-08 591 mrs x0, hcr_el2
31a32b49b80f79 Marc Zyngier 2021-04-08 592 and x0, x0, #HCR_E2H
31a32b49b80f79 Marc Zyngier 2021-04-08 593 cbz x0, 1f
31a32b49b80f79 Marc Zyngier 2021-04-08 594
ae4b7e38e9a947 Marc Zyngier 2022-06-30 595 /* Set a sane SCTLR_EL1, the VHE way */
9d7c13e5dde312 Ard Biesheuvel 2023-01-11 596 pre_disable_mmu_workaround
ae4b7e38e9a947 Marc Zyngier 2022-06-30 597 msr_s SYS_SCTLR_EL12, x1
ae4b7e38e9a947 Marc Zyngier 2022-06-30 598 mov x2, #BOOT_CPU_FLAG_E2H
ae4b7e38e9a947 Marc Zyngier 2022-06-30 599 b 2f
31a32b49b80f79 Marc Zyngier 2021-04-08 600
31a32b49b80f79 Marc Zyngier 2021-04-08 601 1:
9d7c13e5dde312 Ard Biesheuvel 2023-01-11 602 pre_disable_mmu_workaround
ae4b7e38e9a947 Marc Zyngier 2022-06-30 603 msr sctlr_el1, x1
ae4b7e38e9a947 Marc Zyngier 2022-06-30 604 mov x2, xzr
ae4b7e38e9a947 Marc Zyngier 2022-06-30 605 2:
d87a8e65b51011 Mark Rutland 2020-11-13 606 mov w0, #BOOT_CPU_MODE_EL2
ae4b7e38e9a947 Marc Zyngier 2022-06-30 607 orr x0, x0, x2
9703d9d7f77ce1 Catalin Marinas 2012-03-05 608 eret
ecbb11ab3ebc02 Mark Rutland 2020-11-13 609 SYM_FUNC_END(init_kernel_el)
9703d9d7f77ce1 Catalin Marinas 2012-03-05 610
9703d9d7f77ce1 Catalin Marinas 2012-03-05 611 /*
9703d9d7f77ce1 Catalin Marinas 2012-03-05 612 * This provides a "holding pen" for platforms to hold all secondary
9703d9d7f77ce1 Catalin Marinas 2012-03-05 613 * cores are held until we're ready for them to initialise.
9703d9d7f77ce1 Catalin Marinas 2012-03-05 614 */
c63d9f82db9439 Mark Brown 2020-02-18 615 SYM_FUNC_START(secondary_holding_pen)
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 616 mov x0, xzr
ecbb11ab3ebc02 Mark Rutland 2020-11-13 617 bl init_kernel_el // w0=cpu_boot_mode
005e12676af09a Ard Biesheuvel 2022-06-24 618 mrs x2, mpidr_el1
b03cc885328e3c Ard Biesheuvel 2016-04-18 619 mov_q x1, MPIDR_HWID_BITMASK
005e12676af09a Ard Biesheuvel 2022-06-24 620 and x2, x2, x1
b1c98297fe0c6e Ard Biesheuvel 2015-03-10 621 adr_l x3, secondary_holding_pen_release
9703d9d7f77ce1 Catalin Marinas 2012-03-05 622 pen: ldr x4, [x3]
005e12676af09a Ard Biesheuvel 2022-06-24 623 cmp x4, x2
9703d9d7f77ce1 Catalin Marinas 2012-03-05 624 b.eq secondary_startup
9703d9d7f77ce1 Catalin Marinas 2012-03-05 625 wfe
9703d9d7f77ce1 Catalin Marinas 2012-03-05 626 b pen
c63d9f82db9439 Mark Brown 2020-02-18 627 SYM_FUNC_END(secondary_holding_pen)
652af899799354 Mark Rutland 2013-10-24 628
652af899799354 Mark Rutland 2013-10-24 629 /*
652af899799354 Mark Rutland 2013-10-24 630 * Secondary entry point that jumps straight into the kernel. Only to
652af899799354 Mark Rutland 2013-10-24 631 * be used where CPUs are brought online dynamically by the kernel.
652af899799354 Mark Rutland 2013-10-24 632 */
c63d9f82db9439 Mark Brown 2020-02-18 633 SYM_FUNC_START(secondary_entry)
3dcf60bbfd284e Ard Biesheuvel 2023-01-11 634 mov x0, xzr
ecbb11ab3ebc02 Mark Rutland 2020-11-13 635 bl init_kernel_el // w0=cpu_boot_mode
652af899799354 Mark Rutland 2013-10-24 636 b secondary_startup
c63d9f82db9439 Mark Brown 2020-02-18 637 SYM_FUNC_END(secondary_entry)
9703d9d7f77ce1 Catalin Marinas 2012-03-05 638
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests