8796fe40dd
The definition of top_bit used in this function is one higher
than that used in the Arm ARM psuedo-code, which put the error
indication at top_bit - 1 at the wrong place, which meant that
it wasn't visible to Auth.
Fixing the definition of top_bit requires more changes, because
its most common use is for the count of bits in top_bit:bot_bit,
which would then need to be computed as top_bit - bot_bit + 1.
For now, prefer the minimal fix to the error indication alone.
Fixes: 63ff0ca94c
Reported-by: Derrick McKee <derrick.mckee@gmail.com>
Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
Message-id: 20200728195706.11087-1-richard.henderson@linaro.org
Reviewed-by: Peter Maydell <peter.maydell@linaro.org>
[PMM: added comment about the divergence from the pseudocode]
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
78 lines
1.9 KiB
Makefile
78 lines
1.9 KiB
Makefile
# -*- Mode: makefile -*-
|
|
#
|
|
# AArch64 specific tweaks
|
|
|
|
ARM_SRC=$(SRC_PATH)/tests/tcg/arm
|
|
VPATH += $(ARM_SRC)
|
|
|
|
AARCH64_SRC=$(SRC_PATH)/tests/tcg/aarch64
|
|
VPATH += $(AARCH64_SRC)
|
|
|
|
# Float-convert Tests
|
|
AARCH64_TESTS=fcvt
|
|
|
|
fcvt: LDFLAGS+=-lm
|
|
|
|
run-fcvt: fcvt
|
|
$(call run-test,$<,$(QEMU) $<, "$< on $(TARGET_NAME)")
|
|
$(call diff-out,$<,$(AARCH64_SRC)/fcvt.ref)
|
|
|
|
# Pauth Tests
|
|
ifneq ($(DOCKER_IMAGE)$(CROSS_CC_HAS_ARMV8_3),)
|
|
AARCH64_TESTS += pauth-1 pauth-2 pauth-4 pauth-5
|
|
pauth-%: CFLAGS += -march=armv8.3-a
|
|
run-pauth-%: QEMU_OPTS += -cpu max
|
|
run-plugin-pauth-%: QEMU_OPTS += -cpu max
|
|
endif
|
|
|
|
# Semihosting smoke test for linux-user
|
|
AARCH64_TESTS += semihosting
|
|
run-semihosting: semihosting
|
|
$(call run-test,$<,$(QEMU) $< 2> $<.err, "$< on $(TARGET_NAME)")
|
|
|
|
run-plugin-semihosting-with-%:
|
|
$(call run-test, $@, $(QEMU) $(QEMU_OPTS) \
|
|
-plugin $(PLUGIN_LIB)/$(call extract-plugin,$@) \
|
|
$(call strip-plugin,$<) 2> $<.err, \
|
|
"$< on $(TARGET_NAME) with $*")
|
|
|
|
AARCH64_TESTS += semiconsole
|
|
run-semiconsole: semiconsole
|
|
$(call skip-test, $<, "MANUAL ONLY")
|
|
|
|
run-plugin-semiconsole-with-%:
|
|
$(call skip-test, $<, "MANUAL ONLY")
|
|
|
|
ifneq ($(DOCKER_IMAGE)$(CROSS_CC_HAS_SVE),)
|
|
# System Registers Tests
|
|
AARCH64_TESTS += sysregs
|
|
sysregs: CFLAGS+=-march=armv8.1-a+sve
|
|
|
|
# SVE ioctl test
|
|
AARCH64_TESTS += sve-ioctls
|
|
sve-ioctls: CFLAGS+=-march=armv8.1-a+sve
|
|
|
|
ifneq ($(HAVE_GDB_BIN),)
|
|
GDB_SCRIPT=$(SRC_PATH)/tests/guest-debug/run-test.py
|
|
|
|
run-gdbstub-sysregs: sysregs
|
|
$(call run-test, $@, $(GDB_SCRIPT) \
|
|
--gdb $(HAVE_GDB_BIN) \
|
|
--qemu $(QEMU) --qargs "$(QEMU_OPTS)" \
|
|
--bin $< --test $(AARCH64_SRC)/gdbstub/test-sve.py, \
|
|
"basic gdbstub SVE support")
|
|
|
|
run-gdbstub-sve-ioctls: sve-ioctls
|
|
$(call run-test, $@, $(GDB_SCRIPT) \
|
|
--gdb $(HAVE_GDB_BIN) \
|
|
--qemu $(QEMU) --qargs "$(QEMU_OPTS)" \
|
|
--bin $< --test $(AARCH64_SRC)/gdbstub/test-sve-ioctl.py, \
|
|
"basic gdbstub SVE ZLEN support")
|
|
|
|
EXTRA_RUNS += run-gdbstub-sysregs run-gdbstub-sve-ioctls
|
|
endif
|
|
|
|
endif
|
|
|
|
TESTS += $(AARCH64_TESTS)
|