On Tue, Jan 13, 2015 at 05:02:10PM +, Ian Jackson wrote:
> Wei Liu writes ("[PATCH v3 05/19] libxl: add vmemrange to
> libxl__domain_build_state"):
> > A vnode consists of one or more vmemranges (virtual memory range). One
> > example of multiple vmemranges is that there is a hole in one vnode.
>
> I'm finding this series a bit oddly structured. This patch, for
> example, just introduces some new fields to an internal state struct -
> but these fields are not initialised, set, or read.
>
The new fields (and other existing fields) are initialised to zero in
initiate_domain_create, that's why it doesn't need to be explicitly
initialised.
These new fields are accessed in the next patch. I can either explicitly
say so in commit log or squash this patch with the next one. Which way
do you prefer?
TBH I don't think this patch and next one should be squashed into one
patch.
Wei.
> Ian.
___
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel