[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v5 0/4] Xen ABI feature control


  • To: Jürgen Groß <jgross@xxxxxxxx>, Paul Durrant <paul@xxxxxxx>, "xen-devel@xxxxxxxxxxxxxxxxxxxx" <xen-devel@xxxxxxxxxxxxxxxxxxxx>
  • From: Christian Lindig <christian.lindig@xxxxxxxxxx>
  • Date: Thu, 3 Dec 2020 13:32:47 +0000
  • Accept-language: en-GB, en-US
  • Arc-authentication-results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=citrix.com; dmarc=pass action=none header.from=citrix.com; dkim=pass header.d=citrix.com; arc=none
  • Arc-message-signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=gXlpwmINk2mGBNYkik9bxuW71GX8tebXQLV7unc1R7c=; b=RIrYIcj2nWoXfLbhDY2TTAjJDcGie8FSW9J0cTNmnmaxIPhkdsgvIIK7zsChAElWK4eaeeLVQBbwuyb/CCjVRzU9UaNdJcygXveglhnSIt06cS4eZfk7xEsrFEfIIjYRUQuF50PN+mI91xBrGz7iAz1XYM96fg4ZOhjff9QFEBLJlr/W9PWUjYoR+gg3CIHhTEAODlGZKc6rbobLYgGJPx9VU2vLPad4rgK8d4jpPpFvaII/8CxmrTrg0WvMUoybuVMEhXVZLQILZPHJ/L2VAeQo+lfYPgaf2zMXb+CL3f7DGuGLFsnUcSUhbdi0UbS0UENtopc2RGFnDkcctyKjSA==
  • Arc-seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=Yh9SOPutK6WOE//zhyxIsXlXSdao9WVRW10PV8w0SUWCZOP6hn1Gk2G7CVLhXK4URLMiTxPFoKI8LX8nPCKaF2329a7RH+THt8+Mr1YTjj/hKYN02qIXwl3iwOg84VlfRmiqyLT61j91i7lapguu00r30O1mtfwnMPkRltBJlT4kCywsYjD6NLQFF6/1wUiJ/GfQn1AaooNkkCp/EGenficC860nG4kAakd6F6R3yMd/myic+mX+wFQWe48khIefcYD9eE0Zjyf617bJfWG/Y9/E5GTKICnmsw8TzoM1kUG8a6kQlrfSqh60gPEDik1s06tLZwKZyNkvyIqGHBsudQ==
  • Authentication-results: esa2.hc3370-68.iphmx.com; dkim=pass (signature verified) header.i=@citrix.onmicrosoft.com
  • Cc: Paul Durrant <pdurrant@xxxxxxxxxx>, Andrew Cooper <Andrew.Cooper3@xxxxxxxxxx>, Anthony Perard <anthony.perard@xxxxxxxxxx>, David Scott <dave@xxxxxxxxxx>, George Dunlap <George.Dunlap@xxxxxxxxxx>, "Ian Jackson" <iwj@xxxxxxxxxxxxxx>, Jan Beulich <jbeulich@xxxxxxxx>, Julien Grall <julien@xxxxxxx>, Roger Pau Monne <roger.pau@xxxxxxxxxx>, Stefano Stabellini <sstabellini@xxxxxxxxxx>, Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>, "Wei Liu" <wl@xxxxxxx>
  • Delivery-date: Thu, 03 Dec 2020 13:33:09 +0000
  • Ironport-sdr: HPlPa6xiAPFCZraDZfIaMQ161sNyf3o+pS3O/hBbwvuYDSNhwN+b2C9SBaZkn9k0/urNSA+e5+ pxLzXQZcqun8YDpIAQ/2MmXCC8t6+q4Ejg1FsSnvhqY1EgJSyFWeG05jNjiP5HLMkSBZYzEz1t S+eOR2QcRErroJbE1dDXktttFiAkv1W3rKoNNoU+BajT0ty0T+hq0Hf/4QoD5hQN63GQkzQ966 +Zwz8pN50wYyhjZG//OA0yhuxLREo2QKeE7y28WdI/7FQAIT1q3R3TVU8VtzUxNOsmi51E4RjQ f3s=
  • List-id: Xen developer discussion <xen-devel.lists.xenproject.org>
  • Thread-index: AQHWyXjIEJK6osKOu0eZZfeqHV9qfg==
  • Thread-topic: [PATCH v5 0/4] Xen ABI feature control


