diff options
| author | Zhenzhong Duan <zhenzhong.duan@intel.com> | 2023-07-03 15:15:09 +0800 |
|---|---|---|
| committer | Cédric Le Goater <clg@redhat.com> | 2023-07-10 09:52:52 +0200 |
| commit | 0520d63c7701b76560eca7d4a484f3e3f57ccb00 (patch) | |
| tree | 472d119cad45924dae72a51596888b5d7e22d4ae | |
| parent | 2b43b2995b02e73b70d10b5be2f2b29b68c7e2fa (diff) | |
vfio/migration: Remove print of "Migration disabled"
Property enable_migration supports [on/off/auto].
In ON mode, error pointer is passed to errp and logged.
In OFF mode, we doesn't need to log "Migration disabled" as it's intentional.
In AUTO mode, we should only ever see errors or warnings if the device
supports migration and an error or incompatibility occurs while further
probing or configuring it. Lack of support for migration shoundn't
generate an error or warning.
Signed-off-by: Zhenzhong Duan <zhenzhong.duan@intel.com>
Reviewed-by: Joao Martins <joao.m.martins@oracle.com>
Reviewed-by: Cédric Le Goater <clg@redhat.com>
Signed-off-by: Cédric Le Goater <clg@redhat.com>
| -rw-r--r-- | hw/vfio/pci.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/hw/vfio/pci.c b/hw/vfio/pci.c index 9154dd929d..eefd4ec330 100644 --- a/hw/vfio/pci.c +++ b/hw/vfio/pci.c @@ -3209,7 +3209,6 @@ static void vfio_realize(PCIDevice *pdev, Error **errp) if (!pdev->failover_pair_id) { ret = vfio_migration_realize(vbasedev, errp); if (ret) { - error_report("%s: Migration disabled", vbasedev->name); goto out_deregister; } } |