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

Re: [Xen-devel] [PATCH 06/17] PVH xen: Introduce PVH guest type



>>> On 25.04.13 at 01:01, Mukesh Rathor <mukesh.rathor@xxxxxxxxxx> wrote:
> On Wed, 24 Apr 2013 08:07:08 +0100
> "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
> 
>> >>> On 23.04.13 at 23:25, Mukesh Rathor <mukesh.rathor@xxxxxxxxxx>
>> >>> wrote:
>> > This patch introduces the concept of a pvh guest. There are also
>> > other basic changes like creating macros to check for pvh
>> > vcpu/domain, and creating new macros to see if it's pv/pvh/hvm
>> > domain/vcpu. Also, modify copy macros to include pvh. Lastly, we
>> > introduce that PVH uses HVM style event delivery.
>> > 
>> > Chagnes in V2:
>> >   - make is_pvh/is_hvm enum instead of adding is_pvh as a new flag.
>> >   - fix indentation and spacing in guest_kernel_mode macro.
>> >   - add debug only BUG() in GUEST_KERNEL_RPL macro as it should no
>> > longer be called in any PVH paths.
>> > 
>> > Chagnes in V3:
>> >   - Rename enum fields, and add is_pv to it.
>> >   - Get rid if is_hvm_or_pvh_* macros.
>> > 
>> > Chagnes in V4:
>> >   - Move e820 fields out of pv_domain struct.
>> 
>> Is there any reason why this can't be a standalone patch?
> 
> I could move it, the e820 field changes, to an earlier smaller prep
> patch or create a separate one if they are all big.

A separate one would be much preferred.

Jan


_______________________________________________
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®.