Acked-by: Christian Lindig <christian.lindig@xxxxxxxxxx>




From: Jürgen Groß
Sent: Thursday, December 03, 2020 13:15
To: Paul Durrant; xen-devel@xxxxxxxxxxxxxxxxxxxx
Cc: Paul Durrant; Andrew Cooper; Anthony Perard; Christian Lindig; David Scott; George Dunlap; Ian Jackson; Jan Beulich; Julien Grall; Roger Pau Monne; Stefano Stabellini; Volodymyr Babchuk; Wei Liu
Subject: Re: [PATCH v5 0/4] Xen ABI feature control

On 03.12.20 13:41, Paul Durrant wrote:
> From: Paul Durrant <pdurrant@xxxxxxxxxx>
>
> This series was previously called "evtchn: Introduce a per-guest knob to
> control FIFO ABI". It is been extensively re-worked and extended to cover
> another ABI feature.
>
> Paul Durrant (4):
>    domctl: introduce a new domain create flag,
>      XEN_DOMCTL_CDF_evtchn_fifo, ...
>    domctl: introduce a new domain create flag,
>      XEN_DOMCTL_CDF_evtchn_upcall, ...
>    libxl: introduce a 'libxl_xen_abi_features' enumeration...
>    xl: introduce a 'xen-abi-features' option...
>
>   docs/man/xl.cfg.5.pod.in         | 50 ++++++++++++++++++++++++++++++++
>   tools/include/libxl.h            | 10 +++++++
>   tools/libs/light/libxl_arm.c     | 22 +++++++++-----
>   tools/libs/light/libxl_create.c  | 31 ++++++++++++++++++++
>   tools/libs/light/libxl_types.idl |  7 +++++
>   tools/libs/light/libxl_x86.c     | 17 ++++++++++-
>   tools/ocaml/libs/xc/xenctrl.ml   |  2 ++
>   tools/ocaml/libs/xc/xenctrl.mli  |  2 ++
>   tools/xl/xl_parse.c              | 50 ++++++++++++++++++++++++++++++--
>   xen/arch/arm/domain.c            |  3 +-
>   xen/arch/arm/domain_build.c      |  3 +-
>   xen/arch/arm/setup.c             |  3 +-
>   xen/arch/x86/domain.c            |  8 +++++
>   xen/arch/x86/hvm/hvm.c           |  3 ++
>   xen/arch/x86/setup.c             |  4 ++-
>   xen/common/domain.c              |  3 +-
>   xen/common/event_channel.c       | 24 +++++++++++++--
>   xen/include/public/domctl.h      |  6 +++-
>   18 files changed, 229 insertions(+), 19 deletions(-)
> ---
> Cc: Andrew Cooper <andrew.cooper3@xxxxxxxxxx>
> Cc: Anthony PERARD <anthony.perard@xxxxxxxxxx>
> Cc: Christian Lindig <christian.lindig@xxxxxxxxxx>
> Cc: David Scott <dave@xxxxxxxxxx>
> Cc: George Dunlap <george.dunlap@xxxxxxxxxx>
> Cc: Ian Jackson <iwj@xxxxxxxxxxxxxx>
> Cc: Jan Beulich <jbeulich@xxxxxxxx>
> Cc: Julien Grall <julien@xxxxxxx>
> Cc: "Roger Pau Monné" <roger.pau@xxxxxxxxxx>
> Cc: Stefano Stabellini <sstabellini@xxxxxxxxxx>
> Cc: Volodymyr Babchuk <Volodymyr_Babchuk@xxxxxxxx>
> Cc: Wei Liu <wl@xxxxxxx>
>

Do we want to add a create flag for each such feature, or would it be
better to set options like those via hypfs?

It would be fairly easy to ad dynamic hypfs paths, e.g.:

/domain/<domid>/abi-features/evtchn-fifo
/domain/<domid>/abi-features/evtchn-upcall

which would have boolean type and could be set as long as the domain
hasn't been started.

xl support could even be rather generic, without the need to add coding
to xl for each new feature.

This is no objection to this series, but just an idea how to avoid
extending the use of unstable interfaces.

Thoughts?


Juergen

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.