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

Re: [Xen-devel] [PATCH v3 05/13] x86/altp2m: basic data structures and support routines.

On 07/14/2015 01:01 AM, Sahita, Ravi wrote:
>> -----Original Message-----
>> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
>> Sent: Monday, July 13, 2015 1:01 AM
>>>>> On 10.07.15 at 23:48, <ravi.sahita@xxxxxxxxx> wrote:
>>>> From: Jan Beulich [mailto:JBeulich@xxxxxxxx]
>>>> Sent: Thursday, July 09, 2015 6:30 AM
>>>>>>> On 01.07.15 at 20:09, <edmund.h.white@xxxxxxxxx> wrote:
>>>>> ---
>>>>>  xen/arch/x86/hvm/Makefile        |  1 +
>>>>>  xen/arch/x86/hvm/altp2m.c        | 92
>>>> +++++++++++++++++++++++++++++++++++++
>>>> Wouldn't this better go into xen/arch/x86/mm/?
>>> In this case we followed the pattern of nestedhvm - hope that's ok.
>> Not really imo: Nested HVM obviously belongs in hvm/; alt-P2m is more of a
>> mm extension than a HVM one afaict, and hence would rather belong in mm/.
> Would like George's opinion also on this before we make this change (again 
> want to avoid thrashing on things like this).

This is a bit of a murky one, since the whole reason you're doing this
is that you actually do use hardware to do the VMFUNC and p2m switching.

Let me take a look at v5 and see what I think (as it sounds like some of
the hvm_function hooks have disappeared).


Xen-devel mailing list



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