Re: [libvirt] [PATCH 3/8] conf, qemu: Add support for shmem model

On 09/27/2016 08:24 AM, Martin Kletzander wrote: docs/formatdomain.html.in ?? Just so I'm sure I understand ;-)... This is the existing 'model type' for 'shmem' being implemented as the "default" (e.g. 0 entry) because we know at some short time in the future we're going to be adding a new type (or two). Since the <model type='ivshmem'> is now going to be the default on output, we should explain in some way... and encourage choosing something else because "at some point in the future" ;-) we'll deprecate this one (whenever that dire time exists who knows). Making sure #2 - we don't have to care about save files, true? Since the default will be to now to ShmemDefFormat the <model> - a save file read on an older libvirt will have a failure, but that'd be true for any XML change I suppose. I understand it does matter that the model is printed just in case someone decided that's the model they wanted... If we really wanted overkill, some sort of bool could be created to keep track of whether model was read or not and then to not print out if not read in... So I don't forget, once you get to implementing 'plain' and 'doorbell', then perhaps some details from the v2 reply to 6/7 would be helpful to the less than informed user trying to figure out which to use ;-) ACK - in principle your call on the ShmemDefFormat bool or not... John