[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 01/10] s390/ioinst: fix IO_INT_WORD_ISC macro
From: |
Christian Borntraeger |
Subject: |
Re: [Qemu-devel] [PATCH 01/10] s390/ioinst: fix IO_INT_WORD_ISC macro |
Date: |
Mon, 15 Jun 2015 20:40:27 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.7.0 |
Am 15.06.2015 um 17:57 schrieb Aurelien Jarno:
> The I/O-Interruption Subclass field corresponds to bits 2 to 5 (BE
> notation) of the Interruption-Identification Word. The value should
> be shift by 27 instead of 24.
>
> Cc: Christian Borntraeger <address@hidden>
> Cc: Cornelia Huck <address@hidden>
> Cc: Alexander Graf <address@hidden>
> Signed-off-by: Aurelien Jarno <address@hidden>
> ---
> target-s390x/ioinst.h | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/target-s390x/ioinst.h b/target-s390x/ioinst.h
> index 203bdba..013cc91 100644
> --- a/target-s390x/ioinst.h
> +++ b/target-s390x/ioinst.h
> @@ -220,7 +220,7 @@ typedef struct IOIntCode {
> #define IOINST_SCHID_SSID(_schid) ((_schid & 0x00060000) >> 17)
> #define IOINST_SCHID_NR(_schid) (_schid & 0x0000ffff)
>
> -#define IO_INT_WORD_ISC(_int_word) ((_int_word & 0x38000000) >> 24)
> +#define IO_INT_WORD_ISC(_int_word) ((_int_word & 0x38000000) >> 27)
> #define ISC_TO_ISC_BITS(_isc) ((0x80 >> _isc) << 24)
>
> #define IO_INT_WORD_AI 0x80000000
>
Its Connys area of competence, but your change matches the KVM
implementation in the kernel
static u64 int_word_to_isc_bits(u32 int_word)
{
u8 isc = (int_word & 0x38000000) >> 27;
return (0x80 >> isc) << 24;
}
so
Reviewed-by: Christian Borntraeger <address@hidden>
- [Qemu-devel] [PATCH 00/10] target-s390x: add support for CCW in TCG mode, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 02/10] s390/ioinst: fix endianness in ioinst_schib_valid, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 07/10] target-s390x: fix s390_cpu_initial_reset, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 06/10] target-s390x: initialize I/O interrupt queue, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 05/10] target-s390x: correctly initialize ext interrupt queue, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 01/10] s390/ioinst: fix IO_INT_WORD_ISC macro, Aurelien Jarno, 2015/06/15
- Re: [Qemu-devel] [PATCH 01/10] s390/ioinst: fix IO_INT_WORD_ISC macro,
Christian Borntraeger <=
- [Qemu-devel] [PATCH 08/10] target-s390x: wire up DIAG IPL in TCG mode, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 03/10] virtio-ccw: disable ioevent bit when ioeventfds are not enabled, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 10/10] target-s390x: wire up I/O instructions in TCG mode, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 04/10] target-s390x: fix setcc in TCG mode, Aurelien Jarno, 2015/06/15
- [Qemu-devel] [PATCH 09/10] target-s390x: wire up DIAG REIPL in TCG mode, Aurelien Jarno, 2015/06/15
- Re: [Qemu-devel] [PATCH 00/10] target-s390x: add support for CCW in TCG mode, Alexander Graf, 2015/06/16