From: Andrew Jones <drjones@redhat.com>
To: Oliver Upton <oupton@google.com>
Cc: kvmarm@lists.cs.columbia.edu, Marc Zyngier <maz@kernel.org>,
James Morse <james.morse@arm.com>,
Alexandru Elisei <alexandru.elisei@arm.com>,
Suzuki K Poulose <suzuki.poulose@arm.com>,
Peter Shier <pshier@google.com>,
Ricardo Koller <ricarkol@google.com>,
Reiji Watanabe <reijiw@google.com>,
Raghavendra Rao Anata <rananta@google.com>,
kvm@vger.kernel.org
Subject: Re: [PATCH v2 08/11] selftests: KVM: Create helper for making SMCCC calls
Date: Tue, 5 Oct 2021 15:39:57 +0200 [thread overview]
Message-ID: <20211005133957.q2a52s5mbthj6b4k@gator.home> (raw)
In-Reply-To: <20210923191610.3814698-9-oupton@google.com>
On Thu, Sep 23, 2021 at 07:16:07PM +0000, Oliver Upton wrote:
> The PSCI and PV stolen time tests both need to make SMCCC calls within
> the guest. Create a helper for making SMCCC calls and rework the
> existing tests to use the library function.
>
> Signed-off-by: Oliver Upton <oupton@google.com>
> ---
> .../testing/selftests/kvm/aarch64/psci_test.c | 25 ++++++-------------
> .../selftests/kvm/include/aarch64/processor.h | 22 ++++++++++++++++
> .../selftests/kvm/lib/aarch64/processor.c | 25 +++++++++++++++++++
> tools/testing/selftests/kvm/steal_time.c | 13 +++-------
> 4 files changed, 58 insertions(+), 27 deletions(-)
>
> diff --git a/tools/testing/selftests/kvm/aarch64/psci_test.c b/tools/testing/selftests/kvm/aarch64/psci_test.c
> index 018c269990e1..cebea7356e5a 100644
> --- a/tools/testing/selftests/kvm/aarch64/psci_test.c
> +++ b/tools/testing/selftests/kvm/aarch64/psci_test.c
> @@ -26,32 +26,23 @@
> static uint64_t psci_cpu_on(uint64_t target_cpu, uint64_t entry_addr,
> uint64_t context_id)
> {
> - register uint64_t x0 asm("x0") = PSCI_0_2_FN64_CPU_ON;
> - register uint64_t x1 asm("x1") = target_cpu;
> - register uint64_t x2 asm("x2") = entry_addr;
> - register uint64_t x3 asm("x3") = context_id;
> + struct arm_smccc_res res;
>
> - asm("hvc #0"
> - : "=r"(x0)
> - : "r"(x0), "r"(x1), "r"(x2), "r"(x3)
> - : "memory");
> + smccc_hvc(PSCI_0_2_FN64_CPU_ON, target_cpu, entry_addr, context_id,
> + 0, 0, 0, 0, &res);
>
> - return x0;
> + return res.a0;
> }
>
> static uint64_t psci_affinity_info(uint64_t target_affinity,
> uint64_t lowest_affinity_level)
> {
> - register uint64_t x0 asm("x0") = PSCI_0_2_FN64_AFFINITY_INFO;
> - register uint64_t x1 asm("x1") = target_affinity;
> - register uint64_t x2 asm("x2") = lowest_affinity_level;
> + struct arm_smccc_res res;
>
> - asm("hvc #0"
> - : "=r"(x0)
> - : "r"(x0), "r"(x1), "r"(x2)
> - : "memory");
> + smccc_hvc(PSCI_0_2_FN64_AFFINITY_INFO, target_affinity, lowest_affinity_level,
> + 0, 0, 0, 0, 0, &res);
>
> - return x0;
> + return res.a0;
> }
>
> static void guest_main(uint64_t target_cpu)
> diff --git a/tools/testing/selftests/kvm/include/aarch64/processor.h b/tools/testing/selftests/kvm/include/aarch64/processor.h
> index c0273aefa63d..e6b7cb65d158 100644
> --- a/tools/testing/selftests/kvm/include/aarch64/processor.h
> +++ b/tools/testing/selftests/kvm/include/aarch64/processor.h
> @@ -132,4 +132,26 @@ void vm_install_sync_handler(struct kvm_vm *vm,
>
> #define isb() asm volatile("isb" : : : "memory")
>
> +/**
> + * struct arm_smccc_res - Result from SMC/HVC call
> + * @a0-a3 result values from registers 0 to 3
> + */
> +struct arm_smccc_res {
> + unsigned long a0;
> + unsigned long a1;
> + unsigned long a2;
> + unsigned long a3;
> +};
> +
> +/**
> + * smccc_hvc - Invoke a SMCCC function using the hvc conduit
> + * @function_id: the SMCCC function to be called
> + * @arg0-arg6: SMCCC function arguments, corresponding to registers x1-x7
> + * @res: pointer to write the return values from registers x0-x3
> + *
> + */
> +void smccc_hvc(uint32_t function_id, uint64_t arg0, uint64_t arg1,
> + uint64_t arg2, uint64_t arg3, uint64_t arg4, uint64_t arg5,
> + uint64_t arg6, struct arm_smccc_res *res);
> +
> #endif /* SELFTEST_KVM_PROCESSOR_H */
> diff --git a/tools/testing/selftests/kvm/lib/aarch64/processor.c b/tools/testing/selftests/kvm/lib/aarch64/processor.c
> index 632b74d6b3ca..f77430e2d688 100644
> --- a/tools/testing/selftests/kvm/lib/aarch64/processor.c
> +++ b/tools/testing/selftests/kvm/lib/aarch64/processor.c
> @@ -426,3 +426,28 @@ void vm_install_exception_handler(struct kvm_vm *vm, int vector,
> assert(vector < VECTOR_NUM);
> handlers->exception_handlers[vector][0] = handler;
> }
> +
> +void smccc_hvc(uint32_t function_id, uint64_t arg0, uint64_t arg1,
> + uint64_t arg2, uint64_t arg3, uint64_t arg4, uint64_t arg5,
> + uint64_t arg6, struct arm_smccc_res *res)
> +{
> + asm volatile("mov w0, %w[function_id]\n"
> + "mov x1, %[arg0]\n"
> + "mov x2, %[arg1]\n"
> + "mov x3, %[arg2]\n"
> + "mov x4, %[arg3]\n"
> + "mov x5, %[arg4]\n"
> + "mov x6, %[arg5]\n"
> + "mov x7, %[arg6]\n"
> + "hvc #0\n"
> + "mov %[res0], x0\n"
> + "mov %[res1], x1\n"
> + "mov %[res2], x2\n"
> + "mov %[res3], x3\n"
> + : [res0] "=r"(res->a0), [res1] "=r"(res->a1),
> + [res2] "=r"(res->a2), [res3] "=r"(res->a3)
> + : [function_id] "r"(function_id), [arg0] "r"(arg0),
> + [arg1] "r"(arg1), [arg2] "r"(arg2), [arg3] "r"(arg3),
> + [arg4] "r"(arg4), [arg5] "r"(arg5), [arg6] "r"(arg6)
> + : "x0", "x1", "x2", "x3", "x4", "x5", "x6", "x7");
> +}
> diff --git a/tools/testing/selftests/kvm/steal_time.c b/tools/testing/selftests/kvm/steal_time.c
> index ecec30865a74..5d52b82226c5 100644
> --- a/tools/testing/selftests/kvm/steal_time.c
> +++ b/tools/testing/selftests/kvm/steal_time.c
> @@ -120,17 +120,10 @@ struct st_time {
>
> static int64_t smccc(uint32_t func, uint32_t arg)
> {
> - unsigned long ret;
> + struct arm_smccc_res res;
>
> - asm volatile(
> - "mov x0, %1\n"
> - "mov x1, %2\n"
> - "hvc #0\n"
> - "mov %0, x0\n"
> - : "=r" (ret) : "r" (func), "r" (arg) :
> - "x0", "x1", "x2", "x3");
> -
> - return ret;
> + smccc_hvc(func, arg, 0, 0, 0, 0, 0, 0, &res);
> + return res.a0;
> }
>
> static void check_status(struct st_time *st)
> --
> 2.33.0.685.g46640cef36-goog
>
Reviewed-by: Andrew Jones <drjones@redhat.com>
next prev parent reply other threads:[~2021-10-05 13:40 UTC|newest]
Thread overview: 44+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-23 19:15 [PATCH v2 00/11] KVM: arm64: Implement PSCI SYSTEM_SUSPEND support Oliver Upton
2021-09-23 19:16 ` [PATCH v2 01/11] KVM: arm64: Drop unused vcpu param to kvm_psci_valid_affinity() Oliver Upton
2021-10-01 3:50 ` Reiji Watanabe
2021-10-05 13:22 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 02/11] KVM: arm64: Clean up SMC64 PSCI filtering for AArch32 guests Oliver Upton
2021-10-01 3:56 ` Reiji Watanabe
2021-10-05 13:23 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 03/11] KVM: arm64: Encapsulate reset request logic in a helper function Oliver Upton
2021-10-01 6:04 ` Reiji Watanabe
2021-10-01 16:10 ` Oliver Upton
2021-10-05 13:33 ` Andrew Jones
2021-10-05 15:05 ` Oliver Upton
2021-10-05 19:01 ` Andrew Jones
2021-10-13 4:48 ` Reiji Watanabe
2021-10-05 13:35 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 04/11] KVM: arm64: Rename the KVM_REQ_SLEEP handler Oliver Upton
2021-10-05 13:34 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 05/11] KVM: arm64: Defer WFI emulation as a requested event Oliver Upton
2021-09-30 10:50 ` Marc Zyngier
2021-09-30 17:09 ` Sean Christopherson
2021-09-30 17:32 ` Oliver Upton
2021-09-30 18:08 ` Sean Christopherson
2021-09-30 21:57 ` Oliver Upton
2021-10-01 13:57 ` Marc Zyngier
2021-09-23 19:16 ` [PATCH v2 06/11] KVM: arm64: Add support for SYSTEM_SUSPEND PSCI call Oliver Upton
2021-09-30 12:29 ` Marc Zyngier
2021-09-30 17:19 ` Sean Christopherson
2021-09-30 17:35 ` Oliver Upton
2021-09-30 17:40 ` Oliver Upton
2021-10-01 14:02 ` Marc Zyngier
2021-10-05 16:02 ` Oliver Upton
2021-09-23 19:16 ` [PATCH v2 07/11] selftests: KVM: Rename psci_cpu_on_test to psci_test Oliver Upton
2021-10-05 13:36 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 08/11] selftests: KVM: Create helper for making SMCCC calls Oliver Upton
2021-10-05 13:39 ` Andrew Jones [this message]
2021-09-23 19:16 ` [PATCH v2 09/11] selftests: KVM: Use KVM_SET_MP_STATE to power off vCPU in psci_test Oliver Upton
2021-09-23 19:16 ` [PATCH v2 10/11] selftests: KVM: Refactor psci_test to make it amenable to new tests Oliver Upton
2021-10-05 13:45 ` Andrew Jones
2021-10-05 14:54 ` Oliver Upton
2021-10-05 19:05 ` Andrew Jones
2021-09-23 19:16 ` [PATCH v2 11/11] selftests: KVM: Test SYSTEM_SUSPEND PSCI call Oliver Upton
2021-10-05 13:49 ` Andrew Jones
2021-10-05 15:07 ` Oliver Upton
2021-09-23 20:15 ` [PATCH v2 00/11] KVM: arm64: Implement PSCI SYSTEM_SUSPEND support Oliver Upton
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=20211005133957.q2a52s5mbthj6b4k@gator.home \
--to=drjones@redhat.com \
--cc=alexandru.elisei@arm.com \
--cc=james.morse@arm.com \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=maz@kernel.org \
--cc=oupton@google.com \
--cc=pshier@google.com \
--cc=rananta@google.com \
--cc=reijiw@google.com \
--cc=ricarkol@google.com \
--cc=suzuki.poulose@arm.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).