[Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-12-10 Thread Andrew Cooper
Signed-off-by: Andrew Cooper --- CC: Keir Fraser CC: Jan Beulich CC: Tim Deegan CC: Ian Campbell CC: Ian Jackson CC: Stefano Stabellini CC: Shannon Zhao v2: Reformat the comments, and note that GSI 0 cannot be used. --- This ABI is utterly mad. Despite appearing to be wrong, it is compati

Re: [Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-24 Thread Paul Durrant
Beulich > >> Sent: 24 November 2015 07:33 > >> To: Andrew Cooper > >> Cc: Keir (Xen.org); Ian Campbell; Tim (Xen.org); Ian Jackson; Xen-devel; > >> Stefano Stabellini; Shannon Zhao > >> Subject: Re: [Xen-devel] [PATCH] public/hvm: Export the > >>

Re: [Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-24 Thread George Dunlap
On Tue, Nov 24, 2015 at 10:28 AM, Andrew Cooper wrote: >> Could we also perhaps deprecate this whole mechanism at this point? I added >> HVMOP_set_evtchn_upcall_vector quite a while ago now. > > January, so Xen 4.6 is the only release with this hvmop in. Come on -- if you're not rebuilding from

Re: [Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-24 Thread Andrew Cooper
Campbell; Tim (Xen.org); Ian Jackson; Xen-devel; >> Stefano Stabellini; Shannon Zhao >> Subject: Re: [Xen-devel] [PATCH] public/hvm: Export the >> HVM_PARAM_CALLBACK_VIA ABI in the API >> >>>>> On 20.11.15 at 19:20, wrote: >>> --- a/xen/includ

Re: [Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-24 Thread Paul Durrant
bellini; Shannon Zhao > Subject: Re: [Xen-devel] [PATCH] public/hvm: Export the > HVM_PARAM_CALLBACK_VIA ABI in the API > > >>> On 20.11.15 at 19:20, wrote: > > --- a/xen/include/public/hvm/params.h > > +++ b/xen/include/public/hvm/params.h > > @@ -29,18

Re: [Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-23 Thread Jan Beulich
>>> On 20.11.15 at 19:20, wrote: > --- a/xen/include/public/hvm/params.h > +++ b/xen/include/public/hvm/params.h > @@ -29,18 +29,21 @@ > * Parameter space for HVMOP_{set,get}_param. > */ > > +#define HVM_PARAM_CALLBACK_IRQ 0 > + > /* > * How should CPU0 event-channel notifications be deli

[Xen-devel] [PATCH] public/hvm: Export the HVM_PARAM_CALLBACK_VIA ABI in the API

2015-11-20 Thread Andrew Cooper
Signed-off-by: Andrew Cooper CC: Keir Fraser CC: Jan Beulich CC: Tim Deegan CC: Ian Campbell CC: Ian Jackson CC: Stefano Stabellini CC: Shannon Zhao --- This ABI is utterly mad. Despite appearing to be wrong, it is compatible with its implementation of enum { HVMIRQ_callback_