Using 'strcmp(target_name(), "aarch64")' is quite unique in the QEMU code base, and to be honest, I’m not too keen on adding it unless there’s really no other option.The previous versions simply tested the TARGET_ARM macro but commit 5731baee6c3c ("hw/vfio: Compile some common objects once") made the migration-multifd.c file target-independent so it cannot use target-specific macros now. Another option would be to move vfio_load_config_after_iter() to helpers.c since that file is target-dependent and can simply test TARGET_ARM macro (#if defined(TARGET_ARM)) instead of doing strcmp(target_name(), "aarch64") which I agree looks weird.
This would look better than testing target_name(). Let's do that. Thanks, C.
