* [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
@ 2019-01-11 18:09 ` Anthony PERARD
0 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-11 18:09 UTC (permalink / raw)
To: qemu-devel
Cc: Anthony PERARD, Stefano Stabellini, Paul Durrant, open list:X86
Patch "xen: add event channel interface for XenDevice-s" makes use of
the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
4.7. Also the function xen_device_bind_event_channel assign the return
value of xenevtchn_bind_interdomain to channel->local_port but check the
result for error with xendev->local_port.
Fix by:
- removing local_port from struct XenDevice as it isn't use anywere.
- adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
4.6 and earlier.
As extra, replace the type of XenEventChannel->local_port by
evtchn_port_t.
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
---
hw/xen/xen-bus.c | 12 +++++++-----
include/hw/xen/xen-bus.h | 1 -
include/hw/xen/xen_common.h | 1 +
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
index f90bcf2342..3aeccec69c 100644
--- a/hw/xen/xen-bus.c
+++ b/hw/xen/xen-bus.c
@@ -917,7 +917,7 @@ void xen_device_copy_grant_refs(XenDevice *xendev, bool to_domain,
}
struct XenEventChannel {
- unsigned int local_port;
+ evtchn_port_t local_port;
XenEventHandler handler;
void *opaque;
Notifier notifier;
@@ -939,17 +939,19 @@ XenEventChannel *xen_device_bind_event_channel(XenDevice *xendev,
void *opaque, Error **errp)
{
XenEventChannel *channel = g_new0(XenEventChannel, 1);
+ xenevtchn_port_or_error_t local_port;
- channel->local_port = xenevtchn_bind_interdomain(xendev->xeh,
- xendev->frontend_id,
- port);
- if (xendev->local_port < 0) {
+ local_port = xenevtchn_bind_interdomain(xendev->xeh,
+ xendev->frontend_id,
+ port);
+ if (local_port < 0) {
error_setg_errno(errp, errno, "xenevtchn_bind_interdomain failed");
g_free(channel);
return NULL;
}
+ channel->local_port = local_port;
channel->handler = handler;
channel->opaque = opaque;
channel->notifier.notify = event_notify;
diff --git a/include/hw/xen/xen-bus.h b/include/hw/xen/xen-bus.h
index e55a5de5f1..3183f10e3c 100644
--- a/include/hw/xen/xen-bus.h
+++ b/include/hw/xen/xen-bus.h
@@ -29,7 +29,6 @@ typedef struct XenDevice {
xengnttab_handle *xgth;
bool feature_grant_copy;
xenevtchn_handle *xeh;
- xenevtchn_port_or_error_t local_port;
NotifierList event_notifiers;
} XenDevice;
diff --git a/include/hw/xen/xen_common.h b/include/hw/xen/xen_common.h
index 2b91d199a1..9a8155e172 100644
--- a/include/hw/xen/xen_common.h
+++ b/include/hw/xen/xen_common.h
@@ -32,6 +32,7 @@ extern xc_interface *xen_xc;
typedef xc_interface xenforeignmemory_handle;
typedef xc_evtchn xenevtchn_handle;
typedef xc_gnttab xengnttab_handle;
+typedef evtchn_port_or_error_t xenevtchn_port_or_error_t;
#define xenevtchn_open(l, f) xc_evtchn_open(l, f);
#define xenevtchn_close(h) xc_evtchn_close(h)
--
Anthony PERARD
^ permalink raw reply related [flat|nested] 10+ messages in thread
* [PATCH] xen: Fix event channel interface for XenDevice-s
@ 2019-01-11 18:09 ` Anthony PERARD
0 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-11 18:09 UTC (permalink / raw)
To: qemu-devel
Cc: Anthony PERARD, open list:X86, Stefano Stabellini, Paul Durrant
Patch "xen: add event channel interface for XenDevice-s" makes use of
the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
4.7. Also the function xen_device_bind_event_channel assign the return
value of xenevtchn_bind_interdomain to channel->local_port but check the
result for error with xendev->local_port.
Fix by:
- removing local_port from struct XenDevice as it isn't use anywere.
- adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
4.6 and earlier.
As extra, replace the type of XenEventChannel->local_port by
evtchn_port_t.
Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
---
hw/xen/xen-bus.c | 12 +++++++-----
include/hw/xen/xen-bus.h | 1 -
include/hw/xen/xen_common.h | 1 +
3 files changed, 8 insertions(+), 6 deletions(-)
diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
index f90bcf2342..3aeccec69c 100644
--- a/hw/xen/xen-bus.c
+++ b/hw/xen/xen-bus.c
@@ -917,7 +917,7 @@ void xen_device_copy_grant_refs(XenDevice *xendev, bool to_domain,
}
struct XenEventChannel {
- unsigned int local_port;
+ evtchn_port_t local_port;
XenEventHandler handler;
void *opaque;
Notifier notifier;
@@ -939,17 +939,19 @@ XenEventChannel *xen_device_bind_event_channel(XenDevice *xendev,
void *opaque, Error **errp)
{
XenEventChannel *channel = g_new0(XenEventChannel, 1);
+ xenevtchn_port_or_error_t local_port;
- channel->local_port = xenevtchn_bind_interdomain(xendev->xeh,
- xendev->frontend_id,
- port);
- if (xendev->local_port < 0) {
+ local_port = xenevtchn_bind_interdomain(xendev->xeh,
+ xendev->frontend_id,
+ port);
+ if (local_port < 0) {
error_setg_errno(errp, errno, "xenevtchn_bind_interdomain failed");
g_free(channel);
return NULL;
}
+ channel->local_port = local_port;
channel->handler = handler;
channel->opaque = opaque;
channel->notifier.notify = event_notify;
diff --git a/include/hw/xen/xen-bus.h b/include/hw/xen/xen-bus.h
index e55a5de5f1..3183f10e3c 100644
--- a/include/hw/xen/xen-bus.h
+++ b/include/hw/xen/xen-bus.h
@@ -29,7 +29,6 @@ typedef struct XenDevice {
xengnttab_handle *xgth;
bool feature_grant_copy;
xenevtchn_handle *xeh;
- xenevtchn_port_or_error_t local_port;
NotifierList event_notifiers;
} XenDevice;
diff --git a/include/hw/xen/xen_common.h b/include/hw/xen/xen_common.h
index 2b91d199a1..9a8155e172 100644
--- a/include/hw/xen/xen_common.h
+++ b/include/hw/xen/xen_common.h
@@ -32,6 +32,7 @@ extern xc_interface *xen_xc;
typedef xc_interface xenforeignmemory_handle;
typedef xc_evtchn xenevtchn_handle;
typedef xc_gnttab xengnttab_handle;
+typedef evtchn_port_or_error_t xenevtchn_port_or_error_t;
#define xenevtchn_open(l, f) xc_evtchn_open(l, f);
#define xenevtchn_close(h) xc_evtchn_close(h)
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
2019-01-11 18:09 ` Anthony PERARD
@ 2019-01-11 18:11 ` Anthony PERARD
-1 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-11 18:11 UTC (permalink / raw)
To: qemu-devel; +Cc: Stefano Stabellini, Paul Durrant, open list:X86
On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> Patch "xen: add event channel interface for XenDevice-s" makes use of
> the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> 4.7. Also the function xen_device_bind_event_channel assign the return
> value of xenevtchn_bind_interdomain to channel->local_port but check the
> result for error with xendev->local_port.
>
> Fix by:
> - removing local_port from struct XenDevice as it isn't use anywere.
> - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> 4.6 and earlier.
>
> As extra, replace the type of XenEventChannel->local_port by
> evtchn_port_t.
>
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Notes:
This patch fix "xen: add event channel interface for XenDevice-s" that
isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
--
Anthony PERARD
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xen: Fix event channel interface for XenDevice-s
@ 2019-01-11 18:11 ` Anthony PERARD
0 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-11 18:11 UTC (permalink / raw)
To: qemu-devel; +Cc: open list:X86, Stefano Stabellini, Paul Durrant
On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> Patch "xen: add event channel interface for XenDevice-s" makes use of
> the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> 4.7. Also the function xen_device_bind_event_channel assign the return
> value of xenevtchn_bind_interdomain to channel->local_port but check the
> result for error with xendev->local_port.
>
> Fix by:
> - removing local_port from struct XenDevice as it isn't use anywere.
> - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> 4.6 and earlier.
>
> As extra, replace the type of XenEventChannel->local_port by
> evtchn_port_t.
>
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Notes:
This patch fix "xen: add event channel interface for XenDevice-s" that
isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
2019-01-11 18:11 ` Anthony PERARD
@ 2019-01-11 18:16 ` Peter Maydell
-1 siblings, 0 replies; 10+ messages in thread
From: Peter Maydell @ 2019-01-11 18:16 UTC (permalink / raw)
To: Anthony PERARD
Cc: QEMU Developers, open list:X86, Stefano Stabellini, Paul Durrant
On Fri, 11 Jan 2019 at 18:13, Anthony PERARD <anthony.perard@citrix.com> wrote:
>
> On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> > Patch "xen: add event channel interface for XenDevice-s" makes use of
> > the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> > 4.7. Also the function xen_device_bind_event_channel assign the return
> > value of xenevtchn_bind_interdomain to channel->local_port but check the
> > result for error with xendev->local_port.
> >
> > Fix by:
> > - removing local_port from struct XenDevice as it isn't use anywere.
> > - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> > 4.6 and earlier.
> >
> > As extra, replace the type of XenEventChannel->local_port by
> > evtchn_port_t.
> >
> > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
>
> Notes:
> This patch fix "xen: add event channel interface for XenDevice-s" that
> isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
Thanks for the fix. I assume you're going to squash it into the
appropriate patch in that pullrequest ?
thanks
-- PMM
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
@ 2019-01-11 18:16 ` Peter Maydell
0 siblings, 0 replies; 10+ messages in thread
From: Peter Maydell @ 2019-01-11 18:16 UTC (permalink / raw)
To: Anthony PERARD
Cc: open list:X86, Stefano Stabellini, QEMU Developers, Paul Durrant
On Fri, 11 Jan 2019 at 18:13, Anthony PERARD <anthony.perard@citrix.com> wrote:
>
> On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> > Patch "xen: add event channel interface for XenDevice-s" makes use of
> > the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> > 4.7. Also the function xen_device_bind_event_channel assign the return
> > value of xenevtchn_bind_interdomain to channel->local_port but check the
> > result for error with xendev->local_port.
> >
> > Fix by:
> > - removing local_port from struct XenDevice as it isn't use anywere.
> > - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> > 4.6 and earlier.
> >
> > As extra, replace the type of XenEventChannel->local_port by
> > evtchn_port_t.
> >
> > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
>
> Notes:
> This patch fix "xen: add event channel interface for XenDevice-s" that
> isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
Thanks for the fix. I assume you're going to squash it into the
appropriate patch in that pullrequest ?
thanks
-- PMM
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
2019-01-11 18:09 ` Anthony PERARD
` (2 preceding siblings ...)
(?)
@ 2019-01-11 20:16 ` Paul Durrant
-1 siblings, 0 replies; 10+ messages in thread
From: Paul Durrant @ 2019-01-11 20:16 UTC (permalink / raw)
To: Anthony Perard, qemu-devel; +Cc: Stefano Stabellini, open list:X86
> -----Original Message-----
> From: Anthony PERARD [mailto:anthony.perard@citrix.com]
> Sent: 11 January 2019 18:10
> To: qemu-devel@nongnu.org
> Cc: Anthony Perard <anthony.perard@citrix.com>; Stefano Stabellini
> <sstabellini@kernel.org>; Paul Durrant <Paul.Durrant@citrix.com>; open
> list:X86 <xen-devel@lists.xenproject.org>
> Subject: [PATCH] xen: Fix event channel interface for XenDevice-s
>
> Patch "xen: add event channel interface for XenDevice-s" makes use of
> the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> 4.7. Also the function xen_device_bind_event_channel assign the return
> value of xenevtchn_bind_interdomain to channel->local_port but check the
> result for error with xendev->local_port.
>
> Fix by:
> - removing local_port from struct XenDevice as it isn't use anywere.
> - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> 4.6 and earlier.
>
> As extra, replace the type of XenEventChannel->local_port by
> evtchn_port_t.
>
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
> ---
> hw/xen/xen-bus.c | 12 +++++++-----
> include/hw/xen/xen-bus.h | 1 -
> include/hw/xen/xen_common.h | 1 +
> 3 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
> index f90bcf2342..3aeccec69c 100644
> --- a/hw/xen/xen-bus.c
> +++ b/hw/xen/xen-bus.c
> @@ -917,7 +917,7 @@ void xen_device_copy_grant_refs(XenDevice *xendev,
> bool to_domain,
> }
>
> struct XenEventChannel {
> - unsigned int local_port;
> + evtchn_port_t local_port;
> XenEventHandler handler;
> void *opaque;
> Notifier notifier;
> @@ -939,17 +939,19 @@ XenEventChannel
> *xen_device_bind_event_channel(XenDevice *xendev,
> void *opaque, Error
> **errp)
> {
> XenEventChannel *channel = g_new0(XenEventChannel, 1);
> + xenevtchn_port_or_error_t local_port;
>
> - channel->local_port = xenevtchn_bind_interdomain(xendev->xeh,
> - xendev->frontend_id,
> - port);
> - if (xendev->local_port < 0) {
> + local_port = xenevtchn_bind_interdomain(xendev->xeh,
> + xendev->frontend_id,
> + port);
> + if (local_port < 0) {
> error_setg_errno(errp, errno, "xenevtchn_bind_interdomain
> failed");
>
> g_free(channel);
> return NULL;
> }
>
> + channel->local_port = local_port;
> channel->handler = handler;
> channel->opaque = opaque;
> channel->notifier.notify = event_notify;
> diff --git a/include/hw/xen/xen-bus.h b/include/hw/xen/xen-bus.h
> index e55a5de5f1..3183f10e3c 100644
> --- a/include/hw/xen/xen-bus.h
> +++ b/include/hw/xen/xen-bus.h
> @@ -29,7 +29,6 @@ typedef struct XenDevice {
> xengnttab_handle *xgth;
> bool feature_grant_copy;
> xenevtchn_handle *xeh;
> - xenevtchn_port_or_error_t local_port;
> NotifierList event_notifiers;
> } XenDevice;
>
> diff --git a/include/hw/xen/xen_common.h b/include/hw/xen/xen_common.h
> index 2b91d199a1..9a8155e172 100644
> --- a/include/hw/xen/xen_common.h
> +++ b/include/hw/xen/xen_common.h
> @@ -32,6 +32,7 @@ extern xc_interface *xen_xc;
> typedef xc_interface xenforeignmemory_handle;
> typedef xc_evtchn xenevtchn_handle;
> typedef xc_gnttab xengnttab_handle;
> +typedef evtchn_port_or_error_t xenevtchn_port_or_error_t;
>
> #define xenevtchn_open(l, f) xc_evtchn_open(l, f);
> #define xenevtchn_close(h) xc_evtchn_close(h)
> --
> Anthony PERARD
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] xen: Fix event channel interface for XenDevice-s
2019-01-11 18:09 ` Anthony PERARD
(?)
(?)
@ 2019-01-11 20:16 ` Paul Durrant
-1 siblings, 0 replies; 10+ messages in thread
From: Paul Durrant @ 2019-01-11 20:16 UTC (permalink / raw)
To: qemu-devel; +Cc: Anthony Perard, open list:X86, Stefano Stabellini
> -----Original Message-----
> From: Anthony PERARD [mailto:anthony.perard@citrix.com]
> Sent: 11 January 2019 18:10
> To: qemu-devel@nongnu.org
> Cc: Anthony Perard <anthony.perard@citrix.com>; Stefano Stabellini
> <sstabellini@kernel.org>; Paul Durrant <Paul.Durrant@citrix.com>; open
> list:X86 <xen-devel@lists.xenproject.org>
> Subject: [PATCH] xen: Fix event channel interface for XenDevice-s
>
> Patch "xen: add event channel interface for XenDevice-s" makes use of
> the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> 4.7. Also the function xen_device_bind_event_channel assign the return
> value of xenevtchn_bind_interdomain to channel->local_port but check the
> result for error with xendev->local_port.
>
> Fix by:
> - removing local_port from struct XenDevice as it isn't use anywere.
> - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> 4.6 and earlier.
>
> As extra, replace the type of XenEventChannel->local_port by
> evtchn_port_t.
>
> Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
> ---
> hw/xen/xen-bus.c | 12 +++++++-----
> include/hw/xen/xen-bus.h | 1 -
> include/hw/xen/xen_common.h | 1 +
> 3 files changed, 8 insertions(+), 6 deletions(-)
>
> diff --git a/hw/xen/xen-bus.c b/hw/xen/xen-bus.c
> index f90bcf2342..3aeccec69c 100644
> --- a/hw/xen/xen-bus.c
> +++ b/hw/xen/xen-bus.c
> @@ -917,7 +917,7 @@ void xen_device_copy_grant_refs(XenDevice *xendev,
> bool to_domain,
> }
>
> struct XenEventChannel {
> - unsigned int local_port;
> + evtchn_port_t local_port;
> XenEventHandler handler;
> void *opaque;
> Notifier notifier;
> @@ -939,17 +939,19 @@ XenEventChannel
> *xen_device_bind_event_channel(XenDevice *xendev,
> void *opaque, Error
> **errp)
> {
> XenEventChannel *channel = g_new0(XenEventChannel, 1);
> + xenevtchn_port_or_error_t local_port;
>
> - channel->local_port = xenevtchn_bind_interdomain(xendev->xeh,
> - xendev->frontend_id,
> - port);
> - if (xendev->local_port < 0) {
> + local_port = xenevtchn_bind_interdomain(xendev->xeh,
> + xendev->frontend_id,
> + port);
> + if (local_port < 0) {
> error_setg_errno(errp, errno, "xenevtchn_bind_interdomain
> failed");
>
> g_free(channel);
> return NULL;
> }
>
> + channel->local_port = local_port;
> channel->handler = handler;
> channel->opaque = opaque;
> channel->notifier.notify = event_notify;
> diff --git a/include/hw/xen/xen-bus.h b/include/hw/xen/xen-bus.h
> index e55a5de5f1..3183f10e3c 100644
> --- a/include/hw/xen/xen-bus.h
> +++ b/include/hw/xen/xen-bus.h
> @@ -29,7 +29,6 @@ typedef struct XenDevice {
> xengnttab_handle *xgth;
> bool feature_grant_copy;
> xenevtchn_handle *xeh;
> - xenevtchn_port_or_error_t local_port;
> NotifierList event_notifiers;
> } XenDevice;
>
> diff --git a/include/hw/xen/xen_common.h b/include/hw/xen/xen_common.h
> index 2b91d199a1..9a8155e172 100644
> --- a/include/hw/xen/xen_common.h
> +++ b/include/hw/xen/xen_common.h
> @@ -32,6 +32,7 @@ extern xc_interface *xen_xc;
> typedef xc_interface xenforeignmemory_handle;
> typedef xc_evtchn xenevtchn_handle;
> typedef xc_gnttab xengnttab_handle;
> +typedef evtchn_port_or_error_t xenevtchn_port_or_error_t;
>
> #define xenevtchn_open(l, f) xc_evtchn_open(l, f);
> #define xenevtchn_close(h) xc_evtchn_close(h)
> --
> Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
2019-01-11 18:16 ` Peter Maydell
@ 2019-01-14 12:07 ` Anthony PERARD
-1 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-14 12:07 UTC (permalink / raw)
To: Peter Maydell
Cc: QEMU Developers, open list:X86, Stefano Stabellini, Paul Durrant
On Fri, Jan 11, 2019 at 06:16:45PM +0000, Peter Maydell wrote:
> On Fri, 11 Jan 2019 at 18:13, Anthony PERARD <anthony.perard@citrix.com> wrote:
> >
> > On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> > > Patch "xen: add event channel interface for XenDevice-s" makes use of
> > > the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> > > 4.7. Also the function xen_device_bind_event_channel assign the return
> > > value of xenevtchn_bind_interdomain to channel->local_port but check the
> > > result for error with xendev->local_port.
> > >
> > > Fix by:
> > > - removing local_port from struct XenDevice as it isn't use anywere.
> > > - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> > > 4.6 and earlier.
> > >
> > > As extra, replace the type of XenEventChannel->local_port by
> > > evtchn_port_t.
> > >
> > > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> >
> > Notes:
> > This patch fix "xen: add event channel interface for XenDevice-s" that
> > isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
>
> Thanks for the fix. I assume you're going to squash it into the
> appropriate patch in that pullrequest ?
Yes, it's a good idea to squash it. I'll resubmit the pullreq shortly.
--
Anthony PERARD
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s
@ 2019-01-14 12:07 ` Anthony PERARD
0 siblings, 0 replies; 10+ messages in thread
From: Anthony PERARD @ 2019-01-14 12:07 UTC (permalink / raw)
To: Peter Maydell
Cc: open list:X86, Stefano Stabellini, QEMU Developers, Paul Durrant
On Fri, Jan 11, 2019 at 06:16:45PM +0000, Peter Maydell wrote:
> On Fri, 11 Jan 2019 at 18:13, Anthony PERARD <anthony.perard@citrix.com> wrote:
> >
> > On Fri, Jan 11, 2019 at 06:09:41PM +0000, Anthony PERARD wrote:
> > > Patch "xen: add event channel interface for XenDevice-s" makes use of
> > > the type xenevtchn_port_or_error_t, but this isn't avaiable before Xen
> > > 4.7. Also the function xen_device_bind_event_channel assign the return
> > > value of xenevtchn_bind_interdomain to channel->local_port but check the
> > > result for error with xendev->local_port.
> > >
> > > Fix by:
> > > - removing local_port from struct XenDevice as it isn't use anywere.
> > > - adding a compatibility typedef for xenevtchn_port_or_error_t for Xen
> > > 4.6 and earlier.
> > >
> > > As extra, replace the type of XenEventChannel->local_port by
> > > evtchn_port_t.
> > >
> > > Signed-off-by: Anthony PERARD <anthony.perard@citrix.com>
> >
> > Notes:
> > This patch fix "xen: add event channel interface for XenDevice-s" that
> > isn't commited yet, of the patch series "Xen PV backend 'qdevification'".
>
> Thanks for the fix. I assume you're going to squash it into the
> appropriate patch in that pullrequest ?
Yes, it's a good idea to squash it. I'll resubmit the pullreq shortly.
--
Anthony PERARD
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2019-01-14 12:08 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-01-11 18:09 [Qemu-devel] [PATCH] xen: Fix event channel interface for XenDevice-s Anthony PERARD
2019-01-11 18:09 ` Anthony PERARD
2019-01-11 18:11 ` [Qemu-devel] " Anthony PERARD
2019-01-11 18:11 ` Anthony PERARD
2019-01-11 18:16 ` [Qemu-devel] " Peter Maydell
2019-01-11 18:16 ` Peter Maydell
2019-01-14 12:07 ` Anthony PERARD
2019-01-14 12:07 ` Anthony PERARD
2019-01-11 20:16 ` Paul Durrant
2019-01-11 20:16 ` [Qemu-devel] " Paul Durrant
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.