[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] libxc: Defer initialization of start_page for HVM guests



On 12/22/2015 09:36 AM, Roger Pau Monné wrote:
El 22/12/15 a les 15.12, Boris Ostrovsky ha escrit:
On 12/22/2015 04:42 AM, Roger Pau Monné wrote:
Also, why is everything done inside of alloc_magic_pages_hvm moved to
start_info_hvm? AFAICT we should only need to move the code that fills
the hvm_start_info struct, but the rest of the code already present in
alloc_magic_pages_hvm could be left as-is.
That's what I was referring to in the commit message (and in the comment
below it): most, if not all, of the stuff that alloc_magic_pages_hvm()
currently does really has nothing to do with magic pages allocation.
E.g. hvm_params settting, special pages initialization, etc. So I
figured I could move all of this to a later point. But, as I said in the
comment, I am not convinced that start_info() is the right place either.
Some of it has to do with magic pages IMHO (although I have to admit
this is probably a question of taste), for example it makes sense from
my PoV to allocate the hvm_info_page, the special pages (xenstore,
console...), the ioreq server pages and possibly the EPT identity map.
Moving the start_info stuff into it's own function also makes sense.

Since there are many ways to separate this I'll wait for maintainers to express their preference.

(I don't know who is around now and I will disappear too on Friday until January so this may have to wait until then).

-boris

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.