[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/1] s390x/css: fix ilen in IO instruction handl
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH 1/1] s390x/css: fix ilen in IO instruction handlers |
Date: |
Tue, 25 Jul 2017 08:52:15 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
* Halil Pasic <address@hidden> [2017-07-24 16:34:52 +0200]:
> When initiating a program check interruption by calling program_interrupt
> the instruction length (ilen) of the current instruction is supplied as
> the third parameter.
>
> On s390x all the IO instructions are of instruction format S and their
> ilen is 4. The calls to program_interrupt (introduced by commits
> 7b18aad543 ("s390: Add channel I/O instructions.", 2013-01-24) and
> 61bf0dcb2e ("s390x/ioinst: Add missing alignment checks for IO
> instructions", 2013-06-21)) however use ilen == 2.
>
> This is probably due to a confusion between ilen which specifies the
> instruction length in bytes and ILC which does the same but in halfwords.
> If kvm_enabled() this does not actually matter, because the ilen
> parameter of program_interrupt is effectively unused.
>
> Let's provide the correct ilen to program_interrupt.
>
> Signed-off-by: Halil Pasic <address@hidden>
> Fixes: 7b18aad543 ("s390: Add channel I/O instructions.")
> Fixes: 61bf0dcb2e ("s390x/ioinst: Add missing alignment checks for IO
> instructions")
> Reviewed-by: David Hildenbrand <address@hidden>
> ---
> target/s390x/ioinst.c | 42 +++++++++++++++++++++---------------------
> 1 file changed, 21 insertions(+), 21 deletions(-)
>
If I didn't misunderstandd, David will take care of the non-I/O cases.
So:
Reviewed-by: Dong Jia Shi <address@hidden>
[...]
--
Dong Jia Shi