On Thu, Sep 21, 2023 at 04:16:25PM -0400, Michael S. Tsirkin wrote: > On Thu, Sep 21, 2023 at 04:53:45PM -0300, Jason Gunthorpe wrote: > > On Thu, Sep 21, 2023 at 03:34:03PM -0400, Michael S. Tsirkin wrote: > > > > > that's easy/practical. If instead VDPA gives the same speed with just > > > shadow vq then keeping this hack in vfio seems like less of a problem. > > > Finally if VDPA is faster then maybe you will reconsider using it ;) > > > > It is not all about the speed. > > > > VDPA presents another large and complex software stack in the > > hypervisor that can be eliminated by simply using VFIO. > > If all you want is passing through your card to guest > then yes this can be addressed "by simply using VFIO". That is pretty much the goal, yes. > And let me give you a simple example just from this patchset: > it assumes guest uses MSIX and just breaks if it doesn't. It does? Really? Where did you see that? > > VFIO is > > already required for other scenarios. > > Required ... by some people? Most VMs I run don't use anything > outside of virtio. Yes, some people. The sorts of people who run large data centers. > It seems to deal with emulating virtio which seems more like a vdpa > thing. Alex described it right, it creates an SW trapped IO bar that relays the doorbell to an admin queue command. > If you start adding virtio emulation to vfio then won't > you just end up with another vdpa? And if no why not? > And I don't buy the "we already invested in this vfio based solution", > sorry - that's not a reason upstream has to maintain it. I think you would be well justified to object to actual mediation, like processing queues in VFIO or otherwise complex things. Fortunately there is no need to do that with DPU HW. The legacy IO BAR is a weird quirk that just cannot be done without a software trap, and the OASIS standardization effort was for exactly this kind of simplistic transformation. I also don't buy the "upstream has to maintain it" line. The team that submitted it will maintain it just fine, thank you. Jason
