Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class

[Ming Lei]

On Tue, Jun 17, 2014 at 4:46 PM, Cornelia Huck <address@hidden> wrote:
>>
>> Looks like s390 virtio-blk never enables the two common features, is
>> there any reason the two features can't be supported by s390?
>
> Indirect descriptors are fine. event_idx will not work IIUC because we
> always need to do a sync before we see changes, and this needs an
> interrupt to trigger.

Sounds like the old s390 isn't cache coherent? Because you mean
write in one side can only be observed from another side with an
explicit notification or interrupt.

On arm/arm64, we didn't see any problem with event_idx.

>
> The correct way, of course, is to unset VIRTIO_RING_F_EVENT_IDX in the
> guest driver, and I verified that this works - this does not help with
> old guests, however.

Both sides should be ok since it is decided by negotiation.

>
> Note that this applies only to the old s390-virtio transport -
> virtio-ccw is fine.

Could you share me how s390 virtio is supported in kernel since
I only see virtio-mmio and virtio-pci support under drivers/virtio/ of
linux kernel?

Thanks,
--
Ming Lei