[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio propert
From: |
Ming Lei |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class |
Date: |
Tue, 17 Jun 2014 18:40:30 +0800 |
On Tue, Jun 17, 2014 at 6:25 PM, Cornelia Huck <address@hidden> wrote:
> On Tue, 17 Jun 2014 18:15:39 +0800
> Ming Lei <address@hidden> wrote:
>
>> 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.
>
> But you probably have the queues in guest memory, as on other
> transports (including virtio-ccw)? The old s390-virtio transport keeps
> the devices and their virtqueues in a memory area behind the guest
> memory - the guest does not see that memory directly, but a sync has to
> be performed to see virtqueue movement (see s390_virtio_device_sync()).
OK, it looks like a real physical device, :-)
I will keep s390-virtio as it is, thanks for your explanation.
Thanks,
--
Ming Lei
- [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, (continued)
- [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/16
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/16
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/16
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/16
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class,
Ming Lei <=
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Ming Lei, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Alexander Graf, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Cornelia Huck, 2014/06/17
- Re: [Qemu-devel] [PATCH 2/4] s390-virtio-bus: move common virtio properties to virtio s390 device class, Alexander Graf, 2014/06/17
[Qemu-devel] [PATCH 3/4] s390 virtio-ccw: move common virtio properties to virtio ccw device class, Ming Lei, 2014/06/16
[Qemu-devel] [PATCH 4/4] virtio-blk: remove DEFINE_VIRTIO_BLK_FEATURES, Ming Lei, 2014/06/16
Re: [Qemu-devel] [PATCH 0/4] virtio: move common virtio properties to bus class device, Paolo Bonzini, 2014/06/16