Re: [PATCHv2] virtio: Introduce vringh wrappers in virtio_config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Sjur,

Sorry for the delay - just got back from you know where :)

On Wed, Mar 13, 2013 at 12:12 PM, Sjur Brændeland <sjurbren@xxxxxxxxx> wrote:
>> From: Sjur Brændeland <sjur.brandeland@xxxxxxxxxxxxxx>
>>
>> Add wrappers for the host vrings to support loose
>> coupling between the virtio device and driver.
>>
>> A new struct vringh_config_ops with the functions
>> find_vrhs() and del_vrhs() is added to the virtio_device
>> struct. This enables virtio drivers to manage virtio
>> host rings without detailed knowledge of how the
>> vrings are created and deleted.
>>
>> The function vringh_notify() is added so vringh clients
>> can notify the other side that buffers are added to the
>> used-ring.
>>
>> Cc: Ohad Ben-Cohen <ohad@xxxxxxxxxx>
>> Cc: Rusty Russell <rusty@xxxxxxxxxxxxxxx>
>> Signed-off-by: Sjur Brændeland <sjur.brandeland@xxxxxxxxxxxxxx>
>
> As you have been pushing for this wrapper, could you please
> add your Ack or Reviewed tag if you are happy with this patch?

I am happy with this patch: the caif virtio driver is now decoupled
from remoteproc which I think is great. Thanks for doing it!

To provide a Reviewed-by tag though I need to review it from a virtio
point of view, which I can try to do after I'll finish going through
my backlog of pending patches. Let me finish that pile of patches I
have, and if you or Rusty will still want me to provide a Reviewed-by
tag here I'll do it.

Thanks,
Ohad.
_______________________________________________
Virtualization mailing list
Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx
https://lists.linuxfoundation.org/mailman/listinfo/virtualization



[Index of Archives]     [KVM Development]     [Libvirt Development]     [Libvirt Users]     [CentOS Virtualization]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite Forum]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux