mbox series

[0/2] vhost-vDPA: doorbell mapping support

Message ID 20210415080444.44645-1-jasowang@redhat.com
Headers show
Series vhost-vDPA: doorbell mapping support | expand

Message

Jason Wang April 15, 2021, 8:04 a.m. UTC 
Hi All:

This series tries to implement doorbell mapping support for
vhost-vDPA. Tested with virtio-pci vDPA driver.

Please review.

Jason Wang (2):
  vhost-vdpa: skip ram device from the IOTLB mapping
  vhost-vdpa: doorbell mapping support

 hw/virtio/vhost-vdpa.c         | 97 ++++++++++++++++++++++++++++++----
 include/hw/virtio/vhost-vdpa.h |  7 +++
 2 files changed, 94 insertions(+), 10 deletions(-)

Comments

Michael S. Tsirkin May 4, 2021, 8:38 a.m. UTC | #1 
On Thu, Apr 15, 2021 at 04:04:42PM +0800, Jason Wang wrote:
> Hi All:
> 
> This series tries to implement doorbell mapping support for
> vhost-vDPA. Tested with virtio-pci vDPA driver.

BTW I'd rather we used the kick/call terminology from the virtio
spec. doorbell terminology seems to be originating from e1000
which has a register named like this.

> Please review.
> 
> Jason Wang (2):
>   vhost-vdpa: skip ram device from the IOTLB mapping
>   vhost-vdpa: doorbell mapping support
> 
>  hw/virtio/vhost-vdpa.c         | 97 ++++++++++++++++++++++++++++++----
>  include/hw/virtio/vhost-vdpa.h |  7 +++
>  2 files changed, 94 insertions(+), 10 deletions(-)
> 
> -- 
> 2.25.1
Jason Wang May 6, 2021, 7:20 a.m. UTC | #2 
在 2021/5/4 下午4:38, Michael S. Tsirkin 写道:
> On Thu, Apr 15, 2021 at 04:04:42PM +0800, Jason Wang wrote:
>> Hi All:
>>
>> This series tries to implement doorbell mapping support for
>> vhost-vDPA. Tested with virtio-pci vDPA driver.
> BTW I'd rather we used the kick/call terminology from the virtio
> spec. doorbell terminology seems to be originating from e1000
> which has a register named like this.


Ok, will rename in next version.

Thanks


>
>> Please review.
>>
>> Jason Wang (2):
>>    vhost-vdpa: skip ram device from the IOTLB mapping
>>    vhost-vdpa: doorbell mapping support
>>
>>   hw/virtio/vhost-vdpa.c         | 97 ++++++++++++++++++++++++++++++----
>>   include/hw/virtio/vhost-vdpa.h |  7 +++
>>   2 files changed, 94 insertions(+), 10 deletions(-)
>>
>> -- 
>> 2.25.1
>