[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA
From: |
Dong Jia Shi |
Subject: |
Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA |
Date: |
Thu, 21 Sep 2017 09:10:07 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
* Halil Pasic <address@hidden> [2017-09-20 13:13:01 +0200]:
>
>
> On 09/20/2017 10:33 AM, Cornelia Huck wrote:
> > On Wed, 20 Sep 2017 15:42:38 +0800
> > Dong Jia Shi <address@hidden> wrote:
> >
> >> * Halil Pasic <address@hidden> [2017-09-19 20:27:45 +0200]:
> >>
> >>> Let's add indirect data addressing support for our virtual channel
> >>> subsystem. This implementation does not bother with any kind of
> >>> prefetching. We simply step through the IDAL on demand.
> >>>
> >>> Signed-off-by: Halil Pasic <address@hidden>
> >>> Signed-off-by: Cornelia Huck <address@hidden>
> >>> ---
> >>> hw/s390x/css.c | 117
> >>> ++++++++++++++++++++++++++++++++++++++++++++++++++++++++-
> >>> 1 file changed, 116 insertions(+), 1 deletion(-)
> >>>
> >>> diff --git a/hw/s390x/css.c b/hw/s390x/css.c
> >>> index 2d37a9ddde..a3ce6d89b6 100644
> >>> --- a/hw/s390x/css.c
> >>> +++ b/hw/s390x/css.c
> >>> @@ -827,6 +827,121 @@ incr:
> >>> return 0;
> >>> }
> >>>
> >>> +/* returns values between 1 and bsz, where bsz is a power of 2 */
> >>> +static inline uint16_t ida_continuous_left(hwaddr cda, uint64_t bsz)
> >>> +{
> >>> + return bsz - (cda & (bsz - 1));
> >>> +}
> >>> +
> >>> +static inline uint64_t ccw_ida_block_size(uint8_t flags)
> >>> +{
> >>> + if ((flags & CDS_F_C64) && !(flags & CDS_F_I2K)) {
> >>> + return 1ULL << 12;
> >>> + }
> >>> + return 1ULL << 11;
> >>> +}
> >>> +
> >>> +static inline int ida_read_next_idaw(CcwDataStream *cds, bool ccw_fmt1,
> >>> + bool idaw_fmt_2)
> >>> +{
> >>> + union {uint64_t fmt2; uint32_t fmt1; } idaw;
> >>> + int ret;
> >>> + hwaddr idaw_addr;
> >>> +
> >>> + if (idaw_fmt_2) {
> >>> + idaw_addr = cds->cda_orig + sizeof(idaw.fmt2) * cds->at_idaw;
> >>> + if (idaw_addr & 0x07 && cds_ccw_addrs_ok(idaw_addr, 0,
> >>> ccw_fmt1)) {
> >>> + return -EINVAL; /* channel program check */
> >>> + }
> >>> + ret = address_space_rw(&address_space_memory, idaw_addr,
> >> Ahh, just got one question here:
> >> Do we need to considerate endianess for idaw_addr?
> >
> > That is taken care of below.
> >
> > And the previous version worked on my laptop via tcg ;)
>
> Nod.
My fault!
I was thinking of the idaw_addr itself, not the content of it. Now I
realized that, since we already converted (cds->cda_orig) in
copy_ccw_from_guest(), there is no need to convert (idaw_addr +
idaw_size * idaw_index) anymore.
Please ingnore my noise. ;P
>
> >
> >>
> >>> + MEMTXATTRS_UNSPECIFIED, (void *)
> >>> &idaw.fmt2,
> >>> + sizeof(idaw.fmt2), false);
> >>> + cds->cda = be64_to_cpu(idaw.fmt2);
> >>> + } else {
> >>> + idaw_addr = cds->cda_orig + sizeof(idaw.fmt1) * cds->at_idaw;
> >>> + if (idaw_addr & 0x03 && cds_ccw_addrs_ok(idaw_addr, 0,
> >>> ccw_fmt1)) {
> >>> + return -EINVAL; /* channel program check */
> >>> + }
> >>> + ret = address_space_rw(&address_space_memory, idaw_addr,
> >>> + MEMTXATTRS_UNSPECIFIED, (void *)
> >>> &idaw.fmt1,
> >>> + sizeof(idaw.fmt1), false);
> >>> + cds->cda = be64_to_cpu(idaw.fmt1);
[...]
--
Dong Jia Shi
- [Qemu-devel] [PATCH v3 1/5] s390x/css: introduce css data stream, (continued)
- [Qemu-devel] [PATCH v3 1/5] s390x/css: introduce css data stream, Halil Pasic, 2017/09/19
- [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Halil Pasic, 2017/09/19
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Dong Jia Shi, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Cornelia Huck, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Halil Pasic, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Cornelia Huck, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Halil Pasic, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Dong Jia Shi, 2017/09/20
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA, Cornelia Huck, 2017/09/21
- Re: [Qemu-devel] [PATCH v3 5/5] s390x/css: support ccw IDA,
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