diff mbox series

[RFC,for-next,1/1] arm64: sve: Fix some compile errors about sve_cond_update_zcr_vq

Message ID 1617183879-48748-1-git-send-email-tanxiaofei@huawei.com (mailing list archive)
State New, archived
Headers show
Series [RFC,for-next,1/1] arm64: sve: Fix some compile errors about sve_cond_update_zcr_vq | expand

Commit Message

Xiaofei Tan March 31, 2021, 9:44 a.m. UTC
There are some compile errors of ARM64 KVM when ARM64_SVE is not
selected, in the locations of sve_cond_update_zcr_vq used.
One error log is as following:

  CC      arch/arm64/kvm/sys_regs.o
In file included from arch/arm64/kvm/hyp/vhe/switch.c:8:0:
./arch/arm64/kvm/hyp/include/hyp/switch.h: In function ‘__hyp_sve_restore_guest’:
./arch/arm64/kvm/hyp/include/hyp/switch.h:220:2: error: implicit declaration of
function ‘sve_cond_update_zcr_vq’; did you mean
‘kvm_pmu_update_run’? [-Werror=implicit-function-declaration]
  sve_cond_update_zcr_vq(vcpu_sve_max_vq(vcpu) - 1, SYS_ZCR_EL2);
  ^~~~~~~~~~~~~~~~~~~~~~
  kvm_pmu_update_run

The root cause is that sve_cond_update_zcr_vq is not defined when
ARM64_SVE is not selected. Fix it by adding an empty definition
when CONFIG_ARM64_SVE=n.

Signed-off-by: Xiaofei Tan <tanxiaofei@huawei.com>
---
 arch/arm64/include/asm/fpsimd.h | 2 ++
 1 file changed, 2 insertions(+)

Comments

Marc Zyngier March 31, 2021, 10:09 a.m. UTC | #1
On Wed, 31 Mar 2021 17:44:39 +0800, Xiaofei Tan wrote:
> There are some compile errors of ARM64 KVM when ARM64_SVE is not
> selected, in the locations of sve_cond_update_zcr_vq used.
> One error log is as following:
> 
>   CC      arch/arm64/kvm/sys_regs.o
> In file included from arch/arm64/kvm/hyp/vhe/switch.c:8:0:
> ./arch/arm64/kvm/hyp/include/hyp/switch.h: In function ‘__hyp_sve_restore_guest’:
> ./arch/arm64/kvm/hyp/include/hyp/switch.h:220:2: error: implicit declaration of
> function ‘sve_cond_update_zcr_vq’; did you mean
> ‘kvm_pmu_update_run’? [-Werror=implicit-function-declaration]
>   sve_cond_update_zcr_vq(vcpu_sve_max_vq(vcpu) - 1, SYS_ZCR_EL2);
>   ^~~~~~~~~~~~~~~~~~~~~~
>   kvm_pmu_update_run
> 
> [...]

Applied to kvm-arm64/nvhe-sve, thanks!

[1/1] arm64: sve: Fix some compile errors about sve_cond_update_zcr_vq
      commit: a9f8696d4be5228de9d1d4f0e9f027b64d77dab6

Cheers,

	M.
diff mbox series

Patch

diff --git a/arch/arm64/include/asm/fpsimd.h b/arch/arm64/include/asm/fpsimd.h
index 05c9c55..3f93b63 100644
--- a/arch/arm64/include/asm/fpsimd.h
+++ b/arch/arm64/include/asm/fpsimd.h
@@ -168,6 +168,8 @@  static inline int sve_get_current_vl(void)
 static inline void sve_user_disable(void) { BUILD_BUG(); }
 static inline void sve_user_enable(void) { BUILD_BUG(); }
 
+#define sve_cond_update_zcr_vq(val, reg)
+
 static inline void sve_init_vq_map(void) { }
 static inline void sve_update_vq_map(void) { }
 static inline int sve_verify_vq_map(void) { return 0; }