On 10.03.2025 10:23, Cédric Le Goater wrote:
On 3/10/25 09:17, Avihai Horon wrote:

On 07/03/2025 12:57, Maciej S. Szmigiero wrote:
External email: Use caution opening links or attachments


From: "Maciej S. Szmigiero" <maciej.szmigi...@oracle.com>

Wire data commonly use BE byte order (including in the existing migration
protocol), use it also for for VFIO device state packets.

Nit: should we add a sentence about the motivation? Something like:

This will allow VFIO multifd device state transfer between hosts with different 
endianness.
Although currently there is no such use case, it's good to have it now for 
completeness.

Maciej,

Could you please send a v2 with this change ?  and

I've sent v2 now as a standalone patch.


Fixes: 3228d311ab18 ("vfio/migration: Multifd device state transfer support - 
received buffers queuing")
Fixes: 6d644baef203 ("vfio/migration: Multifd device state transfer support - send 
side")

we don't need these Fixes trailers because the feature is not part of
a released QEMU version yet.

Removed these tags now.

I've originally added them because you said last week it's going to be a 
"Fixes" patch:
https://lore.kernel.org/qemu-devel/7eb4dd80-0ae3-4522-bd1d-b004c19c4...@redhat.com/

Signed-off-by: Maciej S. Szmigiero <maciej.szmigi...@oracle.com>

Reviewed-by: Avihai Horon <avih...@nvidia.com>

Thanks.

Thanks Avihai,

C.

Thanks,
Maciej


Reply via email to