Re: [Xen-devel] [PATCH 15/17] PVH xen: Miscellaneous changes

["Jan Beulich" <JBeulich@xxxxxxxx>]

>>> On 10.05.13 at 03:54, Mukesh Rathor <mukesh.rathor@xxxxxxxxxx> wrote:
> On Wed, 24 Apr 2013 10:06:22 +0100
> "Jan Beulich" <JBeulich@xxxxxxxx> wrote:
> 
>> >>> On 23.04.13 at 23:26, Mukesh Rathor <mukesh.rathor@xxxxxxxxxx>
>> >>> wrote:
>> > --- a/xen/include/public/xen.h
>> > +++ b/xen/include/public/xen.h
>> > @@ -693,6 +693,8 @@ typedef struct shared_info shared_info_t;
>> >   *      c. list of allocated page frames [mfn_list, nr_pages]
>> >   *         (unless relocated due to XEN_ELFNOTE_INIT_P2M)
>> >   *      d. start_info_t structure        [register ESI (x86)]
>> > + *      d1. struct shared_info_t                [shared_info]
>> > + *                   (above if auto translated guest only)
>> >   *      e. bootstrap page tables         [pt_base and CR3 (x86)]
>> >   *      f. bootstrap stack               [register ESP (x86)]
>> >   *  4. Bootstrap elements are packed together, but each is
>> > 4kB-aligned.
>> 
>> This adjustment should be done in the patch implementing this.
> 
> Happens in tool stack for domU and construct_dom0 for dom0. So probably
> best to leave in this patch? Or I could move it to last patch changing  
> construct_dom0.

If the adjustment is done in two separate patches, then the
change above logically belongs into the first one (albeit for
ease of committing putting it in the Dom0 one would seem
preferable, even if that's the later one in the series).

Jan


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