[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] s390x/misc_helper.c: wrap s390_virtio_hypercall
From: |
Alexander Graf |
Subject: |
Re: [Qemu-devel] [PATCH] s390x/misc_helper.c: wrap s390_virtio_hypercall in BQL |
Date: |
Sun, 23 Apr 2017 19:19:02 +0200 |
> Am 23.04.2017 um 18:08 schrieb Aurelien Jarno <address@hidden>:
>
> s390_virtio_hypercall can trigger IO events and interrupts, most notably
> when using virtio-ccw devices.
>
> Signed-off-by: Aurelien Jarno <address@hidden>
> ---
> roms/qemu-palcode | 2 +-
> roms/seabios | 2 +-
> target/s390x/misc_helper.c | 2 ++
> 3 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/roms/qemu-palcode b/roms/qemu-palcode
> index f3c7e44c70..c87a92639b 160000
> --- a/roms/qemu-palcode
> +++ b/roms/qemu-palcode
> @@ -1 +1 @@
> -Subproject commit f3c7e44c70254975df2a00af39701eafbac4d471
> +Subproject commit c87a92639b28ac42bc8f6c67443543b405dc479b
> diff --git a/roms/seabios b/roms/seabios
> index 5f4c7b13cd..e2fc41e24e 160000
> --- a/roms/seabios
> +++ b/roms/seabios
> @@ -1 +1 @@
> -Subproject commit 5f4c7b13cdf9c450eb55645f4362ea58fa61b79b
> +Subproject commit e2fc41e24ee0ada60fc511d60b15a41b294538be
I guess you didn't mean to send those hunks?
> diff --git a/target/s390x/misc_helper.c b/target/s390x/misc_helper.c
> index 4946b56ab3..aec737d707 100644
> --- a/target/s390x/misc_helper.c
> +++ b/target/s390x/misc_helper.c
> @@ -307,7 +307,9 @@ void HELPER(diag)(CPUS390XState *env, uint32_t r1,
> uint32_t r3, uint32_t num)
> switch (num) {
> case 0x500:
> /* KVM hypercall */
> + qemu_mutex_lock_iothread();
> r = s390_virtio_hypercall(env);
> + qemu_mutex_unlock_iothread();
That change however looks good to me. So without the subprj commits,
Reviewed-by: Alexander Graf <address@hidden>
Alex
> break;
> case 0x44:
> /* yield */
> --
> 2.11.0
>