linux-kernel - Re: [PATCH V2 00/19] vDPA driver for virtio-pci device

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5347a78d-fce4-f1dc-b4ed-9740e39ad56f@redhat.com>
Date:   Mon, 4 Jan 2021 14:58:35 +0800
From:   Jason Wang <jasowang@...hat.com>
To:     "Michael S. Tsirkin" <mst@...hat.com>
Cc:     virtualization@...ts.linux-foundation.org,
        linux-kernel@...r.kernel.org, shahafs@...lanox.com
Subject: Re: [PATCH V2 00/19] vDPA driver for virtio-pci device


On 2021/1/3 下午8:37, Michael S. Tsirkin wrote:
> On Thu, Dec 31, 2020 at 11:52:14AM +0800, Jason Wang wrote:
>> On 2020/12/4 下午12:03, Jason Wang wrote:
>>> Hi all:
>>>
>>> This series tries to implement a vDPA driver for virtio-pci device
>>> which will bridge between vDPA bus and virtio-pci device.
>>>
>>> This could be used for future feature prototyping and testing.
>>>
>>> Please review
>>>
>>> Changes from V2:
>>>
>>> - don't try to use devres for virtio-pci core
>>> - tweak the commit log
>>> - split the patches furtherly to ease the reviewing
>>>
>>> Changes from V1:
>>>
>>> - Split common codes from virito-pci and share it with vDPA driver
>>> - Use dynamic id in order to be less confusing with virtio-pci driver
>>> - No feature whitelist, supporting any features (mq, config etc)
>>>
>>> Thanks
>>
>> Michael, any comment for this series?
>>
>> It's needed for testing doorbell mapping and config interrupt support.
>>
>> Thanks
> I saw you got some comments back in december so was expecting another
> version. If you'd rather I reviewed this one, let me know.


Right. So I've posted a new version that tries to address the previous 
comments.

Please review that version.

Thanks


>