Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Jan Beulich
>>> On 17.02.16 at 15:39, wrote: > On 02/17/2016 05:05 AM, Roger Pau Monné wrote: >> El 17/2/16 a les 10:58, Jan Beulich ha escrit: >> On 16.02.16 at 22:26, wrote: On 02/16/2016 12:37 PM, Roger Pau Monne wrote: > diff --git

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Boris Ostrovsky
On 02/17/2016 05:05 AM, Roger Pau Monné wrote: El 17/2/16 a les 10:58, Jan Beulich ha escrit: On 16.02.16 at 22:26, wrote: On 02/16/2016 12:37 PM, Roger Pau Monne wrote: diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h index 7b629b1..6ba060f 100644

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Jan Beulich
>>> On 16.02.16 at 18:37, wrote: > --- a/xen/include/public/xen.h > +++ b/xen/include/public/xen.h > @@ -787,25 +787,46 @@ typedef struct start_info start_info_t; > /* > * Start of day structure passed to PVH guests in %ebx. > * > - * NOTE: nothing will be loaded at

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Samuel Thibault
Roger Pau Monne, on Tue 16 Feb 2016 18:37:46 +0100, wrote: > After some discussion around the new boot ABI consensus has been reached > about the layout and contents of the start info. The following patch updates > the layout to what has been agreed. > > Also, the new layout is described in

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Roger Pau Monné
El 17/2/16 a les 10:58, Jan Beulich ha escrit: On 16.02.16 at 22:26, wrote: >> On 02/16/2016 12:37 PM, Roger Pau Monne wrote: >>> diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h >>> index 7b629b1..6ba060f 100644 >>> --- a/xen/include/public/xen.h

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Roger Pau Monné
El 16/2/16 a les 21:06, Konrad Rzeszutek Wilk ha escrit: > On Tue, Feb 16, 2016 at 06:37:46PM +0100, Roger Pau Monne wrote: >> After some discussion around the new boot ABI consensus has been reached >> about the layout and contents of the start info. The following patch updates >> the layout to

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-17 Thread Jan Beulich
>>> On 16.02.16 at 22:26, wrote: > On 02/16/2016 12:37 PM, Roger Pau Monne wrote: >> diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h >> index 7b629b1..6ba060f 100644 >> --- a/xen/include/public/xen.h >> +++ b/xen/include/public/xen.h >> @@ -787,25

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-16 Thread Boris Ostrovsky
On 02/16/2016 12:37 PM, Roger Pau Monne wrote: diff --git a/xen/include/public/xen.h b/xen/include/public/xen.h index 7b629b1..6ba060f 100644 --- a/xen/include/public/xen.h +++ b/xen/include/public/xen.h @@ -787,25 +787,46 @@ typedef struct start_info start_info_t; /* * Start of day

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-16 Thread Konrad Rzeszutek Wilk
On Tue, Feb 16, 2016 at 06:37:46PM +0100, Roger Pau Monne wrote: > After some discussion around the new boot ABI consensus has been reached > about the layout and contents of the start info. The following patch updates > the layout to what has been agreed. .. It would be good to have an URL to

Re: [Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-16 Thread Andrew Cooper
On 16/02/16 17:37, Roger Pau Monne wrote: > After some discussion around the new boot ABI consensus has been reached > about the layout and contents of the start info. The following patch updates > the layout to what has been agreed. > > Also, the new layout is described in binary terms in order

[Xen-devel] [PATCH v4 1/4] x86/HVM: update the start info structure layout

2016-02-16 Thread Roger Pau Monne
After some discussion around the new boot ABI consensus has been reached about the layout and contents of the start info. The following patch updates the layout to what has been agreed. Also, the new layout is described in binary terms in order to avoid issues with alignments when using C