Re: [PATCH] tests: Wait for migration completion on destination QEMU to avoid failures

[Stefan Berger <stefanb@linux.ibm.com>]

On 10/18/24 12:23 PM, Michael Tokarev wrote:
> 16.10.2024 18:21, Stefan Berger wrote:
>> Rather than waiting for the completion of migration on the source side,
>> wait for it on the destination QEMU side to avoid accessing the TPM TIS
>> memory mapped registers before QEMU could restore their state. This
>> error condition could be triggered on busy systems where the destination
>> QEMU did not have enough time to restore the TIS state while the test 
>> case
>> was already reading its registers. The test case was for example reading
>> the STS register and received an unexpected value (0xffffffff), which
>> lead to a segmentation fault later on due to trying to read 0xffff bytes
>> from the TIS into a buffer.
>>
>> Cc: qemu-stable@nongnu.org
>> Reported-by: Fabiano Rosas <farosas@suse.de>
>> Signed-off-by: Stefan Berger <stefanb@linux.ibm.com>
> 
> Reviewed-by: Michael Tokarev <mjt@tls.msk.ru>
> 
> Queued for qemu-stable, though this one might better be applied through
> the test tree.

Thanks.  I actually already sent it as part of my PR today...

> 
> Thanks,
> 
> /mjt