|
From: | Paolo Bonzini |
Subject: | Re: [Qemu-devel] [PULL 2/8] i.MX: Implement a more complete version of the GPT timer. |
Date: | Wed, 26 Jun 2013 09:11:03 +0200 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130514 Thunderbird/17.0.6 |
Il 26/06/2013 09:08, address@hidden ha scritto: > For my own education, could you elaborate on the reason this is not a > good change? Quoting from http://article.gmane.org/gmane.comp.emulators.qemu/218078: > The MemoryRegion name is free text that can be changed as desired and > doesn't need to match the type name. > > The TYPE_* constant serves to keep consistency between usages of the > string literal but does not prohibit changing its value. > > The VMStateDescription should thus not use the TYPE_* constant because > it must not ever change for live migration. Renaming a type is a valid > thing to do if there is no command line compatibility to keep (think > board-level "xlnx.foo" -> "xlnx,foo"). Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |