On 9/29/21 23:15, Olivier Matz wrote:
Hi Maxime, On Wed, Sep 29, 2021 at 10:17:39PM +0200, Maxime Coquelin wrote:This patch removes the simplification in Virtio descriptors handling, where their buffer addresses are IOVAs for Virtio PCI devices, and VA-only for Virtio-user devices, which added a requirement on Virtio-user that it only supported IOVA as VA. This change introduced a regression for applications using Virtio-user and other physical PMDs that require IOVA as PA because they don't use an IOMMU. This patch reverts to the old behaviour, but needed to be reworked because of the refactoring that happened in v21.02. Fixes: 17043a2909bb ("net/virtio: force IOVA as VA mode for virtio-user") Cc: [email protected] Reported-by: Olivier Matz <[email protected]> Signed-off-by: Maxime Coquelin <[email protected]>Tested-by: Olivier Matz <[email protected]> Many thanks for your quick solution on this!
You're welcome, thanks for reporting. I just notice your reply to v1, so I missed to report your Tested-by on v2 (which only has cosmetic changes). Feel free to add it. Maxime
