On 1/21/26 07:34, Jim Shu wrote:
The return value of API is 0 for success and negative error code for failure. We'll check if the return value equals to 0. Also, the MIG_MODE should be CPR_TRANSFER and CPR_EXEC instead of 2 same bits.The API usage is aligned with 'hw/vfio/cpr-legacy.c' after these 2 changes. Fixes: 3ca0a0ab05c ("migration: Use bitset of MigMode instead of variable arguments") Signed-off-by: Jim Shu <[email protected]> --- hw/vfio/cpr-iommufd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/hw/vfio/cpr-iommufd.c b/hw/vfio/cpr-iommufd.c index c244db88fbb..6758c73b6b9 100644 --- a/hw/vfio/cpr-iommufd.c +++ b/hw/vfio/cpr-iommufd.c @@ -159,8 +159,8 @@ bool vfio_iommufd_cpr_register_iommufd(IOMMUFDBackend *be, Error **errp)if (!vfio_cpr_supported(be, cpr_blocker)) {return migrate_add_blocker_modes(cpr_blocker, - BIT(MIG_MODE_CPR_TRANSFER) | BIT(MIG_MODE_CPR_TRANSFER), - errp); + BIT(MIG_MODE_CPR_TRANSFER) | BIT(MIG_MODE_CPR_EXEC), + errp) == 0; }vmstate_register(NULL, -1, &iommufd_cpr_vmstate, be);
Reviewed-by: Cédric Le Goater <[email protected]> Thanks, C.
