Re: [PATCH V3 16/19] virtio-pci: introduce modern device module

From: Michael S. Tsirkin
Date: Mon Feb 08 2021 - 07:24:51 EST


On Mon, Feb 08, 2021 at 01:42:27PM +0800, Jason Wang wrote:
>
> On 2021/2/5 下午11:34, Michael S. Tsirkin wrote:
> > On Mon, Jan 04, 2021 at 02:55:00PM +0800, Jason Wang wrote:
> > > Signed-off-by: Jason Wang<jasowang@xxxxxxxxxx>
> > I don't exactly get why we need to split the modern driver out,
> > and it can confuse people who are used to be seeing virtio-pci.
>
>
> The virtio-pci module still there. No user visible changes. Just some codes
> that could be shared with other driver were split out.
>

What I am saying is this: we can have virtio-vdpa depend on
virtio-pci without splitting the common code out to an
extra module.

> >
> > The vdpa thing so far looks like a development tool, why do
> > we care that it depends on a bit of extra code?
>
>
> If I'm not misunderstanding, trying to share codes is proposed by you here:
>
> https://lkml.org/lkml/2020/6/10/232
>
> We also had the plan to convert IFCVF to use this library.
>
> Thanks

If that happens then an extra module might become useful.

--
MST