Re: [PATCH 2/5] vfio/cpr-legacy: drop an erroneous assert

[Steven Sistare]

On 9/25/2025 10:23 PM, Zhenzhong Duan wrote:

vfio_legacy_cpr_dma_map() is not only used in post_load on destination
but also error recovery path on source side. Assert it for destination
is wrong.

Fixes: 7e9f21411302 ("vfio/container: restore DMA vaddr")
Signed-off-by: Zhenzhong Duan <zhenzhong.d...@intel.com>

Reviewed-by: Steve Sistare <steven.sist...@oracle.com>

---
  hw/vfio/cpr-legacy.c | 2 --
  1 file changed, 2 deletions(-)

diff --git a/hw/vfio/cpr-legacy.c b/hw/vfio/cpr-legacy.c
index 3ea24d60de..19fd8b60d3 100644
--- a/hw/vfio/cpr-legacy.c
+++ b/hw/vfio/cpr-legacy.c
@@ -51,8 +51,6 @@ static int vfio_legacy_cpr_dma_map(const VFIOContainerBase 
*bcontainer,
          .size = size,
      };

- g_assert(cpr_is_incoming());

-
      if (ioctl(container->fd, VFIO_IOMMU_MAP_DMA, &map)) {
          return -errno;
      }