Re: [PATCH v5 28/36] vfio/migration: Multifd device state transfer support - config loading support

["Cédric Le Goater" <clg@redhat.com>]

>>> @@ -728,6 +728,12 @@ static int vfio_load_state(QEMUFile *f, void *opaque, int version_id)
>>>           switch (data) {
>>>           case VFIO_MIG_FLAG_DEV_CONFIG_STATE:
>>>           {
>>> +            if (vfio_multifd_transfer_enabled(vbasedev)) {
>>> +                error_report("%s: got DEV_CONFIG_STATE but doing multifd transfer",
>>> +                             vbasedev->name);
>>
>> To make clearer, maybe change to:
>> "%s: got DEV_CONFIG_STATE in main migration channel but doing multifd transfer"
> 
> That normally would be good idea, however we are already at 83 characters in this
> line here and will not fit that many more words to this string.

The 80 characters "rule" is not strict. A clear error report is
always good to have !

Thanks,

C.