On 3/7/2023 6:32 AM, Dr. David Alan Gilbert wrote:
> * Steve Sistare ([email protected]) wrote:
>> Include CONFIG_DEVICES so that populate_vfio_info is instantiated for
>> CONFIG_VFIO. Without it, the 'info migrate' command never returns
>> info about vfio.
>>
>> Fixes: 43bd0bf30f ("migration: Move populate_vfio_info() into a separate
>> file")
>> Signed-off-by: Steve Sistare <[email protected]>
>> Reviewed-by: Marc-André Lureau <[email protected]>
>> ---
>> migration/target.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/migration/target.c b/migration/target.c
>> index 907ebf0..00ca007 100644
>> --- a/migration/target.c
>> +++ b/migration/target.c
>> @@ -8,6 +8,7 @@
>> #include "qemu/osdep.h"
>> #include "qapi/qapi-types-migration.h"
>> #include "migration.h"
>> +#include CONFIG_DEVICES
>>
>> #ifdef CONFIG_VFIO
>> #include "hw/vfio/vfio-common.h"
>> @@ -17,7 +18,6 @@ void populate_vfio_info(MigrationInfo *info)
>> {
>> #ifdef CONFIG_VFIO
>> if (vfio_mig_active()) {
>> - info->has_vfio = true;
>
> Why this removal?
The has_vfio member was removed by:
720a252c2651 ("qapi migration: Elide redundant has_FOO in generated C")
The stale reference was not noticed at the time because the ifdef was never
true.
- Steve
