Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Dong Jia Shi
* Halil Pasic [2017-09-20 13:02:59 +0200]: > > > On 09/20/2017 10:25 AM, Cornelia Huck wrote: > > On Wed, 20 Sep 2017 15:47:51 +0800 > > Dong Jia Shi wrote: > > > >> * Halil Pasic [2017-09-19 20:27:44 +0200]: > > > >>> @@ -828,7 +836,9 @@ void ccw_dstream_init(CcwDataStream *cds, CCW1 const

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Halil Pasic
On 09/20/2017 10:25 AM, Cornelia Huck wrote: > On Wed, 20 Sep 2017 15:47:51 +0800 > Dong Jia Shi wrote: > >> * Halil Pasic [2017-09-19 20:27:44 +0200]: > >>> @@ -828,7 +836,9 @@ void ccw_dstream_init(CcwDataStream *cds, CCW1 const >>> *ccw, ORB const *orb) >>> g_assert(!(orb->ctrl1 & OR

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Cornelia Huck
On Wed, 20 Sep 2017 13:34:21 +0200 Halil Pasic wrote: > On 09/20/2017 10:06 AM, Cornelia Huck wrote: > > On Tue, 19 Sep 2017 20:27:44 +0200 > > Halil Pasic wrote: > > > >> The architecture mandates the addresses to be accessed on the first > >> indirection level (that is, the data addresses

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Halil Pasic
On 09/20/2017 10:06 AM, Cornelia Huck wrote: > On Tue, 19 Sep 2017 20:27:44 +0200 > Halil Pasic wrote: > >> The architecture mandates the addresses to be accessed on the first >> indirection level (that is, the data addresses without IDA, and the >> (M)IDAW addresses with (M)IDA) to be checked

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Cornelia Huck
On Wed, 20 Sep 2017 15:47:51 +0800 Dong Jia Shi wrote: > * Halil Pasic [2017-09-19 20:27:44 +0200]: > > @@ -828,7 +836,9 @@ void ccw_dstream_init(CcwDataStream *cds, CCW1 const > > *ccw, ORB const *orb) > > g_assert(!(orb->ctrl1 & ORB_CTRL1_MASK_MIDAW)); > > cds->flags = (orb->ctrl0

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Cornelia Huck
On Tue, 19 Sep 2017 20:27:44 +0200 Halil Pasic wrote: > The architecture mandates the addresses to be accessed on the first > indirection level (that is, the data addresses without IDA, and the > (M)IDAW addresses with (M)IDA) to be checked against an CCW format > dependent limit maximum address

Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-20 Thread Dong Jia Shi
* Halil Pasic [2017-09-19 20:27:44 +0200]: > The architecture mandates the addresses to be accessed on the first > indirection level (that is, the data addresses without IDA, and the > (M)IDAW addresses with (M)IDA) to be checked against an CCW format > dependent limit maximum address. If a vio

[Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking

2017-09-19 Thread Halil Pasic
The architecture mandates the addresses to be accessed on the first indirection level (that is, the data addresses without IDA, and the (M)IDAW addresses with (M)IDA) to be checked against an CCW format dependent limit maximum address. If a violation is detected, the storage access is not to be p