From: Alexandru Elisei <alexandru.elisei@arm.com>
To: kvm@vger.kernel.org, kvmarm@lists.cs.columbia.edu
Cc: maz@kernel.org, andre.przywara@arm.com, pbonzini@redhat.com
Subject: [kvm-unit-tests RFC PATCH 12/16] arm64: timer: Add test for EL2 timers
Date: Wed, 28 Aug 2019 14:38:27 +0100 [thread overview]
Message-ID: <1566999511-24916-13-git-send-email-alexandru.elisei@arm.com> (raw)
In-Reply-To: <1566999511-24916-1-git-send-email-alexandru.elisei@arm.com>
When VHE is available, EL2 has two extra timers, the physical and virtual
EL2 timers. Extend the timer test to include them.
Signed-off-by: Alexandru Elisei <alexandru.elisei@arm.com>
---
I'm seeing three failed tests when running at EL2, I'm investigating those
to figure out if it's KVM or kvm-unit-tests related.
lib/arm64/asm/sysreg.h | 8 +++
arm/timer.c | 150 +++++++++++++++++++++++++++++++++++++++++++++++--
2 files changed, 154 insertions(+), 4 deletions(-)
diff --git a/lib/arm64/asm/sysreg.h b/lib/arm64/asm/sysreg.h
index ed407f93330d..02054fbe2763 100644
--- a/lib/arm64/asm/sysreg.h
+++ b/lib/arm64/asm/sysreg.h
@@ -19,6 +19,14 @@
#define SYS_CNTV_CTL_EL02 sys_reg(3, 5, 14, 3, 1)
#define SYS_CNTV_CVAL_EL02 sys_reg(3, 5, 14, 3, 2)
+#define SYS_CNTHP_TVAL_EL2 sys_reg(3, 4, 14, 2, 0)
+#define SYS_CNTHP_CTL_EL2 sys_reg(3, 4, 14, 2, 1)
+#define SYS_CNTHP_CVAL_EL2 sys_reg(3, 4, 14, 2, 2)
+
+#define SYS_CNTHV_TVAL_EL2 sys_reg(3, 4, 14, 3, 0)
+#define SYS_CNTHV_CTL_EL2 sys_reg(3, 4, 14, 3, 1)
+#define SYS_CNTHV_CVAL_EL2 sys_reg(3, 4, 14, 3, 2)
+
#ifdef __ASSEMBLY__
.irp num,0,1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20,21,22,23,24,25,26,27,28,29,30
.equ .L__reg_num_x\num, \num
diff --git a/arm/timer.c b/arm/timer.c
index 29477ee65878..faab671d0fb1 100644
--- a/arm/timer.c
+++ b/arm/timer.c
@@ -23,6 +23,8 @@ static void *gic_icenabler;
static bool ptimer_unsupported;
+static int current_el;
+
static void ptimer_unsupported_handler(struct pt_regs *regs, unsigned int esr)
{
ptimer_unsupported = true;
@@ -159,6 +161,66 @@ static void write_ptimer_ctl_vhe(u64 val)
write_sysreg_s(val, SYS_CNTP_CTL_EL02);
}
+static u64 read_hvtimer_cval(void)
+{
+ return read_sysreg_s(SYS_CNTHV_CVAL_EL2);
+}
+
+static void write_hvtimer_cval(u64 val)
+{
+ write_sysreg_s(val, SYS_CNTHV_CVAL_EL2);
+}
+
+static s32 read_hvtimer_tval(void)
+{
+ return read_sysreg_s(SYS_CNTHV_TVAL_EL2);
+}
+
+static void write_hvtimer_tval(s32 val)
+{
+ write_sysreg_s(val, SYS_CNTHV_TVAL_EL2);
+}
+
+static u64 read_hvtimer_ctl(void)
+{
+ return read_sysreg_s(SYS_CNTHV_CTL_EL2);
+}
+
+static void write_hvtimer_ctl(u64 val)
+{
+ write_sysreg_s(val, SYS_CNTHV_CTL_EL2);
+}
+
+static u64 read_hptimer_cval(void)
+{
+ return read_sysreg_s(SYS_CNTHP_CVAL_EL2);
+}
+
+static void write_hptimer_cval(u64 val)
+{
+ write_sysreg_s(val, SYS_CNTHP_CVAL_EL2);
+}
+
+static s32 read_hptimer_tval(void)
+{
+ return read_sysreg_s(SYS_CNTHP_TVAL_EL2);
+}
+
+static void write_hptimer_tval(s32 val)
+{
+ write_sysreg_s(val, SYS_CNTHP_TVAL_EL2);
+}
+
+static u64 read_hptimer_ctl(void)
+{
+ return read_sysreg_s(SYS_CNTHP_CTL_EL2);
+}
+
+static void write_hptimer_ctl(u64 val)
+{
+ write_sysreg_s(val, SYS_CNTHP_CTL_EL2);
+}
+
struct timer_info {
u32 irq;
u32 irq_flags;
@@ -216,7 +278,29 @@ static struct timer_info ptimer_info_vhe = {
.write_ctl = write_ptimer_ctl_vhe,
};
-static struct timer_info *vtimer, *ptimer;
+static struct timer_info hvtimer_info = {
+ .irq_received = false,
+ .read_counter = read_vtimer_counter,
+ .read_cval = read_hvtimer_cval,
+ .write_cval = write_hvtimer_cval,
+ .read_tval = read_hvtimer_tval,
+ .write_tval = write_hvtimer_tval,
+ .read_ctl = read_hvtimer_ctl,
+ .write_ctl = write_hvtimer_ctl,
+};
+
+static struct timer_info hptimer_info = {
+ .irq_received = false,
+ .read_counter = read_ptimer_counter,
+ .read_cval = read_hptimer_cval,
+ .write_cval = write_hptimer_cval,
+ .read_tval = read_hptimer_tval,
+ .write_tval = write_hptimer_tval,
+ .read_ctl = read_hptimer_ctl,
+ .write_ctl = write_hptimer_ctl,
+};
+
+static struct timer_info *vtimer, *ptimer, *hvtimer, *hptimer;
static void set_timer_irq_enabled(struct timer_info *info, bool enabled)
{
@@ -241,6 +325,10 @@ static void irq_handler(struct pt_regs *regs)
info = vtimer;
} else if (irqnr == PPI(ptimer->irq)) {
info = ptimer;
+ } else if (current_el == CurrentEL_EL2 && irqnr == PPI(hptimer->irq)) {
+ info = hptimer;
+ } else if (current_el == CurrentEL_EL2 && irqnr == PPI(hvtimer->irq)) {
+ info = hvtimer;
} else {
report_info("Unexpected interrupt: %d\n", irqnr);
return;
@@ -362,6 +450,20 @@ static void test_ptimer(void)
report_prefix_pop();
}
+static void test_hvtimer(void)
+{
+ report_prefix_push("hvtimer-busy-loop");
+ test_timer(hvtimer);
+ report_prefix_pop();
+}
+
+static void test_hptimer(void)
+{
+ report_prefix_push("hptimer-busy-loop");
+ test_timer(hptimer);
+ report_prefix_pop();
+}
+
static void test_init(void)
{
const struct fdt_property *prop;
@@ -369,12 +471,14 @@ static void test_init(void)
int node, len;
u32 *data;
- if (current_level() == CurrentEL_EL1) {
+ if (current_el == CurrentEL_EL1) {
vtimer = &vtimer_info;
ptimer = &ptimer_info;
} else {
vtimer = &vtimer_info_vhe;
ptimer = &ptimer_info_vhe;
+ hvtimer = &hvtimer_info;
+ hptimer = &hptimer_info;
}
node = fdt_node_offset_by_compatible(fdt, -1, "arm,armv8-timer");
@@ -390,6 +494,19 @@ static void test_init(void)
vtimer->irq = fdt32_to_cpu(data[7]);
vtimer->irq_flags = fdt32_to_cpu(data[8]);
+ if (current_el == CurrentEL_EL2) {
+ assert(fdt32_to_cpu(data[9]) == 1);
+ hptimer->irq = fdt32_to_cpu(data[10]);
+ hptimer->irq_flags = fdt32_to_cpu(data[11]);
+ /* The hvtimer is not in the DT, assume KVM default. */
+ hvtimer->irq = 28;
+ /*
+ * With VHE, accesses to the vtimer are redirected to the
+ * hvtimer. They should have the same interrupt properties.
+ */
+ hvtimer->irq_flags = vtimer->irq_flags;
+ }
+
install_exception_handler(EL1H_SYNC, ESR_EL1_EC_UNKNOWN, ptimer_unsupported_handler);
ptimer->read_ctl();
install_exception_handler(EL1H_SYNC, ESR_EL1_EC_UNKNOWN, NULL);
@@ -433,12 +550,22 @@ static void print_timer_info(void)
printf("CNTVCT_EL0 : 0x%016lx\n", vtimer->read_counter());
printf("CNTV_CTL_EL0 : 0x%016lx\n", vtimer->read_ctl());
printf("CNTV_CVAL_EL0: 0x%016lx\n", vtimer->read_cval());
+
+ if (current_el == CurrentEL_EL2) {
+ printf("CNTHP_CTL_EL0 : 0x%016lx\n", hptimer->read_ctl());
+ printf("CNTHP_CVAL_EL0: 0x%016lx\n", hptimer->read_cval());
+
+ printf("CNTHV_CTL_EL0 : 0x%016lx\n", hvtimer->read_ctl());
+ printf("CNTHV_CVAL_EL0: 0x%016lx\n", hvtimer->read_cval());
+ }
}
int main(int argc, char **argv)
{
int i;
+ current_el = current_level();
+
test_init();
print_timer_info();
@@ -446,13 +573,28 @@ int main(int argc, char **argv)
if (argc == 1) {
test_vtimer();
test_ptimer();
+ if (current_el == CurrentEL_EL2) {
+ test_hvtimer();
+ test_hptimer();
+ }
}
for (i = 1; i < argc; ++i) {
- if (strcmp(argv[i], "vtimer") == 0)
+ if (strcmp(argv[i], "vtimer") == 0) {
test_vtimer();
- if (strcmp(argv[i], "ptimer") == 0)
+ } if (strcmp(argv[i], "ptimer") == 0) {
test_ptimer();
+ } if (strcmp(argv[i], "hvtimer") == 0) {
+ if (current_el == CurrentEL_EL1)
+ report_info("Skipping hvtimer tests. Boot at EL2 to enable.");
+ else
+ test_hvtimer();
+ } if (strcmp(argv[i], "hptimer") == 0) {
+ if (current_el == CurrentEL_EL1)
+ report_info("Skipping hptimer tests. Boot at EL2 to enable.");
+ else
+ test_hptimer();
+ }
}
return report_summary();
--
2.7.4
_______________________________________________
kvmarm mailing list
kvmarm@lists.cs.columbia.edu
https://lists.cs.columbia.edu/mailman/listinfo/kvmarm
next prev parent reply other threads:[~2019-08-28 13:39 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-08-28 13:38 [kvm-unit-tests RFC PATCH 00/16] arm64: Run at EL2 Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 01/16] arm: selftest.c: Remove redundant check for Exception Level Alexandru Elisei
2019-08-28 14:32 ` Andrew Jones
2019-08-28 15:39 ` Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 02/16] arm/arm64: psci: Don't run C code without stack or vectors Alexandru Elisei
2019-08-28 14:45 ` Andrew Jones
2019-08-28 15:14 ` Alexandru Elisei
2019-09-02 14:55 ` Alexandru Elisei
2019-09-03 6:37 ` Andrew Jones
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 03/16] lib: arm/arm64: Add missing include for alloc_page.h in pgtable.h Alexandru Elisei
2019-08-28 14:47 ` Andrew Jones
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 04/16] arm/arm64: selftest: Add prefetch abort test Alexandru Elisei
2019-08-28 14:09 ` Mark Rutland
2019-08-29 8:18 ` Alexandru Elisei
2019-08-29 10:19 ` Mark Rutland
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 05/16] arm64: timer: Write to ICENABLER to disable timer IRQ Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 06/16] arm64: timer: EOIR the interrupt after masking the timer Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 07/16] arm64: timer: Test behavior when timer disabled or masked Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 08/16] lib: arm/arm64: Refuse to disable the MMU with non-identity stack pointer Alexandru Elisei
2019-08-28 14:55 ` Andrew Jones
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 09/16] lib: arm/arm64: Invalidate TLB before enabling MMU Alexandru Elisei
2019-08-28 14:59 ` Andrew Jones
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 10/16] lib: Add UL and ULL definitions to linux/const.h Alexandru Elisei
2019-08-28 15:10 ` Andrew Jones
2019-08-28 15:46 ` Alexandru Elisei
2019-08-28 16:19 ` Andrew Jones
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 11/16] lib: arm64: Run existing tests at EL2 Alexandru Elisei
2019-08-28 13:38 ` Alexandru Elisei [this message]
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 13/16] arm64: selftest: Add basic test for EL2 Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 14/16] lib: arm64: Add support for disabling and re-enabling VHE Alexandru Elisei
2019-08-28 14:19 ` Mark Rutland
2019-08-29 8:36 ` Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 15/16] arm64: selftest: Expand EL2 test to disable and re-enable VHE Alexandru Elisei
2019-08-28 13:38 ` [kvm-unit-tests RFC PATCH 16/16] arm64: timer: Run tests with VHE disabled Alexandru Elisei
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1566999511-24916-13-git-send-email-alexandru.elisei@arm.com \
--to=alexandru.elisei@arm.com \
--cc=andre.przywara@arm.com \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=maz@kernel.org \
--cc=pbonzini@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).