Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1311516 > unrolled thread
| Started by | Stefano Stabellini <stefano.stabellini@eu.citrix.com> |
|---|---|
| First post | 2016-01-18 13:40 +0100 |
| Last post | 2016-01-20 10:10 +0100 |
| Articles | 7 — 3 participants |
Back to article view | Back to linux.kernel
This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by
below is the oldest one visible, not the original post.
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2016-01-18 13:40 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Andrew Cooper <andrew.cooper3@citrix.com> - 2016-01-18 13:50 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Stefano Stabellini <stefano.stabellini@eu.citrix.com> - 2016-01-18 13:50 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Andrew Cooper <andrew.cooper3@citrix.com> - 2016-01-18 14:00 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Shannon Zhao <zhaoshenglong@huawei.com> - 2016-01-20 07:40 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Shannon Zhao <zhaoshenglong@huawei.com> - 2016-01-20 09:50 +0100
Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ Andrew Cooper <andrew.cooper3@citrix.com> - 2016-01-20 10:10 +0100
| From | Stefano Stabellini <stefano.stabellini@eu.citrix.com> |
|---|---|
| Date | 2016-01-18 13:40 +0100 |
| Subject | Re: [Xen-devel] [PATCH v2 08/16] xen/hvm/params: Add a new delivery type for event-channel in HVM_PARAM_CALLBACK_IRQ |
| Message-ID | <qShgd-Du-3@gated-at.bofh.it> |
On Fri, 15 Jan 2016, Shannon Zhao wrote: > From: Shannon Zhao <shannon.zhao@linaro.org> > > Add a new delivery type: > val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. > To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and > bit 1 stands the interrupt polarity is active low(1) or high(0). > > Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> > --- > include/xen/interface/hvm/params.h | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h > index a6c7991..550688a 100644 > --- a/include/xen/interface/hvm/params.h > +++ b/include/xen/interface/hvm/params.h > @@ -34,6 +34,11 @@ > * Domain = val[47:32], Bus = val[31:16], > * DevFn = val[15: 8], IntX = val[ 1: 0] > * val[63:56] == 2: val[7:0] is a vector number. > + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: > + * bit 0: interrupt is edge(1) or level(0) triggered > + * bit 1: interrupt is active low(1) or high(0) > + * val[7:0] is PPI number used by event-channel. > + * This is only used by ARM/ARM64. > * If val == 0 then CPU0 event-channel notifications are not delivered. > */ > #define HVM_PARAM_CALLBACK_IRQ 0 Andrew, I think that this patch is correct. Looking back at your previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), is it possible that you were confused by enum callback_via_type, which is internal to Xen and offset'ed by 1 compared to the described values in xen/include/public/hvm/params.h? If not, and indeed somebody introduced one more field but failed to document it, then I suggest she sends a patch to fix the issue as soon as possible.
[toc] | [next] | [standalone]
| From | Andrew Cooper <andrew.cooper3@citrix.com> |
|---|---|
| Date | 2016-01-18 13:50 +0100 |
| Message-ID | <qShpT-GJ-9@gated-at.bofh.it> |
| In reply to | #1311516 |
On 18/01/16 12:38, Stefano Stabellini wrote: > On Fri, 15 Jan 2016, Shannon Zhao wrote: >> From: Shannon Zhao <shannon.zhao@linaro.org> >> >> Add a new delivery type: >> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >> bit 1 stands the interrupt polarity is active low(1) or high(0). >> >> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> >> --- >> include/xen/interface/hvm/params.h | 5 +++++ >> 1 file changed, 5 insertions(+) >> >> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h >> index a6c7991..550688a 100644 >> --- a/include/xen/interface/hvm/params.h >> +++ b/include/xen/interface/hvm/params.h >> @@ -34,6 +34,11 @@ >> * Domain = val[47:32], Bus = val[31:16], >> * DevFn = val[15: 8], IntX = val[ 1: 0] >> * val[63:56] == 2: val[7:0] is a vector number. >> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: >> + * bit 0: interrupt is edge(1) or level(0) triggered >> + * bit 1: interrupt is active low(1) or high(0) >> + * val[7:0] is PPI number used by event-channel. >> + * This is only used by ARM/ARM64. >> * If val == 0 then CPU0 event-channel notifications are not delivered. >> */ >> #define HVM_PARAM_CALLBACK_IRQ 0 > Andrew, I think that this patch is correct. Looking back at your > previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), > is it possible that you were confused by enum callback_via_type, which > is internal to Xen and offset'ed by 1 compared to the described values > in xen/include/public/hvm/params.h? > > If not, and indeed somebody introduced one more field but failed to > document it, then I suggest she sends a patch to fix the issue as soon > as possible. I was indeed confused - the ABI is utterly mad. However, this change does need rebasing over c/s ca5c54b, which was the result of the original discussion. ~Andrew
[toc] | [prev] | [next] | [standalone]
| From | Stefano Stabellini <stefano.stabellini@eu.citrix.com> |
|---|---|
| Date | 2016-01-18 13:50 +0100 |
| Message-ID | <qShpT-GJ-7@gated-at.bofh.it> |
| In reply to | #1311521 |
On Mon, 18 Jan 2016, Andrew Cooper wrote: > On 18/01/16 12:38, Stefano Stabellini wrote: > > On Fri, 15 Jan 2016, Shannon Zhao wrote: > >> From: Shannon Zhao <shannon.zhao@linaro.org> > >> > >> Add a new delivery type: > >> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. > >> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and > >> bit 1 stands the interrupt polarity is active low(1) or high(0). > >> > >> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> > >> --- > >> include/xen/interface/hvm/params.h | 5 +++++ > >> 1 file changed, 5 insertions(+) > >> > >> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h > >> index a6c7991..550688a 100644 > >> --- a/include/xen/interface/hvm/params.h > >> +++ b/include/xen/interface/hvm/params.h > >> @@ -34,6 +34,11 @@ > >> * Domain = val[47:32], Bus = val[31:16], > >> * DevFn = val[15: 8], IntX = val[ 1: 0] > >> * val[63:56] == 2: val[7:0] is a vector number. > >> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: > >> + * bit 0: interrupt is edge(1) or level(0) triggered > >> + * bit 1: interrupt is active low(1) or high(0) > >> + * val[7:0] is PPI number used by event-channel. > >> + * This is only used by ARM/ARM64. > >> * If val == 0 then CPU0 event-channel notifications are not delivered. > >> */ > >> #define HVM_PARAM_CALLBACK_IRQ 0 > > Andrew, I think that this patch is correct. Looking back at your > > previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), > > is it possible that you were confused by enum callback_via_type, which > > is internal to Xen and offset'ed by 1 compared to the described values > > in xen/include/public/hvm/params.h? > > > > If not, and indeed somebody introduced one more field but failed to > > document it, then I suggest she sends a patch to fix the issue as soon > > as possible. > > I was indeed confused - the ABI is utterly mad. All right. In that case, Shannon, you can add my Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > However, this change does need rebasing over c/s ca5c54b, which was the > result of the original discussion. c/s ca5c54b is for Xen, while this is a Linux patch (Linux has its own set of Xen headers).
[toc] | [prev] | [next] | [standalone]
| From | Andrew Cooper <andrew.cooper3@citrix.com> |
|---|---|
| Date | 2016-01-18 14:00 +0100 |
| Message-ID | <qShzA-Kd-17@gated-at.bofh.it> |
| In reply to | #1311522 |
On 18/01/16 12:46, Stefano Stabellini wrote: > On Mon, 18 Jan 2016, Andrew Cooper wrote: >> On 18/01/16 12:38, Stefano Stabellini wrote: >>> On Fri, 15 Jan 2016, Shannon Zhao wrote: >>>> From: Shannon Zhao <shannon.zhao@linaro.org> >>>> >>>> Add a new delivery type: >>>> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >>>> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >>>> bit 1 stands the interrupt polarity is active low(1) or high(0). >>>> >>>> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> >>>> --- >>>> include/xen/interface/hvm/params.h | 5 +++++ >>>> 1 file changed, 5 insertions(+) >>>> >>>> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h >>>> index a6c7991..550688a 100644 >>>> --- a/include/xen/interface/hvm/params.h >>>> +++ b/include/xen/interface/hvm/params.h >>>> @@ -34,6 +34,11 @@ >>>> * Domain = val[47:32], Bus = val[31:16], >>>> * DevFn = val[15: 8], IntX = val[ 1: 0] >>>> * val[63:56] == 2: val[7:0] is a vector number. >>>> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: >>>> + * bit 0: interrupt is edge(1) or level(0) triggered >>>> + * bit 1: interrupt is active low(1) or high(0) >>>> + * val[7:0] is PPI number used by event-channel. >>>> + * This is only used by ARM/ARM64. >>>> * If val == 0 then CPU0 event-channel notifications are not delivered. >>>> */ >>>> #define HVM_PARAM_CALLBACK_IRQ 0 >>> Andrew, I think that this patch is correct. Looking back at your >>> previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), >>> is it possible that you were confused by enum callback_via_type, which >>> is internal to Xen and offset'ed by 1 compared to the described values >>> in xen/include/public/hvm/params.h? >>> >>> If not, and indeed somebody introduced one more field but failed to >>> document it, then I suggest she sends a patch to fix the issue as soon >>> as possible. >> I was indeed confused - the ABI is utterly mad. > All right. In that case, Shannon, you can add my > > Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> > > >> However, this change does need rebasing over c/s ca5c54b, which was the >> result of the original discussion. > c/s ca5c54b is for Xen, while this is a Linux patch (Linux has its own > set of Xen headers). All ABI changes need to happen in the Xen public headers first. This patch cannot be accepted yet. ~Andrew
[toc] | [prev] | [next] | [standalone]
| From | Shannon Zhao <zhaoshenglong@huawei.com> |
|---|---|
| Date | 2016-01-20 07:40 +0100 |
| Message-ID | <qSUAW-2Ev-13@gated-at.bofh.it> |
| In reply to | #1311526 |
On 2016/1/18 20:52, Andrew Cooper wrote: > On 18/01/16 12:46, Stefano Stabellini wrote: >> > On Mon, 18 Jan 2016, Andrew Cooper wrote: >>> >> On 18/01/16 12:38, Stefano Stabellini wrote: >>>> >>> On Fri, 15 Jan 2016, Shannon Zhao wrote: >>>>> >>>> From: Shannon Zhao <shannon.zhao@linaro.org> >>>>> >>>> >>>>> >>>> Add a new delivery type: >>>>> >>>> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >>>>> >>>> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >>>>> >>>> bit 1 stands the interrupt polarity is active low(1) or high(0). >>>>> >>>> >>>>> >>>> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> >>>>> >>>> --- >>>>> >>>> include/xen/interface/hvm/params.h | 5 +++++ >>>>> >>>> 1 file changed, 5 insertions(+) >>>>> >>>> >>>>> >>>> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h >>>>> >>>> index a6c7991..550688a 100644 >>>>> >>>> --- a/include/xen/interface/hvm/params.h >>>>> >>>> +++ b/include/xen/interface/hvm/params.h >>>>> >>>> @@ -34,6 +34,11 @@ >>>>> >>>> * Domain = val[47:32], Bus = val[31:16], >>>>> >>>> * DevFn = val[15: 8], IntX = val[ 1: 0] >>>>> >>>> * val[63:56] == 2: val[7:0] is a vector number. >>>>> >>>> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: >>>>> >>>> + * bit 0: interrupt is edge(1) or level(0) triggered >>>>> >>>> + * bit 1: interrupt is active low(1) or high(0) >>>>> >>>> + * val[7:0] is PPI number used by event-channel. >>>>> >>>> + * This is only used by ARM/ARM64. >>>>> >>>> * If val == 0 then CPU0 event-channel notifications are not delivered. >>>>> >>>> */ >>>>> >>>> #define HVM_PARAM_CALLBACK_IRQ 0 >>>> >>> Andrew, I think that this patch is correct. Looking back at your >>>> >>> previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), >>>> >>> is it possible that you were confused by enum callback_via_type, which >>>> >>> is internal to Xen and offset'ed by 1 compared to the described values >>>> >>> in xen/include/public/hvm/params.h? >>>> >>> >>>> >>> If not, and indeed somebody introduced one more field but failed to >>>> >>> document it, then I suggest she sends a patch to fix the issue as soon >>>> >>> as possible. >>> >> I was indeed confused - the ABI is utterly mad. >> > All right. In that case, Shannon, you can add my >> > >> > Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> >> > >> > >>> >> However, this change does need rebasing over c/s ca5c54b, which was the >>> >> result of the original discussion. >> > c/s ca5c54b is for Xen, while this is a Linux patch (Linux has its own >> > set of Xen headers). > All ABI changes need to happen in the Xen public headers first. This > patch cannot be accepted yet. So you mean it should port c/s ca5c54b to Linux first? Then add this change based on that? Thanks, -- Shannon
[toc] | [prev] | [next] | [standalone]
| From | Shannon Zhao <zhaoshenglong@huawei.com> |
|---|---|
| Date | 2016-01-20 09:50 +0100 |
| Message-ID | <qSWCJ-3V7-1@gated-at.bofh.it> |
| In reply to | #1312920 |
On 2016/1/20 16:39, Andrew Cooper wrote: > On 20/01/2016 06:33, Shannon Zhao wrote: >> > >> > On 2016/1/18 20:52, Andrew Cooper wrote: >>> >> On 18/01/16 12:46, Stefano Stabellini wrote: >>>>> >>>> On Mon, 18 Jan 2016, Andrew Cooper wrote: >>>>>>> >>>>>> On 18/01/16 12:38, Stefano Stabellini wrote: >>>>>>>>> >>>>>>>> On Fri, 15 Jan 2016, Shannon Zhao wrote: >>>>>>>>>>> >>>>>>>>>> From: Shannon Zhao <shannon.zhao@linaro.org> >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> Add a new delivery type: >>>>>>>>>>> >>>>>>>>>> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >>>>>>>>>>> >>>>>>>>>> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >>>>>>>>>>> >>>>>>>>>> bit 1 stands the interrupt polarity is active low(1) or high(0). >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> >>>>>>>>>>> >>>>>>>>>> --- >>>>>>>>>>> >>>>>>>>>> include/xen/interface/hvm/params.h | 5 +++++ >>>>>>>>>>> >>>>>>>>>> 1 file changed, 5 insertions(+) >>>>>>>>>>> >>>>>>>>>> >>>>>>>>>>> >>>>>>>>>> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h >>>>>>>>>>> >>>>>>>>>> index a6c7991..550688a 100644 >>>>>>>>>>> >>>>>>>>>> --- a/include/xen/interface/hvm/params.h >>>>>>>>>>> >>>>>>>>>> +++ b/include/xen/interface/hvm/params.h >>>>>>>>>>> >>>>>>>>>> @@ -34,6 +34,11 @@ >>>>>>>>>>> >>>>>>>>>> * Domain = val[47:32], Bus = val[31:16], >>>>>>>>>>> >>>>>>>>>> * DevFn = val[15: 8], IntX = val[ 1: 0] >>>>>>>>>>> >>>>>>>>>> * val[63:56] == 2: val[7:0] is a vector number. >>>>>>>>>>> >>>>>>>>>> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: >>>>>>>>>>> >>>>>>>>>> + * bit 0: interrupt is edge(1) or level(0) triggered >>>>>>>>>>> >>>>>>>>>> + * bit 1: interrupt is active low(1) or high(0) >>>>>>>>>>> >>>>>>>>>> + * val[7:0] is PPI number used by event-channel. >>>>>>>>>>> >>>>>>>>>> + * This is only used by ARM/ARM64. >>>>>>>>>>> >>>>>>>>>> * If val == 0 then CPU0 event-channel notifications are not delivered. >>>>>>>>>>> >>>>>>>>>> */ >>>>>>>>>>> >>>>>>>>>> #define HVM_PARAM_CALLBACK_IRQ 0 >>>>>>>>> >>>>>>>> Andrew, I think that this patch is correct. Looking back at your >>>>>>>>> >>>>>>>> previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), >>>>>>>>> >>>>>>>> is it possible that you were confused by enum callback_via_type, which >>>>>>>>> >>>>>>>> is internal to Xen and offset'ed by 1 compared to the described values >>>>>>>>> >>>>>>>> in xen/include/public/hvm/params.h? >>>>>>>>> >>>>>>>> >>>>>>>>> >>>>>>>> If not, and indeed somebody introduced one more field but failed to >>>>>>>>> >>>>>>>> document it, then I suggest she sends a patch to fix the issue as soon >>>>>>>>> >>>>>>>> as possible. >>>>>>> >>>>>> I was indeed confused - the ABI is utterly mad. >>>>> >>>> All right. In that case, Shannon, you can add my >>>>> >>>> >>>>> >>>> Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> >>>>> >>>> >>>>> >>>> >>>>>>> >>>>>> However, this change does need rebasing over c/s ca5c54b, which was the >>>>>>> >>>>>> result of the original discussion. >>>>> >>>> c/s ca5c54b is for Xen, while this is a Linux patch (Linux has its own >>>>> >>>> set of Xen headers). >>> >> All ABI changes need to happen in the Xen public headers first. This >>> >> patch cannot be accepted yet. >> > So you mean it should port c/s ca5c54b to Linux first? Then add this >> > change based on that? > FIrst, you must get this content and submit a patch against Xen. The > Xen repository is the authoritative version of the ABI. > I've sent a patch to Xen, but not applied yet. > Once that patch is accepted, you will need to port ca5c54b and the new > patch as part of this series. Ok, thanks. -- Shannon
[toc] | [prev] | [next] | [standalone]
| From | Andrew Cooper <andrew.cooper3@citrix.com> |
|---|---|
| Date | 2016-01-20 10:10 +0100 |
| Message-ID | <qSWCJ-3V7-3@gated-at.bofh.it> |
| In reply to | #1312920 |
On 20/01/2016 06:33, Shannon Zhao wrote: > > On 2016/1/18 20:52, Andrew Cooper wrote: >> On 18/01/16 12:46, Stefano Stabellini wrote: >>>> On Mon, 18 Jan 2016, Andrew Cooper wrote: >>>>>> On 18/01/16 12:38, Stefano Stabellini wrote: >>>>>>>> On Fri, 15 Jan 2016, Shannon Zhao wrote: >>>>>>>>>> From: Shannon Zhao <shannon.zhao@linaro.org> >>>>>>>>>> >>>>>>>>>> Add a new delivery type: >>>>>>>>>> val[63:56] == 3: val[15:8] is flag: val[7:0] is a PPI. >>>>>>>>>> To the flag, bit 0 stands the interrupt mode is edge(1) or level(0) and >>>>>>>>>> bit 1 stands the interrupt polarity is active low(1) or high(0). >>>>>>>>>> >>>>>>>>>> Signed-off-by: Shannon Zhao <shannon.zhao@linaro.org> >>>>>>>>>> --- >>>>>>>>>> include/xen/interface/hvm/params.h | 5 +++++ >>>>>>>>>> 1 file changed, 5 insertions(+) >>>>>>>>>> >>>>>>>>>> diff --git a/include/xen/interface/hvm/params.h b/include/xen/interface/hvm/params.h >>>>>>>>>> index a6c7991..550688a 100644 >>>>>>>>>> --- a/include/xen/interface/hvm/params.h >>>>>>>>>> +++ b/include/xen/interface/hvm/params.h >>>>>>>>>> @@ -34,6 +34,11 @@ >>>>>>>>>> * Domain = val[47:32], Bus = val[31:16], >>>>>>>>>> * DevFn = val[15: 8], IntX = val[ 1: 0] >>>>>>>>>> * val[63:56] == 2: val[7:0] is a vector number. >>>>>>>>>> + * val[63:56] == 3: val[15:8] is flag of event-channel interrupt: >>>>>>>>>> + * bit 0: interrupt is edge(1) or level(0) triggered >>>>>>>>>> + * bit 1: interrupt is active low(1) or high(0) >>>>>>>>>> + * val[7:0] is PPI number used by event-channel. >>>>>>>>>> + * This is only used by ARM/ARM64. >>>>>>>>>> * If val == 0 then CPU0 event-channel notifications are not delivered. >>>>>>>>>> */ >>>>>>>>>> #define HVM_PARAM_CALLBACK_IRQ 0 >>>>>>>> Andrew, I think that this patch is correct. Looking back at your >>>>>>>> previous comment (http://marc.info/?l=devicetree&m=144804014214262&w=2), >>>>>>>> is it possible that you were confused by enum callback_via_type, which >>>>>>>> is internal to Xen and offset'ed by 1 compared to the described values >>>>>>>> in xen/include/public/hvm/params.h? >>>>>>>> >>>>>>>> If not, and indeed somebody introduced one more field but failed to >>>>>>>> document it, then I suggest she sends a patch to fix the issue as soon >>>>>>>> as possible. >>>>>> I was indeed confused - the ABI is utterly mad. >>>> All right. In that case, Shannon, you can add my >>>> >>>> Reviewed-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> >>>> >>>> >>>>>> However, this change does need rebasing over c/s ca5c54b, which was the >>>>>> result of the original discussion. >>>> c/s ca5c54b is for Xen, while this is a Linux patch (Linux has its own >>>> set of Xen headers). >> All ABI changes need to happen in the Xen public headers first. This >> patch cannot be accepted yet. > So you mean it should port c/s ca5c54b to Linux first? Then add this > change based on that? FIrst, you must get this content and submit a patch against Xen. The Xen repository is the authoritative version of the ABI. Once that patch is accepted, you will need to port ca5c54b and the new patch as part of this series. ~Andrew
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web