[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking |
Date: |
Thu, 21 Sep 2017 08:39:09 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
* Halil Pasic <address@hidden> [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 <address@hidden> wrote:
> >
> >> * Halil Pasic <address@hidden> [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 & ORB_CTRL0_MASK_I2K ? CDS_F_I2K : 0) |
> >>> (orb->ctrl0 & ORB_CTRL0_MASK_C64 ? CDS_F_C64 : 0) |
> >>> + (orb->ctrl0 & ORB_CTRL0_MASK_FMT ? CDS_F_FMT : 0) |
> >> This reminds me one more question:
> >> Calling ccw_dsteram_init() after copy_ccw_from_guest() may lead to a
> >> fmt-1 @ccw with an @orb that designates fmt-0 ccw. This sounds insane.
> >
> > That's just a consequence of us translating everything to format-1
> > ccws. A bit confusing, but no problem if we pay attention to the format
> > bit everywhere it makes a difference.
> >
>
> Agree.
Ok. I'm fine with this.
>
> Halil
--
Dong Jia Shi
Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Cornelia Huck, 2017/09/20
[Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking, Halil Pasic, 2017/09/19
Re: [Qemu-devel] [PATCH v3 4/5] 390x/css: introduce maximum data address checking, Cornelia Huck, 2017/09/20
[Qemu-devel] [PATCH v3 3/5] virtio-ccw: use ccw data stream, Halil Pasic, 2017/09/19
Re: [Qemu-devel] [PATCH v3 3/5] virtio-ccw: use ccw data stream, Pierre Morel, 2017/09/21