[libvirt] [PATCH v2 17/21] conf: Add target type and model for pl011

[Pavel Hrdina]

On Tue, Nov 21, 2017 at 05:42:27PM +0100, Andrea Bolognani wrote:
> We can finally introduce a specific target model for the pl011 device
> used by mach-virt guests, which means isa-serial will no longer show
> up to confuse users.
> 
> We make sure migration works in both directions by interpreting the
> isa-serial target type, or the lack of target type, appropriately
> when parsing the guest XML, and skipping the newly-introduced type
> when formatting if for migration. We also verify that pl011 is not
> used for non-mach-virt guests and add a bunch of test cases.
> 
> Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=151292
 
I'm not sure that 'system' is a good name for serial type for mach-virt,
it kind of feels like too generic name.  Can we use the "apb-serial" for
mach-virt?  I understand that the "apb" name might not be well known but
at least people that work closely with ARM machines have a chance to
understand it better than "system" name.  Either way, we should document
it better than just listing it as a valid value for type, at least
mention that it's valid for ARM/mach-virt machine and that it's a system
bus or something like that.  The standalone "system" doesn't make it
clear what it actually is.

Otherwise the patch looks good.

Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20171123/b53671b7/attachment-0001.sig>