Re: [Xen-devel] [RFC PATCH V3] Implement 3-level event channel in Xen

[Keir Fraser <keir@xxxxxxx>]

On 27/02/2013 14:33, "Wei Liu" <wei.liu2@xxxxxxxxxx> wrote:

> Hi all
> 
> This is another version of the patch series. Unfortunately the kernel side is
> not available at the moment. :-(
> 
> Keir, Jan, Ian and David, are you happy with this design in general? I would
> like to have explicit ACK / NACK on this if possible, as feature freeze for
> 4.3
> is quite close now.

I'm happy with the design, in that it solves a current need, and the
implementation exists. It seems this is something we really do need for 4.3.

 -- Keir

> Changes since V2:
> * new interface to register extended event channel ABI
> * use vmap to simplify mapping
> * replace MAX_EVTCHNS macro with inline function
> * libxl: evtchn_l3 -> evtchn_extended
> 
> The most notable bit of this series is the interface change. In order to cope
> with future ABIs, the interface is renamed to EVTCHNOP_register_extended. It
> also provides supported ABI query, so that we can remove unused ABI in the
> future.
> 
> The semantic meaning of EVTCHNOP_register_extended changes a bit. The `level'
> in parameter now changes to `cmd', which means we should go down to specific
> ABI routines. ABI-specific structures are still embedded in the union.
> 
> Changes since V1:
> * move all evtchn related macros / struct definitions to event.h
> * only allow 3-level evtchn for Dom0 and driver domains
> * add evtchn_l3 flag in libxl
> 
> 
> Wei.
> 
> Diffstat:
>  tools/libxl/libxl_create.c         |    4 +
>  tools/libxl/libxl_types.idl        |    1 +
>  tools/libxl/xl_cmdimpl.c           |    2 +
>  tools/libxl/xl_sxp.c               |    2 +
>  xen/arch/arm/domain.c              |    1 +
>  xen/arch/x86/domain.c              |    1 +
>  xen/arch/x86/irq.c                 |    7 +-
>  xen/common/domain.c                |    3 +
>  xen/common/domctl.c                |    5 +-
>  xen/common/event_channel.c         |  458
> +++++++++++++++++++++++++++++++++---
>  xen/common/keyhandler.c            |    6 +-
>  xen/common/schedule.c              |    4 +-
>  xen/include/asm-arm/types.h        |    7 +-
>  xen/include/asm-x86/config.h       |    4 +-
>  xen/include/public/domctl.h        |    3 +
>  xen/include/public/event_channel.h |   44 ++++
>  xen/include/public/xen.h           |   35 ++-
>  xen/include/xen/event.h            |   85 ++++++-
>  xen/include/xen/sched.h            |   65 ++---
>  xen/xsm/flask/hooks.c              |    1 +
>  20 files changed, 623 insertions(+), 115 deletions(-)
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel@xxxxxxxxxxxxx
> http://lists.xen.org/xen-devel



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