On 10/5/23 16:16, Laszlo Ersek wrote: > On 10/5/23 14:01, Cédric Le Goater wrote: >> On 10/5/23 13:30, marcandre.lur...@redhat.com wrote: >>> From: Marc-André Lureau <marcandre.lur...@redhat.com> >>> >>> Hi, >>> >>> Implement RAMFB migration, and add properties to enable it only on >= 8.2 >>> machines, + a few related cleanups. >>> >>> Cedric, did you get the chance to test the VFIO display/ramfb code? >> >> Nope. I was busy with VFIO stuff. I haven't even read Laszlo's >> email yet. I will try this or next week. >> >> That said, could we avoid adding another migration property in >> VFIOPCIDevice and use the available "enable-migration" ? > > I'm not entirely sure, but I suspect we can't / shouldn't do that. > "x-ramfb-migrate" is effectively a machine type compat prop, so if it > doesn't *precisely* line up with enable-migration (i.e., if they aren't > equivalent), then we shouldn't merge them. AFAICT, a 8.1 machine type > may have "enable-migration" set, but it should still have > "x-ramfb-migrate" clear.
or more precisely, not clear, but "auto". Laszlo