RE: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Liang, Cunming
ongnu.org; virtio-dev@lists.oasis-open.org; Daly, Dan > ; Tan, Jianfeng ; Wang, Zhihong > ; Wang, Xiao W > Subject: Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering > external host notifiers > > On Thu, Apr 19, 2018 at 04:24:29PM +, Liang, Cunming wrote: > > &g

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
On Thu, Apr 19, 2018 at 07:35:57PM +0200, Paolo Bonzini wrote: > On 19/04/2018 19:27, Michael S. Tsirkin wrote: > > > > That CONFIG_SMP here is clearly wrong but I don't really know what > > to set it to. Also, we probably should switch virtio_wmb to dma_XX > > barriers. > > > > That's actually e

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Paolo Bonzini
On 19/04/2018 19:27, Michael S. Tsirkin wrote: > > That CONFIG_SMP here is clearly wrong but I don't really know what > to set it to. Also, we probably should switch virtio_wmb to dma_XX > barriers. > > That's actually easy. Will try to do. Should it be dma_wmb() before updating the indices, and

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
On Thu, Apr 19, 2018 at 06:59:39PM +0200, Paolo Bonzini wrote: > On 19/04/2018 18:52, Liang, Cunming wrote: > >>> Oh you are right. > >>> > >>> So it's only needed for non-intel platforms or when packets are > >>> in WC memory then. And I don't know whether dpdk ever puts > >>> packets in WC memor

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Paolo Bonzini
On 19/04/2018 18:52, Liang, Cunming wrote: >>> Oh you are right. >>> >>> So it's only needed for non-intel platforms or when packets are >>> in WC memory then. And I don't know whether dpdk ever puts >>> packets in WC memory. >>> >>> I guess we'll cross this bridge when we get to it. >> Non-TSO a

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
Tiwei > > ; > > jasow...@redhat.com; alex.william...@redhat.com; stefa...@redhat.com; > > qemu-de...@nongnu.org; virtio-dev@lists.oasis-open.org; Daly, Dan > > ; Tan, Jianfeng ; Wang, Zhihong > > ; Wang, Xiao W > > Subject: Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user:

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
On Thu, Apr 19, 2018 at 06:07:07PM +0200, Paolo Bonzini wrote: > On 19/04/2018 17:59, Michael S. Tsirkin wrote: > > On Thu, Apr 19, 2018 at 05:51:51PM +0200, Paolo Bonzini wrote: > >> On 19/04/2018 17:19, Michael S. Tsirkin wrote: > - if we make it 1 when weak barriers are needed, the device a

RE: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Liang, Cunming
.@nongnu.org; virtio-dev@lists.oasis-open.org; Daly, Dan > ; Tan, Jianfeng ; Wang, Zhihong > ; Wang, Xiao W > Subject: Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering > external host notifiers > > On Thu, Apr 19, 2018 at 03:02:40PM +0200, Paolo Bonzini wrote: >

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Paolo Bonzini
On 19/04/2018 17:59, Michael S. Tsirkin wrote: > On Thu, Apr 19, 2018 at 05:51:51PM +0200, Paolo Bonzini wrote: >> On 19/04/2018 17:19, Michael S. Tsirkin wrote: - if we make it 1 when weak barriers are needed, the device also needs to nack feature negotiation (not allow setting the FEATU

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
On Thu, Apr 19, 2018 at 05:51:51PM +0200, Paolo Bonzini wrote: > On 19/04/2018 17:19, Michael S. Tsirkin wrote: > >> - if we make it 1 when weak barriers are needed, the device also needs > >> to nack feature negotiation (not allow setting the FEATURES_OK) if the > >> bit is not set by the driver.

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Paolo Bonzini
On 19/04/2018 17:19, Michael S. Tsirkin wrote: >> - if we make it 1 when weak barriers are needed, the device also needs >> to nack feature negotiation (not allow setting the FEATURES_OK) if the >> bit is not set by the driver. >> However, that is not enough. Live >> migration assumes that it is

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Michael S. Tsirkin
On Thu, Apr 19, 2018 at 03:02:40PM +0200, Paolo Bonzini wrote: > On 19/04/2018 14:43, Liang, Cunming wrote: > >> 2. Memory barriers. Right now after updating the avail idx, > >> virtio does smp_wmb() and then the MMIO write. Normal hardware > >> drivers do wmb() which is an sfence. Can a PCI devic

Re: [virtio-dev] RE: [PATCH v3 6/6] vhost-user: support registering external host notifiers

2018-04-19 Thread Paolo Bonzini
On 19/04/2018 14:43, Liang, Cunming wrote: >> 2. Memory barriers. Right now after updating the avail idx, >> virtio does smp_wmb() and then the MMIO write. Normal hardware >> drivers do wmb() which is an sfence. Can a PCI device read bypass >> index write and see a stale index value? > > A compile